Skip to content

Commit 2f93f82

Browse files
DaanHooglanddhslove
authored andcommitted
merge errors fixed
1 parent b843545 commit 2f93f82

File tree

1 file changed

+3
-3
lines changed

1 file changed

+3
-3
lines changed

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

Lines changed: 3 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -1638,7 +1638,7 @@ private boolean deleteVolumeFromStorage(VolumeVO volume, Account caller) throws
16381638
* <li> if the volume is still attached to a VM we throw an {@link InvalidParameterValueException};
16391639
* <li> if volume state is in {@link Volume.State#UploadOp}, we check the {@link VolumeDataStoreVO}. Then, if the {@link VolumeDataStoreVO} for the given volume has download status of {@link VMTemplateStorageResourceAssoc.Status#DOWNLOAD_IN_PROGRESS}, an exception is throw;
16401640
* <li> if the volume state is in {@link Volume.State#NotUploaded} or if the state is {@link Volume.State#UploadInProgress}, an {@link InvalidParameterValueException} is thrown;
1641-
* <li> we also check if the user has access to the given volume using {@link AccountManager#checkAccess(Account, org.apache.cloudstack.acl.SecurityChecker.AccessType, boolean, String)}.
1641+
* <li> we also check if the user has access to the given volume using {@see AccountManager#checkAccess(Account, org.apache.cloudstack.acl.SecurityChecker.AccessType, boolean, String)}.
16421642
* </ul>
16431643
*
16441644
* After all validations we return the volume object.
@@ -3602,7 +3602,7 @@ private void checkVmStateForMigration(VMInstanceVO vm, VolumeVO vol) {
36023602
List<State> suitableVmStatesForMigration = List.of(State.Stopped, State.Running, State.Shutdown);
36033603

36043604
if (!suitableVmStatesForMigration.contains(vm.getState())) {
3605-
s_logger.debug(String.format(
3605+
logger.debug(String.format(
36063606
"Unable to migrate volume: [%s] Id: [%s] because the VM: [%s] Id: [%s] is in state [%s], which is not supported for migration.",
36073607
vol.getName(), vol.getId(), vm.getInstanceName(), vm.getUuid(), vm.getState()
36083608
));
@@ -3665,7 +3665,7 @@ private DiskOfferingVO retrieveAndValidateNewDiskOffering(MigrateVolumeCmd cmd)
36653665
* <ul>
36663666
* <li>We check if the given volume is of ROOT type. We cannot change the disk offering of a ROOT volume. Therefore, we thrown an {@link InvalidParameterValueException};
36673667
* <li>We the disk is being migrated to shared storage and the new disk offering is for local storage (or vice versa), we throw an {@link InvalidParameterValueException}. Bear in mind that we are validating only the new disk offering. If none is provided we can override the current disk offering. This means, placing a volume with shared disk offering in local storage and vice versa;
3668-
* <li>We then proceed checking the target storage pool supports the new disk offering {@link #doesTargetStorageSupportNewDiskOffering(StoragePool, DiskOfferingVO)}.
3668+
* <li>We then proceed checking the target storage pool supports the new disk offering {@see #doesTargetStorageSupportNewDiskOffering(StoragePool, DiskOfferingVO)}.
36693669
* </ul>
36703670
*
36713671
* If all of the above validations pass, we check if the size of the new disk offering is different from the volume. If it is, we log a warning message.

0 commit comments

Comments
 (0)