adoroszlai commented on a change in pull request #2002:
URL: https://github.com/apache/ozone/pull/2002#discussion_r589802440
##########
File path: hadoop-ozone/dist/src/main/smoketest/s3/commonawslib.robot
##########
@@ -39,18 +39,6 @@ Execute AWSS3Cli
${output} = Execute aws s3 --endpoint-url
${ENDPOINT_URL} ${command}
[return] ${output}
-Install aws cli
Review comment:
Thanks @xiaoyuyao for the review. The installation logic is idempotent
(`yum` does nothing if the package is already installed), so we could simply
keep it without any new check. I have considered this option, but we already
have other tools which are not installed during tests, rather assumed to be
present in the environment (eg. `boto3` most recently).
----------------------------------------------------------------
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]
---------------------------------------------------------------------
To unsubscribe, e-mail: [email protected]
For additional commands, e-mail: [email protected]
