Reduce ClusterPermission controller reconcile rate - #77
Conversation
Signed-off-by: fxiang1 <fxiang@redhat.com>
|
/uncc @philipwu08 |
Codecov Report✅ All modified and coverable lines are covered by tests. Additional details and impacted files@@ Coverage Diff @@
## main #77 +/- ##
==========================================
+ Coverage 60.28% 60.71% +0.43%
==========================================
Files 3 3
Lines 924 756 -168
==========================================
- Hits 557 459 -98
+ Misses 337 266 -71
- Partials 30 31 +1
Flags with carried forward coverage won't be shown. Click here to find out more. ☔ View full report in Codecov by Sentry. 🚀 New features to boost your workflow:
|
mikeshng
left a comment
There was a problem hiding this comment.
/approve
/lgtm
The resync period should be optionally exposed to pass in from an ENV or cmd param or something.The default is also a bit strange. Probably something like 12 hrs or 24 hrs better. But these are not important details.
|
[APPROVALNOTIFIER] This PR is APPROVED This pull-request has been approved by: fxiang1, mikeshng The full list of commands accepted by this bot can be found here. The pull request process is described here DetailsNeeds approval from an approver in each of these files:
Approvers can indicate their approval by writing |
af5a756
into
open-cluster-management-io:main
https://issues.redhat.com/browse/ACM-25572