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

Bump PostgreSQL from 14.3.0 to 14.13.0 #414

Merged
merged 1 commit into from
Nov 18, 2024
Merged

Conversation

mitchnielsen
Copy link
Contributor

@mitchnielsen mitchnielsen commented Nov 18, 2024

Summary

Bumps the PostgreSQL image version from 14.3.0 to 14.13.0 to match the version used in production.

This version is currently overridden to downgrade the default version used by the chart, which is PostgreSQL 15.4.0.

When we're ready for newer versions of PostgreSQL, we can look into bumping the PostgreSQL chart version as well. At the moment, there are no newer chart v12 versions, so we'll stick with this for now.

Closes https://linear.app/prefect/issue/PLA-636/update-bitnami-postgresql-helm-chart

Bumps the PostgreSQL image version from 14.3.0 to 14.13.0 to match the
version used in production.

This version is currently overridden to downgrade the default version
used by the chart, which is PostgreSQL 15.4.0.

When we're ready for newer versions of PostgreSQL, we can look into
bumping the PostgreSQL chart version as well. At the moment, there are
no newer chart v12 versions, so we'll stick with this for now.

Closes https://linear.app/prefect/issue/PLA-636/update-bitnami-postgresql-helm-chart
@mitchnielsen mitchnielsen added the dependencies Pull requests that update a dependency file label Nov 18, 2024
@mitchnielsen mitchnielsen self-assigned this Nov 18, 2024
@mitchnielsen
Copy link
Contributor Author

Note: I'm not really sure why updatecli is trying to update the chart to v16, such as in #405. There's a semver pattern setteing here telling it to look for 12.x.x. I don't have much updatecli experience yet so maybe that's expected, but wanted to jot down the thought in case it's helpful later.

@mitchnielsen mitchnielsen marked this pull request as ready for review November 18, 2024 17:18
@mitchnielsen mitchnielsen requested a review from a team as a code owner November 18, 2024 17:18
@jimid27
Copy link
Contributor

jimid27 commented Nov 18, 2024

Note: I'm not really sure why updatecli is trying to update the chart to v16, such as in #405. There's a semver pattern setteing here telling it to look for 12.x.x. I don't have much updatecli experience yet so maybe that's expected, but wanted to jot down the thought in case it's helpful later.

fwiw #405 is the monthly update cli major run so it will just looking for the latest major minor or patch release of a chart

@mitchnielsen
Copy link
Contributor Author

fwiw #405 is the monthly update cli major run so it will just looking for the latest major minor or patch release of a chart

Ahh of course, thanks - suppose we could remove that from the major update cycle until we're ready to update, but maybe good to have it pop up periodically as a reminder

@mitchnielsen mitchnielsen merged commit b118aa4 into main Nov 18, 2024
13 checks passed
@mitchnielsen mitchnielsen deleted the upgrade-postgresql-14-13 branch November 18, 2024 17:26
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
dependencies Pull requests that update a dependency file
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants