NVIDIA / skyhook / 21524977372

81%

push

github

feat(operator): implement per-node priority ordering

Implements per-node priority ordering where each node progresses through
skyhooks independently, preventing slow/blocked nodes from blocking
healthy nodes. Includes comprehensive e2e test documentation, bug fixes,
and validation.

Core Changes:
- Per-node priority ordering: nodes progress independently through priorities
- Nodes no longer wait for all other nodes to complete a priority level
- Added IsNodeReadyForSkyhook() to check per-node priority readiness
- Fixed CollectNodeStatus() to show in_progress if ANY node is progressing
- Updated waiting/blocked logic to be per-node, not global
- Changed runtime-required to be per node, not by skyhook.

Tests:
- Added critical assertions proving concurrent execution at different priorities
- Updated runtime-required test for per-node taint removal
- Added 7 unit tests for CollectNodeStatus per-node behavior

Documentation:
- Added comprehensive README.md for all chainsaw e2e tests
- Updated ordering_of_skyhooks.md with per-node behavior details
- Updated runtime_required.md for per-node taint removal
- Documented test scenarios, key features, and file descriptions

Infrastructure:
- Regenerated mocks for SkyhookNode with new methods
- Fixed license formatter to skip deleted files

This enables per-node parallelism while maintaining strict ordering
guarantees per node, preventing cluster-wide deadlocks from bad nodes.

99 of 99 new or added lines in 2 files covered. (100.0%)

6605 of 8138 relevant lines covered (81.16%)

4.34 hits per line