NVIDIA / k8s-operator-libs
63%
main: 62%

LAST BUILD BRANCH: pull-request/145
DEFAULT BRANCH: main
Repo Added
Token FQB9I1vDYh7d5xWkpcB0REXSEYE1rsUtp regen
Build 306 Last
Files 20
Badge
Embed ▾
README BADGES
x

If you need to use a raster PNG badge, change the '.svg' to '.png' in the link

Markdown

Textile

RDoc

HTML

Rst

LAST BUILD ON BRANCH pull-request/145
branch: pull-request/145
CHANGE BRANCH
x
Sync Branches

coverage: 62.553% (+0.4%) from 62.114%
27644467942

push

github

rajathagasthya 
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, nodes with an explicit upgrade-requested annotation, or
nodes waiting for safe driver load -- the safe-load handshake relies on
the full flow to evict workloads before the driver load is unblocked at
pod-restart-required. 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>

41 of 48 new or added lines in 2 files covered. (85.42%)

1465 of 2342 relevant lines covered (62.55%)

0.7 hits per line

Relevant lines Covered
2342 RELEVANT LINES 1465 COVERED LINES
0.7 HITS PER LINE
Source Files on pull-request/145

Recent builds

Builds Branch Commit Type Ran Committer Via Coverage
27644467942 pull-request/145 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 t... push rajathagasthya github
62.55
27376940706 pull-request/145 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 t... push rajathagasthya github
62.59
27365593432 pull-request/145 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 t... push rajathagasthya github
62.67
27364405118 pull-request/145 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 t... push rajathagasthya github
62.67
See All Builds (268)

Badge your Repo: k8s-operator-libs

We detected this repo isn’t badged! Grab the embed code to the right, add it to your repo to show off your code coverage, and when the badge is live hit the refresh button to remove this message.
Embed ▾
README BADGES
x

If you need to use a raster PNG badge, change the '.svg' to '.png' in the link

Markdown

Textile

RDoc

HTML

Rst

Refresh