Add E2E test which shows adding deny rule blocks kas traffic - #6424
Add E2E test which shows adding deny rule blocks kas traffic#6424Mansi Mandhare (m-mandhare) wants to merge 4 commits into
Conversation
|
Skipping CI for Draft Pull Request. |
|
/ok-to-test |
|
/assign Martin Bukatovic (@mbukatov) This is still a draft, but I'm making sure it's on our radar. |
|
/assign Martin Bukatovic (@mbukatov) |
|
/assign Miquel Simon (@miquelsi) |
|
/retest |
|
[APPROVALNOTIFIER] This PR is NOT APPROVED This pull-request has been approved by: m-mandhare The full list of commands accepted by this bot can be found here. DetailsNeeds approval from an approver in each of these files:Approvers can indicate their approval by writing |
|
/retest |
7d3c5ef to
f0f5b3b
Compare
|
/retest |
|
/retest |
1 similar comment
|
/retest |
|
Mansi Mandhare (@m-mandhare): The following test failed, say
Full PR test history. Your PR dashboard. DetailsInstructions for interacting with me using PR comments are available here. If you have questions or suggestions related to my behavior, please file an issue against the kubernetes-sigs/prow repository. I understand the commands that are listed here. |
What
Creates a Swift HCP cluster (worker subnet + vnet-integration/Swift subnet, NSG + Red Hat delegation).
Case 1 — outbound Deny Any → worker (ports 443/6443): blocks kubelet → KAS ILB. Creates a node pool and asserts it stays stuck (never Succeeded) for ~15 minutes.
Removes that rule.
Case 2 — inbound Deny worker subnet → Swift subnet -: blocks worker(port 8443) → private-router / KAS / ignition / konnectivity. Same stuck-provisioning assertion.
Removes that rule, creates a third node pool, and asserts it reaches Provisioning/Succeeded (positive control).
Why
Misconfigured customer NSG rules that block worker bootstrap paths can leave node pools stuck in Provisioning. This e2e proves those two network paths matter and that the node pool recovers once the rules are removed.
Testing
Ignition error in nodepool VM -

Testing is required for feature completion and tests should be part of the pull
request along with the feature changes.
Describe the testing provided. If you did not add tests, provide a clear
justification.
Special notes for your reviewer
PR Checklist
If E2E tests are included:
demonstrate that the test is able to detect a defect/error and fail with
proper error message and logs which communicates nature of the problem.