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

Specify sslrootcert in database options #146

Open
wants to merge 2 commits into
base: main
Choose a base branch
from

Conversation

drodowic
Copy link
Collaborator

SUMMARY

Adds sslrootcert in the database options pointing to the volume mount of the bundle_cacert_secret. This allows a custom CA to be used if sslmode is set to verify-ca or verify-full in the postgres_configuration_secret.

Issue: AAP-32390

@@ -24,6 +24,7 @@
CONN_MAX_AGE: 0
OPTIONS:
sslmode: '{{ postgres_sslmode }}'
sslrootcert: /etc/pki/ca-trust/source/anchors/bundle-ca.crt
Copy link
Contributor

Choose a reason for hiding this comment

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

Suggested change
sslrootcert: /etc/pki/ca-trust/source/anchors/bundle-ca.crt
sslrootcert: /etc/pki/tls/certs/ca-bundle.crt

https://github.com/ansible/awx-operator/blob/devel/roles/installer/defaults/main.yml#L443

I thought we needed sslrootcert only when the sslmode value was either verify-full or verify-ca, isn't the case ?

Copy link
Contributor

@dsavineau dsavineau left a comment

Choose a reason for hiding this comment

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

Could you rebase your PR with latest changes from the main branch ? this should fix the CI errors

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