From a113a0a27490956100cd153ea89c1d5e012a858d Mon Sep 17 00:00:00 2001
From: Jon Bates <jonathan.bates@team.bumble.com>
Date: Fri, 23 Jun 2023 15:27:09 +0100
Subject: [PATCH] Make otlp ports configurable on deployment

Signed-off-by: Jon Bates <jonathan.bates@team.bumble.com>
---
 charts/jaeger/templates/collector-deploy.yaml | 4 ++--
 charts/jaeger/values.yaml                     | 2 +-
 2 files changed, 3 insertions(+), 3 deletions(-)

diff --git a/charts/jaeger/templates/collector-deploy.yaml b/charts/jaeger/templates/collector-deploy.yaml
index 211dc3f9..4cd0bac0 100644
--- 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
index cd3494c3..7f6a9f6b 100644
--- a/charts/jaeger/values.yaml
+++ b/charts/jaeger/values.yaml
@@ -383,7 +383,7 @@ collector:
         # port: 4317
         # nodePort:
       http:
-        name: otlp-grpc
+        name: otlp-http
         # port: 4318
         # nodePort:
   ingress: