Change in ovirt-engine[ovirt-engine-3.3]: core: Remove non-existent Storage Domain will be rolled forw...

[iheim at redhat.com]

Itamar Heim has submitted this change and it was merged.

Change subject: core: Remove non-existent Storage Domain will be rolled forward.
......................................................................


core: Remove non-existent Storage Domain will be rolled forward.

Removal of a Storage Domain which no longer exists causes a failure in
the engine and blocks the user from removing the storage.

The following patch validates the storage domain was not removed due
to the reason of non existence, and if so it will roll-forward and delete
it any how.

Change-Id: I9515a5ddfcaa965c2bad1883f0543494915b8c7d
Bug-Url: https://bugzilla.redhat.com/1001626
Signed-off-by: Maor Lipchuk <mlipchuk at redhat.com>
---
M backend/manager/modules/bll/src/main/java/org/ovirt/engine/core/bll/storage/RemoveStorageDomainCommand.java
1 file changed, 15 insertions(+), 3 deletions(-)

Approvals:
  Itamar Heim: Verified; Looks good to me, approved



-- 
To view, visit http://gerrit.ovirt.org/20333
To unsubscribe, visit http://gerrit.ovirt.org/settings

Gerrit-MessageType: merged
Gerrit-Change-Id: I9515a5ddfcaa965c2bad1883f0543494915b8c7d
Gerrit-PatchSet: 2
Gerrit-Project: ovirt-engine
Gerrit-Branch: ovirt-engine-3.3
Gerrit-Owner: Maor Lipchuk <mlipchuk at redhat.com>
Gerrit-Reviewer: Allon Mureinik <amureini at redhat.com>
Gerrit-Reviewer: Itamar Heim <iheim at redhat.com>
Gerrit-Reviewer: Maor Lipchuk <mlipchuk at redhat.com>
Gerrit-Reviewer: Sergey Gotliv <sgotliv at redhat.com>
Gerrit-Reviewer: automation at ovirt.org
Gerrit-Reviewer: oVirt Jenkins CI Server