fix: 🐛 ingressroute default name

traefik · Jul 24, 2024 · a494617 · a494617
1 parent 50c24e5
commit a494617
Show file tree

Hide file tree

Showing 3 changed files with 8 additions and 2 deletions.
diff --git a/traefik/templates/ingressroute.yaml b/traefik/templates/ingressroute.yaml
@@ -4,7 +4,7 @@
 apiVersion: traefik.io/v1alpha1
 kind: IngressRoute
 metadata:
-  name: {{ template "traefik.fullname" $ }}-$name
+  name: {{ $.Release.Name }}-{{ $name }}
   namespace: {{ template "traefik.namespace" $ }}
   annotations:
     {{- if and $.Values.ingressClass.enabled $.Values.providers.kubernetesCRD.enabled $.Values.providers.kubernetesCRD.ingressClass }}

diff --git a/traefik/tests/dashboard-ingressroute_test.yaml b/traefik/tests/dashboard-ingressroute_test.yaml
@@ -14,11 +14,14 @@ tests:
   asserts:
   - hasDocuments:
       count: 0
-- it: should have the expected default route match rule
+- it: should have the expected default
   asserts:
   - equal:
       path: spec.routes[0].match
       value: PathPrefix(`/dashboard`) || PathPrefix(`/api`)
+  - equal:
+      path: metadata.name
+      value: RELEASE-NAME-dashboard
 - it: should support overwriting the route match rule
   set:
     ingressRoute:

diff --git a/traefik/tests/healthcheck-ingressroute_test.yaml b/traefik/tests/healthcheck-ingressroute_test.yaml
@@ -10,6 +10,9 @@ tests:
   asserts:
   - hasDocuments:
       count: 1
+  - equal:
+      path: metadata.name
+      value: RELEASE-NAME-healthcheck
 - it: should have the expected default route match rule
   set:
     ingressRoute: