neils-dev commented on a change in pull request #2110:
URL: https://github.com/apache/ozone/pull/2110#discussion_r613444873



##########
File path: 
hadoop-ozone/integration-test/src/test/java/org/apache/hadoop/ozone/shell/TestOzoneShellHA.java
##########
@@ -558,6 +560,86 @@ public void testDeleteToTrashOrSkipTrash() throws 
Exception {
     }
   }
 
+  @Test
+  public void testDeleteTrashNoSkipTrash() throws Exception {
+
+    // Test delete from Trash directory removes item from filesystem
+
+    // setup configuration to use TrashPolicyOzone
+    // (default is TrashPolicyDefault)

Review comment:
       Thanks for your comment **@sadanand48** .  Commit a change for the 
service id as suggested.  Thanks for following up.  BTW, the host _set to_ 
`localhost`, as was previously used for the clientconf _should not_ be a 
problem with the integration test.  It was tested both individually` 
hadoop-ozone/integration-test$ mvn 
-Dtest=TestOzoneShellHA#testDeleteTrashNoSkipTrash test` and when the entire 
suite is run, `hadoop-ozone/integration-test$ mvn -Dtest=TestOzoneShellHA 
test`.  See attached image.
   
   ` hadoop-ozone/integration-test$ mvn 
-Dtest=TestOzoneShellHA#testDeleteTrashNoSkipTrash test`
   
![skipTrash](https://user-images.githubusercontent.com/81126310/114753821-24128680-9d15-11eb-81e5-c47b9bc75eb6.png)
   




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

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