chore(helm): omit normal events by default (kyverno#9493)

* chore(helm): omit normal events by default Signed-off-by: Khaled Emara <khaled.emara@nirmata.com> * fix(tests): fix tests related to events Signed-off-by: Khaled Emara <khaled.emara@nirmata.com> --------- Signed-off-by: Khaled Emara <khaled.emara@nirmata.com>
bdun1013 · Jan 24, 2024 · 3ef598c · 3ef598c
1 parent 0b7a6a1
commit 3ef598c
Show file tree

Hide file tree

Showing 4 changed files with 9 additions and 4 deletions.
diff --git a/charts/kyverno/README.md b/charts/kyverno/README.md
@@ -327,7 +327,7 @@ The chart values are organised per component.
 | features.generateValidatingAdmissionPolicy.enabled | bool | `false` | Enables the feature |
 | features.logging.format | string | `"text"` | Logging format |
 | features.logging.verbosity | int | `2` | Logging verbosity |
-| features.omitEvents.eventTypes | list | `[]` | Events which should not be emitted (possible values `PolicyViolation`, `PolicyApplied`, `PolicyError`, and `PolicySkipped`) |
+| features.omitEvents.eventTypes | list | `["PolicyApplied","PolicySkipped"]` | Events which should not be emitted (possible values `PolicyViolation`, `PolicyApplied`, `PolicyError`, and `PolicySkipped`) |
 | features.policyExceptions.enabled | bool | `true` | Enables the feature |
 | features.policyExceptions.namespace | string | `""` | Restrict policy exceptions to a single namespace |
 | features.protectManagedResources.enabled | bool | `false` | Enables the feature |

diff --git a/charts/kyverno/values.yaml b/charts/kyverno/values.yaml
@@ -601,11 +601,11 @@ features:
     verbosity: 2
   omitEvents:
     # -- Events which should not be emitted (possible values `PolicyViolation`, `PolicyApplied`, `PolicyError`, and `PolicySkipped`)
-    eventTypes: []
+    eventTypes:
+      - PolicyApplied
+      - PolicySkipped
       # - PolicyViolation
-      # - PolicyApplied
       # - PolicyError
-      # - PolicySkipped
   policyExceptions:
     # -- Enables the feature
     enabled: true

diff --git a/config/install-latest-testing.yaml b/config/install-latest-testing.yaml
@@ -50395,6 +50395,7 @@ spec:
             - --generateValidatingAdmissionPolicy=false
             - --loggingFormat=text
             - --v=2
+            - --omit-events=PolicyApplied,PolicySkipped
             - --enablePolicyException=true
             - --protectManagedResources=false
             - --allowInsecureRegistry=false
@@ -50545,6 +50546,7 @@ spec:
             - --enableDeferredLoading=true
             - --loggingFormat=text
             - --v=2
+            - --omit-events=PolicyApplied,PolicySkipped
             - --enablePolicyException=true
 
           env:
@@ -50790,6 +50792,7 @@ spec:
             - --enableDeferredLoading=true
             - --loggingFormat=text
             - --v=2
+            - --omit-events=PolicyApplied,PolicySkipped
             - --enablePolicyException=true
             - --reportsChunkSize=1000
             - --allowInsecureRegistry=false

diff --git a/scripts/config/standard/kyverno.yaml b/scripts/config/standard/kyverno.yaml
@@ -1,6 +1,8 @@
 features:
   policyExceptions:
     enabled: true
+  omitEvents:
+    eventTypes: []
 
 backgroundController:
   rbac: