khos2ow commented on a change in pull request #2817: Remove 'iam' projects
URL: https://github.com/apache/cloudstack/pull/2817#discussion_r211612784
 
 

 ##########
 File path: api/src/main/java/org/apache/cloudstack/api/ApiConstants.java
 ##########
 @@ -652,19 +652,7 @@
     public static final String MAX_CONNECTIONS = "maxconnections";
     public static final String SERVICE_STATE = "servicestate";
 
-    public static final String IAM_ACCOUNT_IDS = "accountids";
-    public static final String IAM_MEMBER_ACCOUNTS = "memberaccounts";
-    public static final String IAM_PARENT_POLICY_ID = "parentpolicyid";
-    public static final String IAM_PARENT_POLICY_NAME = "parentpolicyname";
-    public static final String IAM_POLICY_IDS = "policyids";
-    public static final String IAM_POLICIES = "policies";
-    public static final String IAM_APIS = "apis";
     public static final String IAM_GROUPS = "groups";
 
 Review comment:
   I removed it at first, but it is used in [AccountResponse][ar], and since 
that's an API response I moved it back.
   
   [ar]: 
https://github.com/apache/cloudstack/blob/master/api/src/main/java/org/apache/cloudstack/api/response/AccountResponse.java#L254

----------------------------------------------------------------
This is an automated message from the Apache Git Service.
To respond to the message, please log on GitHub and use the
URL above to go to the specific comment.
 
For queries about this service, please contact Infrastructure at:
[email protected]


With regards,
Apache Git Services

Reply via email to