raju-balpande commented on code in PR #6223:
URL: https://github.com/apache/ozone/pull/6223#discussion_r1535888764


##########
hadoop-ozone/integration-test/src/test/java/org/apache/hadoop/ozone/recon/TestReconTasks.java:
##########
@@ -219,9 +230,10 @@ public void testEmptyMissingContainerDownNode() throws 
Exception {
     ContainerManager scmContainerManager = scm.getContainerManager();
     ReconContainerManager reconContainerManager =
         (ReconContainerManager) reconScm.getContainerManager();
+    int previousContainerCount = reconContainerManager.getContainers().size();
     ContainerInfo containerInfo =
         scmContainerManager
-            .allocateContainer(RatisReplicationConfig.getInstance(ONE), 
"test");
+            .allocateContainer(RatisReplicationConfig.getInstance(ONE), 
"testEmptyMissingContainer");

Review Comment:
   I tried changing it to HddsProtos.ReplicationFactor.THREE but it seems 
having problem with number of pipelines..
   java.io.IOException: Could not allocate container. Cannot get any matching 
pipeline for replicationConfig: RATIS/THREE, State:PipelineState.OPEN
   
        at 
org.apache.hadoop.hdds.scm.container.ContainerManagerImpl.allocateContainer(ContainerManagerImpl.java:202)
        at 
org.apache.hadoop.ozone.recon.TestReconTasks.testEmptyMissingContainerDownNode(TestReconTasks.java:236)
        at java.lang.reflect.Method.invoke(Method.java:498)



-- 
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]


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

Reply via email to