[GitHub] [ozone] prashantpogde commented on a change in pull request #2564: HDDS-5647. [Multi-Tenant] Implement AssignUserToTenant

[GitBox <gi...@apache.org>]

prashantpogde commented on a change in pull request #2564:
URL: https://github.com/apache/ozone/pull/2564#discussion_r696085012



##########
File path: hadoop-ozone/client/src/main/java/org/apache/hadoop/ozone/client/protocol/ClientProtocol.java
##########
@@ -573,13 +573,14 @@ void cancelDelegationToken(Token<OzoneTokenIdentifier> token)
 //  void deleteTenant(String tenantName) throws IOException;
 
   /**
-   * Create tenant user.
-   * @param tenantUsername tenant user name.
+   * Assign user to tenant.
+   * @param username user name to be assigned.
    * @param tenantName tenant name.
+   * @param accessId access ID.
    * @throws IOException
    */
-  S3SecretValue createTenantUser(String tenantUsername, String tenantName)
-      throws IOException;
+  S3SecretValue assignUserToTenant(String username, String tenantName,

Review comment:
       accessID as input is optional. If its not given, Ozone should create and return the unique accessID.




-- 
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: issues-unsubscribe@ozone.apache.org

For queries about this service, please contact Infrastructure at:
users@infra.apache.org



---------------------------------------------------------------------
To unsubscribe, e-mail: issues-unsubscribe@ozone.apache.org
For additional commands, e-mail: issues-help@ozone.apache.org