Re: [vpp-dev] Change to Gerrit

2019-11-21 Thread Vratko Polak -X (vrpolak - PANTHEON TECHNOLOGIES at Cisco) via Lists.Fd.Io
> implementing something like [0] +1, but... > skip the jobs and set verify label after the codestyle checks if no files > were changed ... how do you imagine this being implemented in ci-management? Currently, the other jobs are triggered by checkstyle_success Gerrit comment, but how would th

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 abo

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 codes

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? To enf

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 label