add note to include how to use the route

localstack · Aug 15, 2024 · 53911e9 · 53911e9
1 parent 19261f8
commit 53911e9
Show file tree

Hide file tree

Showing 4 changed files with 11 additions and 4 deletions.
diff --git a/charts/localstack/templates/NOTES.txt b/charts/localstack/templates/NOTES.txt
@@ -17,6 +17,11 @@
 {{- else if contains "ClusterIP" .Values.service.type }}
   export POD_NAME=$(kubectl get pods --namespace {{ .Release.Namespace | quote }} -l "app.kubernetes.io/name={{ include "localstack.name" . }},app.kubernetes.io/instance={{ .Release.Name }}" -o jsonpath="{.items[0].metadata.name}")
   export CONTAINER_PORT=$(kubectl get pod --namespace {{ .Release.Namespace | quote }} $POD_NAME -o jsonpath="{.spec.containers[0].ports[0].containerPort}")
-  echo "Visit http://127.0.0.1:8080 to use your application"
+  echo "visit http://127.0.0.1:8080 to use your application"
   kubectl --namespace {{ .Release.Namespace | quote }} port-forward $POD_NAME 8080:$CONTAINER_PORT
 {{- end }}
+{{- if and .Values.openshift .Values.route.enabled }}
+  export ROUTE_URL=$(oc get route localstack-fork --namespace "localstack" -o jsonpath="{.spec.host}")
+  echo http://$ROUTE_URL
+  echo "visit http://$ROUTE_URL to use your application"
+{{- end }}
diff --git a/charts/localstack/templates/role.yaml b/charts/localstack/templates/role.yaml
@@ -19,7 +19,7 @@ rules:
 - apiGroups: [""]
   resources: ["services"]
   verbs: ["get", "list"]
-{{- if .Values.openshift -}}
+{{- if .Values.openshift }}
 - apiGroups: ["security.openshift.io"]
   resources: ["securitycontextconstraints"]
   resourceNames: ["anyuid"]

diff --git a/charts/localstack/templates/route.yaml b/charts/localstack/templates/route.yaml
@@ -14,6 +14,7 @@ spec:
     weight: 100
   port:
     targetPort: {{ .Values.route.port }}
+  wildcardPolicy: None
   tls:
     {{- toYaml .Values.route.tls | nindent  4 }}
 {{- end }}

diff --git a/charts/localstack/values.yaml b/charts/localstack/values.yaml
@@ -159,10 +159,11 @@ ingress:
   #    hosts:
   #      - chart-example.local
 
-## Create a route if using OpenShift
+## Create a Route resource if using OpenShift
+## Defaults to the 'edge' port of the service on 4566
 route:
   enabled: false
-  port: 4566
+  port: edge
 
 persistence:
   ## @param persistence.enabled Enable persistence using Persistent Volume Claims