Skip to content

Commit 19deded

Browse files
author
Henrique Sato
committed
Adjusts
1 parent fcdeaa2 commit 19deded

File tree

2 files changed

+8
-6
lines changed

2 files changed

+8
-6
lines changed

server/src/main/java/com/cloud/deploy/DeploymentPlanningManagerImpl.java

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -31,7 +31,7 @@
3131
import java.util.Set;
3232
import java.util.Timer;
3333
import java.util.TreeSet;
34-
import java.util.stream.Collectors;Res
34+
import java.util.stream.Collectors;
3535

3636
import javax.inject.Inject;
3737
import javax.naming.ConfigurationException;

server/src/main/java/com/cloud/storage/VolumeApiServiceImpl.java

Lines changed: 7 additions & 5 deletions
Original file line numberDiff line numberDiff line change
@@ -1318,10 +1318,8 @@ public VolumeVO resizeVolume(ResizeVolumeCmd cmd) throws ResourceAllocationExcep
13181318
volumeMigrateRequired = true;
13191319
}
13201320

1321-
boolean volumeResizeRequired = false;
1322-
if (currentSize != newSize || !compareEqualsIncludingNullOrZero(newMaxIops, volume.getMaxIops()) || !compareEqualsIncludingNullOrZero(newMinIops, volume.getMinIops())) {
1323-
volumeResizeRequired = true;
1324-
}
1321+
boolean volumeResizeRequired = currentSize != newSize || !compareEqualsIncludingNullOrZero(newMaxIops, volume.getMaxIops()) || !compareEqualsIncludingNullOrZero(newMinIops, volume.getMinIops())
1322+
|| !compareEqualsIncludingNullOrZero(newMaxIops, diskOffering.getIopsWriteRate()) || !compareEqualsIncludingNullOrZero(newMinIops, diskOffering.getIopsReadRate());
13251323
if (!volumeMigrateRequired && !volumeResizeRequired && newDiskOffering != null) {
13261324
_volsDao.updateDiskOffering(volume.getId(), newDiskOffering.getId());
13271325
volume = _volsDao.findById(volume.getId());
@@ -1386,7 +1384,11 @@ public VolumeVO resizeVolume(ResizeVolumeCmd cmd) throws ResourceAllocationExcep
13861384
} else if (jobResult instanceof Throwable) {
13871385
throw new RuntimeException("Unexpected exception", (Throwable) jobResult);
13881386
} else if (jobResult instanceof Long) {
1389-
return _volsDao.findById((Long) jobResult);
1387+
Long volumeId = (Long) jobResult;
1388+
if (newDiskOffering != null) {
1389+
_volsDao.updateDiskOffering(volumeId, newDiskOffering.getId());
1390+
}
1391+
return _volsDao.findById(volumeId);
13901392
}
13911393
}
13921394

0 commit comments

Comments
 (0)