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@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@redhat.com> Gerrit-Reviewer: Liron Aravot <laravot@redhat.com> Gerrit-Reviewer: Maor Lipchuk <mlipchuk@redhat.com> Gerrit-Reviewer: Tal Nisan <tnisan@redhat.com>