Skip to content

Commit 1259202

Browse files
committed
Clarify more around the downgrade and feature disablement scenarios
1 parent a6a044e commit 1259202

File tree

1 file changed

+6
-3
lines changed
  • keps/sig-scheduling/5194-reserved-for-workloads

1 file changed

+6
-3
lines changed

keps/sig-scheduling/5194-reserved-for-workloads/README.md

Lines changed: 6 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -485,9 +485,12 @@ this:
485485
leading to a situation where the claim will never be deallocated.
486486
- For new pods that gets scheduled, the scheduler will add pod references in the
487487
`ReservedFor` list, despite there being a non-pod reference here. So it ends up with
488-
both pod and non-pod references in the list. We need to make sure the system can
489-
handle this, as it might also happen as a result of disablement and the enablement
490-
of the feature.
488+
both pod and non-pod references in the list. We can manage both pod and non-pod
489+
references in the list by letting the workload controllers add the non-pod reference
490+
even if it sees pod references and making sure that the resourceclaim controller removes
491+
pod references even if there are non-pod references in the list. For deallocation, it is
492+
only safe when no pods are consuming the claim, so both workload and pod reference should
493+
be removed once that is true.
491494

492495
### Version Skew Strategy
493496

0 commit comments

Comments
 (0)