Merge pull request #1262 from NanaKhadija1980j/cleanup/1221-remove-unused-env-vars

cleanup: remove unused deployment environment variables (#1221)

Author: OtowoSamuel

charts/stellar-operator/examples/values-production.yaml

@@ -85,7 +85,6 @@ byzantineWatcher:
 otel:
   enabled: true
   endpoint: "http://tempo.monitoring.svc.cluster.local:4317"
-  samplingRatio: "0.1"
   collector:
     enabled: true
     config:

charts/stellar-operator/templates/deployment.yaml

@@ -65,12 +65,6 @@ spec:
             {{- if .Values.otel.enabled }}
             - name: OTEL_EXPORTER_OTLP_ENDPOINT
               value: {{ .Values.otel.endpoint | quote }}
-            - name: OTEL_SERVICE_NAME
-              value: "stellar-operator"
-            - name: OTEL_TRACES_SAMPLER
-              value: "parentbased_traceidratio"
-            - name: OTEL_TRACES_SAMPLER_ARG
-              value: {{ .Values.otel.samplingRatio | quote }}
             {{- end }}
           volumeMounts:
             - name: operator-config

charts/stellar-operator/values.yaml

@@ -484,6 +484,8 @@ byzantineWatcher:
 
 # OpenTelemetry configuration
 # Controls OTLP trace export from the operator and the bundled OTel Collector.
+# The operator sets service.name and sampling in code (AlwaysOn); only the OTLP
+# endpoint is injected via OTEL_EXPORTER_OTLP_ENDPOINT when enabled.
 otel:
   # Set to true to enable OTLP trace export from the operator.
   # When enabled, OTEL_EXPORTER_OTLP_ENDPOINT is injected into the operator pod.
@@ -494,9 +496,6 @@ otel:
   # Example: "http://stellar-operator-otel-collector:4317"
   endpoint: "http://stellar-operator-otel-collector:4317"
 
-  # Sampling ratio (0.0–1.0). 1.0 = always sample, 0.1 = 10% of traces.
-  samplingRatio: "1.0"
-
   # OTel Collector sidecar/deployment bundled with the Helm chart.
   collector:
     # Deploy the OTel Collector alongside the operator.