Skip to content

Commit 4dac294

Browse files
authored
Merge pull request #1004 from github/copilot/allow-priorityclassname-in-helm-chart
feat(helm): add priorityClassName support
2 parents 254bad3 + f69fffa commit 4dac294

3 files changed

Lines changed: 6 additions & 0 deletions

File tree

helm/safe-settings/README.md

Lines changed: 1 addition & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -37,6 +37,7 @@ A Helm chart for Kubernetes
3737
| nodeSelector | object | `{}` | |
3838
| podAnnotations | object | `{}` | |
3939
| podSecurityContext | object | `{}` | |
40+
| priorityClassName | string | `""` | Priority class name for the pod. |
4041
| replicaCount | int | `1` | |
4142
| resources | object | `{}` | |
4243
| securityContext.allowPrivilegeEscalation | bool | `false` | |

helm/safe-settings/templates/deployment.yaml

Lines changed: 3 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -77,3 +77,6 @@ spec:
7777
tolerations:
7878
{{- toYaml . | nindent 8 }}
7979
{{- end }}
80+
{{- if .Values.priorityClassName }}
81+
priorityClassName: {{ .Values.priorityClassName }}
82+
{{- end }}

helm/safe-settings/values.yaml

Lines changed: 2 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -118,6 +118,8 @@ tolerations: []
118118

119119
affinity: {}
120120

121+
priorityClassName: ""
122+
121123
deploymentConfig:
122124
restrictedRepos:
123125
# You can exclude certain repos from safe-settings processing

0 commit comments

Comments
 (0)