traefik / mesh

Traefik Mesh - Simpler Service Mesh
https://traefik.io/traefik-mesh
Apache License 2.0
2.03k stars 141 forks source link

CoreDNS config used by the Helm chart and the prepare command is not the same #549

Closed kevinpollet closed 4 years ago

kevinpollet commented 4 years ago

Bug Report

The CoreDNS configuration in the Helm chart is not in sync with the one used by the prepare command. I think the Helm chart configuration is out of date.

rewrite continue {
            name regex ([a-zA-Z0-9-_]*)\.([a-zv0-9-_]*)\.maesh maesh-{1}-6d61657368-{2}.maesh.svc.{{ default "cluster.local" .Values.clusterDomain }}
            answer name maesh-([a-zA-Z0-9-_]*)-6d61657368-([a-zA-Z0-9-_]*)\.maesh\.svc\.{{ default "cluster.local" .Values.clusterDomain | replace "." "\\." }} {1}.{2}.maesh
        }
rewrite continue {
        name regex ([a-zA-Z0-9-_]*)\.([a-zv0-9-_]*)\.maesh %[3]s-{1}-6d61657368-{2}.%[3]s.svc.%[1]s
        answer name %[3]s-([a-zA-Z0-9-_]*)-6d61657368-([a-zA-Z0-9-_]*)\.%[3]s\.svc\.%[2]s {1}.{2}.maesh
    }
dtomcej commented 4 years ago

This is from when Maesh needed to be installed in its own statically named namespace to properly resolve.

This could definitely be updated to ensure they are the same regex.