[openstack-dev] [neutron] -2'ing all patches on every gate breakage
Ihar Hrachyshka
ihrachys at redhat.com
Mon Apr 4 16:22:54 UTC 2016
Armando M. <armamig at gmail.com> wrote:
>
>
> On 4 April 2016 at 09:01, Ihar Hrachyshka <ihrachys at redhat.com> wrote:
> Hi all,
>
> I noticed that often times we go and -2 all the patches in the review
> queue on every neutron specific gate breakage spotted. This is allegedly
> done to make sure that nothing known to be broken land in merge gate
> until we fix the breakage on our side.
>
> This is not allegedly done. When I do it, I put a comment next to my
> action.
>
>
>
> While I share the goal of not resetting the gate if we can avoid it, I
> find the way we do it a bit too aggressive. Especially considering that
> often times those -2 votes sit there not cleared even days after the
> causing breakage is fixed, needlessly blocking patches landing.
>
> That's a blatant lie: I am aggressive at putting -2s as well as removing
> them. Other changes for those the -2 stick is probably because they
> aren't worth the hassle. We've been also in feature freeze so slowing
> things down should have hurt anyway.
>
>
> I suggest we either make sure that we remove those -2 votes right after
> gate fixes land, or we use other means to communicate to core reviewers
> that there is a time window when nothing should land in the merge queue.
>
> Initially I tried sending emails ahead of time alerting for gate
> breakages, but that doesn't work for obvious reasons: there is a lag that
> can still be fatal.
>
> On the specific circumstance, gate broke on Friday late afternoon PDT. It
> didn't seem that was anything critical worth merging at all cost that
> couldn't wait until Monday morning and I didn't bother check that things
> merged safely in the middle of my weekend.
Yeah, but it’s already up to two working days in some places.
Note that I don’t mean you should check anything on your weekend. Instead,
I think we should avoid -2’s in this case and teach core reviewers to check
some source of gate state truth. An email would actually work as long as
everyone actively checks it [if for some reason people are not reading
openstack-dev@, let’s To: everyone in the group].
Ihar
More information about the OpenStack-dev
mailing list