From jira at ovirt-jira.atlassian.net Sun Feb 25 15:41:27 2018 Content-Type: multipart/mixed; boundary="===============2989541375534173939==" MIME-Version: 1.0 From: eyal edri (oVirt JIRA) To: infra at ovirt.org Subject: [JIRA] (OVIRT-1911) RFE: Add relevant git commit messages as comments in the relevant Bugzillas Date: Sun, 25 Feb 2018 15:41:24 +0000 Message-ID: In-Reply-To: JIRA.33718.1519571949000@Atlassian.JIRA --===============2989541375534173939== 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_1519573284-30715-140 Content-Type: text/plain; charset=3DUTF-8 Content-Transfer-Encoding: 7bit [ https://ovirt-jira.atlassian.net/browse/OVIRT-1911?page=3Dcom.atlass= ian.jira.plugin.system.issuetabpanels:all-tabpanel ] eyal edri reassigned OVIRT-1911: -------------------------------- Assignee: Anton Marchukov (was: infra) Anton, how much effort do you think this will need to add it to Gerrit Hook= s? = [~ykaul] I also think this should be communicated before adding, people som= etimes don't like new automatic info added to bugs without a good reason. We'll also need to think what do you do in case there are multiple external= trackers attached to the bug, which is the case most of the time, do you s= tart "spamming" the bug with commit messages from each tracker? sometimes i= t can be as much as 10 different patches. = What about if there are trackers from multiple projects, a quite common sce= nario with oVirt bugs (e.g vdsm + engine + jsonrpc) , do you add all the co= mmit messages then? = We've seen issues with current new comments added to bugs when they can't m= ove to ON_QA, it has the potential of causing more noise than good, so we r= eally need to think if and how we want to implement this before moving forw= ard. cc [~dfodor] > RFE: Add relevant git commit messages as comments in the relevant Bugzill= as > -------------------------------------------------------------------------= -- > > Key: OVIRT-1911 > URL: https://ovirt-jira.atlassian.net/browse/OVIRT-1911 > Project: oVirt - virtualization made easy > Issue Type: New Feature > Components: Gerrit/git > Reporter: Yaniv Kaul > Assignee: Anton Marchukov > > When a bug moves to MODIFIED, can we add the relevant commit message(s) t= o the bug? > I see it happening in RHEL from time to time, and I think it's easier (fo= r us and QE) then clicking on the link to Gerrit. -- This message was sent by Atlassian Jira (v1001.0.0-SNAPSHOT#100080) ------------=3D_1519573284-30715-140 Content-Type: text/html; charset=3D"UTF-8" Content-Disposition: inline Content-Transfer-Encoding: 7bit 
[ https://ovirt-jira.atlassian.net/browse/OVIRT-1911?page=3Dcom.atlass=
ian.jira.plugin.system.issuetabpanels:all-tabpanel ]

eyal edri reassigned OVIRT-1911:

Assignee: Anton Marchukov  (was: infra)

Anton, how much effort do you think this will need to add it to Gerrit H= ooks?

[~ykaul] I also think this should be communicated before adding, people = sometimes don't like new automatic info added to bugs without a good reason= .

We'll also need to think what do you do in case there are multiple exter= nal trackers attached to the bug, which is the case most of the time, do yo= u start “spamming” the bug with commit messages from each track= er? sometimes it can be as much as 10 different patches.

What about if there are trackers from multiple projects, a quite common = scenario with oVirt bugs (e.g vdsm + engine + jsonrpc) , do you add all the= commit messages then?

We've seen issues with current new comments added to bugs when they can'= t move to ON_QA, it has the potential of causing more noise than good, so w= e really need to think if and how we want to implement this before moving f= orward.

cc [~dfodor]

RFE: Add relevant git commit messages as comments in the re= levant Bugzillas

     Key: OVIRT-1911
     URL: https://ovirt-jira.atlassian.net/browse/OVIRT-1911
 Project: oVirt - virtualization made easy
         Issue Type: New Feature
         Components: Gerrit/git
Reporter: Yaniv Kaul
Assignee: Anton Marchukov

When a bug moves to MODIFIED, can we add the relevant commit message(s) = to the bug? I see it happening in RHEL from time to time, and I think it's = easier (for us and QE) then clicking on the link to Gerrit.

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

3D"" ------------=3D_1519573284-30715-140-- --===============2989541375534173939== Content-Type: multipart/alternative MIME-Version: 1.0 Content-Transfer-Encoding: base64 Content-Disposition: attachment; filename="attachment.bin" VGhpcyBpcyBhIG11bHRpLXBhcnQgbWVzc2FnZSBpbiBNSU1FIGZvcm1hdC4uLgoKLS0tLS0tLS0t LS0tPV8xNTE5NTczMjg0LTMwNzE1LTE0MApDb250ZW50LVR5cGU6IHRleHQvcGxhaW47IGNoYXJz ZXQ9VVRGLTgKQ29udGVudC1UcmFuc2Zlci1FbmNvZGluZzogN2JpdAoKCiAgICAgWyBodHRwczov L292aXJ0LWppcmEuYXRsYXNzaWFuLm5ldC9icm93c2UvT1ZJUlQtMTkxMT9wYWdlPWNvbS5hdGxh c3NpYW4uamlyYS5wbHVnaW4uc3lzdGVtLmlzc3VldGFicGFuZWxzOmFsbC10YWJwYW5lbCBdCgpl eWFsIGVkcmkgcmVhc3NpZ25lZCBPVklSVC0xOTExOgotLS0tLS0tLS0tLS0tLS0tLS0tLS0tLS0t LS0tLS0tLQoKICAgIEFzc2lnbmVlOiBBbnRvbiBNYXJjaHVrb3YgICh3YXM6IGluZnJhKQoKQW50 b24sIGhvdyBtdWNoIGVmZm9ydCBkbyB5b3UgdGhpbmsgdGhpcyB3aWxsIG5lZWQgdG8gYWRkIGl0 IHRvIEdlcnJpdCBIb29rcz8gCgpbfnlrYXVsXSBJIGFsc28gdGhpbmsgdGhpcyBzaG91bGQgYmUg Y29tbXVuaWNhdGVkIGJlZm9yZSBhZGRpbmcsIHBlb3BsZSBzb21ldGltZXMgZG9uJ3QgbGlrZSBu ZXcgYXV0b21hdGljIGluZm8gYWRkZWQgdG8gYnVncyB3aXRob3V0IGEgZ29vZCByZWFzb24uCgpX ZSdsbCBhbHNvIG5lZWQgdG8gdGhpbmsgd2hhdCBkbyB5b3UgZG8gaW4gY2FzZSB0aGVyZSBhcmUg bXVsdGlwbGUgZXh0ZXJuYWwgdHJhY2tlcnMgYXR0YWNoZWQgdG8gdGhlIGJ1Zywgd2hpY2ggaXMg dGhlIGNhc2UgbW9zdCBvZiB0aGUgdGltZSwgZG8geW91IHN0YXJ0ICJzcGFtbWluZyIgdGhlIGJ1 ZyB3aXRoIGNvbW1pdCBtZXNzYWdlcyBmcm9tIGVhY2ggdHJhY2tlcj8gc29tZXRpbWVzIGl0IGNh biBiZSBhcyBtdWNoIGFzIDEwIGRpZmZlcmVudCBwYXRjaGVzLiAKCldoYXQgYWJvdXQgaWYgdGhl cmUgYXJlIHRyYWNrZXJzIGZyb20gbXVsdGlwbGUgcHJvamVjdHMsIGEgcXVpdGUgY29tbW9uIHNj ZW5hcmlvIHdpdGggb1ZpcnQgYnVncyAoZS5nIHZkc20gKyBlbmdpbmUgKyBqc29ucnBjKSAsIGRv IHlvdSBhZGQgYWxsIHRoZSBjb21taXQgbWVzc2FnZXMgdGhlbj8gCgpXZSd2ZSBzZWVuIGlzc3Vl cyB3aXRoIGN1cnJlbnQgbmV3IGNvbW1lbnRzIGFkZGVkIHRvIGJ1Z3Mgd2hlbiB0aGV5IGNhbid0 IG1vdmUgdG8gT05fUUEsIGl0IGhhcyB0aGUgcG90ZW50aWFsIG9mIGNhdXNpbmcgbW9yZSBub2lz ZSB0aGFuIGdvb2QsIHNvIHdlIHJlYWxseSBuZWVkIHRvIHRoaW5rIGlmIGFuZCBob3cgd2Ugd2Fu dCB0byBpbXBsZW1lbnQgdGhpcyBiZWZvcmUgbW92aW5nIGZvcndhcmQuCgpjYyBbfmRmb2Rvcl0K Cgo+IFJGRTogQWRkIHJlbGV2YW50IGdpdCBjb21taXQgbWVzc2FnZXMgYXMgY29tbWVudHMgaW4g dGhlIHJlbGV2YW50IEJ1Z3ppbGxhcwo+IC0tLS0tLS0tLS0tLS0tLS0tLS0tLS0tLS0tLS0tLS0t LS0tLS0tLS0tLS0tLS0tLS0tLS0tLS0tLS0tLS0tLS0tLS0tLS0tLS0tLQo+Cj4gICAgICAgICAg ICAgICAgIEtleTogT1ZJUlQtMTkxMQo+ICAgICAgICAgICAgICAgICBVUkw6IGh0dHBzOi8vb3Zp cnQtamlyYS5hdGxhc3NpYW4ubmV0L2Jyb3dzZS9PVklSVC0xOTExCj4gICAgICAgICAgICAgUHJv amVjdDogb1ZpcnQgLSB2aXJ0dWFsaXphdGlvbiBtYWRlIGVhc3kKPiAgICAgICAgICBJc3N1ZSBU eXBlOiBOZXcgRmVhdHVyZQo+ICAgICAgICAgIENvbXBvbmVudHM6IEdlcnJpdC9naXQKPiAgICAg ICAgICAgIFJlcG9ydGVyOiBZYW5pdiBLYXVsCj4gICAgICAgICAgICBBc3NpZ25lZTogQW50b24g TWFyY2h1a292Cj4KPiBXaGVuIGEgYnVnIG1vdmVzIHRvIE1PRElGSUVELCBjYW4gd2UgYWRkIHRo ZSByZWxldmFudCBjb21taXQgbWVzc2FnZShzKSB0byB0aGUgYnVnPwo+IEkgc2VlIGl0IGhhcHBl bmluZyBpbiBSSEVMIGZyb20gdGltZSB0byB0aW1lLCBhbmQgSSB0aGluayBpdCdzIGVhc2llciAo Zm9yIHVzIGFuZCBRRSkgdGhlbiBjbGlja2luZyBvbiB0aGUgbGluayB0byBHZXJyaXQuCgoKCi0t ClRoaXMgbWVzc2FnZSB3YXMgc2VudCBieSBBdGxhc3NpYW4gSmlyYQoodjEwMDEuMC4wLVNOQVBT SE9UIzEwMDA4MCkKCi0tLS0tLS0tLS0tLT1fMTUxOTU3MzI4NC0zMDcxNS0xNDAKQ29udGVudC1U eXBlOiB0ZXh0L2h0bWw7IGNoYXJzZXQ9IlVURi04IgpDb250ZW50LURpc3Bvc2l0aW9uOiBpbmxp bmUKQ29udGVudC1UcmFuc2Zlci1FbmNvZGluZzogN2JpdAoKPGh0bWw+PGJvZHk+CjxwcmU+WyBo dHRwczovL292aXJ0LWppcmEuYXRsYXNzaWFuLm5ldC9icm93c2UvT1ZJUlQtMTkxMT9wYWdlPWNv bS5hdGxhc3NpYW4uamlyYS5wbHVnaW4uc3lzdGVtLmlzc3VldGFicGFuZWxzOmFsbC10YWJwYW5l bCBdPC9wcmU+CjxoMz5leWFsIGVkcmkgcmVhc3NpZ25lZCBPVklSVC0xOTExOjwvaDM+CjxwcmU+ QXNzaWduZWU6IEFudG9uIE1hcmNodWtvdiAgKHdhczogaW5mcmEpPC9wcmU+CjxwPkFudG9uLCBo b3cgbXVjaCBlZmZvcnQgZG8geW91IHRoaW5rIHRoaXMgd2lsbCBuZWVkIHRvIGFkZCBpdCB0byBH ZXJyaXQgSG9va3M/PC9wPgo8cD5bfnlrYXVsXSBJIGFsc28gdGhpbmsgdGhpcyBzaG91bGQgYmUg Y29tbXVuaWNhdGVkIGJlZm9yZSBhZGRpbmcsIHBlb3BsZSBzb21ldGltZXMgZG9uJ3QgbGlrZSBu ZXcgYXV0b21hdGljIGluZm8gYWRkZWQgdG8gYnVncyB3aXRob3V0IGEgZ29vZCByZWFzb24uPC9w Pgo8cD5XZSdsbCBhbHNvIG5lZWQgdG8gdGhpbmsgd2hhdCBkbyB5b3UgZG8gaW4gY2FzZSB0aGVy ZSBhcmUgbXVsdGlwbGUgZXh0ZXJuYWwgdHJhY2tlcnMgYXR0YWNoZWQgdG8gdGhlIGJ1Zywgd2hp Y2ggaXMgdGhlIGNhc2UgbW9zdCBvZiB0aGUgdGltZSwgZG8geW91IHN0YXJ0ICZsZHF1bztzcGFt bWluZyZyZHF1bzsgdGhlIGJ1ZyB3aXRoIGNvbW1pdCBtZXNzYWdlcyBmcm9tIGVhY2ggdHJhY2tl cj8gc29tZXRpbWVzIGl0IGNhbiBiZSBhcyBtdWNoIGFzIDEwIGRpZmZlcmVudCBwYXRjaGVzLjwv cD4KPHA+V2hhdCBhYm91dCBpZiB0aGVyZSBhcmUgdHJhY2tlcnMgZnJvbSBtdWx0aXBsZSBwcm9q ZWN0cywgYSBxdWl0ZSBjb21tb24gc2NlbmFyaW8gd2l0aCBvVmlydCBidWdzIChlLmcgdmRzbSAr IGVuZ2luZSArIGpzb25ycGMpICwgZG8geW91IGFkZCBhbGwgdGhlIGNvbW1pdCBtZXNzYWdlcyB0 aGVuPzwvcD4KPHA+V2UndmUgc2VlbiBpc3N1ZXMgd2l0aCBjdXJyZW50IG5ldyBjb21tZW50cyBh ZGRlZCB0byBidWdzIHdoZW4gdGhleSBjYW4ndCBtb3ZlIHRvIE9OX1FBLCBpdCBoYXMgdGhlIHBv dGVudGlhbCBvZiBjYXVzaW5nIG1vcmUgbm9pc2UgdGhhbiBnb29kLCBzbyB3ZSByZWFsbHkgbmVl ZCB0byB0aGluayBpZiBhbmQgaG93IHdlIHdhbnQgdG8gaW1wbGVtZW50IHRoaXMgYmVmb3JlIG1v dmluZyBmb3J3YXJkLjwvcD4KPHA+Y2MgW35kZm9kb3JdPC9wPgo8YmxvY2txdW90ZT48aDM+UkZF OiBBZGQgcmVsZXZhbnQgZ2l0IGNvbW1pdCBtZXNzYWdlcyBhcyBjb21tZW50cyBpbiB0aGUgcmVs ZXZhbnQgQnVnemlsbGFzPC9oMz4KPHByZT4gICAgIEtleTogT1ZJUlQtMTkxMQogICAgIFVSTDog aHR0cHM6Ly9vdmlydC1qaXJhLmF0bGFzc2lhbi5uZXQvYnJvd3NlL09WSVJULTE5MTEKIFByb2pl Y3Q6IG9WaXJ0IC0gdmlydHVhbGl6YXRpb24gbWFkZSBlYXN5CiAgICAgICAgIElzc3VlIFR5cGU6 IE5ldyBGZWF0dXJlCiAgICAgICAgIENvbXBvbmVudHM6IEdlcnJpdC9naXQKUmVwb3J0ZXI6IFlh bml2IEthdWwKQXNzaWduZWU6IEFudG9uIE1hcmNodWtvdjwvcHJlPgo8cD5XaGVuIGEgYnVnIG1v dmVzIHRvIE1PRElGSUVELCBjYW4gd2UgYWRkIHRoZSByZWxldmFudCBjb21taXQgbWVzc2FnZShz KSB0byB0aGUgYnVnPyBJIHNlZSBpdCBoYXBwZW5pbmcgaW4gUkhFTCBmcm9tIHRpbWUgdG8gdGlt ZSwgYW5kIEkgdGhpbmsgaXQncyBlYXNpZXIgKGZvciB1cyBhbmQgUUUpIHRoZW4gY2xpY2tpbmcg b24gdGhlIGxpbmsgdG8gR2Vycml0LjwvcD48L2Jsb2NrcXVvdGU+CjxwPiZtZGFzaDsgVGhpcyBt ZXNzYWdlIHdhcyBzZW50IGJ5IEF0bGFzc2lhbiBKaXJhICh2MTAwMS4wLjAtU05BUFNIT1QjMTAw MDgwKTwvcD4KCjxpbWcgc3JjPSJodHRwczovL3U0MDQzNDAyLmN0LnNlbmRncmlkLm5ldC93Zi9v cGVuP3Vwbj1pNVRNV0dWOTlhbUpiTnhKcFNwMi0yQkozM0JTTTN0dWlVZlJUazY0Sy0yQk9qRlRL ZDFlUWswamVhSWNCSmxMeVdpTmN2R0lzQ2Z4YVlJYm5TU1o1bjVnLTJGRkF3c2UzNDltS0t4RlFK OGgtMkZZNmdWLTJCS2hweVVmQU8yeHN6d3o0Q2NJTFhhTlg2WGMtMkZsRGdtZmxVS250Y3R1bk1Z a1BueEt0LTJGaEJKWDNQMHg5WkNVRjBHZU9ORVFJVW4tMkZGMUtCZjFmQVh2b1hwSVgtMkI4UmlT N3hKUTBYbWUyRUxHYTZRRFo1VnlvQnh1ejA5WGVzVTBOV3k2Y0NOUnJtQkw0VC0yRkxxeEFIODRq MEpkcS0yRjdzeHM0LTJGWlV0Q054S0RHRS0yRkxRcXJOLTJGVU83ME1MUHhITFZyUVE4S3g5Z3FP S2ZjVHl4dDlZRDROdEdSMGRoS2d4TWN2R2kyZjJ2ajZxVThNTUpCVEM4Q3pwYlpTbE9xZ094WVF2 SWU4NGt2VDIxbmEtMkJIUng5SjBqaXk4YnNlQWxMSFdqeTZPZ0ZrcHB1R3ZLemtrT2xxdnZVZGJT QUdpUkloQ0hxbGtPaE5Oc0U5MXIyYW80dzRsVnBZIiBhbHQ9IiIgd2lkdGg9IjEiIGhlaWdodD0i MSIgYm9yZGVyPSIwIiBzdHlsZT0iaGVpZ2h0OjFweCAhaW1wb3J0YW50O3dpZHRoOjFweCAhaW1w b3J0YW50O2JvcmRlci13aWR0aDowICFpbXBvcnRhbnQ7bWFyZ2luLXRvcDowICFpbXBvcnRhbnQ7 bWFyZ2luLWJvdHRvbTowICFpbXBvcnRhbnQ7bWFyZ2luLXJpZ2h0OjAgIWltcG9ydGFudDttYXJn aW4tbGVmdDowICFpbXBvcnRhbnQ7cGFkZGluZy10b3A6MCAhaW1wb3J0YW50O3BhZGRpbmctYm90 dG9tOjAgIWltcG9ydGFudDtwYWRkaW5nLXJpZ2h0OjAgIWltcG9ydGFudDtwYWRkaW5nLWxlZnQ6 MCAhaW1wb3J0YW50OyIvPgo8L2JvZHk+PC9odG1sPgoKLS0tLS0tLS0tLS0tPV8xNTE5NTczMjg0 LTMwNzE1LTE0MC0tCg== --===============2989541375534173939==--