fix: 🐛 improve error message on additional service without ports

traefik · Jun 27, 2024 · d4cab24 · d4cab24
1 parent 0449b0b
commit d4cab24
Show file tree

Hide file tree

Showing 3 changed files with 4 additions and 19 deletions.
diff --git a/traefik/templates/service.yaml b/traefik/templates/service.yaml
@@ -11,13 +11,11 @@
 {{- $exposedPorts := false -}}
 {{- range $portName, $config := $.Values.ports -}}
   {{- if $config -}}
-    {{- if $config.http3 -}}
-    {{- if $config.http3.enabled -}}
+    {{- if ($config.http3).enabled -}}
       {{- if (not $config.tls.enabled) -}}
         {{- fail "ERROR: You cannot enable http3 without enabling tls" -}}
       {{- end -}}
     {{- end -}}
-    {{- end -}}
     {{- if eq (toString $config.protocol) "UDP" -}}
       {{ $_ := set $udpPorts $portName $config -}}
     {{- end -}}
@@ -31,7 +29,7 @@
 {{- end -}}
 
 {{- if (eq $exposedPorts false) -}}
-  {{- fail "You need to expose at least one port or set enabled=false to service" -}}
+  {{- fail (printf "ERROR: Cannot create Service %s without ports" $fullname) -}}
 {{- end -}}
 
 {{- if and $exposedPorts (or $tcpPorts $service.single) }}

diff --git a/traefik/tests/service-config-custom_test.yaml b/traefik/tests/service-config-custom_test.yaml
@@ -46,7 +46,7 @@ tests:
             internal: false
     asserts:
       - failedTemplate:
-          errorMessage: "You need to expose at least one port or set enabled=false to service"
+          errorMessage: "ERROR: Cannot create Service RELEASE-NAME-traefik-internal without ports"
   - it: should have customized annotations when specified via values
     set:
       service:

diff --git a/traefik/tests/service-config_test.yaml b/traefik/tests/service-config_test.yaml
@@ -24,7 +24,7 @@ tests:
             default: false
     asserts:
       - failedTemplate:
-          errorMessage: "You need to expose at least one port or set enabled=false to service"
+          errorMessage: "ERROR: Cannot create Service RELEASE-NAME-traefik without ports"
   - it: should be a type LoadBalancer by default
     asserts:
       - equal:
@@ -273,19 +273,6 @@ tests:
     - equal:
         path: spec.ports[2].protocol
         value: UDP
-  - it: should fail when there is no exposed port
-    set:
-      ports:
-        web:
-          expose:
-            default: false
-        websecure:
-          expose:
-            default: false
-    asserts:
-    - failedTemplate:
-        errorMessage: "You need to expose at least one port or set enabled=false to service"
-        documentIndex: 1
   - it: should not be possible to use http3 without enabling tls
     set:
       ports: