-
Notifications
You must be signed in to change notification settings - Fork 8.5k
CI: Update Helm to v4.0.1. #14256
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: release-1.14
Are you sure you want to change the base?
CI: Update Helm to v4.0.1. #14256
Conversation
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.
Pull request overview
This pull request updates Helm from v3.19.2 to v4.0.1, which is a major version upgrade.
Key Changes:
- Updates Helm version references in CI/CD workflows, Makefiles, and tool configuration
- Updates Go module dependencies for Helm v4
- Changes import path for Helm chartutil package
Reviewed changes
Copilot reviewed 7 out of 8 changed files in this pull request and generated 2 comments.
Show a summary per file
| File | Description |
|---|---|
magefiles/steps/helm.go |
Updates import path for chartutil from v3 to v4 (contains incorrect path) |
images/test-runner/Makefile |
Updates HELM_VERSION build argument from 3.19.2 to 4.0.1 |
hack/.tool-versions |
Updates helm tool version from 3.19.2 to 4.0.1 |
go.sum |
Updates go.sum with new Helm v4 module checksums and removes obsolete dependencies |
go.mod |
Updates helm.sh/helm dependency from v3.19.2 to v4.0.1 and removes indirect dependencies |
charts/Makefile |
Updates helm installation command to use v4 module path |
.github/workflows/ci.yaml |
Updates Helm version in CI workflow from v3.19.2 to v4.0.1 |
.github/workflows/chart.yaml |
Updates Helm version in chart workflow from v3.19.2 to v4.0.1 |
💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.
cpanato
left a comment
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.
/lgtm
/approve
tao12345666333
left a comment
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.
/lgtm
|
[APPROVALNOTIFIER] This PR is APPROVED This pull-request has been approved by: cpanato, Gacko, tao12345666333 The full list of commands accepted by this bot can be found here. The pull request process is described here
Needs approval from an approver in each of these files:
Approvers can indicate their approval by writing |
/triage accepted
/kind cleanup
/priority backlog
/cc @cpanato @strongjz @tao12345666333
/hold