DaveTeng0 commented on PR #4630: URL: https://github.com/apache/ozone/pull/4630#issuecomment-1536782362
> Hi @DaveTeng0 , Thanks for the patch. Can you add this check in TestMultiTenantVolume as well which is meant for S3 multi tenant volume scenario. Yup, I add one test case in TestMultiTenantVolume where ozone should reject non-S3 compliant tenant Id when the default cluster's StrictS3 is true. And for other more different cases, I added in unit test TestOMTenantCreateRequest. Thanks for the review. -- 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]
