concourse / concourse-chart

Helm chart to install Concourse
Apache License 2.0
143 stars 174 forks source link

Add the complete list of supported options for postgres.sslmode #309

Closed hoffmc closed 7 months ago

hoffmc commented 1 year ago

Existing Issue

Why do we need this PR?

It's unclear which value needs to be set to enable SSL encryption for the postgres database connection. The most reasonable value - 'enable' - does not exist and the Concourse source code needs to be read to understand the available choices.

Changes proposed in this pull request

Contributor Checklist

Reviewer Checklist

This section is intended for the core maintainers only, to track review progress. Please do not fill out this section.

  • [ ] Code reviewed
  • [ ] Topgun tests run
  • [ ] Back-port if needed
  • [ ] Is the correct branch targeted? (master or dev)
xtremerui commented 1 year ago

LGTM. Could you fix the DCO check error?