nic-6443 commented on PR #2743: URL: https://github.com/apache/apisix-ingress-controller/pull/2743#issuecomment-4234321394
Pushed e2e test fix: replaced 2 direct HTTP assertions with `Eventually` retry in `test/e2e/crds/v2/route.go`. **Root cause**: In APISIX standalone mode, config application via `/apisix/admin/configs` is async (202 response). The Basic test and WebSocket test asserted HTTP status immediately after config push without retry, causing intermittent 404s when APISIX hadn't applied the new config yet. **Fix**: Use `Eventually` with 20s timeout + 1s polling (same pattern as other assertions in the test suite). The previous Benchmark test failure (APISIX 500 on ListServices) was an infrastructure issue — APISIX standalone failed to initialize. This should resolve on re-run. -- This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and use the URL above to go to the specific comment. To unsubscribe, e-mail: [email protected] For queries about this service, please contact Infrastructure at: [email protected]
