Change in ovirt-engine[master]: engine: Set boot order to more then one CDROM if selected

shavivi at redhat.com shavivi at redhat.com
Thu Mar 6 13:49:33 UTC 2014


Shahar Havivi has submitted this change and it was merged.

Change subject: engine: Set boot order to more then one CDROM if selected
......................................................................


engine: Set boot order to more then one CDROM if selected

When dealing with more then one CDROM we set only the first in list if
user select boot order from CDROM, which can cause a problem if the
first CDROM is payload or cloud-init which are not bootable.
When setting all CDROM to be bootable qemu will try to boot from one
of them.

Change-Id: I7d4783f084623bc59096d87514a6f143e056d5a2
Bug-Url: https://bugzilla.redhat.com/1064927
Signed-off-by: Shahar Havivi <shaharh at redhat.com>
---
M backend/manager/modules/common/src/main/java/org/ovirt/engine/core/common/utils/VmDeviceCommonUtils.java
1 file changed, 2 insertions(+), 5 deletions(-)

Approvals:
  Shahar Havivi: Verified; Looks good to me, approved
  Omer Frenkel: Looks good to me, approved



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

Gerrit-MessageType: merged
Gerrit-Change-Id: I7d4783f084623bc59096d87514a6f143e056d5a2
Gerrit-PatchSet: 4
Gerrit-Project: ovirt-engine
Gerrit-Branch: master
Gerrit-Owner: Shahar Havivi <shavivi at redhat.com>
Gerrit-Reviewer: Eli Mesika <elimesika at gmail.com>
Gerrit-Reviewer: Michal Skrivanek <michal.skrivanek at redhat.com>
Gerrit-Reviewer: Omer Frenkel <ofrenkel at redhat.com>
Gerrit-Reviewer: Shahar Havivi <shavivi at redhat.com>
Gerrit-Reviewer: automation at ovirt.org
Gerrit-Reviewer: oVirt Jenkins CI Server



More information about the Engine-commits mailing list