Change in ovirt-engine[master]: core: fix vm pools behavior when DC is missing

rgolan at redhat.com rgolan at redhat.com
Mon Feb 10 13:10:22 UTC 2014


Roy Golan has submitted this change and it was merged.

Change subject: core: fix vm pools behavior when DC is missing
......................................................................


core: fix vm pools behavior when DC is missing

When DC is missing (for example force-removed)
then if any vm pools still exist, they don't return in queries, because
of bad join in the view.
 this was fixed by changing the vm_pools_view to use left-join with
storage-pools

in addition, this should not happen, as empty vm pools are not
supported.
but when using force-remove DC, all vms are deleted, but vm-pools are
left.
 this was fixed by changing the stored procedure for remove storage
pool, to remove vm-pools as well, after removing vms.

Change-Id: I47c963b6b66efa05e96cb9b9f80e034d8070a30b
Bug-Url: https://bugzilla.redhat.com/show_bug.cgi?id=1031717
Signed-off-by: Omer Frenkel <ofrenkel at redhat.com>
---
M packaging/dbscripts/create_views.sql
M packaging/dbscripts/storages_sp.sql
2 files changed, 11 insertions(+), 1 deletion(-)

Approvals:
  Eli Mesika: Looks good to me, but someone else must approve
  Roy Golan: Verified; Looks good to me, approved
  Omer Frenkel: Verified
  Arik Hadas: Looks good to me, but someone else must approve



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

Gerrit-MessageType: merged
Gerrit-Change-Id: I47c963b6b66efa05e96cb9b9f80e034d8070a30b
Gerrit-PatchSet: 2
Gerrit-Project: ovirt-engine
Gerrit-Branch: master
Gerrit-Owner: Omer Frenkel <ofrenkel at redhat.com>
Gerrit-Reviewer: Allon Mureinik <amureini at redhat.com>
Gerrit-Reviewer: Arik Hadas <ahadas at redhat.com>
Gerrit-Reviewer: Eli Mesika <emesika at redhat.com>
Gerrit-Reviewer: Omer Frenkel <ofrenkel at redhat.com>
Gerrit-Reviewer: Roy Golan <rgolan at redhat.com>



More information about the Engine-commits mailing list