[GitHub] [ozone] iamabug commented on a change in pull request #2549: HDDS-5604. Intermittent failure in TestPipelineClose.

[GitBox <gi...@apache.org>]

iamabug commented on a change in pull request #2549:
URL: https://github.com/apache/ozone/pull/2549#discussion_r691858687



##########
File path: hadoop-ozone/integration-test/src/test/java/org/apache/hadoop/hdds/scm/pipeline/TestPipelineClose.java
##########
@@ -242,7 +238,7 @@ public void testPipelineCloseWithLogFailure() throws IOException {
     xceiverRatis.handleNodeLogFailure(groupId, null);
 
     // verify SCM receives a pipeline action report "immediately"
-    Mockito.verify(pipelineActionTest, Mockito.timeout(100))
+    Mockito.verify(pipelineActionTest, Mockito.timeout(500))

Review comment:
       I notice that there is a global Timeout variable in this class, which applies to all the methods in this class.
   ```
    public class TestPipelineClose {
   
      /**
        * Set a timeout for each test.
        */
      @Rule
      public Timeout timeout = Timeout.seconds(300);
   ```
   
   Not sure if this has something to do with the failures.




-- 
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: issues-unsubscribe@ozone.apache.org

For queries about this service, please contact Infrastructure at:
users@infra.apache.org



---------------------------------------------------------------------
To unsubscribe, e-mail: issues-unsubscribe@ozone.apache.org
For additional commands, e-mail: issues-help@ozone.apache.org