Skip to content

fix: fetch kb version from helm release chart before upgrade#466

Merged
yipeng1030 merged 2 commits intomainfrom
bugfix/fix_method_of_fetch_version_when_upgrade
Oct 14, 2024
Merged

fix: fetch kb version from helm release chart before upgrade#466
yipeng1030 merged 2 commits intomainfrom
bugfix/fix_method_of_fetch_version_when_upgrade

Conversation

@yipeng1030
Copy link
Copy Markdown
Contributor

fix #452

What's changed?
get the KB version from the helm release, instead of from KB deployment.

@github-actions github-actions Bot added the size/M Denotes a PR that changes 30-99 lines. label Oct 11, 2024
@codecov-commenter
Copy link
Copy Markdown

codecov-commenter commented Oct 11, 2024

Codecov Report

❌ Patch coverage is 62.06897% with 11 lines in your changes missing coverage. Please review.
✅ Project coverage is 41.59%. Comparing base (65f8876) to head (2ba8975).
⚠️ Report is 108 commits behind head on main.

Files with missing lines Patch % Lines
pkg/cmd/kubeblocks/upgrade.go 64.00% 6 Missing and 3 partials ⚠️
pkg/util/helm/helm.go 50.00% 2 Missing ⚠️
Additional details and impacted files
@@            Coverage Diff             @@
##             main     #466      +/-   ##
==========================================
+ Coverage   41.58%   41.59%   +0.01%     
==========================================
  Files         165      165              
  Lines       22883    22893      +10     
==========================================
+ Hits         9515     9522       +7     
- Misses      12253    12255       +2     
- Partials     1115     1116       +1     
Flag Coverage Δ
unittests 41.59% <62.06%> (+0.01%) ⬆️

Flags with carried forward coverage won't be shown. Click here to find out more.

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

🚀 New features to boost your workflow:
  • ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.

@yipeng1030 yipeng1030 merged commit b291757 into main Oct 14, 2024
@yipeng1030 yipeng1030 deleted the bugfix/fix_method_of_fetch_version_when_upgrade branch October 14, 2024 08:32
@github-actions github-actions Bot added this to the Release 1.0 milestone Oct 14, 2024
@yipeng1030
Copy link
Copy Markdown
Contributor Author

/cherry-pick release-0.9

@github-actions
Copy link
Copy Markdown

🤖 says: cherry pick action finished successfully 🎉!
See: https://github.com/apecloud/kbcli/actions/runs/11323857849

github-actions Bot pushed a commit that referenced this pull request Oct 14, 2024
@shanshanying
Copy link
Copy Markdown
Contributor

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

size/M Denotes a PR that changes 30-99 lines.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

[BUG] "kbcli kb upgrade" fails when interrupted due to network issues.

4 participants