Re: [vpp-dev] Change to Gerrit

2019-11-21 Thread Vratko Polak -X (vrpolak - PANTHEON TECHNOLOGIES at Cisco) via Lists.Fd.Io
nciguerra Sent: Wednesday, November 20, 2019 7:33 PM To: vpp-dev@lists.fd.io Subject: [vpp-dev] Change to Gerrit How would the group feel about implementing something like [0], so that changes to the commit message don't trigger rebuilds? To enforce the commit message structure, we could skip

Re: [vpp-dev] Change to Gerrit

2019-11-20 Thread Paul Vinciguerra
Hi Andrew. That used to be the case, not anymore. See: https://gerrit.fd.io/r/c/vpp/+/23521 . Changing just the commit via the UI, triggers everything. Paul On Wed, Nov 20, 2019 at 6:56 PM Andrew  Yourtchenko wrote: > Does this idea keep in mind a corner case of editing the commit message >

Re: [vpp-dev] Change to Gerrit

2019-11-20 Thread Andrew Yourtchenko
Does this idea keep in mind a corner case of editing the commit message over the gerrit UI? That operation currently doesn’t trigger the rebuilds... (Though i tend to use it for a quick edit to recover a checkstyle error, so I would rather have it trigger the checkstyle :-) If these things

Re: [vpp-dev] Change to Gerrit

2019-11-20 Thread Damjan Marion via Lists.Fd.Io
+1 > On 20 Nov 2019, at 19:32, Paul Vinciguerra wrote: > > How would the group feel about implementing something like [0], so that > changes to the commit message don't trigger rebuilds? > > To enforce the commit message structure, we could skip the jobs and set > verify label after the

Re: [EXTERNAL] [vpp-dev] Change to Gerrit

2019-11-20 Thread Chris Luke
+1 From: vpp-dev@lists.fd.io On Behalf Of Paul Vinciguerra Sent: Wednesday, November 20, 2019 1:33 PM To: vpp-dev@lists.fd.io Subject: [EXTERNAL] [vpp-dev] Change to Gerrit How would the group feel about implementing something like [0], so that changes to the commit message don't trigger

Re: [vpp-dev] Change to Gerrit

2019-11-20 Thread Dave Barach via Lists.Fd.Io
+1 From: vpp-dev@lists.fd.io On Behalf Of Paul Vinciguerra Sent: Wednesday, November 20, 2019 1:33 PM To: vpp-dev@lists.fd.io Subject: [vpp-dev] Change to Gerrit How would the group feel about implementing something like [0], so that changes to the commit message don't trigger rebuilds

Re: [vpp-dev] Change to Gerrit

2019-11-20 Thread Florin Coras
+1 Florin > On Nov 20, 2019, at 10:32 AM, Paul Vinciguerra > wrote: > > How would the group feel about implementing something like [0], so that > changes to the commit message don't trigger rebuilds? > > To enforce the commit message structure, we could skip the jobs and set > verify

[vpp-dev] Change to Gerrit

2019-11-20 Thread Paul Vinciguerra
How would the group feel about implementing something like [0], so that changes to the commit message don't trigger rebuilds? To enforce the commit message structure, we could skip the jobs and set verify label after the codestyle checks if no files were changed. Maybe others don't care, but I