-
Notifications
You must be signed in to change notification settings - Fork 410
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: set transform to strip managedfields for informer #2149
feat: set transform to strip managedfields for informer #2149
Conversation
Codecov ReportAttention: Patch coverage is
Additional details and impacted files@@ Coverage Diff @@
## master #2149 +/- ##
==========================================
+ Coverage 56.09% 58.97% +2.87%
==========================================
Files 186 211 +25
Lines 18092 19026 +934
==========================================
+ Hits 10149 11220 +1071
+ Misses 6910 6721 -189
- Partials 1033 1085 +52
Flags with carried forward coverage won't be shown. Click here to find out more. ☔ View full report in Codecov by Sentry. |
@vie-serendipity Thank you for posting this pull request. I think it is very useful to reduce the RAM cost for yurt-manager and yurthub component. but
please add |
/PTAL @rambohe-ch |
|
/LGTM |
What type of PR is this?
/kind feature
What this PR does / why we need it:
Reduce memory of yurt-manager and yurthub by stripping managedfields of objects. Because managed fields are often quite large, taking up around 30%. We usually don't need them, so we can strip them for memory efficiency.
Which issue(s) this PR fixes:
Fixes #
Special notes for your reviewer:
Does this PR introduce a user-facing change?
other Note