Skip to content

Commit

Permalink
update nginx ingress controller 1.9.0 (#2005)
Browse files Browse the repository at this point in the history
  • Loading branch information
pgvishnuram committed Oct 4, 2023
1 parent 7784ba0 commit b84ad68
Show file tree
Hide file tree
Showing 4 changed files with 14 additions and 2 deletions.
2 changes: 1 addition & 1 deletion .circleci/config.yml
Original file line number Diff line number Diff line change
Expand Up @@ -366,7 +366,7 @@ workflows:
- quay.io/astronomer/ap-nats-server:2.8.4-4
- quay.io/astronomer/ap-nats-streaming:0.24.6-4
- quay.io/astronomer/ap-nginx-es:1.25.2
- quay.io/astronomer/ap-nginx:1.8.1
- quay.io/astronomer/ap-nginx:1.9.0
- quay.io/astronomer/ap-node-exporter:1.6.1
- quay.io/astronomer/ap-openresty:1.21.4-10
- quay.io/astronomer/ap-pgbouncer-krb:1.17.0-9
Expand Down
1 change: 1 addition & 0 deletions charts/nginx/templates/nginx-configmap.yaml
Original file line number Diff line number Diff line change
Expand Up @@ -11,6 +11,7 @@ metadata:
release: {{ .Release.Name }}
heritage: {{ .Release.Service }}
data:
allow-snippet-annotations: "{{ .Values.allowSnippetAnnotations }}"
add-headers: {{ .Release.Namespace }}/{{ template "nginx.fullname" . }}-ingress-controller-headers
proxy-add-original-uri-header: "true"
proxy-connect-timeout: {{ .Values.proxyConnectTimeout | quote }}
Expand Down
3 changes: 3 additions & 0 deletions charts/nginx/templates/nginx-deployment.yaml
Original file line number Diff line number Diff line change
Expand Up @@ -52,6 +52,9 @@ spec:
{{- if .Values.global.singleNamespace }}
- --watch-namespace={{ .Release.Namespace }}
{{- end }}
{{- if .Values.enableAnnotationValidations }}
- --enable-annotation-validation=true
{{- end }}
- --election-id={{ .Values.electionId }}-{{ template "nginx.fullname" . }}
- --publish-service={{ .Release.Namespace }}/{{ template "nginx.fullname" . }}
- --http-port={{ .Values.ports.http }}
Expand Down
10 changes: 9 additions & 1 deletion charts/nginx/values.yaml
Original file line number Diff line number Diff line change
Expand Up @@ -9,7 +9,7 @@ tolerations: []
images:
nginx:
repository: quay.io/astronomer/ap-nginx
tag: 1.8.1
tag: 1.9.0
pullPolicy: IfNotPresent
defaultBackend:
repository: quay.io/astronomer/ap-default-backend
Expand Down Expand Up @@ -82,6 +82,14 @@ ports:
# Election ID to use for status update
electionId: ingress-controller-leader

# Enables the annotation validation feature
enableAnnotationValidations: false

# -- This configuration defines if Ingress Controller should allow users to set
# their own *-snippet annotations, otherwise this is forbidden / dropped
# when users add those annotations.
# Global snippets in ConfigMap are still respected
allowSnippetAnnotations: false

defaultBackend:
resources: {}

0 comments on commit b84ad68

Please sign in to comment.