Skip to content

Commit

Permalink
Merge pull request #451 from mapk-amazon/exposing_priorityClass_value…
Browse files Browse the repository at this point in the history
…s_for_jobs

Exposes job.PriorityClass.value as modifiable value
  • Loading branch information
nuwang authored Feb 16, 2024
2 parents 9c6eb6e + cdd1f39 commit adfda49
Show file tree
Hide file tree
Showing 2 changed files with 3 additions and 2 deletions.
4 changes: 2 additions & 2 deletions galaxy/templates/priorityclass-job.yaml
Original file line number Diff line number Diff line change
Expand Up @@ -5,8 +5,8 @@ metadata:
name: {{ include "galaxy.pod-priority-class" . }}
labels:
{{- include "galaxy.labels" . | nindent 4 }}
value: -1000
value: {{ default "-1000" .Values.jobs.priorityClass.value }}
preemptionPolicy: Never
globalDefault: false
description: "Galaxy jobs will run when cluster resources are available and will not preempt existing workloads."
description: "By default, Galaxy jobs will not preempt existing workloads but will run when cluster resources are available. Depending on the cluster setup, the preemption value can be modified to prompt scaling of the cluster for the computation."
{{ end }}
1 change: 1 addition & 0 deletions galaxy/values.yaml
Original file line number Diff line number Diff line change
Expand Up @@ -542,6 +542,7 @@ jobs:
#- Assign a [priorityClass](https://kubernetes.io/docs/concepts/scheduling-eviction/pod-priority-preemption/#priorityclass) to the dispatched jobs.
enabled: true
existingClass: ""
value: -1000
rules:
tpv_rules_local.yml:
global:
Expand Down

0 comments on commit adfda49

Please sign in to comment.