Skip to content

Commit fa9de7d

Browse files
authored
Merge pull request #652 from imrenagi/feature/pod-priority
Add priorityClassName to flagger and loadtester chart
2 parents 744b832 + 23e8059 commit fa9de7d

File tree

6 files changed

+13
-0
lines changed

6 files changed

+13
-0
lines changed

charts/flagger/README.md

+1
Original file line numberDiff line numberDiff line change
@@ -140,6 +140,7 @@ Parameter | Description | Default
140140
`istio.kubeconfig.key` | The name of Kubernetes secret data key that contains the Istio control plane kubeconfig | `kubeconfig`
141141
`ingressAnnotationsPrefix` | Annotations prefix for NGINX ingresses | None
142142
`ingressClass` | Ingress class used for annotating HTTPProxy objects, e.g. `contour` | None
143+
`podPriorityClassName` | PriorityClass name for pod priority configuration | ""
143144

144145
Specify each parameter using the `--set key=value[,key=value]` argument to `helm upgrade`. For example,
145146

charts/flagger/templates/deployment.yaml

+3
Original file line numberDiff line numberDiff line change
@@ -48,6 +48,9 @@ spec:
4848
secret:
4949
secretName: "{{ .Values.istio.kubeconfig.secretName }}"
5050
{{- end }}
51+
{{- if .Values.podPriorityClassName }}
52+
priorityClassName: {{ .Values.podPriorityClassName }}
53+
{{- end }}
5154
containers:
5255
- name: flagger
5356
{{- if .Values.securityContext.enabled }}

charts/flagger/values.yaml

+3
Original file line numberDiff line numberDiff line change
@@ -14,6 +14,9 @@ podAnnotations:
1414
prometheus.io/port: "8080"
1515
appmesh.k8s.aws/sidecarInjectorWebhook: disabled
1616

17+
# priority class name for pod priority configuration
18+
podPriorityClassName: ""
19+
1720
metricsServer: "http://prometheus:9090"
1821

1922
# accepted values are kubernetes, istio, linkerd, appmesh, nginx, gloo or supergloo:mesh.namespace (defaults to istio)

charts/loadtester/README.md

+1
Original file line numberDiff line numberDiff line change
@@ -67,6 +67,7 @@ Parameter | Description | Default
6767
`istio.gateway.enabled` | Create Istio gateway in namespace | `false`
6868
`istio.tls.enabled` | Enable TLS in gateway ( TLS secrets should be in namespace ) | `false`
6969
`istio.tls.httpsRedirect` | Redirect traffic to TLS port | `false`
70+
`podPriorityClassName` | PriorityClass name for pod priority configuration | ""
7071

7172
Specify each parameter using the `--set key=value[,key=value]` argument to `helm upgrade`. For example,
7273

charts/loadtester/templates/deployment.yaml

+3
Original file line numberDiff line numberDiff line change
@@ -27,6 +27,9 @@ spec:
2727
{{- else if .Values.rbac.create }}
2828
serviceAccountName: {{ include "loadtester.fullname" . }}
2929
{{- end }}
30+
{{- if .Values.podPriorityClassName }}
31+
priorityClassName: {{ .Values.podPriorityClassName }}
32+
{{- end }}
3033
containers:
3134
- name: {{ .Chart.Name }}
3235
image: "{{ .Values.image.repository }}:{{ .Values.image.tag }}"

charts/loadtester/values.yaml

+2
Original file line numberDiff line numberDiff line change
@@ -9,6 +9,8 @@ podAnnotations:
99
prometheus.io/scrape: "true"
1010
prometheus.io/port: "8080"
1111

12+
podPriorityClassName: ""
13+
1214
logLevel: info
1315
cmd:
1416
timeout: 1h

0 commit comments

Comments
 (0)