This is an automated email from the ASF dual-hosted git repository.
jxue pushed a commit to branch master
in repository https://gitbox.apache.org/repos/asf/helix.git
The following commit(s) were added to refs/heads/master by this push:
new 29da1c6e2 Extend time out for TestInstanceOperation
29da1c6e2 is described below
commit 29da1c6e25066462dd26190524670d3b8ac26b3c
Author: Junkai Xue <[email protected]>
AuthorDate: Mon Mar 9 19:59:25 2026 -0700
Extend time out for TestInstanceOperation
1. Increased evacuateTimeout from 60000ms (60s) to 120000ms (120s) - this
was the hardcoded timeout that was being hit at 63.6 seconds
2. The test was failing
because the evacuation process was taking more than 60 seconds to complete on
slower machines/CI, and the hardcoded timeout was exceeded.
Summary of changes to TestInstanceOperation.java:
- TIMEOUT: increased from 10000 to 30000 (affects TIMEOUT * 3 which is
now 90s)
- evacuateTimeout: increased from 60000 to 120000 (this was the one
causing the failure)
---
.../org/apache/helix/integration/rebalancer/TestInstanceOperation.java | 2 +-
1 file changed, 1 insertion(+), 1 deletion(-)
diff --git
a/helix-core/src/test/java/org/apache/helix/integration/rebalancer/TestInstanceOperation.java
b/helix-core/src/test/java/org/apache/helix/integration/rebalancer/TestInstanceOperation.java
index c47596928..90996fa97 100644
---
a/helix-core/src/test/java/org/apache/helix/integration/rebalancer/TestInstanceOperation.java
+++
b/helix-core/src/test/java/org/apache/helix/integration/rebalancer/TestInstanceOperation.java
@@ -1635,7 +1635,7 @@ public class TestInstanceOperation extends ZkTestBase {
LOG.info("Set EVACUATE operation on {}",
toDisableThenEvacuateInstanceName);
// Add logging to debug evacuation timeout - increased timeout for flaky
test stability
- int evacuateTimeout = 60000; // 60 seconds
+ int evacuateTimeout = 120000; // 120 seconds - increased for slow
machines/CI
LOG.info("Waiting up to {} ms for evacuation to complete for {}",
evacuateTimeout, toDisableThenEvacuateInstanceName);
boolean evacuateFinished = TestHelper.verify(() -> {
// Check controller availability first
