From jira at ovirt-jira.atlassian.net Mon Jan 1 15:04:30 2018 Content-Type: multipart/mixed; boundary="===============2181133240087531301==" MIME-Version: 1.0 From: Barak Korren (oVirt JIRA) To: infra at ovirt.org Subject: [JIRA] (OVIRT-1821) jenkins 'check-patch.sh' does not always generate differences.html in downstream repos Date: Mon, 01 Jan 2018 15:04:19 +0000 Message-ID: In-Reply-To: JIRA.33368.1514819058000@Atlassian.JIRA --===============2181133240087531301== 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_1514819059-25953-112 Content-Type: text/plain; charset=3DUTF-8 Content-Transfer-Encoding: 7bit [ https://ovirt-jira.atlassian.net/browse/OVIRT-1821?page=3Dcom.atlass= ian.jira.plugin.system.issuetabpanels:all-tabpanel ] Barak Korren updated OVIRT-1821: -------------------------------- Epic Link: OVIRT-400 > jenkins 'check-patch.sh' does not always generate differences.html in dow= nstream repos > -------------------------------------------------------------------------= ------------- > > Key: OVIRT-1821 > URL: https://ovirt-jira.atlassian.net/browse/OVIRT-1821 > Project: oVirt - virtualization made easy > Issue Type: Bug > Reporter: Barak Korren > Assignee: infra > Labels: standard-ci > > The 'check-patch.sh' script in the 'jenkins' repo knows how to generate a= 'differences.html' file when job YAML changes. > The 'jenkins' repo is designed to be used as an upstream source repo for = other repos, and it is designed that 'check-patch.sh' will be usable on tho= se DS repos. > When YAML changes from the 'jenkins' repo make it into the DS repos via = a patch to the 'upstream-sources.yaml' file in the DS repo, and 'check-patc= h.sh' runs on that repo, 'differences.html' is not being generated, because= the change to 'upstream-sources.yaml' does not look like a job YAML change. > We need to change 'check-patch.sh' so that when running in a DS repo, it= knows how to look and diagnose changes coming from the upstream repo. -- This message was sent by Atlassian Jira (v1001.0.0-SNAPSHOT#100075) ------------=3D_1514819059-25953-112 Content-Type: text/html; charset=3D"UTF-8" Content-Disposition: inline Content-Transfer-Encoding: 7bit 
[ https://ovirt-jira.atlassian.net/browse/OVIRT-1821?page=3Dcom.atlass=
ian.jira.plugin.system.issuetabpanels:all-tabpanel ]

Barak Korren updated OVIRT-1821:

Epic Link: OVIRT-400

jenkins ‘check-patch.sh’ does not always genera= te differences.html in downstream repos

     Key: OVIRT-1821
     URL: https://ovirt-jira.atlassian.net/browse/OVIRT-1821
 Project: oVirt - virtualization made easy
         Issue Type: Bug
Reporter: Barak Korren
Assignee: infra
  Labels: standard-ci

The ‘check-patch.sh’ script in the ‘jenkins’ rep= o knows how to generate a ‘differences.html’ file when job YAML= changes. The ‘jenkins’ repo is designed to be used as an upstr= eam source repo for other repos, and it is designed that ‘check-patch= .sh’ will be usable on those DS repos. When YAML changes from the &ls= quo;jenkins’ repo make it into the DS repos via a patch to the &lsqu= o;upstream-sources.yaml’ file in the DS repo, and ‘check-patch.= sh’ runs on that repo, ‘differences.html’ is not being ge= nerated, because the change to ‘upstream-sources.yaml’ does not= look like a job YAML change. We need to change ‘check-patch.sh&rsqu= o; so that when running in a DS repo, it knows how to look and diagnose cha= nges coming from the upstream repo.

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

3D"" ------------=3D_1514819059-25953-112-- --===============2181133240087531301== Content-Type: multipart/alternative MIME-Version: 1.0 Content-Transfer-Encoding: base64 Content-Disposition: attachment; filename="attachment.bin" VGhpcyBpcyBhIG11bHRpLXBhcnQgbWVzc2FnZSBpbiBNSU1FIGZvcm1hdC4uLgoKLS0tLS0tLS0t LS0tPV8xNTE0ODE5MDU5LTI1OTUzLTExMgpDb250ZW50LVR5cGU6IHRleHQvcGxhaW47IGNoYXJz ZXQ9VVRGLTgKQ29udGVudC1UcmFuc2Zlci1FbmNvZGluZzogN2JpdAoKCiAgICAgWyBodHRwczov L292aXJ0LWppcmEuYXRsYXNzaWFuLm5ldC9icm93c2UvT1ZJUlQtMTgyMT9wYWdlPWNvbS5hdGxh c3NpYW4uamlyYS5wbHVnaW4uc3lzdGVtLmlzc3VldGFicGFuZWxzOmFsbC10YWJwYW5lbCBdCgpC YXJhayBLb3JyZW4gdXBkYXRlZCBPVklSVC0xODIxOgotLS0tLS0tLS0tLS0tLS0tLS0tLS0tLS0t LS0tLS0tLQogICAgRXBpYyBMaW5rOiBPVklSVC00MDAKCj4gamVua2lucyAnY2hlY2stcGF0Y2gu c2gnIGRvZXMgbm90IGFsd2F5cyBnZW5lcmF0ZSBkaWZmZXJlbmNlcy5odG1sIGluIGRvd25zdHJl YW0gcmVwb3MKPiAtLS0tLS0tLS0tLS0tLS0tLS0tLS0tLS0tLS0tLS0tLS0tLS0tLS0tLS0tLS0t LS0tLS0tLS0tLS0tLS0tLS0tLS0tLS0tLS0tLS0tLS0tLS0tLS0tLQo+Cj4gICAgICAgICAgICAg ICAgIEtleTogT1ZJUlQtMTgyMQo+ICAgICAgICAgICAgICAgICBVUkw6IGh0dHBzOi8vb3ZpcnQt amlyYS5hdGxhc3NpYW4ubmV0L2Jyb3dzZS9PVklSVC0xODIxCj4gICAgICAgICAgICAgUHJvamVj dDogb1ZpcnQgLSB2aXJ0dWFsaXphdGlvbiBtYWRlIGVhc3kKPiAgICAgICAgICBJc3N1ZSBUeXBl OiBCdWcKPiAgICAgICAgICAgIFJlcG9ydGVyOiBCYXJhayBLb3JyZW4KPiAgICAgICAgICAgIEFz c2lnbmVlOiBpbmZyYQo+ICAgICAgICAgICAgICBMYWJlbHM6IHN0YW5kYXJkLWNpCj4KPiBUaGUg J2NoZWNrLXBhdGNoLnNoJyBzY3JpcHQgaW4gdGhlICdqZW5raW5zJyByZXBvIGtub3dzIGhvdyB0 byBnZW5lcmF0ZSBhICdkaWZmZXJlbmNlcy5odG1sJyBmaWxlIHdoZW4gam9iIFlBTUwgY2hhbmdl cy4KPiBUaGUgJ2plbmtpbnMnIHJlcG8gaXMgZGVzaWduZWQgdG8gYmUgdXNlZCBhcyBhbiB1cHN0 cmVhbSBzb3VyY2UgcmVwbyBmb3Igb3RoZXIgcmVwb3MsIGFuZCBpdCBpcyBkZXNpZ25lZCB0aGF0 ICdjaGVjay1wYXRjaC5zaCcgd2lsbCBiZSB1c2FibGUgb24gdGhvc2UgRFMgcmVwb3MuCj4gV2hl biBZQU1MIGNoYW5nZXMgZnJvbSB0aGUgJ2plbmtpbnMnIHJlcG8gIG1ha2UgaXQgaW50byB0aGUg RFMgcmVwb3MgdmlhIGEgcGF0Y2ggdG8gdGhlICd1cHN0cmVhbS1zb3VyY2VzLnlhbWwnIGZpbGUg aW4gdGhlIERTIHJlcG8sIGFuZCAnY2hlY2stcGF0Y2guc2gnIHJ1bnMgb24gdGhhdCByZXBvLCAn ZGlmZmVyZW5jZXMuaHRtbCcgaXMgbm90IGJlaW5nIGdlbmVyYXRlZCwgYmVjYXVzZSB0aGUgY2hh bmdlIHRvICd1cHN0cmVhbS1zb3VyY2VzLnlhbWwnIGRvZXMgbm90IGxvb2sgbGlrZSBhIGpvYiBZ QU1MIGNoYW5nZS4KPiBXZSBuZWVkIHRvICBjaGFuZ2UgJ2NoZWNrLXBhdGNoLnNoJyBzbyB0aGF0 IHdoZW4gcnVubmluZyBpbiBhIERTIHJlcG8sIGl0IGtub3dzIGhvdyB0byBsb29rIGFuZCBkaWFn bm9zZSBjaGFuZ2VzIGNvbWluZyBmcm9tIHRoZSB1cHN0cmVhbSByZXBvLgoKCgotLQpUaGlzIG1l c3NhZ2Ugd2FzIHNlbnQgYnkgQXRsYXNzaWFuIEppcmEKKHYxMDAxLjAuMC1TTkFQU0hPVCMxMDAw NzUpCgotLS0tLS0tLS0tLS09XzE1MTQ4MTkwNTktMjU5NTMtMTEyCkNvbnRlbnQtVHlwZTogdGV4 dC9odG1sOyBjaGFyc2V0PSJVVEYtOCIKQ29udGVudC1EaXNwb3NpdGlvbjogaW5saW5lCkNvbnRl bnQtVHJhbnNmZXItRW5jb2Rpbmc6IDdiaXQKCjxodG1sPjxib2R5Pgo8cHJlPlsgaHR0cHM6Ly9v dmlydC1qaXJhLmF0bGFzc2lhbi5uZXQvYnJvd3NlL09WSVJULTE4MjE/cGFnZT1jb20uYXRsYXNz aWFuLmppcmEucGx1Z2luLnN5c3RlbS5pc3N1ZXRhYnBhbmVsczphbGwtdGFicGFuZWwgXTwvcHJl Pgo8aDM+QmFyYWsgS29ycmVuIHVwZGF0ZWQgT1ZJUlQtMTgyMTo8L2gzPgo8cHJlPkVwaWMgTGlu azogT1ZJUlQtNDAwPC9wcmU+CjxibG9ja3F1b3RlPjxoMz5qZW5raW5zICZsc3F1bztjaGVjay1w YXRjaC5zaCZyc3F1bzsgZG9lcyBub3QgYWx3YXlzIGdlbmVyYXRlIGRpZmZlcmVuY2VzLmh0bWwg aW4gZG93bnN0cmVhbSByZXBvczwvaDM+CjxwcmU+ICAgICBLZXk6IE9WSVJULTE4MjEKICAgICBV Ukw6IGh0dHBzOi8vb3ZpcnQtamlyYS5hdGxhc3NpYW4ubmV0L2Jyb3dzZS9PVklSVC0xODIxCiBQ cm9qZWN0OiBvVmlydCAtIHZpcnR1YWxpemF0aW9uIG1hZGUgZWFzeQogICAgICAgICBJc3N1ZSBU eXBlOiBCdWcKUmVwb3J0ZXI6IEJhcmFrIEtvcnJlbgpBc3NpZ25lZTogaW5mcmEKICBMYWJlbHM6 IHN0YW5kYXJkLWNpPC9wcmU+CjxwPlRoZSAmbHNxdW87Y2hlY2stcGF0Y2guc2gmcnNxdW87IHNj cmlwdCBpbiB0aGUgJmxzcXVvO2plbmtpbnMmcnNxdW87IHJlcG8ga25vd3MgaG93IHRvIGdlbmVy YXRlIGEgJmxzcXVvO2RpZmZlcmVuY2VzLmh0bWwmcnNxdW87IGZpbGUgd2hlbiBqb2IgWUFNTCBj aGFuZ2VzLiBUaGUgJmxzcXVvO2plbmtpbnMmcnNxdW87IHJlcG8gaXMgZGVzaWduZWQgdG8gYmUg dXNlZCBhcyBhbiB1cHN0cmVhbSBzb3VyY2UgcmVwbyBmb3Igb3RoZXIgcmVwb3MsIGFuZCBpdCBp cyBkZXNpZ25lZCB0aGF0ICZsc3F1bztjaGVjay1wYXRjaC5zaCZyc3F1bzsgd2lsbCBiZSB1c2Fi bGUgb24gdGhvc2UgRFMgcmVwb3MuIFdoZW4gWUFNTCBjaGFuZ2VzIGZyb20gdGhlICZsc3F1bztq ZW5raW5zJnJzcXVvOyByZXBvICBtYWtlIGl0IGludG8gdGhlIERTIHJlcG9zIHZpYSBhIHBhdGNo IHRvIHRoZSAmbHNxdW87dXBzdHJlYW0tc291cmNlcy55YW1sJnJzcXVvOyBmaWxlIGluIHRoZSBE UyByZXBvLCBhbmQgJmxzcXVvO2NoZWNrLXBhdGNoLnNoJnJzcXVvOyBydW5zIG9uIHRoYXQgcmVw bywgJmxzcXVvO2RpZmZlcmVuY2VzLmh0bWwmcnNxdW87IGlzIG5vdCBiZWluZyBnZW5lcmF0ZWQs IGJlY2F1c2UgdGhlIGNoYW5nZSB0byAmbHNxdW87dXBzdHJlYW0tc291cmNlcy55YW1sJnJzcXVv OyBkb2VzIG5vdCBsb29rIGxpa2UgYSBqb2IgWUFNTCBjaGFuZ2UuIFdlIG5lZWQgdG8gIGNoYW5n ZSAmbHNxdW87Y2hlY2stcGF0Y2guc2gmcnNxdW87IHNvIHRoYXQgd2hlbiBydW5uaW5nIGluIGEg RFMgcmVwbywgaXQga25vd3MgaG93IHRvIGxvb2sgYW5kIGRpYWdub3NlIGNoYW5nZXMgY29taW5n IGZyb20gdGhlIHVwc3RyZWFtIHJlcG8uPC9wPjwvYmxvY2txdW90ZT4KPHA+Jm1kYXNoOyBUaGlz IG1lc3NhZ2Ugd2FzIHNlbnQgYnkgQXRsYXNzaWFuIEppcmEgKHYxMDAxLjAuMC1TTkFQU0hPVCMx MDAwNzUpPC9wPgoKPGltZyBzcmM9Imh0dHBzOi8vdTQwNDM0MDIuY3Quc2VuZGdyaWQubmV0L3dm L29wZW4/dXBuPWk1VE1XR1Y5OWFtSmJOeEpwU3AyLTJCSjMzQlNNM3R1aVVmUlRrNjRLLTJCT2pH cEYtMkJ1TXpvSlJSQjFpZnpaSUVydEl4VGNjTHk1MjF6ejdPeFpWaUI1bW5wYmJ3WUpGejZmbGdl dlhPelVKVkgtMkZxQnF2Y3FxcUtJcDNwNE9jeURGT3NSd3ZhTGsxcjdYOEpwTEViYll0US0yRjhz ZS0yRkFvRzNOcU15Nk1FRjk2MGZNNFdDZkFySVBpcERBeFYzSTlRdER4d0laa2VOV0Q5eUFwUHNk SnJhM1Y0TlQ3aElPb1RpZjQ2aE45QTJOYS0yQklzZUhuZ1NTT3pyU1BvODdYczVkbFRPcHhpaXlU UzlCT2pVM1l2QjkyNUdMS21XR0I1NG9aeDRLcmhYclJjR2NuV1NXeUU2N1FGbUtwQVBtdU1pdTB5 QVp4S0VKY1lMSGZ6Vkg0YmRJM1g5Q1duOVotMkZCM2cwUmlIYzgxNEFVZFVhYXUzV2FMMEVrNEw4 cFhsTGFMbzI4LTJCanNzblU2eDhYeHdGd1UtMkZIYzNUblZqazB5LTJGRmw3SXpvbGRZM2E2Mkhi dEtzTlJxcm5tczJRTTEzUlNRNmRwWU1VIiBhbHQ9IiIgd2lkdGg9IjEiIGhlaWdodD0iMSIgYm9y ZGVyPSIwIiBzdHlsZT0iaGVpZ2h0OjFweCAhaW1wb3J0YW50O3dpZHRoOjFweCAhaW1wb3J0YW50 O2JvcmRlci13aWR0aDowICFpbXBvcnRhbnQ7bWFyZ2luLXRvcDowICFpbXBvcnRhbnQ7bWFyZ2lu LWJvdHRvbTowICFpbXBvcnRhbnQ7bWFyZ2luLXJpZ2h0OjAgIWltcG9ydGFudDttYXJnaW4tbGVm dDowICFpbXBvcnRhbnQ7cGFkZGluZy10b3A6MCAhaW1wb3J0YW50O3BhZGRpbmctYm90dG9tOjAg IWltcG9ydGFudDtwYWRkaW5nLXJpZ2h0OjAgIWltcG9ydGFudDtwYWRkaW5nLWxlZnQ6MCAhaW1w b3J0YW50OyIvPgo8L2JvZHk+PC9odG1sPgoKLS0tLS0tLS0tLS0tPV8xNTE0ODE5MDU5LTI1OTUz LTExMi0tCg== --===============2181133240087531301==--