added targetPort support

traefik · Apr 19, 2023 · 0f1eef1 · 0f1eef1
1 parent 7a6c6f1
commit 0f1eef1
Show file tree

Hide file tree

Showing 4 changed files with 34 additions and 3 deletions.
diff --git a/traefik/templates/_service-internal.tpl b/traefik/templates/_service-internal.tpl
@@ -38,7 +38,7 @@
   {{- if $config.expose }}
   - port: {{ default $config.port $config.exposedPort }}
     name: {{ $name | quote }}
-    targetPort: {{ $name }}
+    targetPort: {{ default $name $config.targetPort }}
     protocol: {{ default "TCP" $config.protocol }}
     {{- if $config.nodePort }}
     nodePort: {{ $config.nodePort }}

diff --git a/traefik/templates/_service.tpl b/traefik/templates/_service.tpl
@@ -38,7 +38,7 @@
   {{- if $config.expose }}
   - port: {{ default $config.port $config.exposedPort }}
     name: {{ $name | quote }}
-    targetPort: {{ $name }}
+    targetPort: {{ default $name $config.targetPort }}
     protocol: {{ default "TCP" $config.protocol }}
     {{- if $config.nodePort }}
     nodePort: {{ $config.nodePort }}
@@ -49,7 +49,7 @@
   {{- $http3Port := default $config.exposedPort $config.http3.advertisedPort }}
   - port: {{ $http3Port }}
     name: "{{ $name }}-http3"
-    targetPort: {{ $config.port }}
+    targetPort: {{ default $config.port $config.targetPort }}
     protocol: UDP
     {{- if $config.nodePort }}
     nodePort: {{ $config.nodePort }}

diff --git a/traefik/tests/service-config_test.yaml b/traefik/tests/service-config_test.yaml
@@ -280,3 +280,30 @@ tests:
       - equal:
           path: metadata.namespace
           value: "traefik-ns-override"
+  - it: should have different target port when targetPort is different from port
+    set:
+      ports:
+        web:
+          port: 8080
+          exposedPort: 80
+          expose: true
+          targetPort: 80
+        websecure:
+          port: 443
+          exposedPort: 443
+          expose: true
+          targetPort: 80
+    documentIndex: 0
+    asserts:
+      - equal:
+          path: spec.ports[0].port
+          value: 80
+      - equal:
+          path: spec.ports[0].targetPort
+          value: 80
+      - equal:
+          path: spec.ports[1].port
+          value: 443
+      - equal:
+          path: spec.ports[1].targetPort
+          value: 80
diff --git a/traefik/values.yaml b/traefik/values.yaml
@@ -574,6 +574,8 @@ ports:
     # containerPort: 8000
     expose: true
     exposedPort: 80
+    ## Different target traefik port on the cluster, useful for IP type LB
+    # targetPort: 80
     # The port protocol (TCP/UDP)
     protocol: TCP
     # Use nodeport if set. This is useful if you have configured Traefik in a
@@ -601,6 +603,8 @@ ports:
     # containerPort: 8443
     expose: true
     exposedPort: 443
+    ## Different target traefik port on the cluster, useful for IP type LB
+    # targetPort: 80
     ## The port protocol (TCP/UDP)
     protocol: TCP
     # nodePort: 32443