On 08/01/2016 11:02 AM, Yedidyah Bar David wrote: > On Sun, Jul 31, 2016 at 6:35 PM, Jenkins CI <gerrit2(a)gerrit.ovirt.org&gt; wrote: >> Jenkins CI has posted comments on this change. >> >> Change subject: build: ovirt-engine-4.0.2.3 >> ...................................................................... >> >> >> Patch Set 2: >> >> Build Failed >> >> http://jenkins.ovirt.org/job/ovirt-engine_4.0.2_check-merged-el7-x86_64/64/ : FAILURE > > 15:07:43 Running upgrade sql script > './packaging/dbscripts/upgrade/03_05_0580_add_default_instance_types.sql'... > 15:07:43 psql:./packaging/dbscripts/upgrade/03_05_0580_add_default_instance_types.sql:131: > ERROR: duplicate key value violates unique constraint > "pk_permissions_id" > 15:07:43 DETAIL: Key (id)=() already exists. > > Didn't try reproducing, but my best guess is that above uuid > was generated by a call to uuid_generate_v1() in above file after it > was already inserted to the table in > dbscripts/data/00600_insert_permissions.sql. > > Perhaps we should be using something more random. > > [1] https://www.postgresql.org/docs/9.5/static/uuid-ossp.html > Note that we did use that long time ago, and we decided to stop using it because enabling that extension required an additional RPM package (postgrersql-contrib) and different mechanisms to enable it in PostgreSQL 8 and PostgreSQL 9, which was cumbersome. As we should now support only PostgreSQL 9.2 or newer, this may be no longer a problem. In that case you can just remove our "uuid_generate_v1" function and enable the extension, the name of the function is the same. However, it would be better, in general, to generate the identifiers outside of the database, and pass them as parameters to the stored procedures or SQL statements, that way we have less dependencies. See bug 870056 for additional information. >> >> http://jenkins.ovirt.org/job/ovirt-engine_4.0.2_check-merged-fc23-x86_64/64/ : SUCCESS >> >> -- >> To view, visit https://gerrit.ovirt.org/61733 >> To unsubscribe, visit https://gerrit.ovirt.org/settings >> >> Gerrit-MessageType: comment >> Gerrit-Change-Id: Ice9518eea22dfa19126e04e49f3a76ee86f3e3c0 >> Gerrit-PatchSet: 2 >> Gerrit-Project: ovirt-engine >> Gerrit-Branch: ovirt-engine-4.0.2 >> Gerrit-Owner: Yedidyah Bar David <didi(a)redhat.com&gt; >> Gerrit-Reviewer: Eyal Edri <eedri(a)redhat.com&gt; >> Gerrit-Reviewer: Jenkins CI >> Gerrit-Reviewer: Oved Ourfali <oourfali(a)redhat.com&gt; >> Gerrit-Reviewer: Yedidyah Bar David <didi(a)redhat.com&gt; >> Gerrit-Reviewer: gerrit-hooks <automation(a)ovirt.org&gt; >> Gerrit-HasComments: No > > > -- Dirección Comercial: C/Jose Bardasano Baos, 9, Edif. Gorbea 3, planta 3ºD, 28016 Madrid, Spain Inscrita en el Reg. Mercantil de Madrid – C.I.F. B82657941 - Red Hat S.L.