shouldAuditGlobalSnapshot property (#402)

* shouldAuditGlobalSnapshot property
allegro · Dec 13, 2023 · c256699 · c256699
1 parent 977567b
commit c256699
Show file tree

Hide file tree

Showing 3 changed files with 11 additions and 1 deletion.
diff --git a/CHANGELOG.md b/CHANGELOG.md
@@ -7,6 +7,11 @@ The format is based on [Keep a Changelog](http://keepachangelog.com/en/1.0.0/).
 ### Changed
 - Added possibility to add response header for weighted secondary cluster 
 
+## [0.20.4]
+
+### Changed
+- Fix `shouldAuditGlobalSnapshot` property
+
 ## [0.20.3]
 
 ### Changed

diff --git a/envoy-control-core/src/main/kotlin/pl/allegro/tech/servicemesh/envoycontrol/ControlPlane.kt b/envoy-control-core/src/main/kotlin/pl/allegro/tech/servicemesh/envoycontrol/ControlPlane.kt
@@ -337,7 +337,11 @@ class ControlPlane private constructor(
         }
 
         fun withSnapshotChangeAuditor(snapshotChangeAuditor: SnapshotChangeAuditor): ControlPlaneBuilder {
-            this.snapshotChangeAuditor = snapshotChangeAuditor
+            if (properties.envoy.snapshot.shouldAuditGlobalSnapshot) {
+                this.snapshotChangeAuditor = snapshotChangeAuditor
+            } else {
+                this.snapshotChangeAuditor = NoopSnapshotChangeAuditor
+            }
             return this
         }
 

diff --git a/...e/src/main/kotlin/pl/allegro/tech/servicemesh/envoycontrol/snapshot/SnapshotProperties.kt b/...e/src/main/kotlin/pl/allegro/tech/servicemesh/envoycontrol/snapshot/SnapshotProperties.kt
@@ -36,6 +36,7 @@ class SnapshotProperties {
     var deltaXdsEnabled = false
     var retryPolicy = RetryPolicyProperties()
     var tcpDumpsEnabled: Boolean = true
+    var shouldAuditGlobalSnapshot: Boolean = true
 }
 
 class MetricsProperties {