[GitHub] [ozone] JyotinderSingh commented on pull request #2832: HDDS-5937. Inaccurate bucket info returned from bucket list command i…

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

JyotinderSingh commented on pull request #2832:
URL: https://github.com/apache/ozone/pull/2832#issuecomment-966841796


   > Thanks @JyotinderSingh for the contribution.
   > 
   > Can you please automate/add test case to verify the behavior. Below is one idea for test verification:
   > 
   > step-1) Add assert statement in [TestRootedOzoneFileSystem.java - line no# 432](https://github.com/apache/ozone/blob/master/hadoop-ozone/integration-test/src/test/java/org/apache/hadoop/fs/ozone/TestRootedOzoneFileSystem.java#L432) `Assert.assertEquals(bucketLayout, ozoneBucket.getBucketLayout());`
   > 
   > step-2) Presently [TestRootedOzoneFileSystemWithFSO#testMkdirNonExistentVolumeBucket](https://github.com/apache/ozone/blob/master/hadoop-ozone/integration-test/src/test/java/org/apache/hadoop/fs/ozone/TestRootedOzoneFileSystemWithFSO.java#L97) is ignored and this has been implemented now. Please delete this overridden method from TestRootedOzoneFileSystemWithFSO.java class, so that it will run along with other tests.
   
   Thanks for the feedback @rakeshadr. I have added the assertion to verify bucket layout and unblocked testMkdirNonExistentVolumeBucket. 


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