Change in ovirt-engine[master]: 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(a)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(a)redhat.com>
Gerrit-Reviewer: Jenkins CI
Gerrit-Reviewer: Liron Aravot <laravot(a)redhat.com>
Gerrit-Reviewer: Maor Lipchuk <mlipchuk(a)redhat.com>
Gerrit-Reviewer: Tal Nisan <tnisan(a)redhat.com>
Gerrit-Reviewer: automation(a)ovirt.org