Change in ovirt-engine[master]: core: fixes to import/OvfDataUpdater issues

[tnisan at redhat.com]

Tal Nisan has submitted this change and it was merged.

Change subject: core: fixes to import/OvfDataUpdater issues
......................................................................


core: fixes to import/OvfDataUpdater issues

The following patch fixes a case in which we attempt to insert a record
to vm_ovf_generations table when we already have a record for the same
guid, which will cause us to pk violation error.
This issue currently occurs in import vm/template flows when we attempt
to use
the same id as exported and the record for that key in
vm_ovf_generations table hasn't been deleted yet due to failure
during the import (compensation doesn't remove record from the ovf
table) or because the vm/template has been removed from the
DB, but not from the storage side by ovf data updater.

Change-Id: I272b39f99d582462c0d1ea4cf268edd7e5c42b6d
Bug-Url: https://bugzilla.redhat.com/show_bug.cgi?id=890951
Signed-off-by: Liron Aravot <laravot at redhat.com>
---
M backend/manager/dbscripts/vm_templates_sp.sql
M backend/manager/dbscripts/vms_sp.sql
2 files changed, 7 insertions(+), 1 deletion(-)

Approvals:
  Tal Nisan: Verified; Looks good to me, approved


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

Gerrit-MessageType: merged
Gerrit-Change-Id: I272b39f99d582462c0d1ea4cf268edd7e5c42b6d
Gerrit-PatchSet: 5
Gerrit-Project: ovirt-engine
Gerrit-Branch: master
Gerrit-Owner: Liron Aravot <laravot at redhat.com>
Gerrit-Reviewer: Liron Aravot <laravot at redhat.com>
Gerrit-Reviewer: Maor Lipchuk <mlipchuk at redhat.com>
Gerrit-Reviewer: Tal Nisan <tnisan at redhat.com>