adoroszlai commented on a change in pull request #3167:
URL: https://github.com/apache/ozone/pull/3167#discussion_r820106821



##########
File path: hadoop-ozone/dist/src/main/smoketest/s3/awss3.robot
##########
@@ -47,7 +47,13 @@ File upload and directory list
                         Should not contain        ${result}         dir1
                         Should contain            ${result}         file
 
+List buckets with empty access id
+                        Execute                    aws configure set 
aws_access_key_id ''
+    ${result} =         Execute AWSS3APICli and checkrc         list-buckets   
 255

Review comment:
       S3 tests are grouped, mostly based on operation type.  `list-buckets` 
belongs to `bucketlist.robot`.
   
   (`awss3.robot` is for `aws s3` command, while other tests are for `aws 
s3api`.)
   
   Also, adding it as last test case avoids the need to run `Setup s3 tests` 
explicitly in the next test case.




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