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

adoroszlai pushed a commit to branch master
in repository https://gitbox.apache.org/repos/asf/ozone.git


The following commit(s) were added to refs/heads/master by this push:
     new c7d83a7b1e6 HDDS-15605. Fix flaky 
testContainerExclusionWithClosedContainerException (#10621)
c7d83a7b1e6 is described below

commit c7d83a7b1e65b37b9c0dc5277e2288c202c77261
Author: Chi-Hsuan Huang <[email protected]>
AuthorDate: Sun Jul 5 23:03:05 2026 +0800

    HDDS-15605. Fix flaky testContainerExclusionWithClosedContainerException 
(#10621)
---
 .../hadoop/ozone/client/rpc/TestFailureHandlingByClient.java       | 7 ++++++-
 1 file changed, 6 insertions(+), 1 deletion(-)

diff --git 
a/hadoop-ozone/integration-test/src/test/java/org/apache/hadoop/ozone/client/rpc/TestFailureHandlingByClient.java
 
b/hadoop-ozone/integration-test/src/test/java/org/apache/hadoop/ozone/client/rpc/TestFailureHandlingByClient.java
index 4d20d29791c..01cb7e92544 100644
--- 
a/hadoop-ozone/integration-test/src/test/java/org/apache/hadoop/ozone/client/rpc/TestFailureHandlingByClient.java
+++ 
b/hadoop-ozone/integration-test/src/test/java/org/apache/hadoop/ozone/client/rpc/TestFailureHandlingByClient.java
@@ -394,7 +394,12 @@ public void 
testContainerExclusionWithClosedContainerException()
 
       assertThat(keyOutputStream.getExcludeList().getContainerIds())
           .contains(ContainerID.valueOf(containerId));
-      assertThat(keyOutputStream.getExcludeList().getDatanodes()).isEmpty();
+      // Datanodes are not asserted here. Under the default ALL_COMMITTED watch
+      // level a slow-but-healthy follower can be recorded in the exclude 
list, so
+      // an empty datanode set is not an invariant for this config (the watch
+      // level is configurable via RatisClientConfig watchType, HDDS-2887).
+      // Watch-level datanode exclusion is covered by
+      // testDatanodeExclusionWithMajorityCommit.
       assertThat(keyOutputStream.getExcludeList().getPipelineIds()).isEmpty();
 
       // The close will just write to the buffer


---------------------------------------------------------------------
To unsubscribe, e-mail: [email protected]
For additional commands, e-mail: [email protected]

Reply via email to