-
Notifications
You must be signed in to change notification settings - Fork 211
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
Implement namespace support for exceptions #421
Implement namespace support for exceptions #421
Conversation
Signed-off-by: Markus Blaschke <mblaschke82@gmail.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.
Great work, and thanks for adding a test! Just one little note
Can you also add documentation here: https://github.com/FairwindsOps/polaris/blob/master/docs/usage.md#exemptions |
Signed-off-by: Markus Blaschke <mblaschke82@gmail.com>
Signed-off-by: Markus Blaschke <mblaschke82@gmail.com>
Done |
With this new feature is it possible to ignore completely a namespace? Because I would like to get rid of the kube-system namespace in the dashboard. |
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.
This looks really good to me! Thank you!
@marcemv90 this should allow all controllers within a namespace to be exempted (since it matches on prefix you could add a controller-name of |
Add namespace support for exceptions for shared clusters.