[webkit-dev] Frustrated at inconsiderate behavior

Jeremy Orlow jorlow at chromium.org
Thu Jul 8 14:38:18 PDT 2010


On Fri, Jul 9, 2010 at 2:14 AM, Xan Lopez <xan at gnome.org> wrote:

> On Thu, Jul 8, 2010 at 8:09 PM, Adam Barth <abarth at webkit.org> wrote:
> > Originally, we had the commit-queue to land only if all the core
> > builders were green.  One recent change we made to make the
> > commit-queue more agressive is to land when the commit-queue itself
> > sees 100% passing tests on itself.  When we were waiting for all the
> > core builders to be green, the commit-queue usually had to wait for me
> > or Eric or another contributor to clean up the entire tree and ping
> > the folks who maintain bots that had gotten sick.
> >
> > My view is, generally, that the commit-queue should act like a
> > contentious committer, which means it should act the way most
> > committers act.  Given that folks generally don't wait for all the
> > bots to be green before committing, I felt this change was worth
> > experimenting with.  (Note that sheriff-bot still monitors all the
> > core builders and alerts folks of bustage.)
>
> I see, I interpreted you meant it saw 100% passing tests in the core
> bots (as it used to be). The new behavior seems to make it easier to
> go back to how things were before, when as soon as one port (say GTK+)
> breaks, people will keep piling things on top and if nobody is around
> at that moment then it's much harder to figure out what happened, who
> was responsible, etc, because you only get clear data of the first
> breakage.
>
> So, not a fan.
>
> Of course I see the point that you make about most people committing
> manually doing the same thing, but maybe we should create a rule of
> not committing code touching all ports if any of the trees is red.
> That would improve things, I feel this switch won't do that.
>

I think the point that's been made in this thread is that whatever policy
the commit queue uses should be in line with whatever policies all
committers should be using.  Otherwise you run into the problems Maciej
noted.

If you think we should all follow such a policy, well that's an entirely
different topic.  And one that seems to keep coming up.  If you want to
raise it again, you should probably start by looking at the archives.

J
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://lists.webkit.org/pipermail/webkit-dev/attachments/20100709/ee514697/attachment.html>


More information about the webkit-dev mailing list