Stabilize Amphora serial LB tests: per-spec teardown and sourceRanges wait fix - #320
tusharjadhav3302 wants to merge 1 commit into
Conversation
|
[APPROVALNOTIFIER] This PR is NOT APPROVED This pull-request has been approved by: 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 |
|
@tusharjadhav3302: all tests passed! 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. |
… fix Register Amphora-only per-spec Service/Octavia cleanup for create, ETP:Local+monitors, and UDP sourceRanges. Fix provider case mismatch so Amphora waits for allowed_cidrs allow-all, and dump diagnostics on UDP connectivity failure. Co-authored-by: Cursor <cursoragent@cursor.com>
94a3bbf to
bd44dc7
Compare
Summary
[lb]specs can leave Octavia LBs behind between Its, which undermax-shared-lb/ amphora capacity pressure contributes to ensure-LB timeouts inosp_verification.DeferCleanupfor the create, ETP:Local+monitors, and UDP sourceRanges Its (delete Service + cascade-delete Octavia LB if it remains). Registered beforeCreateLoadBalancerServiceso a timed-out ensure still cleans up. OVN / other[lb]specs unchanged.lbProviderUnderTest == "amphora"case mismatch (EqualFold) so Amphora actually waits for listenerallowed_cidrs→0.0.0.0/0after clearingloadBalancerSourceRanges(previously took the OVN 30s sleep path). On post-open UDP failure, dump listener ACL / LB / pool / member state and asourceRanges_diag_verdict.Test plan
Related