opf / helm-charts

OPF helm chart repository
https://charts.openproject.org
GNU General Public License v3.0
12 stars 22 forks source link

Improve secret management #46

Closed machisuji closed 9 months ago

machisuji commented 10 months ago

Takes the changes from #8 and #20, adapts them and unifies them. Thanks to @johanneskastl and @pitwegner for the PRs!

In the course of this process the following changes were made as well.

Moreover, installations or upgrades will fail if no postgres password is provided.

Error: INSTALLATION FAILED: execution error at (openproject/templates/worker-deployment.yaml:80:14): Database password is required. Please set postgresql.auth.existingSecret (recommended) or postgresql.auth.password

Important

For this to work it needs the latest OpenProject 13.1 images (to be released tomorrow, 13th Dec 2023). For the released images to work correctly, the database.yml bug has to be fixed as well. Working on that. Until this is released the test will fail with the following.

connection to server at "10.96.170.25", port 5432 failed: fe_sendauth: no password supplied (ActiveRecord::ConnectionNotEstablished)

So this PR will only go green once 13.1 is released including that bug fix which is when this will be merged.

changeset-bot[bot] commented 10 months ago

🦋 Changeset detected

Latest commit: 33c7ab022b7537ece39ef2404f6eaf0e7a633e55

The changes in this PR will be included in the next version bump.

This PR includes changesets to release 1 package | Name | Type | | ------------------------ | ----- | | @openproject/helm-charts | Major |

Not sure what this means? Click here to learn what changesets are.

Click here if you're a maintainer who wants to add another changeset to this PR