Merge pull request #241 from vshn/redis_exporter_bugfix

disable certificates verification in redis metrics exporter
vshn · Oct 12, 2023 · 7aea368 · 7aea368
2 parents 8084c44 + d11e7c2
commit 7aea368
Show file tree

Hide file tree

Showing 2 changed files with 13 additions and 0 deletions.
diff --git a/component/component/vshn_redis.jsonnet b/component/component/vshn_redis.jsonnet
@@ -398,6 +398,16 @@ local composition =
           values: {
             metrics: {
               enabled: true,
+              // before all Your warning lamps start blinking,
+              // this is internal communication on loopback interface
+              // full mTLS isn't necessary
+              extraEnvVars: [
+                {
+                  name: 'REDIS_EXPORTER_SKIP_TLS_VERIFICATION',
+                  value: 'true',
+
+                },
+              ],
               containerSecurityContext: {
                 enabled: securityContext,
               },

diff --git a/component/tests/golden/vshn/appcat/appcat/21_composition_vshn_redis.yaml b/component/tests/golden/vshn/appcat/appcat/21_composition_vshn_redis.yaml
@@ -703,6 +703,9 @@ spec:
                 containerSecurityContext:
                   enabled: true
                 enabled: true
+                extraEnvVars:
+                  - name: REDIS_EXPORTER_SKIP_TLS_VERIFICATION
+                    value: 'true'
                 serviceMonitor:
                   enabled: true
                   namespace: ''