Change in ovirt-engine[master]: core: fix to AddVdsSpmIdCommand
Daniel Erez has submitted this change and it was merged.
Change subject: core: fix to AddVdsSpmIdCommand
......................................................................
core: fix to AddVdsSpmIdCommand
related to https://bugzilla.redhat.com/show_bug.cgi?id=771699
Currently AddVdsSpmIdCommand relies on that the vds was previously
removed from it's previous DC (if present), the fix to this bug
rearranged the order of the commands so that the vds is added to the new
DC at first and then removed from it's previous DC, so this check caused
that hosts couldn't be moved between DC.
Change-Id: Id311f2ae71eda95d122ce186f0da1d87768eab0d
Signed-off-by: Liron Aravot <laravot(a)redhat.com>
---
M
backend/manager/modules/bll/src/main/java/org/ovirt/engine/core/bll/AddVdsSpmIdCommand.java
1 file changed, 1 insertion(+), 2 deletions(-)
Approvals:
Daniel Erez: Verified; Looks good to me, approved
Allon Mureinik: Looks good to me, but someone else must approve
--
To view, visit http://gerrit.ovirt.org/8278
To unsubscribe, visit http://gerrit.ovirt.org/settings
Gerrit-MessageType: merged
Gerrit-Change-Id: Id311f2ae71eda95d122ce186f0da1d87768eab0d
Gerrit-PatchSet: 2
Gerrit-Project: ovirt-engine
Gerrit-Branch: master
Gerrit-Owner: Liron Aravot <laravot(a)redhat.com>
Gerrit-Reviewer: Allon Mureinik <amureini(a)redhat.com>
Gerrit-Reviewer: Daniel Erez <derez(a)redhat.com>
Gerrit-Reviewer: Liron Aravot <laravot(a)redhat.com>
Gerrit-Reviewer: Tal Nisan <tnisan(a)redhat.com>
Gerrit-Reviewer: Vered Volansky <vvolansk(a)redhat.com>
Gerrit-Reviewer: Yair Zaslavsky <yzaslavs(a)redhat.com>