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: Use the matchMode specified in the rbac configmap #9419
Conversation
Codecov Report
@@ Coverage Diff @@
## master #9419 +/- ##
=======================================
Coverage 45.86% 45.87%
=======================================
Files 221 221
Lines 26309 26311 +2
=======================================
+ Hits 12067 12069 +2
Misses 12586 12586
Partials 1656 1656
Continue to review full report at Codecov.
|
@chetan-rns looks like a pretty important bugfix. Do you think it should be cherry-picked to 2.4? |
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.
@chetan-rns The PR LGTM!
A minor comment, should we add a test case for below scenarios too?
- match mode = "glob" , and we pass in a regex
- match mode = "regex", and we pass in a glob
@crenshaw-dev I don't have a strong opinion about cherry picking, since the bug only affects the |
Ah gotcha. Let's let it wait for 2.5 then. |
@chetan-rns just checking in on this one. Will you have a chance to add tests? |
Signed-off-by: Chetan Banavikalmutt <chetanrns1997@gmail.com>
Signed-off-by: Chetan Banavikalmutt <chetanrns1997@gmail.com>
@crenshaw-dev I've included unit tests. PTAL |
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!
Read the
matchMode
field from the argocd-rbac-cm before checking the policyCloses: #9363
Signed-off-by: Chetan Banavikalmutt chetanrns1997@gmail.com
Note on DCO:
If the DCO action in the integration test fails, one or more of your commits are not signed off. Please click on the Details link next to the DCO action for instructions on how to resolve this.
Checklist: