Skip to content

Commit ab76d3c

Browse files
committed
merge errors fixed
1 parent 4f9c349 commit ab76d3c

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
@@ -1609,7 +1609,7 @@ private boolean deleteVolumeFromStorage(VolumeVO volume, Account caller) throws
16091609
* <li> if the volume is still attached to a VM we throw an {@link InvalidParameterValueException};
16101610
* <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;
16111611
* <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;
1612-
* <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)}.
1612+
* <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)}.
16131613
* </ul>
16141614
*
16151615
* After all validations we return the volume object.
@@ -3470,7 +3470,7 @@ private void checkVmStateForMigration(VMInstanceVO vm, VolumeVO vol) {
34703470
List<State> suitableVmStatesForMigration = List.of(State.Stopped, State.Running, State.Shutdown);
34713471

34723472
if (!suitableVmStatesForMigration.contains(vm.getState())) {
3473-
s_logger.debug(String.format(
3473+
logger.debug(String.format(
34743474
"Unable to migrate volume: [%s] Id: [%s] because the VM: [%s] Id: [%s] is in state [%s], which is not supported for migration.",
34753475
vol.getName(), vol.getId(), vm.getInstanceName(), vm.getUuid(), vm.getState()
34763476
));
@@ -3533,7 +3533,7 @@ private DiskOfferingVO retrieveAndValidateNewDiskOffering(MigrateVolumeCmd cmd)
35333533
* <ul>
35343534
* <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};
35353535
* <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;
3536-
* <li>We then proceed checking the target storage pool supports the new disk offering {@link #doesTargetStorageSupportNewDiskOffering(StoragePool, DiskOfferingVO)}.
3536+
* <li>We then proceed checking the target storage pool supports the new disk offering {@see #doesTargetStorageSupportNewDiskOffering(StoragePool, DiskOfferingVO)}.
35373537
* </ul>
35383538
*
35393539
* 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)