shreemaan-abhishek opened a new pull request, #13332: URL: https://github.com/apache/apisix/pull/13332
### Description Follow-up to #13266. After that PR landed I re-ran the cross-branch CI flake analysis on the past week of failures (apache/apisix master + several feature branches) and four flakes survived. This PR fixes them. | Test | Symptom | Fix | |---|---|---| | `t/cli/test_etcd_sync_event_handle.sh` | `failed: Round 2 Request 1 unexpected` (curl `/1` returned 503/stale route instead of 204) | Replace `sleep 5; curl /1; ...` with a 30s deadline poll on `/1` until the new `fault-injection` plugin is applied | | `t/core/config_etcd.t` TEST 10 | `timeout when waiting for the process N to exit at Test/Nginx/Util.pm line 683` | Bump `TEST_NGINX_TIMEOUT` to 30 for the file (test leaves a `run_watch` background timer; default 3s kill-wait is too short on slow runners) | | `t/admin/plugins-reload.t` TEST 1 + TEST 2 | `grep_error_log_out` empty for `sync local conf to etcd` / `reload plugins on node before reload` | `ngx.sleep(1)` → `ngx.sleep(2)` (post-reload sync log occasionally lands after the 1s grace) | | `t/discovery/eureka.t` TEST 4 | `error_log` pattern `failed to fetch registry from 127.0.0.1:20997: status=502` not matched | Add `--- wait: 2` so the 1s eureka `fetch_interval` has time to fire before the grep | #### Why these flakes weren't caught in #13266 The previous bundle was scoped to test-files that flaked at high frequency on master before the fix; these four either flaked in branches not yet sampled, or showed up only after the prior fixes landed and freed up other slow paths. Each one has been seen on master/main + at least one feature branch within the past 7 days. #### Which issue(s) this PR fixes: Fixes # ### Checklist - [x] I have explained the need for this PR and the problem it solves - [x] I have explained the changes or the new features added to this PR - [x] I have added tests corresponding to this change - [ ] I have updated the documentation to reflect this change - [x] I have verified that this change is backward compatible -- 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]
