Oved Ourfali has submitted this change and it was merged.
Change subject: core, engine: Avoid creating the CD device multiple times
......................................................................
core, engine: Avoid creating the CD device multiple times
When there are more than one device attached to the blank template, the
AddVmCommand creates multiple CD devices which conflict with each other.
This change fixes this issue, but still assumes that the blank template
has at least one device attached to it.
This patch has already been pushed to 3.4
(Iff3c5a34dd0fe6cfb6b4ab3bc77d34c4c0b57939) but was abandoned, now it seems to
be indeed relevant so pushing again under a different change id.
Change-Id: I34c0ea6d32ca378f22b9309b4c532d65c59337ba
Bug-Url:
https://bugzilla.redhat.com/show_bug.cgi?id=1075102
Signed-off-by: Vitor de Lima <vitor.lima(a)eldorado.org.br>
---
M
backend/manager/modules/bll/src/main/java/org/ovirt/engine/core/bll/utils/VmDeviceUtils.java
1 file changed, 2 insertions(+), 0 deletions(-)
Approvals:
Tomas Jelinek: Verified
Omer Frenkel: Looks good to me, approved
--
To view, visit
http://gerrit.ovirt.org/32929
To unsubscribe, visit
http://gerrit.ovirt.org/settings
Gerrit-MessageType: merged
Gerrit-Change-Id: I34c0ea6d32ca378f22b9309b4c532d65c59337ba
Gerrit-PatchSet: 1
Gerrit-Project: ovirt-engine
Gerrit-Branch: ovirt-engine-3.4
Gerrit-Owner: Tomas Jelinek <tjelinek(a)redhat.com>
Gerrit-Reviewer: Omer Frenkel <ofrenkel(a)redhat.com>
Gerrit-Reviewer: Oved Ourfali <oourfali(a)redhat.com>
Gerrit-Reviewer: Tomas Jelinek <tjelinek(a)redhat.com>
Gerrit-Reviewer: Vitor de Lima <vitordelima(a)gmail.com>
Gerrit-Reviewer: automation(a)ovirt.org
Gerrit-Reviewer: oVirt Jenkins CI Server