Update dependencies #91
Closed
Conversation
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Each major version of the chart increments the default pg image to the next major version as well. While this is overridable by the end user anyway, we currently don't assert an image version beyond the bitnami default so I don't want to introduce unnecessary churn without consensus. Additionally, last I looked, pgAdmin didn't support pg 15 yet, so it may not be wise to lock people out of a very common monitoring tool without an active choice on their part.
|
I had to add a AMD64 machine to my cluster to support Postgresql and Elasticseach. Getting this merged would be great |
|
i'd recommend switching to the OCI repositories as well: https://blog.bitnami.com/2023/04/httpsblog.bitnami.com202304bitnami-helm-charts-now-oci.html |
|
Already updated the dependencies (and moved to OCI) in another PR, so closing this one. |
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
Upgrade all three of our dependency charts. I upgraded major version for the redis chart because we specify the major version of the app to pull, while our elasticsearch chart is already on the latest major version. I limited myself to a minor-patch upgrade for postgresql for three reasons:
This PR addresses #90, #57, and #23. I believe it also addresses #25, but would like confirmation from someone running on an ARM cluster.