NVIDIA / k8s-operator-libs / 27365593432

63%

main: 63%

Build Type

push

github

Committed by rajathagasthya

Commit Message

Add optional restart-only predicate to inplace upgrade flow

Consumers occasionally need a DaemonSet's pods rolled to a new revision
without the disruptive pod-eviction and drain sequence -- for example
when a pod-template change does not affect the managed software (a
label-only change) yet still changes the controller revision hash, so
the node is otherwise driven through the full upgrade flow.

Add an optional RestartOnlyPredicate hook, registered via
WithRestartOnlyPredicate. In the inplace ProcessUpgradeRequiredNodes,
when the predicate reports that a full upgrade is not needed, cordon the
node and route it straight to pod-restart-required, skipping
wait-for-jobs, pod-deletion, and drain. The pod is still restarted so
the DaemonSet converges to the new revision. Cordoning keeps the node
unschedulable if the restart fails, matching the full flow; the
uncordon-required state uncordons it on success.

The change is additive and backward compatible: a nil predicate (the
default, and every existing consumer) preserves current behavior, and
podInSyncWithDS is unchanged. If the predicate returns an error or the
cordon fails, the node is kept in upgrade-required and retried on a
later reconcile, with a Warning event recorded -- an upgrade is never
started on an unknown answer. The predicate is not consulted for
orphaned pods or nodes with an explicit upgrade-requested annotation,
and the maxParallelUpgrades throttle applies to both paths. The
upgrade-requested state is captured before the annotation is cleared,
because the node provider re-fetches and overwrites the in-memory node
object.

Signed-off-by: Rajath Agasthya <ragasthya@nvidia.com>

Coverage Stats

38 of 39 new or added lines in 2 files covered. (97.44%)

195 existing lines in 3 files now uncovered.

1462 of 2333 relevant lines covered (62.67%)

0.7 hits per line