Change in ovirt-engine[master]: core: fix to AddVdsSpmIdCommand

[derez at redhat.com]

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 at 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 at redhat.com>
Gerrit-Reviewer: Allon Mureinik <amureini at redhat.com>
Gerrit-Reviewer: Daniel Erez <derez at redhat.com>
Gerrit-Reviewer: Liron Aravot <laravot at redhat.com>
Gerrit-Reviewer: Tal Nisan <tnisan at redhat.com>
Gerrit-Reviewer: Vered Volansky <vvolansk at redhat.com>
Gerrit-Reviewer: Yair Zaslavsky <yzaslavs at redhat.com>