merlimat opened a new pull request, #25596: URL: https://github.com/apache/pulsar/pull/25596
### Motivation `testLoadBalancerServiceUnitTableViewSyncer` is flaky with `expected [true] but found [false]` after the 30s wait for `primaryLoadManager.getServiceUnitStateTableViewSyncer().isActive()`. The `loadManagerExecutor` is a single-threaded scheduled executor. After `makeSecondaryAsLeader()` followed immediately by `makePrimaryAsLeader()`, the new leader's `playLeader()` is queued behind any still-running `playFollower()` from the prior demotion. `syncer.start()` itself opens both table views and runs `syncExistingItems` + `syncTailItems`. Under CI load this serial chain can exceed 30s. ### Modifications Increase the timeout from 30s to 60s for the syncer-state checks after both leader transitions, and apply the same to the symmetric pair in the cleanup section after the dynamic config is removed. ### Verifying this change - [x] Make sure that the change passes the CI checks. Locally verified with 3 successful back-to-back runs of the test. -- 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]
