Re: Gerrit review, submit type and Jenkins testing

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

 



On Fri, Jan 8, 2016 at 12:03 PM, Raghavendra Talur <rtalur@xxxxxxxxxx> wrote:
> Top posting, this is a very old thread.
>
> Keeping in view the recent NetBSD problems and the number of bugs creeping
> in, I suggest we do these things right now:
>
> a. Change the gerrit merge type to fast forward only.
> As explained below in the thread, with our current setup even if both PatchA
> and PatchB pass regression separately when both are merged it is possible
> that a functional bug creeps in.
> This is the only solution to prevent that from happening.
> I will work with Kaushal to get this done.
>
> b. In Jenkins, remove gerrit trigger and make it a manual operation

Making it manual might be too much work for maintainers. I suggest (as
I've suggested before) we make regressions trigger when a change has
been reviewed +2 by a maintainer.

>
> Too many developers use the upstream infra as a test cluster and it is
> *not*.
> It is a verification mechanism for maintainers to ensure that the patch does
> not cause regression.
> It is required that all developers run full regression on their machines
> before asking for reviews.
> Reviewers should review the patch only when the developer has given a +1
> verified on the patch.
> Again, I will work with Kaushal to get this done.
>
> P.S: Stop using the "universal" jenkins account to trigger jenkins build if
> you are not a maintainer.
> If you are a maintainer and don't have your own jenkins account then get one
> soon!

I think I'll go ahead and remove this account.

>
> Thanks,
> Raghavendra Talur
>
_______________________________________________
Gluster-devel mailing list
Gluster-devel@xxxxxxxxxxx
http://www.gluster.org/mailman/listinfo/gluster-devel



[Index of Archives]     [Gluster Users]     [Ceph Users]     [Linux ARM Kernel]     [Linux ARM]     [Linux Omap]     [Fedora ARM]     [IETF Annouce]     [Security]     [Bugtraq]     [Linux]     [Linux OMAP]     [Linux MIPS]     [eCos]     [Asterisk Internet PBX]     [Linux API]

  Powered by Linux