Skip to content

Commit afb31fa

Browse files
committed
CLOUDSTACK-6660: Dont assume every vm will have a root volume now.
(cherry picked from commit 076bbdc)
1 parent 8d9bdb1 commit afb31fa

1 file changed

Lines changed: 10 additions & 1 deletion

File tree

server/src/com/cloud/vm/UserVmManagerImpl.java

Lines changed: 10 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -1854,7 +1854,16 @@ public UserVm updateVirtualMachine(UpdateVMCmd cmd) throws ResourceUnavailableEx
18541854
saveUsageEvent(vmInstance);
18551855

18561856
// take care of the root volume as well.
1857-
_volumeService.updateDisplay(_volsDao.findByInstanceAndType(id, Volume.Type.ROOT).get(0), isDisplayVm);
1857+
List<VolumeVO> rootVols = _volsDao.findByInstanceAndType(id, Volume.Type.ROOT);
1858+
if(!rootVols.isEmpty()){
1859+
_volumeService.updateDisplay(rootVols.get(0), isDisplayVm);
1860+
}
1861+
1862+
// take care of the data volumes as well.
1863+
List<VolumeVO> dataVols = _volsDao.findByInstanceAndType(id, Volume.Type.DATADISK);
1864+
for(Volume dataVol : dataVols){
1865+
_volumeService.updateDisplay(dataVol, isDisplayVm);
1866+
}
18581867

18591868
}
18601869

0 commit comments

Comments
 (0)