Skip to content

Commit 462309e

Browse files
committed
fixup: reword the section about feature's disabling vs. scheduler extension point
1 parent 6805e7a commit 462309e

File tree

1 file changed

+10
-4
lines changed
  • keps/sig-scheduling/3521-pod-scheduling-readiness

1 file changed

+10
-4
lines changed

keps/sig-scheduling/3521-pod-scheduling-readiness/README.md

Lines changed: 10 additions & 4 deletions
Original file line numberDiff line numberDiff line change
@@ -313,10 +313,16 @@ API Server will return a validation error.
313313
| unscheduled Pod<br>(nil <tt>nodeName</tt>) | ✅ create<br>❌ update | ✅ create<br>✅ update |
314314
| scheduled Pod<br>(non-nil <tt>nodeName</tt>) | ❌ create<br>❌ update | ✅ create<br>✅ update |
315315

316-
- **New field disabled in Alpha but not scheduler extension:** In Alpha, the new Pod field is disabled
317-
by default. However, the scheduler's extension point is activated no matter the feature gate is enabled
318-
or not. This enables scheduler plugin developers to tryout the feature even in Alpha, by crafting
319-
different enqueue plugins and wire with custom fields or conditions.
316+
- **New field disabled in Alpha but not scheduler extension:** In a high level, this feature contains
317+
the following parts:
318+
319+
- a. Pod's spec, feature gate, and other misc bits describing the field `schedulingGates`
320+
- b. the `SchedulingGates` scheduler plugin
321+
- c. the underlying scheduler framework, i.e., the new PreEnqueue extension point
322+
323+
If the feature is disabled, part `a` and `b` are disabled, but `c` is enabled anyways. This implies
324+
a scheduler plugin developer can leverage part `c` _only_ to craft their own `a'` and `b'` to
325+
accomplish their own feature set.
320326

321327
- **New phase literal in kubectl:** To provide better UX, we're going to add a new phase literal
322328
`SchedulingPaused` to the "phase" column of `kubectl get pod`. This new literal indicates whether it's

0 commit comments

Comments
 (0)