harikrishna-patnala commented on code in PR #11616:
URL: https://github.com/apache/cloudstack/pull/11616#discussion_r2371314641


##########
server/src/main/java/com/cloud/vm/snapshot/VMSnapshotManagerImpl.java:
##########
@@ -399,7 +399,7 @@ public VMSnapshot allocVMSnapshot(Long vmId, String 
vsDisplayName, String vsDesc
         _accountMgr.checkAccess(caller, null, true, userVmVo);
 
         // check max snapshot limit for per VM
-        int vmSnapshotMax = VMSnapshotManager.VMSnapshotMax.value();
+        int vmSnapshotMax = 
VMSnapshotManager.VMSnapshotMax.valueIn(userVmVo.getAccountId());

Review Comment:
   @bernardodemarco I just got a doubt, what happens in the case if the VM is 
created in a project. As far as I know, VM will be mapped to a project account 
where it won't be having any configurations. any thoughts on this, may be 
consider the caller account id ?



-- 
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]

Reply via email to