From 53911e956c15255e7a96cc00f36a42fa4cc2d3ad Mon Sep 17 00:00:00 2001
From: Jon Mosco <jmosco@redhat.com>
Date: Thu, 15 Aug 2024 08:50:44 -0400
Subject: [PATCH] add note to include how to use the route

---
 charts/localstack/templates/NOTES.txt  | 7 ++++++-
 charts/localstack/templates/role.yaml  | 2 +-
 charts/localstack/templates/route.yaml | 1 +
 charts/localstack/values.yaml          | 5 +++--
 4 files changed, 11 insertions(+), 4 deletions(-)

diff --git a/charts/localstack/templates/NOTES.txt b/charts/localstack/templates/NOTES.txt
index 3132aae..14ad76f 100644
--- 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
index 29a549c..f3a18ba 100644
--- 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
index 9f308e3..2d4be6d 100644
--- 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
index 86e4b4c..a41756a 100644
--- 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