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

fix: Skip mounting Postgres TLS certificats from secret when sslSource is 'manual' #189

Conversation

jsirianni
Copy link
Member

@jsirianni jsirianni commented Dec 12, 2024

Description of Changes

When Postgres sslSource is not "source", the config map for initializing the Postgres TLS emptydir volume should still be used, but it should skip copying the files from the secret to the emptyDir.

Please check that the PR fulfills these requirements

  • Tests for the changes have been added (for bug fixes / features)
  • Docs have been added / updated (for bug fixes / features)
  • CI passes
  • Changes to ports, services, or other networking have been tested with istio

@jsirianni jsirianni marked this pull request as ready for review December 12, 2024 21:11
@jsirianni jsirianni requested a review from tbm48813 as a code owner December 12, 2024 21:11
Base automatically changed from joesirianni/bpop-1278-helm-allow-setting-terminationgraceperiodsecondsfor-all to main December 12, 2024 21:13
@jsirianni jsirianni force-pushed the joesirianni/bpop-1279-helm-allow-postgres-tls-configuration-to-use-volume-mount.2 branch from 3ca4a6a to 727466b Compare December 12, 2024 21:13
@jsirianni jsirianni merged commit 73278e7 into main Dec 12, 2024
22 checks passed
@jsirianni jsirianni deleted the joesirianni/bpop-1279-helm-allow-postgres-tls-configuration-to-use-volume-mount.2 branch December 12, 2024 21:27
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