Change in ovirt-engine[master]: core: Fix import of VMs with memory snapshot from domain

Code Review gerrit at ovirt.org
Sun Apr 23 22:41:31 UTC 2017


>From Tal Nisan <tnisan at redhat.com>:

Tal Nisan has submitted this change and it was merged.

Change subject: core: Fix import of VMs with memory snapshot from domain
......................................................................


core: Fix import of VMs with memory snapshot from domain

The stored procudure used to delete the entities from the storage domain
referred to a view that does not contain memory images resulting in those
remaining in the database after detaching a domain.
When trying to import a VM that contains memory snapshots to a setup
which the same domain was detached from that included those stale entities
it resulted in a unique constraint violation preventing the import of the
VM.

Change-Id: I6b30ca71d7ed7a390961cbc0ed32c03d6e8b8c01
Bug-Url: https://bugzilla.redhat.com/1434917
Signed-off-by: Tal Nisan <tnisan at redhat.com>
---
M packaging/dbscripts/storages_sp.sql
1 file changed, 4 insertions(+), 4 deletions(-)

Approvals:
  Tal Nisan: Verified
  Jenkins CI: Passed CI tests
  Allon Mureinik: Looks good to me, approved

Objections:
  Maor Lipchuk: I would prefer that you didn't submit this



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

Gerrit-MessageType: merged
Gerrit-Change-Id: I6b30ca71d7ed7a390961cbc0ed32c03d6e8b8c01
Gerrit-PatchSet: 2
Gerrit-Project: ovirt-engine
Gerrit-Branch: master
Gerrit-Owner: Tal Nisan <tnisan at redhat.com>
Gerrit-Reviewer: Allon Mureinik <amureini 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: gerrit-hooks <automation at ovirt.org>


More information about the Engine-commits mailing list