(accumulo) branch 2.1 updated: Fixed ShutdownIT after recent Admin.stop change (#5757)

[dlmarion]

This is an automated email from the ASF dual-hosted git repository.

dlmarion pushed a commit to branch 2.1
in repository https://gitbox.apache.org/repos/asf/accumulo.git


The following commit(s) were added to refs/heads/2.1 by this push:
     new b800cef5d7 Fixed ShutdownIT after recent Admin.stop change (#5757)
b800cef5d7 is described below

commit b800cef5d70ce031c187d101488ef367b8e9c002
Author: Dave Marion <dlmar...@apache.org>
AuthorDate: Fri Jul 25 11:34:20 2025 +0000

    Fixed ShutdownIT after recent Admin.stop change (#5757)
    
    The IT was expecting the tserver to be down, but now
    it may take a little time for the tserver to stop.
    Modified the test to wait for the tserver to stop.
---
 test/src/main/java/org/apache/accumulo/test/functional/ShutdownIT.java | 2 ++
 1 file changed, 2 insertions(+)

diff --git 
a/test/src/main/java/org/apache/accumulo/test/functional/ShutdownIT.java 
b/test/src/main/java/org/apache/accumulo/test/functional/ShutdownIT.java
index a6267c08b2..29b895519a 100644
--- a/test/src/main/java/org/apache/accumulo/test/functional/ShutdownIT.java
+++ b/test/src/main/java/org/apache/accumulo/test/functional/ShutdownIT.java
@@ -35,6 +35,7 @@ import org.apache.accumulo.server.util.Admin;
 import org.apache.accumulo.test.TestIngest;
 import org.apache.accumulo.test.TestRandomDeletes;
 import org.apache.accumulo.test.VerifyIngest;
+import org.apache.accumulo.test.util.Wait;
 import org.junit.jupiter.api.Test;
 
 public class ShutdownIT extends ConfigurableMacBase {
@@ -124,6 +125,7 @@ public class ShutdownIT extends ConfigurableMacBase {
     String doomed = tabletServers.get(0);
     log.info("Stopping " + doomed);
     assertEquals(0, cluster.exec(Admin.class, "stop", 
doomed).getProcess().waitFor());
+    Wait.waitFor(() -> c.instanceOperations().getTabletServers().size() == 1);
     tabletServers = c.instanceOperations().getTabletServers();
     assertEquals(1, tabletServers.size());
     assertNotEquals(tabletServers.get(0), doomed);