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


##########
plugins/backup/nas/src/main/java/org/apache/cloudstack/backup/NASBackupProvider.java:
##########
@@ -115,27 +130,38 @@ public class NASBackupProvider extends AdapterBase 
implements BackupProvider, Co
     @Inject
     private DiskOfferingDao diskOfferingDao;
 
+    protected Long getClusterIdFromRootVolume(VirtualMachine vm) {
+        VolumeVO rootVolume = _volsDao.getInstanceRootVolume(vm.getId());
+        StoragePoolVO rootDiskPool = 
_storagePoolDao.findById(rootVolume.getPoolId());

Review Comment:
   Ideally volume should be  allocated and placed in a storage pool by this 
time but I'll add the check to be safe. 



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