-
Notifications
You must be signed in to change notification settings - Fork 219
Update PostgreSQL version used in Helm charts #6786
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
Conversation
If I understand bitnami/containers#83267 correctly, we can only use the latest version, not a specific version anymore. |
That was also a bit the idea behind this change and thus the version I put there is the latest¹. I don't know how to specify "latest" in these YAML files but that would probably be the best solution. ¹ Or at least the version mentioned on https://artifacthub.io/packages/helm/bitnami/postgresql?modal=install but that's probably the latest. |
hm, it did not work after all :(
|
Maybe just |
No, but using |
This still doesn't work:
A local
|
Maybe we can use https://hub.docker.com/r/bitnamilegacy/postgresql for now. I don't fully understand what they're doing, but that should give us a working check for now... https://hub.docker.com/r/bitnami/postgresql is completely empty |
f07281b
to
fcef4b8
Compare
Looks like the container from |
Can you clarify what is missing? I can't seem to see the workflow, it just loads forever and remains empty |
Firefox also has problems showing the logs but it eventually shows:
and then many errors like this one further down below:
|
I changed it to use the legacy repo now. Let's see whether that works at least. |
I'm getting the same with the legacy repo now. I guess finding an exact constellation of Char and image that work together is taking too long now. So I'll create a PR disabling the check for now (so other PRs can at least be merged automatically again). |
Use https://hub.docker.com/r/bitnamisecure/postgresql instead of deprecated/removed https://hub.docker.com/r/bitnami/postgresql. Related ticket: https://progress.opensuse.org/issues/183056
One more attempt where I just change the container and not update the version of the Helm Chart. |
It still shows the same errors as in #6786 (comment). So I guess I can close this and we'll have to go with #6788 for now. |
Codecov Report✅ All modified and coverable lines are covered by tests. Additional details and impacted files@@ Coverage Diff @@
## master #6786 +/- ##
==========================================
- Coverage 99.25% 99.25% -0.01%
==========================================
Files 402 402
Lines 41285 41285
==========================================
- Hits 40979 40978 -1
- Misses 306 307 +1 ☔ View full report in Codecov by Sentry. 🚀 New features to boost your workflow:
|
Maybe this helps with https://progress.opensuse.org/issues/188982.