Change in ovirt-engine[ovirt-engine-3.4]: core, engine: Avoid creating the CD device multiple times

oourfali at redhat.com oourfali at redhat.com
Mon Sep 15 12:14:16 UTC 2014


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 at 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 at redhat.com>
Gerrit-Reviewer: Omer Frenkel <ofrenkel at redhat.com>
Gerrit-Reviewer: Oved Ourfali <oourfali at redhat.com>
Gerrit-Reviewer: Tomas Jelinek <tjelinek at redhat.com>
Gerrit-Reviewer: Vitor de Lima <vitordelima at gmail.com>
Gerrit-Reviewer: automation at ovirt.org
Gerrit-Reviewer: oVirt Jenkins CI Server



More information about the Engine-commits mailing list