NVIDIA / nodewright / 27426773396

82%

Build Type

push

github

Committed by web-flow

Commit Message

feat(controller): re-apply runtime-required taint on reboot when autoTaintNewNodes=true (#273)

When REAPPLY_ON_REBOOT=true, runtimeRequired=true, and autoTaintNewNodes=true
are all set, a rebooted node must be re-tainted so workloads cannot schedule on
it until Skyhook finishes re-applying. Previously, node.Reset() cleared the
nodeState/cordon/status annotations but left the autoTaint annotation intact,
so HasSkyhookAnnotations() remained true and HandleAutoTaint never re-tainted
the node.

The fix appends the runtime-required taint to the node inside the existing
ReapplyOnReboot block, after node.Reset() and before the StrategicMerge patch,
so the taint re-application and state reset land in one atomic apiserver write.
Taint removal on completion is unchanged (HandleRuntimeRequiredTaint).

Closes #180

Signed-off-by: Brian Lockwood <lockwobr@gmail.com>

Coverage Stats

11 of 11 new or added lines in 1 file covered. (100.0%)

6 existing lines in 1 file now uncovered.

8096 of 9914 relevant lines covered (81.66%)

9.31 hits per line