Muli Salem has posted comments on this change.
Change subject: core: Block Hotplug Nic in Certain Vm States (#826931)
......................................................................
Patch Set 4: (3 inline comments)
....................................................
File
backend/manager/modules/bll/src/main/java/org/ovirt/engine/core/bll/AddVmInterfaceCommand.java
Line 76:
Line 77: boolean succeded = true;
Line 78: VmDynamic vmDynamic = getVm().getDynamicData();
Line 79: if (getParameters().getInterface().isActive()
Line 80: &&
HotPlugUnplugVmNicCommand.isHotPlugNicAllowedForVmStatus(vmDynamic.getstatus()))
{
Done
Line 81: succeded = hotPlugNic(vmDevice, vmDynamic);
Line 82: if (!succeded) {
Line 83:
getReturnValue().getExecuteFailedMessages().add("Failed hot-plugging nic to
VM");
Line 84: }
....................................................
File
backend/manager/modules/bll/src/main/java/org/ovirt/engine/core/bll/HotPlugUnplugVmNicCommand.java
Line 29: @Override
Line 30: protected boolean canDoAction() {
Line 31: boolean returnValue = true;
Line 32:
Line 33: if (isAttachDetachAllowedForVmStatus(getVm().getstatus())) {
Done
Line 34: setVdsId(getVm().getrun_on_vds().getValue());
Line 35: // HotPlug in the host needs to be called only if the Vm
is UP
Line 36: if (isHotPlugNicAllowedForVmStatus(getVm().getstatus())) {
Line 37: returnValue = canPerformHotPlug();
....................................................
File backend/manager/modules/dal/src/main/resources/bundles/AppErrors.properties
Line 685: ACTION_TYPE_FAILED_GUEST_OS_VERSION_IS_NOT_SUPPORTED=Cannot ${action}
${type}. Guest OS version is not supported.
Line 686: HOT_PLUG_DISK_IS_NOT_VIRTIO=Can plug only virtIO disks.
Line 687: HOT_PLUG_DISK_IS_NOT_UNPLUGGED=Disk is already activated.
Line 688: HOT_UNPLUG_DISK_IS_NOT_PLUGGED=Disk is already deactivated.
Line 689: HOT_PLUG_NIC_VM_STATUS_ILLEGAL=Cannot activate/deactivate VM Network
Interface due to VM status. The VM status must be Down, Powering Up or Up.
Done
Line 690: VM_CANNOT_RUN_FROM_DISK_WITHOUT_PLUGGED_DISK=Cannot ${action} ${type}
without at least one active disk.\nPlease activate a disk and rerun the VM.
Line 691: SHAREABLE_DISK_IS_NOT_SUPPORTED_FOR_DISK=Cannot ${action} ${type}.
Disk cannot be shareable if it depends on a snapshot. In order to share it,
remove the disk's snapshots.
Line 692: ERROR_CANNOT_DETACH_DISK_WITH_SNAPSHOT=Cannot ${action} ${type}. The
disk is already configured in a snapshot. In order to detach it, remove the
disk's snapshots.
Line 693: DISK_IS_ALREADY_SHARED_BETWEEN_VMS=Cannot ${action} ${type}. Disk is
shared between vms and cannot become unshareable . Detach the disk from the
rest of the vms it is attached to and then update the disk to be unshareable.
--
To view, visit http://gerrit.ovirt.org/7176
To unsubscribe, visit http://gerrit.ovirt.org/settings
Gerrit-MessageType: comment
Gerrit-Change-Id: Id759ca8152b455047c9c83447f526aaccdd77d6c
Gerrit-PatchSet: 4
Gerrit-Project: ovirt-engine
Gerrit-Branch: master
Gerrit-Owner: Muli Salem <[email protected]>
Gerrit-Reviewer: Moti Asayag <[email protected]>
Gerrit-Reviewer: Muli Salem <[email protected]>
Gerrit-Reviewer: oVirt Jenkins CI Server
_______________________________________________
Engine-patches mailing list
[email protected]
http://lists.ovirt.org/mailman/listinfo/engine-patches
