Merge pull request #1828 from alphagov/sengi/otel-nil-deref

Disable OpenTelemetry in prod until #1827 is fixed.
alphagov · Feb 8, 2024 · 7df4cf3 · 7df4cf3
2 parents 80ff5f5 + 0c5afa2
commit 7df4cf3
Showing 1 changed file with 3 additions and 0 deletions.
diff --git a/charts/app-config/templates/env-configmap.yaml b/charts/app-config/templates/env-configmap.yaml
@@ -41,11 +41,14 @@ data:
   GOVUK_DATA_SYNC_PERIOD: "22:0-8:0"
   {{- end }}
 
+  {{- if ne .Values.govukEnvironment "production" }}
+  # TODO(#1827): fix OTel nil dereference so we can re-enable it in production.
   ENABLE_OPEN_TELEMETRY: "true"
   OTEL_EXPORTER_OTLP_ENDPOINT: "http://tempo-distributor.monitoring.svc.cluster.local:4318"
   OTEL_TRACES_SAMPLER: "traceidratio"
   OTEL_TRACES_SAMPLER_ARG: '0.001'
   OTEL_RUBY_INSTRUMENTATION_RACK_CONFIG_OPTS: "untraced_endpoints=/healthcheck/live"
+  {{- end }}
 
   PLEK_SERVICE_ASSETS_URI: https://{{ .Values.assetsDomain }}
   PLEK_SERVICE_DRAFT_ASSETS_URI: https://draft-assets.{{ .Values.publishingDomainSuffix }}