abh1sar commented on code in PR #9208:
URL: https://github.com/apache/cloudstack/pull/9208#discussion_r1710074070


##########
server/src/main/java/com/cloud/server/ManagementServerImpl.java:
##########
@@ -4470,6 +4471,9 @@ public Map<String, Object> listCapabilities(final 
ListCapabilitiesCmd cmd) {
             regionSecondaryEnabled = true;
         }
 
+        final Integer fsVmMinCpu = 
Integer.parseInt(_configDao.getValue("storagefsvm.min.cpu.count"));
+        final Integer fsVmMinRam = 
Integer.parseInt(_configDao.getValue("storagefsvm.min.ram.size"));

Review Comment:
   StorageFsVmFileShareProvider class in plugins is not directly accessible 
from here. We can put the settings in FileShareProvider but that won't be 
totally accurate as these settings are only pertaining to storagefsvm provider.



-- 
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: commits-unsubscr...@cloudstack.apache.org

For queries about this service, please contact Infrastructure at:
us...@infra.apache.org

Reply via email to