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 grant on the tenant admin which 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]
