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

mlipchuk at redhat.com mlipchuk at redhat.com
Thu Sep 17 12:30:57 UTC 2015


Maor Lipchuk 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:
  Daniel Erez: Looks good to me, approved
  Jenkins CI: Passed CI tests
  Maor Lipchuk: Verified



-- 
To view, visit https://gerrit.ovirt.org/46216
To unsubscribe, visit https://gerrit.ovirt.org/settings

Gerrit-MessageType: merged
Gerrit-Change-Id: I9e4e603301a3ec893e6de5804a11d136dd1965ec
Gerrit-PatchSet: 2
Gerrit-Project: ovirt-engine
Gerrit-Branch: master
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: automation at ovirt.org



More information about the Engine-commits mailing list