Skip to content

Commit 8d5ecd4

Browse files
docs(configuration): note that higher priority alone does not guarantee preemption
Preemption also requires an evictable lower-priority pod on a node that would then fit, and preemptionPolicy: Never raises queue order without evicting. Both built-in classes recommended here are PreemptLowerPriority. Signed-off-by: Lonnie Friedman <lfriedman@together.ai>
1 parent 9e7b262 commit 8d5ecd4

1 file changed

Lines changed: 4 additions & 0 deletions

File tree

docs/configuration/README.md

Lines changed: 4 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -106,6 +106,10 @@ gpu-health-monitor:
106106
The priority classes must already exist in the cluster. `system-node-critical` and
107107
`system-cluster-critical` are built in; anything else has to be created first.
108108

109+
A higher priority is necessary but not sufficient: preemption also needs an evictable
110+
lower-priority pod on a node that would then fit, and a class with `preemptionPolicy: Never`
111+
only improves queue order without evicting anything. Both built-in classes above preempt.
112+
109113
### Image Pull Secrets
110114

111115
Credentials for pulling images from private registries.

0 commit comments

Comments
 (0)