Make otlp ports configurable on deployment

Signed-off-by: Jon Bates <[email protected]>
jaegertracing · Jun 23, 2023 · 3475f96 · 3475f96
1 parent 8500d1b
commit 3475f96
Show file tree

Hide file tree

Showing 2 changed files with 5 additions and 5 deletions.
diff --git a/charts/jaeger/templates/collector-deploy.yaml b/charts/jaeger/templates/collector-deploy.yaml
@@ -119,10 +119,10 @@ spec:
         {{- end }}
         {{- if or .Values.collector.service.otlp.grpc .Values.collector.service.otlp.http }}
         - containerPort: {{ default 4317 .Values.collector.service.otlp.grpc.port }}
-          name: otlp-grpc
+          name: {{ .Values.collector.service.otlp.grpc.name }}
           protocol: TCP
         - containerPort: {{ default 4318 .Values.collector.service.otlp.http.port }}
-          name: otlp-http
+          name: {{ .Values.collector.service.otlp.http.name }}
           protocol: TCP
         {{- end }}
         readinessProbe:

diff --git a/charts/jaeger/values.yaml b/charts/jaeger/values.yaml
@@ -36,9 +36,9 @@ allInOne:
     collector:
       otlp:
         grpc:
-          name: otlp-grpc
+          name: grpc-otlp
         http:
-          name: otlp-http
+          name: http-otlp
   ingress:
     enabled: false
     # For Kubernetes >= 1.18 you should specify the ingress-controller via the field ingressClassName
@@ -383,7 +383,7 @@ collector:
         # port: 4317
         # nodePort:
       http:
-        name: otlp-grpc
+        name: otlp-http
         # port: 4318
         # nodePort:
   ingress: