test: add deployment scale validation when nodehardening=true - #9372
test: add deployment scale validation when nodehardening=true#9372Thibault Cohen (titilambert) wants to merge 9 commits into
Conversation
|
This change is part of the following stack: Change managed by git-spice. |
Windows Unit Test Results 3 files 13 suites 53s ⏱️ Results for commit 4064f6d. ♻️ This comment has been updated with latest results. |
3f390b7 to
2ef545f
Compare
There was a problem hiding this comment.
🟡 Changes recommended
The replica readiness target and intended default validation integration must be corrected before approval.
Once you've addressed the issues Copilot identified, you can request another Copilot review.
Pull request overview
Adds E2E pod-capacity scale validation and helper tests, currently enabled for one Ubuntu 26.04 node-hardening scenario.
Changes:
- Creates and cleans up a node-pinned pause Deployment.
- Adds readiness, capacity, ownership, and construction helpers with tests.
- Invokes validation for one node-hardening scenario.
File summaries
| File | Review |
|---|---|
e2e/scenario_test.go |
Adds one validator call site. Moderate (2 votes): The default validation path and documented SkipScaleValidation escape hatch are not implemented. |
e2e/scale_validation.go |
Implements scale-up, readiness monitoring, and cleanup. Moderate (2 votes): Completion can succeed while requested replicas remain Pending because existing pod occupancy is not correctly reflected in the target. |
e2e/scale_validation_test.go |
Tests Deployment construction and scale-validation helper logic. |
Review details
- Files reviewed: 3/3 changed files
- Comments generated: 2
- Review effort level: Balanced
💡 Configure MCP servers for context-aware, tailored reviews. Learn more in the docs.
There was a problem hiding this comment.
🟡 Changes recommended
Capacity accounting and readiness stabilization issues can cause false failures or miss node failures.
Once you've addressed the issues Copilot identified, you can request another Copilot review.
Review details
- Files reviewed: 3/3 changed files
- Comments generated: 2
- Review effort level: Balanced
What this PR does / why we need it:
Adds an e2e scale validation to
Test_Ubuntu2604Minimal_NodeHardening_KubeReservedSlice_CLIFlags.The validation:
This specifically exercises the Node Hardening configuration under full pod-capacity load and detects the node failure that is not exposed by the existing single-pod smoke validation.
The validation is intentionally scoped to this Node Hardening scenario rather than added to the default validation path for every e2e scenario.
Which issue(s) this PR fixes:
Fixes #