Change in ovirt-engine[master]: core: Don't erroneously remove template->storage mappings

mlipchuk at redhat.com mlipchuk at redhat.com
Sun Sep 2 11:31:56 UTC 2012


Maor Lipchuk has submitted this change and it was merged.

Change subject: core: Don't erroneously remove template->storage mappings
......................................................................


core: Don't erroneously remove template->storage mappings

If a storage domain is force removed and a template has a disk on that
SD, the image is removed regardless of whether that image also resides
on other SDs.  In the case where the template disk has been copied, the
template and images should remain intact and only the references for the
specific SD being removed should be deleted.

Change-Id: I058d160622990d2b42ccc02a61551c6f61550ade
Signed-off-by: Greg Padgett <gpadgett at redhat.com>
---
M backend/manager/dbscripts/storages_sp.sql
1 file changed, 9 insertions(+), 3 deletions(-)

Approvals:
  Maor Lipchuk: Verified; Looks good to me, approved


--
To view, visit http://gerrit.ovirt.org/7192
To unsubscribe, visit http://gerrit.ovirt.org/settings

Gerrit-MessageType: merged
Gerrit-Change-Id: I058d160622990d2b42ccc02a61551c6f61550ade
Gerrit-PatchSet: 3
Gerrit-Project: ovirt-engine
Gerrit-Branch: master
Gerrit-Owner: Greg Padgett <gpadgett at redhat.com>
Gerrit-Reviewer: Allon Mureinik <amureini at redhat.com>
Gerrit-Reviewer: Ayal Baron <abaron at redhat.com>
Gerrit-Reviewer: Eli Mesika <emesika at redhat.com>
Gerrit-Reviewer: Greg Padgett <gpadgett at redhat.com>
Gerrit-Reviewer: Maor Lipchuk <mlipchuk at redhat.com>
Gerrit-Reviewer: Omer Frenkel <ofrenkel at redhat.com>
Gerrit-Reviewer: oVirt Jenkins CI Server



More information about the Engine-commits mailing list