Change in ovirt-engine[ovirt-engine-3.6]: core: Support detach of local Storage Domain

tnisan at redhat.com tnisan at redhat.com
Sun Sep 20 12:07:15 UTC 2015


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 at 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 at redhat.com>
Gerrit-Reviewer: Allon Mureinik <amureini at redhat.com>
Gerrit-Reviewer: Daniel Erez <derez at redhat.com>
Gerrit-Reviewer: Jenkins CI
Gerrit-Reviewer: Maor Lipchuk <mlipchuk at redhat.com>
Gerrit-Reviewer: Tal Nisan <tnisan at redhat.com>
Gerrit-Reviewer: automation at ovirt.org



More information about the Engine-commits mailing list