-
Notifications
You must be signed in to change notification settings - Fork 947
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
fix(helm): Explicitly set divisor in rbac-manager resources #5526
Conversation
…sources Signed-off-by: Maxence Boutet <52334444+mboutet@users.noreply.github.com>
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 - thanks @mboutet
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.
Looks like this is the 3rd PR in a series of fixes like this, thanks for being lucky number 3 @mboutet 😇
previous 2 PRs:
- fix(chart): explicitly set resourceFieldRef.divisor to avoid flapping #4538
- [bug] helm chart: explicitly set divisor in container resources #5198
Do you feel like this covers the full extent of where this divisor field should be set? or are there any potentially lingering locations to still consider? 🤔
@jbw976, seems like it was the only remaining place those were missing. |
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.
awesome, thank you for your thorough follow-up @mboutet! 💪
Could this be backported to release-1.15 branch? For future release of 1.15 helm chart to have the fix? |
/backport |
Successfully created backport PR for |
Description of your changes
This PR adds a divisor section in the
rbac-manager
Deployment of the Helm Chart, solving a continuous sync when Crossplane is deployed with ArgoCD.This is an extension of #5198.
I have:
make reviewable
to ensure this PR is ready for review.Added or updated unit tests.Added or updated e2e tests.Addedbackport release-x.y
labels to auto-backport this PR.