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

[tnisan at redhat.com]

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 at 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
  Maor Lipchuk: Looks good to me, approved
  Liron Aravot: Looks good to me, but someone else must approve



-- 
To view, visit https://gerrit.ovirt.org/47987
To unsubscribe, visit https://gerrit.ovirt.org/settings

Gerrit-MessageType: merged
Gerrit-Change-Id: I0e97919385e5ffc4db0d4d5b5a2ed78935b16bfd
Gerrit-PatchSet: 3
Gerrit-Project: ovirt-engine
Gerrit-Branch: master
Gerrit-Owner: Tal Nisan <tnisan at redhat.com>
Gerrit-Reviewer: Jenkins CI
Gerrit-Reviewer: Liron Aravot <laravot at redhat.com>
Gerrit-Reviewer: Maor Lipchuk <mlipchuk at redhat.com>
Gerrit-Reviewer: Tal Nisan <tnisan at redhat.com>
Gerrit-Reviewer: automation at ovirt.org