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

Web-app expects incorrect secret key #289

Closed
grenader opened this issue Feb 3, 2022 · 4 comments
Closed

Web-app expects incorrect secret key #289

grenader opened this issue Feb 3, 2022 · 4 comments

Comments

@grenader
Copy link

grenader commented Feb 3, 2022

Steps to reproduce:

helm repo add concourse https://concourse-charts.storage.googleapis.com/
helm install con1 concourse/concourse

... wait
kubectl get pod

The web-application pod will not start.
"describe" it and see the following error:

Error: couldn't find key postgresql-password in Secret default/con1-postgresql

"describe" the secret, and observe that it has a different key.

kubectl describe secret con1-postgresql

Output:

....
Data
====
postgres-password:  10 bytes

The problem is that web app is expecting an incorrect key in the secret.

@chris-sanders
Copy link

chris-sanders commented Feb 5, 2022

I've just hit this too, the reason this is happening is that the Bitnami chart was given a major version change inside of the concourse charts patch version change.

During that change Bitnami changed the password field, but that's not all that was changed. See the reference https://docs.bitnami.com/kubernetes/infrastructure/postgresql/administration/upgrade/

Anyone attempting an upgrade is going to fail due to this change. Keys were relocated for authentication, persistence, and the version of postgres was updated. Even if the user gets all of the mapping correct the new chart won't read the database w/o explicitly setting the version or performing a manual upgrade. Finally, the new chart changes the statefulset which you can't do.

Honestly I think this chart should revert the Bitnmai sub-chart bump, and release a new major version of the chart that includes the bump tested for the new values.

@grenader
Copy link
Author

grenader commented Feb 7, 2022

Right, @chris-sanders
I have also found that the new Bitnamy Postgres has created a problem. Manual recreation of the secret with additional keys did not help.
We are successfully using v16.1.16 of your chart now.

@xtremerui
Copy link
Contributor

@chris-sanders thx for the input. We will revert the bump for concourse chart v16.

@xtremerui
Copy link
Contributor

16.1.19 is released with the revert of postgresql bump.

#290 is created to track the work for concourse-chart next major release.

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

No branches or pull requests

3 participants