Merge pull request #479 from spadger/jaeger-0.71.6

[jaeger] Fix otlp service regression in 0.71.5
jaegertracing · Jun 23, 2023 · b02bfb7 · b02bfb7
2 parents 5f5546e + 4cf24b0
commit b02bfb7
Show file tree

Hide file tree

Showing 3 changed files with 7 additions and 7 deletions.
diff --git a/charts/jaeger/Chart.yaml b/charts/jaeger/Chart.yaml
@@ -3,7 +3,7 @@ appVersion: 1.45.0
 description: A Jaeger Helm chart for Kubernetes
 name: jaeger
 type: application
-version: 0.71.5
+version: 0.71.6
 # CronJobs require v1.21
 kubeVersion: '>= 1.21-0'
 keywords:

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: gtpc-otlp
+          name: otlp-grpc
         http:
-          name: http-otlp
+          name: otlp-http
   ingress:
     enabled: false
     # For Kubernetes >= 1.18 you should specify the ingress-controller via the field ingressClassName
@@ -379,11 +379,11 @@ collector:
       # nodePort:
     otlp:
       grpc:
-        name: grpc-otlp
+        name: otlp-grpc
         # port: 4317
         # nodePort:
       http:
-        name: http-otlp
+        name: otlp-http
         # port: 4318
         # nodePort:
   ingress: