-
Notifications
You must be signed in to change notification settings - Fork 152
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 helm version in kanister build #2206
Conversation
The previous version of helm used inside kanister binary is causing integration tests to fail. This happen for the latest Postgresql helm charts. This is necessary as the bitnami helm chart for psotgresql has undergone changes in their handling of secrets and setting passwords during helm install. Signed-off-by: mabhi <abhijit.mukherjee@infracloud.io>
Thanks for submitting this pull request 🎉. The team will review it soon and get back to you. If you haven't already, please take a moment to review our project contributing guideline and Code of Conduct document. |
@@ -62,7 +62,8 @@ IMAGE_NAME := $(BIN) | |||
|
|||
IMAGE := $(REGISTRY)/$(IMAGE_NAME) | |||
|
|||
BUILD_IMAGE ?= ghcr.io/kanisterio/build:v0.0.23 |
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.
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.
Sure @pavannd1 , I actually was looking for this. For the time being I tested with a docker image to validate. Will push a new commit after running the workflow.
Reverted old reference of test build image. Signed-off-by: mabhi <abhijit.mukherjee@infracloud.io>
* Initial commit to bump helm version. The previous version of helm used inside kanister binary is causing integration tests to fail. This happen for the latest Postgresql helm charts. This is necessary as the bitnami helm chart for psotgresql has undergone changes in their handling of secrets and setting passwords during helm install. Signed-off-by: mabhi <abhijit.mukherjee@infracloud.io> * Makefile points to new build image with latest helm version Reverted old reference of test build image. Signed-off-by: mabhi <abhijit.mukherjee@infracloud.io> --------- Signed-off-by: mabhi <abhijit.mukherjee@infracloud.io> Co-authored-by: mergify[bot] <37929162+mergify[bot]@users.noreply.github.com>
Change Overview
The previous version of helm used inside kanister binary is causing integration tests to fail. This happen for the latest PostgreSql helm charts.
This PR is necessary as the bitnami helm chart for PostgreSql has undergone changes in their handling of secrets and setting passwords during helm install causing parsing errors during integration test. The bitnami fix is now available in the latest charts.
This bumps version from
3.2.0
->3.12.2
Pull request type
Please check the type of change your PR introduces:
Issues
Test Plan