Tal Nisan has submitted this change and it was merged. Change subject: core: Support detach of local Storage Domain ...................................................................... core: Support detach of local Storage Domain When detaching a local Storage Domain, the engine will automatically try to remove it. The remove process in the engine, checks if local Storage Domain is already attached to a Data Center so the Host will not fail during the remove process. This validation is meanningless for local Storage Domain since the local Storage Domain will first get detached and only after it will be removed as part of the Remove process. The detach operation should remove its storage pool metadata, and that is why the local Storage Domain should not be obligated to this validation. Change-Id: I9e4e603301a3ec893e6de5804a11d136dd1965ec Bug-Url: https://bugzilla.redhat.com/1263589 Signed-off-by: Maor Lipchuk <mlipchuk@redhat.com> --- M backend/manager/modules/bll/src/main/java/org/ovirt/engine/core/bll/storage/RemoveStorageDomainCommand.java 1 file changed, 2 insertions(+), 2 deletions(-) Approvals: Jenkins CI: Passed CI tests Allon Mureinik: Looks good to me, approved Maor Lipchuk: Verified -- To view, visit https://gerrit.ovirt.org/46295 To unsubscribe, visit https://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: I9e4e603301a3ec893e6de5804a11d136dd1965ec Gerrit-PatchSet: 5 Gerrit-Project: ovirt-engine Gerrit-Branch: ovirt-engine-3.6 Gerrit-Owner: Maor Lipchuk <mlipchuk@redhat.com> Gerrit-Reviewer: Allon Mureinik <amureini@redhat.com> Gerrit-Reviewer: Daniel Erez <derez@redhat.com> Gerrit-Reviewer: Jenkins CI Gerrit-Reviewer: Maor Lipchuk <mlipchuk@redhat.com> Gerrit-Reviewer: Tal Nisan <tnisan@redhat.com> Gerrit-Reviewer: automation@ovirt.org