-
Notifications
You must be signed in to change notification settings - Fork 0
chore(deps): update postgresql docker tag to v18 #148
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
base: main
Are you sure you want to change the base?
Conversation
01_infrastructure/postgresql.tf
Outdated
| repository = "oci://registry-1.docker.io/bitnamicharts" | ||
| chart = "postgresql" | ||
| version = "16.7.26" # Stable version with PostgreSQL 16.x | ||
| version = "18.0.7" # Stable version with PostgreSQL 16.x |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Potential bug: The PostgreSQL Helm chart is undergoing a major version upgrade from 16.x to 18.x without an accompanying data migration strategy, risking database startup failure on existing deployments.
-
Description: The PostgreSQL Helm chart version is being upgraded from
16.7.26to18.0.7, which is a major version jump. While persistence is not explicitly configured, the Bitnami chart enables it by default. The change lacks any data migration strategy, such aspg_dump/pg_restoreorpg_upgradeprocedures. If this infrastructure is applied to an environment with existing data in a persistent volume, the new PostgreSQL 18.x instance will fail to start due to data format incompatibility with the old 16.x data, causing the database to crash loop and leading to a service outage. -
Suggested fix: Implement a data migration strategy for the major version upgrade. This typically involves using pre-upgrade and post-upgrade hooks to perform a backup (e.g.,
pg_dump) before the chart is upgraded and a restore (pg_restore) after the new version is deployed. The upgrade should be a carefully managed process, not an automated dependency update without migration logic.
severity: 0.95, confidence: 0.98
Did we get this right? 👍 / 👎 to inform future reviews.
ac1d696 to
ddbc76a
Compare
1dcee96 to
de5ddc1
Compare
384b1de to
f1fd638
Compare
6be589f to
063fa37
Compare
18c4da1 to
6fc8be5
Compare
df877de to
e3e92af
Compare
90bd0be to
88ba6af
Compare
01_infrastructure/postgresql.tf
Outdated
| # Provides regularly updated, security-patched PostgreSQL deployments | ||
| repository = "oci://registry-1.docker.io/bitnamicharts" | ||
| chart = "postgresql" | ||
| version = "16.7.26" # Stable version with PostgreSQL 16.x | ||
| version = "18.1.15" # Stable version with PostgreSQL 16.x |
This comment was marked as outdated.
This comment was marked as outdated.
Sorry, something went wrong.
2fb2bd2 to
2a0b439
Compare
Signed-off-by: renovate[bot] <29139614+renovate[bot]@users.noreply.github.com>
2a0b439 to
357b306
Compare
This PR contains the following updates:
16.7.26→18.2.0Release Notes
bitnami/charts (postgresql)
v16.7.27Configuration
📅 Schedule: Branch creation - At 12:00 AM through 04:59 AM and 10:00 PM through 11:59 PM, Monday through Friday ( * 0-4,22-23 * * 1-5 ), Only on Sunday and Saturday ( * * * * 0,6 ) (UTC), Automerge - At any time (no schedule defined).
🚦 Automerge: Disabled by config. Please merge this manually once you are satisfied.
♻ Rebasing: Whenever PR becomes conflicted, or you tick the rebase/retry checkbox.
🔕 Ignore: Close this PR and you won't be reminded about this update again.
This PR was generated by Mend Renovate. View the repository job log.