hadoop-yetus commented on a change in pull request #973: HDDS-1611. Evaluate
ACL on volume bucket key and prefix to authorize access. Contributed by Ajay
Kumar.
URL: https://github.com/apache/hadoop/pull/973#discussion_r298385836
##########
File path: hadoop-ozone/dist/src/main/smoketest/basic/ozone-shell.robot
##########
@@ -44,16 +44,17 @@ RpcClient without scheme
*** Keywords ***
Test ozone shell
[arguments] ${protocol} ${server} ${volume}
- ${result} = Execute ozone sh volume create
${protocol}${server}/${volume} --user bilbo --quota 100TB --root
+ ${result} = Execute ozone sh volume create
${protocol}${server}/${volume} --quota 100TB
Should not contain ${result} Failed
Should contain ${result} Creating Volume:
${volume}
- ${result} = Execute ozone sh volume list
${protocol}${server}/ --user bilbo | grep -Ev
'Removed|WARN|DEBUG|ERROR|INFO|TRACE' | jq -r '.[] |
select(.volumeName=="${volume}")'
+ ${result} = Execute ozone sh volume list
${protocol}${server}/ | grep -Ev 'Removed|WARN|DEBUG|ERROR|INFO|TRACE' | jq -r
'.[] | select(.volumeName=="${volume}")'
Should contain ${result} createdOn
- ${result} = Execute ozone sh volume list --user bilbo |
grep -Ev 'Removed|DEBUG|ERROR|INFO|TRACE|WARN' | jq -r '.[] |
select(.volumeName=="${volume}")'
+ ${result} = Execute ozone sh volume list | grep -Ev
'Removed|DEBUG|ERROR|INFO|TRACE|WARN' | jq -r '.[] |
select(.volumeName=="${volume}")'
Review comment:
whitespace:end of line
----------------------------------------------------------------
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]
With regards,
Apache Git Services
---------------------------------------------------------------------
To unsubscribe, e-mail: [email protected]
For additional commands, e-mail: [email protected]
