harikrishna-patnala commented on code in PR #11616:
URL: https://github.com/apache/cloudstack/pull/11616#discussion_r2412545287
##########
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:
Yeah that makes sense too @bernardodemarco we can consider it only when VM
belongs to a project. On the other we should follow the same pattern for other
account level settings. We can start with this setting first. Thanks for the
solution.
--
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]
