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

chore: bump the base image versions #12

Merged
merged 1 commit into from
Feb 25, 2024
Merged

Conversation

budimanjojo
Copy link
Contributor

@budimanjojo budimanjojo commented Feb 24, 2024

I got a problem with pgbackrest after upgrading my chart version to 5.5.1 because of the upgraded pgbackrest version (2.47-2.49) and the one came with this base image is still having the old version, causing the backup to fail with

ERROR: [082]: WAL segment <segment> was not archived before the 60000ms timeout
       HINT: check the archive_command to ensure that all options are correct (especially --stanza).
       HINT: check the PostgreSQL server log for errors.
       HINT: run the 'start' command if the stanza was previously stopped.

And this commit should be done by renovatebot, are you sure you have it configured in this repo? Because looking at the commit history seems like the bot never create a PR or commit bumping version.

@budimanjojo
Copy link
Contributor Author

budimanjojo commented Feb 24, 2024

Weird that the postgres 14 is not found. I use the same value from helm show values oci://registry.developers.crunchydata.com/crunchydata/pgo --version 5.5.1.

@budimanjojo
Copy link
Contributor Author

Created an issue to CrunchyData: CrunchyData/postgres-operator-examples#255

Signed-off-by: budimanjojo <budimanjojo@gmail.com>
@budimanjojo
Copy link
Contributor Author

I reverted the image for version 14 so the test pass. Can this get merged? And should I also create a PR for the renovatebot so this can be done by the bot in the future?

@chkpwd
Copy link
Owner

chkpwd commented Feb 25, 2024

Yea I'll merge this now. Please create a PR for the rest. If you can't today. I can do it Monday.

@chkpwd chkpwd merged commit 67dd112 into chkpwd:main Feb 25, 2024
4 checks passed
@budimanjojo budimanjojo deleted the patch-1 branch February 25, 2024 14:25
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.

None yet

2 participants