Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Add c-ares to the run image for all stacks #307

Merged
merged 1 commit into from
May 22, 2024
Merged

Conversation

edmorley
Copy link
Member

@edmorley edmorley commented May 22, 2024

So that PgBouncer can use its recommended c-ares based DNS backend:
heroku/heroku-buildpack-pgbouncer#188
https://www.pgbouncer.org/install.html
https://c-ares.org/

The package is named libcares2 on Heroku-24 and libc-ares2 on older stacks:
https://packages.ubuntu.com/noble/libcares2
https://packages.ubuntu.com/jammy/libc-ares2
https://packages.ubuntu.com/focal/libc-ares2

This only adds ~200 KB to the run image.

I've not added the corresponding headers to the build image, since:

  • For the PgBouncer use-case they can instead be installed in the build environment used to compile the PgBouncer binaries.
  • There don't seem to be many popular c-ares language bindings that people might use directly in their apps.

GUS-W-15826387.

So that PgBouncer can use its recommended `c-ares` based DNS backend:
heroku/heroku-buildpack-pgbouncer#188
https://www.pgbouncer.org/install.html

The package is named `libcares2` on Heroku-24 and `libc-ares2` on older stacks:
https://packages.ubuntu.com/noble/libcares2
https://packages.ubuntu.com/jammy/libc-ares2
https://packages.ubuntu.com/focal/libc-ares2

This only adds ~200 KB to the run image.

I've not added the corresponding headers to the build image, since:
- For the PgBouncer use-case they can instead be installed in the build
  environment used to compile the PgBouncer binaries.
- There don't seem to be any popular `c-ares` language bindings that
  people might use directly in their apps.
@edmorley edmorley self-assigned this May 22, 2024
@edmorley edmorley marked this pull request as ready for review May 22, 2024 09:55
@edmorley edmorley requested a review from a team as a code owner May 22, 2024 09:55
@edmorley edmorley enabled auto-merge (squash) May 22, 2024 09:58
@edmorley edmorley merged commit 6577a2d into main May 22, 2024
4 checks passed
@edmorley edmorley deleted the edmorley/c-ares branch May 22, 2024 09:59
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants