Change in ovirt-engine[master]: core:Boot from CD doesn't work on 3.1 compatibility(820127)

emesika at redhat.com emesika at redhat.com
Thu May 17 15:04:47 UTC 2012


Eli Mesika has submitted this change and it was merged.

Change subject: core:Boot from CD doesn't work on 3.1 compatibility(820127)
......................................................................


core:Boot from CD doesn't work on 3.1 compatibility(820127)

https://bugzilla.redhat.com/show_bug.cgi?id=820127

Fix addresses basically three issues:
1) Make sure that no null values are added to the structure passed to vdsm
2) Take the boot sequence from dynamic parameters in run-once.
3) Pass in CD/Floppy full path and not only file name.

Seems that all problems are regressions of massive refactoring on the stable device addresses feature code.

Change-Id: Ida06b34f94e791a344e017cd5a6915e31b827cc9
Signed-off-by: Eli Mesika <emesika at redhat.com>
---
M backend/manager/modules/common/src/main/java/org/ovirt/engine/core/common/utils/VmDeviceCommonUtils.java
M backend/manager/modules/vdsbroker/src/main/java/org/ovirt/engine/core/vdsbroker/vdsbroker/VmInfoBuilder.java
2 files changed, 9 insertions(+), 29 deletions(-)

Approvals:
  Eli Mesika: Verified; Looks good to me, approved


--
To view, visit http://gerrit.ovirt.org/4378
To unsubscribe, visit http://gerrit.ovirt.org/settings

Gerrit-MessageType: merged
Gerrit-Change-Id: Ida06b34f94e791a344e017cd5a6915e31b827cc9
Gerrit-PatchSet: 4
Gerrit-Project: ovirt-engine
Gerrit-Branch: master
Gerrit-Owner: Eli Mesika <emesika at redhat.com>
Gerrit-Reviewer: Eli Mesika <emesika at redhat.com>
Gerrit-Reviewer: Mike Kolesnik <mkolesni at redhat.com>
Gerrit-Reviewer: Yair Zaslavsky <yzaslavs at redhat.com>



More information about the Engine-commits mailing list