Fix dnsNames in ace cert for acme (#1222)

Signed-off-by: Arnob kumar saha <arnob@appscode.com>

Author: ArnobKumarSaha

charts/ace/templates/gateway/route-nats.yaml

@@ -54,7 +54,11 @@ spec:
     - group: ""
       kind: Secret
       name: {{ include "ace.fullname" . }}-cert
+  {{ if eq (index .Values "global" "platform" "hostType") "ip" }}
     hostname: {{ include "ace.fullname" . }}-nats
+  {{- else }}
+    hostname: {{ .Values.global.platform.host }}
+  {{- end }}
 
 ---
 apiVersion: gateway.networking.k8s.io/v1alpha2

charts/ace/templates/ingress/certificate.yaml

@@ -17,12 +17,14 @@ spec:
     kind: Issuer
     name: {{ include "ace.fullname" . }}
   dnsNames:
+  {{ if eq (index .Values "global" "platform" "hostType") "ip" }}
     - {{ include "ace.fullname" . }}-nats
     - {{ include "ace.fullname" . }}-nats.{{ .Release.Namespace }}.svc
     - {{ include "ace.fullname" . }}-nats.{{ .Release.Namespace }}.svc.cluster.local
     - {{ include "ace.fullname" . }}-s3proxy
     - {{ include "ace.fullname" . }}-s3proxy.{{ .Release.Namespace }}.svc
     - {{ include "ace.fullname" . }}-s3proxy.{{ .Release.Namespace }}.svc.cluster.local
+  {{- end }}
   {{- if eq .Values.global.platform.hostType "domain" }}
     - "{{ .Values.global.platform.host }}"
     {{- if and .Values.nats.enabled .Values.nats.nats.externalAccess }}

charts/ace/templates/ingress/issuer.yaml

@@ -75,9 +75,9 @@ spec:
           parentRefs:
           - group: gateway.networking.k8s.io
             kind: Gateway
-            name: {{ include "ace.fullname" . }}
-            namespace: {{ .Release.Namespace }}
-            sectionName: api
+            name: {{ include "ace.fullname" . }}-gw-seed-backend
+            namespace: {{ .Release.Namespace }}-gw
+            sectionName: http
         {{- else }}
         ingress:
           ingressClassName: {{ index .Values "ingress-nginx" "controller" "ingressClassResource" "name" }}

charts/s3proxy/templates/httproute.yaml

@@ -59,7 +59,7 @@ spec:
     - group: ""
       kind: Secret
       name: {{ include "ace.fullname" . }}-cert
-    hostname: {{ include "ace.fullname" . }}-s3proxy
+    hostname: {{ index .Values.gateway.hosts 0 }}
 {{- end }}
 
 ---