[Buildbot] #2743: Create a hierarchy of review / merging

Buildbot trac trac at buildbot.net
Sun Sep 7 16:50:31 UTC 2014


#2743: Create a hierarchy of review / merging
------------------------+-------------------------
Reporter:  dustin       |       Owner:  dustin
    Type:  enhancement  |      Status:  assigned
Priority:  minor        |   Milestone:  botherders
 Version:               |  Resolution:
Keywords:               |
------------------------+-------------------------

Old description:

> When I get busy for 5-10 days at a time, Buildbot pull reqs pile up.
> This sucks for contributors, and when I *do* have time to work on
> Buildbot, it still takes me a few days to dig through the backlog.
>
> Options I can think of:
>
>  * help other committers learn to merge pull requests
>  * require contributors to get reviews done by someone else, against
> [wiki:PatchReview patch review guidelines], so I only do a cursory check
> once that other review is complete
>  * two-stage merges, where pull requests go to another committer's repos
> for testing, and I pull from that repo periodically

New description:

 When I get busy for 5-10 days at a time, Buildbot pull reqs pile up.  This
 sucks for contributors, and when I *do* have time to work on Buildbot, it
 still takes me a few days to dig through the backlog.

 Options I can think of:

 1. help other committers learn to merge pull requests
 1. require contributors to get reviews done by someone else, against
 [wiki:PatchReview patch review guidelines], so I only do a cursory check
 once that other review is complete
 1. two-stage merges, where pull requests go to another committer's repos
 for testing, and I pull from that repo periodically

--

Comment (by sa2ajj):

 BTW, option 3 sounds a bit too complicated.

--
Ticket URL: <http://trac.buildbot.net/ticket/2743#comment:3>
Buildbot <http://buildbot.net/>
Buildbot: build/test automation


More information about the bugs mailing list