jojochuang commented on code in PR #10792:
URL: https://github.com/apache/ozone/pull/10792#discussion_r3646822154


##########
hadoop-ozone/dist/src/main/smoketest/ozonefs/ozonefs-obs.robot:
##########
@@ -51,10 +51,10 @@ Verify ls fails on OBS bucket
 Create key in OBS bucket
     Execute             ozone sh key put /${volume}/${bucket}/testfile 
NOTICE.txt
 
-Verify ls fails on OBS bucket key
+Verify ls succeeds on OBS bucket key
     ${url} =            Format FS URL         ${SCHEME}    ${volume}    
${bucket}    testfile
-    ${result} =         Execute and checkrc   ozone fs -ls ${url}     255
-    Should contain      ${result}             Bucket: ${bucket} has layout: 
OBJECT_STORE, which does not support file system semantics. Bucket Layout must 
be FILE_SYSTEM_OPTIMIZED or LEGACY.

Review Comment:
   this is a breaking change. Interestingly, OM does not reject getFileStatus() 
call.
   
   Without breaking this behavior, it would require a client side cache to 
store bucket info. But that's more complex.



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