-
Notifications
You must be signed in to change notification settings - Fork 498
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 to v2.17.0 #3611
Bump helm to v2.17.0 #3611
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.
LGTM
Since this only affects CI tests, IMO it's better to leave the release notes as |
Codecov Report
@@ Coverage Diff @@
## master #3611 +/- ##
==========================================
- Coverage 62.62% 62.43% -0.19%
==========================================
Files 164 164
Lines 17300 17354 +54
==========================================
+ Hits 10834 10835 +1
- Misses 5427 5477 +50
- Partials 1039 1042 +3
|
Author: shonge <[email protected]>
/run-all-tests |
LGTM, nice work 👍 |
/merge |
This pull request has been accepted and is ready to merge. Commit hash: abfb0b3
|
Signed-off-by: ti-srebot <[email protected]>
cherry pick to release-1.1 in PR #3650 |
* cherry pick #3611 to release-1.1 * update e2e Verify Upgrading Operator from v1.1.7 Co-authored-by: Ge Song <[email protected]> Co-authored-by: Yilong Li <[email protected]> Co-authored-by: DanielZhangQD <[email protected]>
What problem does this PR solve?
bump helm to v2.17.0
Fix #3610
What is changed and how does it work?
Code changes
Tests
Side effects
Related changes
Release Notes
Please refer to Release Notes Language Style Guide before writing the release note.