GerritChangeFilter support for branch filtering is broken

[sa2ajj]

Basically, current implementation expects every event coming from Gerrit to have a change included. As ​documentation shows, there're some events do not have any changes included, most notablbe ref-updated.

As a practical example, a set of properties for an event with a change:

event.change.branch
event.change.commitMessage
event.change.id
event.change.number
event.change.owner.email
event.change.owner.name
event.change.owner.username
event.change.project
event.change.status
event.change.subject
event.change.topic
event.change.url
event.patchSet.author.email
event.patchSet.author.name
event.patchSet.author.username
event.patchSet.createdOn
event.patchSet.isDraft
event.patchSet.number
event.patchSet.parents
event.patchSet.ref
event.patchSet.revision
event.patchSet.sizeDeletions
event.patchSet.sizeInsertions
event.patchSet.uploader.email
event.patchSet.uploader.name
event.patchSet.uploader.username
event.type
event.uploader.email
event.uploader.name
event.uploader.username

and here's a set of properties for an event that does not have a change (ref-updated):

event.refUpdate.newRev
event.refUpdate.oldRev
event.refUpdate.project
event.refUpdate.refName
event.submitter.email
event.submitter.name
event.submitter.username
event.type

[tardyp]

fyi, this is how I solved the problem on buildbot_travis.

​https://github.com/tardyp/buildbot_travis/blob/master/buildbot_travis/vcs/gerrit.py#L32

I did not find the time to upstream it yet.

[sa2ajj]

@tardyp, I'm not sure: the problem is in GerritChangeFilter, you are modifying GerritChangeSource. I assume that you use "normal" ChangeFilter, right?

[tardyp]

Its an integration problem. You can fix it in either end of the interface.