OpenTelemetry Environment Variable from ConfigMap variable (#23)

* add conditional otel env variables in deployment.yaml * add changelog and v0.7.2 bump * remove conditional otel hostname env var * change changelog entry
lockerstock · Aug 17, 2023 · b7e4601 · b7e4601
1 parent a0c4136
commit b7e4601
Show file tree

Hide file tree

Showing 3 changed files with 11 additions and 2 deletions.
diff --git a/charts/microservice/CHANGELOG.md b/charts/microservice/CHANGELOG.md
@@ -19,6 +19,12 @@ and this project adheres to [Semantic Versioning](https://semver.org/spec/v2.0.0
 
 ### Security
 
+## [v0.7.2] - 2023-08-16
+
+### Fixed
+
+- OpenTelemetry environment variables to reference ConfigMap variable (https://github.com/lockerstock/helm-charts/pull/23).
+
 ## [v0.7.1] - 2023-07-28
 
 ### Fixed

diff --git a/charts/microservice/Chart.yaml b/charts/microservice/Chart.yaml
@@ -20,7 +20,7 @@ type: application
 # This is the chart version. This version number should be incremented each time you make changes
 # to the chart and its templates, including the app version.
 # Versions are expected to follow Semantic Versioning (https://semver.org/)
-version: 'v0.7.1'
+version: 'v0.7.2'
 
 # This is the version number of the application being deployed. This version number should be
 # incremented each time you make changes to the application. Versions are not expected to

diff --git a/charts/microservice/templates/deployment.yaml b/charts/microservice/templates/deployment.yaml
@@ -77,7 +77,10 @@ spec:
 
             {{- if .Values.deployment.opentelemetry.enabled }}
             - name: OTEL_EXPORTER_OTLP_ENDPOINT
-              value: k8s-app-opentelemetry:4317
+              valueFrom:
+                configMapKeyRef:
+                  name: otel-collector-routing
+                  key: grpcAddress
             {{- end }}
 
             {{- if .Values.deployment.cache.enabled }}