Skip to content

Commit

Permalink
[FIX] two annotations for two ingress (#17)
Browse files Browse the repository at this point in the history
  • Loading branch information
etaques authored Apr 22, 2022
1 parent 6043ef2 commit d6c019d
Show file tree
Hide file tree
Showing 2 changed files with 7 additions and 5 deletions.
8 changes: 4 additions & 4 deletions charts/orb/templates/ingress.yaml
Original file line number Diff line number Diff line change
Expand Up @@ -4,9 +4,9 @@
apiVersion: networking.k8s.io/v1
kind: Ingress
metadata:
{{- if .Values.ingress.annotations }}
{{- if .Values.ingress.annotations-nginx }}
annotations:
{{ toYaml .Values.ingress.annotations | indent 4 }}
{{ toYaml .Values.ingress.annotations-nginx | indent 4 }}
{{- end }}
name: {{ .Release.Name }}-nginx-ingress
spec:
Expand Down Expand Up @@ -37,9 +37,9 @@ spec:
apiVersion: networking.k8s.io/v1
kind: Ingress
metadata:
{{- if .Values.ingress.annotations }}
{{- if .Values.ingress.annotations-nginx-rewrite }}
annotations:
{{ toYaml .Values.ingress.annotations | indent 4 }}
{{ toYaml .Values.ingress.annotations-nginx-rewrite | indent 4 }}
{{- end }}
name: {{ .Release.Name }}-nginx-rewrite-ingress
spec:
Expand Down
4 changes: 3 additions & 1 deletion charts/orb/values.yaml
Original file line number Diff line number Diff line change
Expand Up @@ -26,7 +26,9 @@ smtp:

ingress:
ingressClassName: "nginx"
annotations:
annotations-nginx:
cert-manager.io/issuer: "letsencrypt-prod"
annotations-nginx-rewrite:
cert-manager.io/issuer: "letsencrypt-prod"
nginx.ingress.kubernetes.io/rewrite-target: /$1
hostname: ""
Expand Down

0 comments on commit d6c019d

Please sign in to comment.