GutoVeronezi commented on a change in pull request #4978:
URL: https://github.com/apache/cloudstack/pull/4978#discussion_r629568870



##########
File path: 
plugins/hypervisors/kvm/src/main/java/org/apache/cloudstack/kvm/ha/KVMHostActivityChecker.java
##########
@@ -213,6 +278,18 @@ protected boolean 
verifyActivityOfStorageOnHost(HashMap<StoragePool, List<Volume
         return poolVolMap;
     }
 
+    private boolean isHostServedByNfsPool(Host agent) {
+        List<StoragePoolHostVO> storagesOnHost = 
storagePoolHostDao.listByHostId(agent.getId());
+        for (StoragePoolHostVO storagePoolHostRef : storagesOnHost) {
+            StoragePoolVO storagePool = 
this.storagePool.findById(storagePoolHostRef.getPoolId());
+            if(Storage.StoragePoolType.NetworkFilesystem == 
storagePool.getPoolType()
+                    || Storage.StoragePoolType.ManagedNFS == 
storagePool.getPoolType()) {

Review comment:
       The collection cited in 
https://github.com/apache/cloudstack/pull/4978/files#r629568449 could be used 
here too.




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

For queries about this service, please contact Infrastructure at:
[email protected]


Reply via email to