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