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:28 PM, Kaushal M <kshlmster@xxxxxxxxx> wrote:
On Fri, Jan 8, 2016 at 12:10 PM, Kaushal M <kshlmster@xxxxxxxxx> wrote:
> 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.
>

Makes sense. I have disabled it completely for now and lets keep it that way till
developers realize it(a day should be enough). We will change this trigger to on Code Review +2 by tomorrow.

>>
>> 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.

This is done now.

>
>>
>> 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