GerritStatusPush: add filters on builderName, project, branch

[PiotrSikora]

Currently GerritStatusPush? will try to push status update to Gerrit after build of every project in the system, perhaps it would be wise to add some filtering capabilities to it?

builderName and builderName_re should be enough, but maybe we should add filters for project and branch as well?

I'll take care of this, but I need some feedback on this feature.

[dustin]

Sounds good. Could it use a ChangeFilter?

[deignacio]

I could see this being useful for all StatusReceiver? subclasses. i don't think you could use a ChangeFilter?, because that has a bunch of code that is specific to changes, where as this is filtering BuildStatuses?. in buildbot.status.builder whenever you do something like

for w in self.watchers:
    try:
        if not w.filter(name, s, ...):
            continue
        w.someStep(name, s, ...)
    except:
        ....

i realize that this isn't super important since there is already a filter to only perform this for gerrit-based builds, but if perhaps you were doing builds on patchset-created but not ref-update, this would be helpful

how's your progress on this? i could probably put in some work into this if you wanted.

[PiotrSikora]

Unfortunately, I'm a bit busy lately and I didn't touch Buildbot in months. Feel free to takeover this.

[dustin]

Ticket retargeted after milestone closed

[sa2ajj]

MailNotifier? has a parameter builders that allows to limit the set of builders it sends notifications for. The other parameter (conflicting with builders) is categories (to use a different criterion).

IRC, on the other hand, has only categories.

I believe a more general agreement what should be the ways and make them available for all status targets.