monokle
monokle copied to clipboard
Add a CTA to disable a validation from the audit pane
When reviewing the validation pane errors and warning, in some cases the errors are not applicable or the user does not care about them. Currently, in order to mute those errors and warnings they need to move into setting, abandoning what they are doing, find the error, disable it, and then come back to the panel.
It should be possible to do this inside the validation pane:
- Disable the validation permanently as a CTA for the error itself
- Disable the validation for this resource (add the configuration to .monokle)
- Link to validation configuration
SARIF has both inline and external suppressions for problems. There are always exceptions to the rules which is why suppressions are essential to get to zero misconfigurations. Otherwise it will keep bothering you to keep an issue that you know is not an issue
Some way to suppress warnings would be useful for me. I use nginx for one of my apps and one of the warnings I get is that the container's file system is read only, but setting the FS to read only prevents the container from starting up.
Perhaps some kind of container label would work to suppress whatever validation checks are unwanted for that container? Like so
- name: nginx
image: nginx:stable-alpine3.17-slim
labels:
monokle: "@linter-suppress:KSV003,KSV014"
This way it's clear in the deployments themselves that some warnings are intentionally being suppressed and if multiple people are working on the same project they don't need to keep track of what warnings can be suppressed since it's all there in the code.