search

sighupio / fury-kubernetes-opa

Kubernetes Fury Distribution OPA Core Module: Policy enforcement for your Kubernetes Cluster
https://kubernetesfury.com
BSD 3-Clause "New" or "Revised" License
38 stars 10 forks source link

bugfix: Audit skipped on provided Constraints #61

Closed ralgozino closed 2 years ago

ralgozino commented 2 years ago

Update provided ConstraintTemplates to consider the case that input.review.operation is not set when the audit process runs.

Fixes #59

ralgozino commented 2 years ago

Did you test all the cases? Should maybe add add this testing in the future also on the CI?

yeap, I did. Previously all the numbers in the left bar where in 0:

image

We have tests that check the admission phase, but we don't have them for the audit phase, we only check that the audit pod is healthy. It makes sense to add them, this is the second time we had audit not working as expected.

I'll create the issue. Thanks!