[Buildbot] #2946: Excessuve Twistd Logging for GitPoller

[Buildbot trac]

#2946: Excessuve Twistd Logging for GitPoller
---------------------------+------------------------
Reporter:  Jeremy.Cornett  |       Owner:
    Type:  undecided       |      Status:  new
Priority:  major           |   Milestone:  undecided
 Version:  0.8.9           |  Resolution:
Keywords:                  |
---------------------------+------------------------

Old description:

> Example GitPoller:
>
> c['change_source'].append(
>     GitPoller(
>         repourl='ssh://git@git/AwesomeProduct',
>         branches=True,
>         workdir="gitpoller-work/AwesomeProduct",
>         pollinterval=120))
>
> The branches=True indicates to poll all branches in the repository. This
> is wonderful functionality and I very much like it. Unfortunately, I've
> found that for EACH branch in the repository, you get a line similar to
> this in your twistd.log file.
>
> 2014-10-15 11:24:16-0600 [-] gitpoller: processing 0 changes: [] from
> "ssh://git@git/AwesomeProduct"
>
> When you're using BuildBot to poll repositories with 50+ branches, those
> lines in the twistd.log add no value and spam the log repeatedly. It's be
> much better if only one line was added per repository, instead of per
> branch, when polling all branches in a repository.

New description:

 Example !GitPoller:

 {{{
 c['change_source'].append(
     GitPoller(
         repourl='ssh://git@git/AwesomeProduct',
         branches=True,
         workdir="gitpoller-work/AwesomeProduct",
         pollinterval=120))
 }}}

 The branches=True indicates to poll all branches in the repository. This
 is wonderful functionality and I very much like it. Unfortunately, I've
 found that for EACH branch in the repository, you get a line similar to
 this in your twistd.log file.

 {{{
 2014-10-15 11:24:16-0600 [-] gitpoller: processing 0 changes: [] from
 "ssh://git@git/AwesomeProduct"
 }}}

 When you're using Buildbot to poll repositories with 50+ branches, those
 lines in the twistd.log add no value and spam the log repeatedly. It's be
 much better if only one line was added per repository, instead of per
 branch, when polling all branches in a repository.

--

Comment (by sa2ajj):

 (edited description)

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