Skip to content

Commit 4d55c62

Browse files
Update goals and alternatives
Signed-off-by: Aldo Culquicondor <[email protected]>
1 parent 241e9e2 commit 4d55c62

File tree

1 file changed

+12
-2
lines changed

1 file changed

+12
-2
lines changed

keps/sig-scheduling/20190926-default-even-pod-spreading.md

Lines changed: 12 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -85,10 +85,13 @@ them suitable to provide default spreading constraints for all workloads in thei
8585
`pod.spec.topologySpreadConstraints`.
8686
- Workloads are spread with the default constraints if they belong to the same service, replication controller,
8787
replica set or stateful set, and if they don't define `pod.spec.topologySpreadConstraints`.
88+
- Provide a k8s default for `topologySpreadConstraints` that produces a priority equivalent to
89+
`SelectorSpreadPriority`, so that this algorithm can be removed from the default algorithms' provider.
8890

8991
### Non-Goals
9092

91-
- Removal of `SelectorSpreadPriority`, `ServiceSpreadingPriority` or `ServiceAntiAffinity` priorities.
93+
- Set defaults for specific namespaces or according to other selectors.
94+
- Removal of `ServiceSpreadingPriority` or `ServiceAntiAffinity` priorities.
9295

9396
## Proposal
9497

@@ -281,4 +284,11 @@ Alpha (v1.17):
281284
While this moves the scheduler in the right direction, there are two problems:
282285

283286
1. We can only support one topology key.
284-
1. It makes it hard for pods to override the operator-provided spreading rules.
287+
1. It makes it hard for pods to override the operator-provided spreading rules.
288+
289+
- Implement a mutating controller that sets defaults.
290+
291+
This approach would likely allow us to provide a more flexible interface that
292+
can set defaults for specific namespaces or with other selectors. However, that
293+
wouldn't allow us to replace `SelectorSpreadingPriority` with
294+
`EvenPodsSpreading`.

0 commit comments

Comments
 (0)