From jira at ovirt-jira.atlassian.net Mon Nov 20 14:07:50 2017 Content-Type: multipart/mixed; boundary="===============5115257895772490218==" MIME-Version: 1.0 From: Barak Korren (oVirt JIRA) To: infra at ovirt.org Subject: [JIRA] (OVIRT-1774) Upstream source collector can fail to push on multi-branch projects Date: Mon, 20 Nov 2017 14:07:49 +0000 Message-ID: In-Reply-To: JIRA.33307.1511186803000@Atlassian.JIRA --===============5115257895772490218== Content-Type: text/plain; charset="utf-8" MIME-Version: 1.0 Content-Transfer-Encoding: quoted-printable This is a multi-part message in MIME format... ------------=3D_1511186869-30492-188 Content-Type: text/plain; charset=3DUTF-8 Content-Transfer-Encoding: 7bit [ https://ovirt-jira.atlassian.net/browse/OVIRT-1774?page=3Dcom.atlass= ian.jira.plugin.system.issuetabpanels:all-tabpanel ] Barak Korren updated OVIRT-1774: -------------------------------- Epic Link: OVIRT-400 > Upstream source collector can fail to push on multi-branch projects > ------------------------------------------------------------------- > > Key: OVIRT-1774 > URL: https://ovirt-jira.atlassian.net/browse/OVIRT-1774 > Project: oVirt - virtualization made easy > Issue Type: Bug > Components: oVirt CI > Reporter: Barak Korren > Assignee: infra > Labels: poll-upstream-sources, upstream-source-collector > > When the upstream source collection code looks for similar patches in ord= er to avoid pushing a new patch, it lookes for patches that contain the sam= e change to the '{{upstream-sources.yaml}}' file, regardless of the branch = they may belong to. > The code currently ignores the possibility that similar changes might be = required for different branches, in which case different patches may need t= o be pushed. > The way of detecting similar patches should be changed so that either the= branch name is included in the checksum that are used to identify a patch,= or the query is limited to patches of the branch being handled. -- This message was sent by Atlassian Jira (v1001.0.0-SNAPSHOT#100072) ------------=3D_1511186869-30492-188 Content-Type: text/html; charset=3D"UTF-8" Content-Disposition: inline Content-Transfer-Encoding: 7bit
[ https://ovirt-jira.atlassian.net/browse/OVIRT-1774?page=3Dcom.atlass=
ian.jira.plugin.system.issuetabpanels:all-tabpanel ]

Barak Korren updated OVIRT-1774:

Epic Link: OVIRT-400

Upstream source collector can fail to push on multi-branch = projects

     Key: OVIRT-1774
     URL: https://ovirt-jira.atlassian.net/browse/OVIRT-1774
 Project: oVirt - virtualization made easy
         Issue Type: Bug
         Components: oVirt CI
Reporter: Barak Korren
Assignee: infra
  Labels: poll-upstream-sources, upstream-source-collector

When the upstream source collection code looks for similar patches in or= der to avoid pushing a new patch, it lookes for patches that contain the sa= me change to the ‘{{upstream-sources.yaml}}’ file, regardless o= f the branch they may belong to. The code currently ignores the possibility= that similar changes might be required for different branches, in which ca= se different patches may need to be pushed. The way of detecting similar pa= tches should be changed so that either the branch name is included in the c= hecksum that are used to identify a patch, or the query is limited to patch= es of the branch being handled.

— This message was sent by Atlassian Jira (v1001.0.0-SNAPSHOT#1000= 72)

3D"" ------------=3D_1511186869-30492-188-- --===============5115257895772490218== Content-Type: multipart/alternative MIME-Version: 1.0 Content-Transfer-Encoding: base64 Content-Disposition: attachment; filename="attachment.bin" VGhpcyBpcyBhIG11bHRpLXBhcnQgbWVzc2FnZSBpbiBNSU1FIGZvcm1hdC4uLgoKLS0tLS0tLS0t LS0tPV8xNTExMTg2ODY5LTMwNDkyLTE4OApDb250ZW50LVR5cGU6IHRleHQvcGxhaW47IGNoYXJz ZXQ9VVRGLTgKQ29udGVudC1UcmFuc2Zlci1FbmNvZGluZzogN2JpdAoKCiAgICAgWyBodHRwczov L292aXJ0LWppcmEuYXRsYXNzaWFuLm5ldC9icm93c2UvT1ZJUlQtMTc3ND9wYWdlPWNvbS5hdGxh c3NpYW4uamlyYS5wbHVnaW4uc3lzdGVtLmlzc3VldGFicGFuZWxzOmFsbC10YWJwYW5lbCBdCgpC YXJhayBLb3JyZW4gdXBkYXRlZCBPVklSVC0xNzc0OgotLS0tLS0tLS0tLS0tLS0tLS0tLS0tLS0t LS0tLS0tLQogICAgRXBpYyBMaW5rOiBPVklSVC00MDAKCj4gVXBzdHJlYW0gc291cmNlIGNvbGxl Y3RvciBjYW4gZmFpbCB0byBwdXNoIG9uIG11bHRpLWJyYW5jaCBwcm9qZWN0cwo+IC0tLS0tLS0t LS0tLS0tLS0tLS0tLS0tLS0tLS0tLS0tLS0tLS0tLS0tLS0tLS0tLS0tLS0tLS0tLS0tLS0tLS0t LS0KPgo+ICAgICAgICAgICAgICAgICBLZXk6IE9WSVJULTE3NzQKPiAgICAgICAgICAgICAgICAg VVJMOiBodHRwczovL292aXJ0LWppcmEuYXRsYXNzaWFuLm5ldC9icm93c2UvT1ZJUlQtMTc3NAo+ ICAgICAgICAgICAgIFByb2plY3Q6IG9WaXJ0IC0gdmlydHVhbGl6YXRpb24gbWFkZSBlYXN5Cj4g ICAgICAgICAgSXNzdWUgVHlwZTogQnVnCj4gICAgICAgICAgQ29tcG9uZW50czogb1ZpcnQgQ0kK PiAgICAgICAgICAgIFJlcG9ydGVyOiBCYXJhayBLb3JyZW4KPiAgICAgICAgICAgIEFzc2lnbmVl OiBpbmZyYQo+ICAgICAgICAgICAgICBMYWJlbHM6IHBvbGwtdXBzdHJlYW0tc291cmNlcywgdXBz dHJlYW0tc291cmNlLWNvbGxlY3Rvcgo+Cj4gV2hlbiB0aGUgdXBzdHJlYW0gc291cmNlIGNvbGxl Y3Rpb24gY29kZSBsb29rcyBmb3Igc2ltaWxhciBwYXRjaGVzIGluIG9yZGVyIHRvIGF2b2lkIHB1 c2hpbmcgYSBuZXcgcGF0Y2gsIGl0IGxvb2tlcyBmb3IgcGF0Y2hlcyB0aGF0IGNvbnRhaW4gdGhl IHNhbWUgY2hhbmdlIHRvIHRoZSAne3t1cHN0cmVhbS1zb3VyY2VzLnlhbWx9fScgZmlsZSwgcmVn YXJkbGVzcyBvZiB0aGUgYnJhbmNoIHRoZXkgbWF5IGJlbG9uZyB0by4KPiBUaGUgY29kZSBjdXJy ZW50bHkgaWdub3JlcyB0aGUgcG9zc2liaWxpdHkgdGhhdCBzaW1pbGFyIGNoYW5nZXMgbWlnaHQg YmUgcmVxdWlyZWQgZm9yIGRpZmZlcmVudCBicmFuY2hlcywgaW4gd2hpY2ggY2FzZSBkaWZmZXJl bnQgcGF0Y2hlcyBtYXkgbmVlZCB0byBiZSBwdXNoZWQuCj4gVGhlIHdheSBvZiBkZXRlY3Rpbmcg c2ltaWxhciBwYXRjaGVzIHNob3VsZCBiZSBjaGFuZ2VkIHNvIHRoYXQgZWl0aGVyIHRoZSBicmFu Y2ggbmFtZSBpcyBpbmNsdWRlZCBpbiB0aGUgY2hlY2tzdW0gdGhhdCBhcmUgdXNlZCB0byBpZGVu dGlmeSBhIHBhdGNoLCBvciB0aGUgcXVlcnkgaXMgbGltaXRlZCB0byBwYXRjaGVzIG9mIHRoZSBi cmFuY2ggYmVpbmcgaGFuZGxlZC4KCgoKLS0KVGhpcyBtZXNzYWdlIHdhcyBzZW50IGJ5IEF0bGFz c2lhbiBKaXJhCih2MTAwMS4wLjAtU05BUFNIT1QjMTAwMDcyKQoKLS0tLS0tLS0tLS0tPV8xNTEx MTg2ODY5LTMwNDkyLTE4OApDb250ZW50LVR5cGU6IHRleHQvaHRtbDsgY2hhcnNldD0iVVRGLTgi CkNvbnRlbnQtRGlzcG9zaXRpb246IGlubGluZQpDb250ZW50LVRyYW5zZmVyLUVuY29kaW5nOiA3 Yml0Cgo8aHRtbD48Ym9keT4KPHByZT5bIGh0dHBzOi8vb3ZpcnQtamlyYS5hdGxhc3NpYW4ubmV0 L2Jyb3dzZS9PVklSVC0xNzc0P3BhZ2U9Y29tLmF0bGFzc2lhbi5qaXJhLnBsdWdpbi5zeXN0ZW0u aXNzdWV0YWJwYW5lbHM6YWxsLXRhYnBhbmVsIF08L3ByZT4KPGgzPkJhcmFrIEtvcnJlbiB1cGRh dGVkIE9WSVJULTE3NzQ6PC9oMz4KPHByZT5FcGljIExpbms6IE9WSVJULTQwMDwvcHJlPgo8Ymxv Y2txdW90ZT48aDM+VXBzdHJlYW0gc291cmNlIGNvbGxlY3RvciBjYW4gZmFpbCB0byBwdXNoIG9u IG11bHRpLWJyYW5jaCBwcm9qZWN0czwvaDM+CjxwcmU+ICAgICBLZXk6IE9WSVJULTE3NzQKICAg ICBVUkw6IGh0dHBzOi8vb3ZpcnQtamlyYS5hdGxhc3NpYW4ubmV0L2Jyb3dzZS9PVklSVC0xNzc0 CiBQcm9qZWN0OiBvVmlydCAtIHZpcnR1YWxpemF0aW9uIG1hZGUgZWFzeQogICAgICAgICBJc3N1 ZSBUeXBlOiBCdWcKICAgICAgICAgQ29tcG9uZW50czogb1ZpcnQgQ0kKUmVwb3J0ZXI6IEJhcmFr IEtvcnJlbgpBc3NpZ25lZTogaW5mcmEKICBMYWJlbHM6IHBvbGwtdXBzdHJlYW0tc291cmNlcywg dXBzdHJlYW0tc291cmNlLWNvbGxlY3RvcjwvcHJlPgo8cD5XaGVuIHRoZSB1cHN0cmVhbSBzb3Vy Y2UgY29sbGVjdGlvbiBjb2RlIGxvb2tzIGZvciBzaW1pbGFyIHBhdGNoZXMgaW4gb3JkZXIgdG8g YXZvaWQgcHVzaGluZyBhIG5ldyBwYXRjaCwgaXQgbG9va2VzIGZvciBwYXRjaGVzIHRoYXQgY29u dGFpbiB0aGUgc2FtZSBjaGFuZ2UgdG8gdGhlICZsc3F1bzt7e3Vwc3RyZWFtLXNvdXJjZXMueWFt bH19JnJzcXVvOyBmaWxlLCByZWdhcmRsZXNzIG9mIHRoZSBicmFuY2ggdGhleSBtYXkgYmVsb25n IHRvLiBUaGUgY29kZSBjdXJyZW50bHkgaWdub3JlcyB0aGUgcG9zc2liaWxpdHkgdGhhdCBzaW1p bGFyIGNoYW5nZXMgbWlnaHQgYmUgcmVxdWlyZWQgZm9yIGRpZmZlcmVudCBicmFuY2hlcywgaW4g d2hpY2ggY2FzZSBkaWZmZXJlbnQgcGF0Y2hlcyBtYXkgbmVlZCB0byBiZSBwdXNoZWQuIFRoZSB3 YXkgb2YgZGV0ZWN0aW5nIHNpbWlsYXIgcGF0Y2hlcyBzaG91bGQgYmUgY2hhbmdlZCBzbyB0aGF0 IGVpdGhlciB0aGUgYnJhbmNoIG5hbWUgaXMgaW5jbHVkZWQgaW4gdGhlIGNoZWNrc3VtIHRoYXQg YXJlIHVzZWQgdG8gaWRlbnRpZnkgYSBwYXRjaCwgb3IgdGhlIHF1ZXJ5IGlzIGxpbWl0ZWQgdG8g cGF0Y2hlcyBvZiB0aGUgYnJhbmNoIGJlaW5nIGhhbmRsZWQuPC9wPjwvYmxvY2txdW90ZT4KPHA+ Jm1kYXNoOyBUaGlzIG1lc3NhZ2Ugd2FzIHNlbnQgYnkgQXRsYXNzaWFuIEppcmEgKHYxMDAxLjAu MC1TTkFQU0hPVCMxMDAwNzIpPC9wPgoKPGltZyBzcmM9Imh0dHBzOi8vdTQwNDM0MDIuY3Quc2Vu ZGdyaWQubmV0L3dmL29wZW4/dXBuPWk1VE1XR1Y5OWFtSmJOeEpwU3AyLTJCQ21wWUx5ellTM3Ix UTkzOVQyc2d6M0tYMEdvQUl3dWtpM3B2UWxQLTJGME9sckZ3Z1c5NkMyckZlLTJCdzhWaHJweko0 N3BkN2Y2dHctMkJpNXFvcVF2WUQycDFQU21DOUZyeU9KMVVWaUpra2xWQ09OZ2QxWjN0M0VVZzRl ZXRndjY3clJ0eFJEMUpxWVlaaEZqU2FQMUNVd3N2UmVZMGxJRGxiMnJKZjJib25RT0hMcmpHMnY4 RHBqQ2JEaHhzdWIzLTJGNU10V20ySW5UekhBWjNDMG5LdDlKVnIwLTNEIiBhbHQ9IiIgd2lkdGg9 IjEiIGhlaWdodD0iMSIgYm9yZGVyPSIwIiBzdHlsZT0iaGVpZ2h0OjFweCAhaW1wb3J0YW50O3dp ZHRoOjFweCAhaW1wb3J0YW50O2JvcmRlci13aWR0aDowICFpbXBvcnRhbnQ7bWFyZ2luLXRvcDow ICFpbXBvcnRhbnQ7bWFyZ2luLWJvdHRvbTowICFpbXBvcnRhbnQ7bWFyZ2luLXJpZ2h0OjAgIWlt cG9ydGFudDttYXJnaW4tbGVmdDowICFpbXBvcnRhbnQ7cGFkZGluZy10b3A6MCAhaW1wb3J0YW50 O3BhZGRpbmctYm90dG9tOjAgIWltcG9ydGFudDtwYWRkaW5nLXJpZ2h0OjAgIWltcG9ydGFudDtw YWRkaW5nLWxlZnQ6MCAhaW1wb3J0YW50OyIvPgo8L2JvZHk+PC9odG1sPgoKLS0tLS0tLS0tLS0t PV8xNTExMTg2ODY5LTMwNDkyLTE4OC0tCg== --===============5115257895772490218==--