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(a)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(a)redhat.com>
Gerrit-Reviewer: Allon Mureinik <amureini(a)redhat.com>
Gerrit-Reviewer: Itamar Heim <iheim(a)redhat.com>
Gerrit-Reviewer: Maor Lipchuk <mlipchuk(a)redhat.com>
Gerrit-Reviewer: Sergey Gotliv <sgotliv(a)redhat.com>
Gerrit-Reviewer: automation(a)ovirt.org
Gerrit-Reviewer: oVirt Jenkins CI Server