[ 
https://issues.apache.org/jira/browse/HDDS-997?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=16753705#comment-16753705
 ] 

Shashikant Banerjee commented on HDDS-997:
------------------------------------------

Thanks [~nilotpalnandi] for working on this. The patch looks good to me. Some 
minor comments:

1.While adding the tests in #test_scm_isolation_one_node and 
#test_scm_isolation_two_node, let's also add the expected results in the 
comments to make it more understandable.

2. Add a more meaningful log info while tearing down the blockade cluster 
logging the cluster FILE location.

3.The test seem to waiting for 900 seconds to check the container status after 
running freon? Can it made configurable in the tests?

4.ozoneblockade.json : -> is this file required?

5.test_datanode_isolation_all :-> empty line added. Please revert.

 

> Add blockade Tests for scm isolation and mixed node isolation
> -------------------------------------------------------------
>
>                 Key: HDDS-997
>                 URL: https://issues.apache.org/jira/browse/HDDS-997
>             Project: Hadoop Distributed Data Store
>          Issue Type: Bug
>    Affects Versions: 0.4.0
>            Reporter: Nilotpal Nandi
>            Assignee: Nilotpal Nandi
>            Priority: Major
>         Attachments: HDDS-997.001.patch
>
>




--
This message was sent by Atlassian JIRA
(v7.6.3#76005)

---------------------------------------------------------------------
To unsubscribe, e-mail: hdfs-issues-unsubscr...@hadoop.apache.org
For additional commands, e-mail: hdfs-issues-h...@hadoop.apache.org

Reply via email to