DaanHoogland commented on code in PR #14293:
URL: https://github.com/apache/cloudstack/pull/14293#discussion_r4323370654
##########
server/src/main/java/com/cloud/api/ApiDBUtils.java:
##########
@@ -986,8 +986,8 @@ public static long findCorrectResourceLimitForDomain(Long
limit, ResourceType re
if (domainId == Domain.ROOT_DOMAIN) {
return Resource.RESOURCE_UNLIMITED;
}
- //--If limit doesn't have a value then fetch default limit from the
configs
- return (limit == null) ?
s_resourceLimitMgr.findDefaultResourceLimitForDomain(resourceType) : limit;
+ //--If limit doesn't have a value then fetch the limit that is
enforced: a parent domain's limit or the default from the configs
Review Comment:
```suggestion
```
I think this comment doesn't say anything beyond what the code below says
--
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]
