abh1sar commented on code in PR #11384:
URL: https://github.com/apache/cloudstack/pull/11384#discussion_r2251150597
##########
server/src/main/java/org/apache/cloudstack/backup/BackupManagerImpl.java:
##########
@@ -1284,17 +1284,6 @@ public boolean restoreBackupToVM(final Long backupId,
final Long vmId) throws Re
throw new CloudRuntimeException("Unable to create Instance from
backup as the backup has a different number of disks than the Instance");
}
- int index = 0;
- for (VolumeVO vmVolume: vmVolumes) {
- Backup.VolumeInfo backupVolume = backupVolumes.get(index);
- if (vmVolume.getSize() < backupVolume.getSize()) {
- throw new CloudRuntimeException(String.format(
- "Instance volume size %d[GiB] for volume (%s) is less than
the backed-up volume size %d[GiB] for backed-up volume (%s).",
- vmVolume.getSize(), vmVolume.getUuid(),
backupVolume.getSize(), backupVolume.getUuid()));
- }
- index++;
- }
Review Comment:
Users can configure a different disk offering, or a larger size in case of
custom disk offering.
Only limitation is that the volume size should not be less than the
backed-up volume size.
If not configured, the volume will use the same disk offering and size as
the backed-up volume.
So, nothing to configure at this point.
--
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: [email protected]
For queries about this service, please contact Infrastructure at:
[email protected]