cli: Remove redundant GetVolumeSimpleInfo when setting user's perm

There is no need to call GetVolumeSimpleInfo separately when setting user's permission, since removeUserPolicy() or updateUserPolicy() still checks if specific volume exists. Signed-off-by: Sheng Yong <[email protected]>
web-logs2 · Nov 3, 2021 · f3bc4b1 · f3bc4b1
1 parent b605340
commit f3bc4b1
Showing 1 changed file with 0 additions and 3 deletions.
diff --git a/cli/cmd/user.go b/cli/cmd/user.go
@@ -360,9 +360,6 @@ func newUserPermCmd(client *master.MasterClient) *cobra.Command {
 			if userInfo, err = client.UserAPI().GetUserInfo(userID); err != nil {
 				return
 			}
-			if _, err = client.AdminAPI().GetVolumeSimpleInfo(volume); err != nil {
-				return
-			}
 			if perm.IsNone() {
 				param := proto.NewUserPermRemoveParam(userID, volume)
 				userInfo, err = client.UserAPI().RemovePolicy(param)