-
Notifications
You must be signed in to change notification settings - Fork 14
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
feat: addon support clean sub-resource of multi-versions #512
Conversation
e1a9b63
to
5b08751
Compare
Codecov ReportAttention: Patch coverage is
Additional details and impacted files@@ Coverage Diff @@
## main #512 +/- ##
==========================================
+ Coverage 41.16% 41.31% +0.14%
==========================================
Files 185 186 +1
Lines 24356 24540 +184
==========================================
+ Hits 10026 10138 +112
- Misses 13183 13238 +55
- Partials 1147 1164 +17
Flags with carried forward coverage won't be shown. Click here to find out more. ☔ View full report in Codecov by Sentry. |
93fbbdd
to
68919d7
Compare
f47f081
to
b766150
Compare
By far, the resource types which should be checked and cleaned are types.CompDefGVR(), types.ConfigmapGVR(), types.ConfigConstraintGVR(), types.ConfigConstraintOldGVR(). |
/cherry-pick release-1.0 |
🤖 says: |
/cherry-pick release-1.0 |
🤖 says: cherry pick action finished successfully 🎉! |
Co-authored-by: yipeng1030 <[email protected]> (cherry picked from commit e18cf6b)
fix #441
usage:
add a cmd
kbcli addon delete-resources-with-version [addonName] --versions= or --all-unused-versions=
example:
kbcli addon delete-resources-with-version postgresql --versions=0.9.1
kbcli addon delete-resources-with-version postgresql --all-unused-versions=true
effect:
clean resources which are belong to the specified addon, and not the newest version defined in the helm release, and not currently used by any cluster.
By far, the reource type will be checked and cleaned are
types.CompDefGVR(), types.ConfigmapGVR(), types.ConfigConstraintGVR(), types.ConfigConstraintOldGVR(),