Change in ovirt-engine[master]: core: Release mac pools only VMs with disks on removed SD.

Code Review gerrit at ovirt.org
Sun Aug 20 12:42:08 UTC 2017


>From Maor Lipchuk <mlipchuk at redhat.com>:

Maor Lipchuk has submitted this change and it was merged. ( https://gerrit.ovirt.org/80752 )

Change subject: core: Release mac pools only VMs with disks on removed SD.
......................................................................


core: Release mac pools only VMs with disks on removed SD.

The force remove of storage domain currently releases mac pools of VMs
which are related to the entire storage pool.
The fix is to release the mac pools only for VMs which have disks
which are reside the the removed storage domain.

Change-Id: Ic9bfb91ad7c1683111ad83e377217d8afe0c416c
Bug-Url: https://bugzilla.redhat.com/1482569
Signed-off-by: Maor Lipchuk <mlipchuk at redhat.com>
---
M backend/manager/modules/bll/src/main/java/org/ovirt/engine/core/bll/storage/domain/DetachStorageDomainFromPoolCommand.java
M backend/manager/modules/bll/src/main/java/org/ovirt/engine/core/bll/storage/domain/ForceRemoveStorageDomainCommand.java
M backend/manager/modules/bll/src/main/java/org/ovirt/engine/core/bll/storage/domain/StorageDomainCommandBase.java
3 files changed, 12 insertions(+), 12 deletions(-)

Approvals:
  Jenkins CI: Passed CI tests
  Allon Mureinik: Looks good to me, approved
  Maor Lipchuk: Verified



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

Gerrit-MessageType: merged
Gerrit-Change-Id: Ic9bfb91ad7c1683111ad83e377217d8afe0c416c
Gerrit-PatchSet: 4
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: gerrit-hooks <automation at ovirt.org>


More information about the Engine-commits mailing list