Skip to content

Commit

Permalink
feat(deployment): allow specifying a priorityClassName (#2654)
Browse files Browse the repository at this point in the history
* allow specifying a priorityClassName

* maybe fix

* another fix

* update

* Update _possiblePriorityClassName.tpl

* Update values.yaml

* update to pod priority class name
  • Loading branch information
theosanderson committed Aug 30, 2024
1 parent 2a99315 commit 3fbbac2
Show file tree
Hide file tree
Showing 8 changed files with 12 additions and 0 deletions.
5 changes: 5 additions & 0 deletions kubernetes/loculus/templates/_possiblePriorityClassName.tpl
Original file line number Diff line number Diff line change
@@ -0,0 +1,5 @@
{{- define "possiblePriorityClassName" -}}
{{- if .Values.podPriorityClassName }}
priorityClassName: {{ .Values.podPriorityClassName }}
{{- end -}}
{{- end -}}
Original file line number Diff line number Diff line change
Expand Up @@ -20,6 +20,7 @@ spec:
app: loculus
component: loculus-ena-submission
spec:
{{- include "possiblePriorityClassName" . | nindent 6 }}
initContainers:
- name: ena-submission-flyway
image: "ghcr.io/loculus-project/ena-submission-flyway:{{ $dockerTag }}"
Expand Down
1 change: 1 addition & 0 deletions kubernetes/loculus/templates/ingest-deployment.yaml
Original file line number Diff line number Diff line change
Expand Up @@ -23,6 +23,7 @@ spec:
annotations:
argocd.argoproj.io/sync-options: Force=true,Replace=true
spec:
{{- include "possiblePriorityClassName" $ | nindent 6 }}
containers:
- name: ingest-{{ $key }}
image: {{ $value.ingest.image}}:{{ $dockerTag }}
Expand Down
1 change: 1 addition & 0 deletions kubernetes/loculus/templates/keycloak-deployment.yaml
Original file line number Diff line number Diff line change
Expand Up @@ -29,6 +29,7 @@ spec:
app: loculus
component: keycloak
spec:
{{- include "possiblePriorityClassName" . | nindent 6 }}
initContainers:
{{- include "loculus.configProcessor" (dict "name" "keycloak-config" "dockerTag" $dockerTag) | nindent 8 }}
- name: keycloak-theme-prep
Expand Down
1 change: 1 addition & 0 deletions kubernetes/loculus/templates/lapis-silo-deployment.yaml
Original file line number Diff line number Diff line change
Expand Up @@ -22,6 +22,7 @@ spec:
app: loculus
component: lapis-silo-{{ $key }}
spec:
{{- include "possiblePriorityClassName" $ | nindent 6 }}
initContainers:
{{- include "loculus.configProcessor" (dict "name" "lapis-silo-database-config" "dockerTag" $dockerTag) | nindent 8 }}
containers:
Expand Down
1 change: 1 addition & 0 deletions kubernetes/loculus/templates/loculus-backend.yaml
Original file line number Diff line number Diff line change
Expand Up @@ -20,6 +20,7 @@ spec:
app: loculus
component: backend
spec:
{{- include "possiblePriorityClassName" . | nindent 6 }}
initContainers:
{{- include "loculus.configProcessor" (dict "name" "loculus-backend-config" "dockerTag" $dockerTag) | nindent 8 }}
containers:
Expand Down
Original file line number Diff line number Diff line change
Expand Up @@ -30,6 +30,7 @@ spec:
app: loculus
component: loculus-preprocessing-{{ $organism }}-v{{ $processingConfig.version }}-{{ $processingIndex }}
spec:
{{- include "possiblePriorityClassName" $ | nindent 6 }}
containers:
- name: preprocessing-{{ $organism }}
image: {{ $processingConfig.image}}:{{ $dockerTag }}
Expand Down
1 change: 1 addition & 0 deletions kubernetes/loculus/templates/loculus-website.yaml
Original file line number Diff line number Diff line change
Expand Up @@ -20,6 +20,7 @@ spec:
app: loculus
component: website
spec:
{{- include "possiblePriorityClassName" . | nindent 6 }}
initContainers:
{{- include "loculus.configProcessor" (dict "name" "loculus-website-config" "dockerTag" $dockerTag) | nindent 8 }}
containers:
Expand Down

0 comments on commit 3fbbac2

Please sign in to comment.