fix: keycloak controller can see multiple namespaces and therefore ke…

…ycloak name must be unique across namespaces as well
CloudNativeEntrepreneur · May 5, 2020 · 82fdbe5 · 82fdbe5
1 parent d926243
commit 82fdbe5
Show file tree

Hide file tree

Showing 2 changed files with 2 additions and 2 deletions.
diff --git a/charts/demo-app/templates/keycloak-client.yaml b/charts/demo-app/templates/keycloak-client.yaml
@@ -3,7 +3,7 @@ kind: KeycloakClient
 metadata:
   name: {{ template "keycloakclient" . }}
 spec:
-  keycloak: {{ .Values.keycloak.keycloak }}-{{ .Release.Namespace }}
+  keycloak: {{ .Values.keycloak.keycloak }}
   realm: {{ .Values.keycloak.realm }}
   clientId: {{ template "keycloakclient" . }}
   clientType: confidential

diff --git a/charts/demo-app/values.yaml b/charts/demo-app/values.yaml
@@ -47,7 +47,7 @@ canary:
 keycloak:
   proxyPort: 3000
   url: https://auth-jx-staging.cloudnativeentrepreneur.dev # no trailing slash
-  keycloak: keycloak
+  keycloak: keycloak-jx-staging
   realm: members
   redirectUris:
     allowAll: true