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

pgbouncer extension points #104

Merged
merged 2 commits into from
Aug 24, 2021
Merged

pgbouncer extension points #104

merged 2 commits into from
Aug 24, 2021

Conversation

benbz
Copy link
Contributor

@benbz benbz commented Aug 24, 2021

e.g. setting in values.yaml:

pgbouncer:
  env:
  - name: SERVER_TLS_SSLMODE
    value: require
  extraVolumeMounts:
  - name: config
    mountPath: /etc/pgbouncer
  extraVolumes:
  - name: config
    emptyDir: {}

now results in server_tls_sslmode = require in pgbouncer.ini and uses an emptyDir rather than writing files in the container. This is useful for OpenShift as the entrypoint script can't write files to /etc/pgbouncer. It is not sufficient on its own - a custom entrypoint script or pgbouncer.ini is also required so that user = postgres is replaced with user = as per edoburu/docker-pgbouncer#23

@tiina303 tiina303 added the bump patch Updates chart version by 0.0.1 label Aug 24, 2021
Copy link
Contributor

@tiina303 tiina303 left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Thanks for improving our charts 😃

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bump patch Updates chart version by 0.0.1
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants