This is an automated email from the ASF dual-hosted git repository.

dahn pushed a commit to branch main
in repository https://gitbox.apache.org/repos/asf/cloudstack.git


The following commit(s) were added to refs/heads/main by this push:
     new 4e826d5  system_use column is now moved from disk_offering table to 
service_offering table, because it is applicable only to service offering. 
(#6015)
4e826d5 is described below

commit 4e826d504e06952a31ed83cd5daf52ef0f3fead3
Author: Harikrishna <[email protected]>
AuthorDate: Tue Feb 22 13:14:36 2022 +0530

    system_use column is now moved from disk_offering table to service_offering 
table, because it is applicable only to service offering. (#6015)
    
    There is a left over usage of system_use when user tries to list disk 
offering. Now we deleted that reference.
---
 server/src/main/java/com/cloud/api/query/QueryManagerImpl.java | 3 ---
 1 file changed, 3 deletions(-)

diff --git a/server/src/main/java/com/cloud/api/query/QueryManagerImpl.java 
b/server/src/main/java/com/cloud/api/query/QueryManagerImpl.java
index 0ae776b..75ad4b0 100644
--- a/server/src/main/java/com/cloud/api/query/QueryManagerImpl.java
+++ b/server/src/main/java/com/cloud/api/query/QueryManagerImpl.java
@@ -2932,10 +2932,7 @@ public class QueryManagerImpl extends 
MutualExclusiveIdsManagerBase implements Q
                 if (account.getType() == Account.ACCOUNT_TYPE_NORMAL) {
                     throw new InvalidParameterValueException("Only ROOT admins 
and Domain admins can list disk offerings with isrecursive=true");
                 }
-            } else { // domain + all ancestors
-                sc.addAnd("systemUse", SearchCriteria.Op.EQ, false); // 
non-root users should not see system offering at all
             }
-
         }
 
         if (volumeId != null && storagePoolId != null) {

Reply via email to