Skip to content

Stabilize Amphora serial LB tests: per-spec teardown and sourceRanges wait fix - #320

Open
tusharjadhav3302 wants to merge 1 commit into
openshift:mainfrom
tusharjadhav3302:amphora-isolated-per-spec-teardown
Open

tusharjadhav3302 wants to merge 1 commit into
openshift:mainfrom
tusharjadhav3302:amphora-isolated-per-spec-teardown

Conversation

@tusharjadhav3302

@tusharjadhav3302 tusharjadhav3302 commented Sep 19, 2026

Copy link
Copy Markdown
Contributor

Summary

  • Serial Amphora [lb] specs can leave Octavia LBs behind between Its, which under max-shared-lb / amphora capacity pressure contributes to ensure-LB timeouts in osp_verification.
  • Register Amphora-only DeferCleanup for the create, ETP:Local+monitors, and UDP sourceRanges Its (delete Service + cascade-delete Octavia LB if it remains). Registered before CreateLoadBalancerService so a timed-out ensure still cleans up. OVN / other [lb] specs unchanged.
  • Fix lbProviderUnderTest == "amphora" case mismatch (EqualFold) so Amphora actually waits for listener allowed_cidrs0.0.0.0/0 after clearing loadBalancerSourceRanges (previously took the OVN 30s sleep path). On post-open UDP failure, dump listener ACL / LB / pool / member state and a sourceRanges_diag_verdict.

Test plan

  • Review Amphora-gated teardown limited to the three Its above
  • CI: build / images / test / verify (pre-squash; re-check after squash)
  • serval71: combined with shiftstack-qa PR #41 — late sourceRanges phase PASS (ACL allow-all ~2s); teardown helpers present in binary

Related

  • Complements shiftstack/shiftstack-qa#41 (late Amphora sourceRanges phase under less OCCM contention). Does not fix OCCM allow-all stall under heavy load; improves wait correctness and leftover LB hygiene.

@openshift-ci

openshift-ci Bot commented Sep 19, 2026

Copy link
Copy Markdown
Contributor

[APPROVALNOTIFIER] This PR is NOT APPROVED

This pull-request has been approved by:
Once this PR has been reviewed and has the lgtm label, please assign gryf for approval. For more information see the Code Review Process.

The full list of commands accepted by this bot can be found here.

Details Needs approval from an approver in each of these files:

Approvers can indicate their approval by writing /approve in a comment
Approvers can cancel approval by writing /approve cancel in a comment

@openshift-ci

openshift-ci Bot commented Sep 20, 2026

Copy link
Copy Markdown
Contributor

@tusharjadhav3302: all tests passed!

Full PR test history. Your PR dashboard.

Details

Instructions 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>
@tusharjadhav3302
tusharjadhav3302 force-pushed the amphora-isolated-per-spec-teardown branch from 94a3bbf to bd44dc7 Compare September 21, 2026 07:22
@tusharjadhav3302 tusharjadhav3302 changed the title Add Amphora-only per-spec teardown for serial LB tests Stabilize Amphora serial LB tests: per-spec teardown and sourceRanges wait fix Sep 21, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant