smengcl commented on a change in pull request #2649:
URL: https://github.com/apache/ozone/pull/2649#discussion_r713390044
##########
File path:
hadoop-ozone/tools/src/main/java/org/apache/hadoop/ozone/shell/s3/GetS3SecretHandler.java
##########
@@ -35,8 +35,8 @@
public class GetS3SecretHandler extends S3Handler {
@Option(names = "-u",
- description = "Specify the user name to perform the operation on "
- + "(admins only)'")
+ description = "Specify the user to perform the operation on "
+ + "(Admins only)'")
Review comment:
Good question. I'm not sure if we should allow **tenant** admins to get
secrets for it's users though. @prashantpogde Maybe we could optionally allow
that (via a special ACL on the tenant admin that is assigned by Ozone admin for
example)?
--
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]
