From 6d6860170235ff1366b171efa5a64cb3e1b41bb7 Mon Sep 17 00:00:00 2001 From: Ryan Luckie Date: Sun, 15 Jul 2018 12:49:15 -0500 Subject: [PATCH] [stable/nginx-ingress] Add support for priorityClasses (#6611) - add support for priorityClasses --- stable/nginx-ingress/Chart.yaml | 2 +- stable/nginx-ingress/README.md | 2 ++ stable/nginx-ingress/templates/controller-daemonset.yaml | 3 +++ stable/nginx-ingress/templates/controller-deployment.yaml | 3 +++ .../nginx-ingress/templates/default-backend-deployment.yaml | 3 +++ stable/nginx-ingress/values.yaml | 4 ++++ 6 files changed, 16 insertions(+), 1 deletion(-) diff --git a/stable/nginx-ingress/Chart.yaml b/stable/nginx-ingress/Chart.yaml index a183470057..c11a3585e3 100755 --- a/stable/nginx-ingress/Chart.yaml +++ b/stable/nginx-ingress/Chart.yaml @@ -1,5 +1,5 @@ name: nginx-ingress -version: 0.22.1 +version: 0.23.0 appVersion: 0.15.0 home: https://github.com/kubernetes/ingress-nginx description: An nginx Ingress controller that uses ConfigMap to store the nginx configuration. diff --git a/stable/nginx-ingress/README.md b/stable/nginx-ingress/README.md index 0a28913f91..f2da20618d 100644 --- a/stable/nginx-ingress/README.md +++ b/stable/nginx-ingress/README.md @@ -74,6 +74,7 @@ Parameter | Description | Default `controller.replicaCount` | desired number of controller pods | `1` `controller.minAvailable` | minimum number of available controller pods for PodDisruptionBudget | `1` `controller.resources` | controller pod resource requests & limits | `{}` +`controller.priorityClassName` | controller priorityClassName | `nil` `controller.lifecycle` | controller pod lifecycle hooks | `{}` `controller.service.annotations` | annotations for controller service | `{}` `controller.service.labels` | labels for controller service | `{}` @@ -136,6 +137,7 @@ Parameter | Description | Default `defaultBackend.replicaCount` | desired number of default backend pods | `1` `defaultBackend.minAvailable` | minimum number of available default backend pods for PodDisruptionBudget | `1` `defaultBackend.resources` | default backend pod resource requests & limits | `{}` +`defaultBackend.priorityClassName` | default backend priorityClassName | `nil` `defaultBackend.service.annotations` | annotations for default backend service | `{}` `defaultBackend.service.clusterIP` | internal default backend cluster service IP | `""` `defaultBackend.service.externalIPs` | default backend service external IP addresses | `[]` diff --git a/stable/nginx-ingress/templates/controller-daemonset.yaml b/stable/nginx-ingress/templates/controller-daemonset.yaml index 756c3f5eb6..64f95c38f9 100644 --- a/stable/nginx-ingress/templates/controller-daemonset.yaml +++ b/stable/nginx-ingress/templates/controller-daemonset.yaml @@ -34,6 +34,9 @@ spec: imagePullSecrets: {{ toYaml .Values.imagePullSecrets | indent 8 }} {{- end }} +{{- if .Values.controller.priorityClassName }} + priorityClassName: "{{ .Values.controller.priorityClassName }}" +{{- end }} containers: - name: {{ template "nginx-ingress.name" . }}-{{ .Values.controller.name }} image: "{{ .Values.controller.image.repository }}:{{ .Values.controller.image.tag }}" diff --git a/stable/nginx-ingress/templates/controller-deployment.yaml b/stable/nginx-ingress/templates/controller-deployment.yaml index 12f8001388..5380285614 100644 --- a/stable/nginx-ingress/templates/controller-deployment.yaml +++ b/stable/nginx-ingress/templates/controller-deployment.yaml @@ -34,6 +34,9 @@ spec: imagePullSecrets: {{ toYaml .Values.imagePullSecrets | indent 8 }} {{- end }} +{{- if .Values.controller.priorityClassName }} + priorityClassName: "{{ .Values.controller.priorityClassName }}" +{{- end }} containers: - name: {{ template "nginx-ingress.name" . }}-{{ .Values.controller.name }} image: "{{ .Values.controller.image.repository }}:{{ .Values.controller.image.tag }}" diff --git a/stable/nginx-ingress/templates/default-backend-deployment.yaml b/stable/nginx-ingress/templates/default-backend-deployment.yaml index ed7d70e066..0e7467b87f 100644 --- a/stable/nginx-ingress/templates/default-backend-deployment.yaml +++ b/stable/nginx-ingress/templates/default-backend-deployment.yaml @@ -30,6 +30,9 @@ spec: imagePullSecrets: {{ toYaml .Values.imagePullSecrets | indent 8 }} {{- end }} +{{- if .Values.defaultBackend.priorityClassName }} + priorityClassName: "{{ .Values.defaultBackend.priorityClassName }}" +{{- end }} containers: - name: {{ template "nginx-ingress.name" . }}-{{ .Values.defaultBackend.name }} image: "{{ .Values.defaultBackend.image.repository }}:{{ .Values.defaultBackend.image.tag }}" diff --git a/stable/nginx-ingress/values.yaml b/stable/nginx-ingress/values.yaml index 0e402aec13..12960a5839 100755 --- a/stable/nginx-ingress/values.yaml +++ b/stable/nginx-ingress/values.yaml @@ -274,6 +274,8 @@ controller: lifecycle: {} + priorityClassName: "" + ## Rollback limit ## revisionHistoryLimit: 10 @@ -344,6 +346,8 @@ defaultBackend: servicePort: 80 type: ClusterIP + priorityClassName: "" + ## Enable RBAC as per https://github.com/kubernetes/ingress/tree/master/examples/rbac/nginx and https://github.com/kubernetes/ingress/issues/266 rbac: create: true