JoaoJandre commented on PR #7441: URL: https://github.com/apache/cloudstack/pull/7441#issuecomment-1527472022
> code looks good, but we have a functional change here. Before the global setting was a cap on the usage. Now it is just a default. I don´t mind either way but backwards compatibilty dictates that the cap should be lowest of the local and the global configuration. What do you think @JoaoJandre ? For the other limits, that is how it currently works, the most specific setting is the one that prevails. For user VMs for example, if you have the global setting at 1 and the account limit at 2, that account will be able to create two accounts. In the VM creation process, the `getCheckedUserVmResource` method will be called, which eventually will call the `findCorrectResourceLimitForAccount` method to get the account's limit, which is the same as I propose to use with the secondary storage. -- 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]
