Omer Frenkel has uploaded a new change for review. Change subject: core: load only managed device payload on run vm ......................................................................
core: load only managed device payload on run vm run vm command loads the vm payload from the devices in order to send it to vdsm. we should only load devices that are managed, as unmanaged devices were not created in the engine or not meant to be persisted and sent on later runs. Change-Id: Icc0c1876228d9b8cc72cb5698b8d8f12135594d0 Bug-Url: https://bugzilla.redhat.com/show_bug.cgi?id=1190696 Signed-off-by: Omer Frenkel <[email protected]> --- M backend/manager/modules/bll/src/main/java/org/ovirt/engine/core/bll/RunVmCommand.java 1 file changed, 1 insertion(+), 1 deletion(-) git pull ssh://gerrit.ovirt.org:29418/ovirt-engine refs/changes/41/38541/1 diff --git a/backend/manager/modules/bll/src/main/java/org/ovirt/engine/core/bll/RunVmCommand.java b/backend/manager/modules/bll/src/main/java/org/ovirt/engine/core/bll/RunVmCommand.java index 6a63925..e1ea86d 100644 --- a/backend/manager/modules/bll/src/main/java/org/ovirt/engine/core/bll/RunVmCommand.java +++ b/backend/manager/modules/bll/src/main/java/org/ovirt/engine/core/bll/RunVmCommand.java @@ -729,7 +729,7 @@ VmDeviceGeneralType.DISK, deviceType.getName()); for (VmDevice vmDevice : vmDevices) { - if (VmPayload.isPayload(vmDevice.getSpecParams())) { + if (vmDevice.getIsManaged() && VmPayload.isPayload(vmDevice.getSpecParams())) { return new VmPayload(vmDevice); } } -- To view, visit https://gerrit.ovirt.org/38541 To unsubscribe, visit https://gerrit.ovirt.org/settings Gerrit-MessageType: newchange Gerrit-Change-Id: Icc0c1876228d9b8cc72cb5698b8d8f12135594d0 Gerrit-PatchSet: 1 Gerrit-Project: ovirt-engine Gerrit-Branch: master Gerrit-Owner: Omer Frenkel <[email protected]> _______________________________________________ Engine-patches mailing list [email protected] http://lists.ovirt.org/mailman/listinfo/engine-patches
