Change in ovirt-engine[ovirt-engine-3.6]: core: Change order of actions in remove data center flow

Tal Nisan has submitted this change and it was merged. Change subject: core: Change order of actions in remove data center flow ...................................................................... core: Change order of actions in remove data center flow When removing a data center the master storage domain was marked as a data domain and then the DestroyStoragePool command was sent to VDSM, since commit 2105394209362347655facec80b61c6b632932c9 a connect to storage pool is being sent as a part of the spm start flow, since the destroy command triggers the spm start after the master domain was marked as data the connect command could not find the master domain resulting in an error connecting to the pool and failure of the remove data center operation, the order of the actions was reversed to to avoid this fault Change-Id: I0e97919385e5ffc4db0d4d5b5a2ed78935b16bfd Bug-Url: https://bugzilla.redhat.com/1272110 Signed-off-by: Tal Nisan <tnisan@redhat.com> --- M backend/manager/modules/bll/src/main/java/org/ovirt/engine/core/bll/storage/RemoveStoragePoolCommand.java 1 file changed, 15 insertions(+), 13 deletions(-) Approvals: Tal Nisan: Verified Jenkins CI: Passed CI tests Liron Aravot: Looks good to me, approved -- To view, visit https://gerrit.ovirt.org/48030 To unsubscribe, visit https://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: I0e97919385e5ffc4db0d4d5b5a2ed78935b16bfd Gerrit-PatchSet: 3 Gerrit-Project: ovirt-engine Gerrit-Branch: ovirt-engine-3.6 Gerrit-Owner: Tal Nisan <tnisan@redhat.com> Gerrit-Reviewer: Jenkins CI Gerrit-Reviewer: Liron Aravot <laravot@redhat.com> Gerrit-Reviewer: Maor Lipchuk <mlipchuk@redhat.com> Gerrit-Reviewer: Tal Nisan <tnisan@redhat.com>
participants (1)
-
tnisan@redhat.com