[webkit-dev] Bugzilla flood by the commitbot ?(was: First Early Warning System (EWS) bot online)
tonikitoo (Antonio Gomes)
tonikitoo at gmail.com
Tue Jan 5 12:38:26 PST 2010
Speaking of which, I also love the commit bot thing, but IMO when it
commits a R+ and CQ+ patch, it submits too much (three) entries to the
bugzilla:
1) one cc'ing eseidel at chromium.org
2) one doing the commit itself
3) and finally marking the bug as FIXED.
@Eric, Adam, is there a way to reduce that to two or even on submit to
bugzilla ? e.g. merge _1_ and _3_ maybe ...
On Tue, Jan 5, 2010 at 3:51 PM, Maciej Stachowiak <mjs at apple.com> wrote:
>
> On Jan 5, 2010, at 10:21 AM, Adam Barth wrote:
>
>> On Tue, Jan 5, 2010 at 9:42 AM, tonikitoo (Antonio Gomes)
>> <tonikitoo at gmail.com> wrote:
>>>>
>>>> Q) Why doesn't the EWS post a success message? I like positive
>>>> re-enforcement.
>>>> A) I'm worried about spamming bugs with too many happy status
>>>> messages. If we have N ports with EWS bots, we don't want to have N
>>>> happy status messages. Eric and I have some ideas for a more passive
>>>> success indicator. Once things are running smoothly, we can share
>>>> some mocks with the list.
>>>
>>> I particularly like not spamming bugzilla in case of successful
>>> builds, and IMO the code-style bot should not post a message either in
>>> case of no errors.
>>
>> I agree, but this behavior was specifically required by Maciej when we
>> brought the bot online.
>
> s/required/requested/
>
>> I'd be happy to change it if that's what
>> folks would prefer. One possibility is to incorporate the status
>> bubbles directly into bugs.webkit.org directly instead of requiring
>> the extension. I find the status bubbles easier to use than hunting
>> for the success / failure message when reviewing a patch.
>
> I think something like the status bubbles would be more useful than the
> success comment, though I also find the success comment more useful than
> nothing.
>
> - Maciej
>
>
--
--Antonio Gomes
More information about the webkit-dev
mailing list