[Buildbot] #2185: GerritChangeSource: owner email can be missing

[Buildbot trac]

#2185: GerritChangeSource: owner email can be missing
-----------------------------+------------------------
Reporter:  sanxiyn           |       Owner:
    Type:  defect            |      Status:  closed
Priority:  patches-accepted  |   Milestone:  0.9.+
 Version:  0.8.5             |  Resolution:  duplicate
Keywords:  gerrit, sprint    |
-----------------------------+------------------------
Changes (by sa2ajj):

 * status:  new => closed
 * resolution:   => duplicate


Old description:

> I got:
>
> {{{
> Traceback (most recent call last):
>   File "/home/tinuviel/buildbot/master/gerritchangesource.py", line 125,
> in eventReceived_patchset_created
>     author="%s <%s>" % (change["owner"]["name"],
> change["owner"]["email"]),
> exceptions.KeyError: 'email'
> }}}
>
> In general, as you can read at [http://gerrit-
> documentation.googlecode.com/svn/Documentation/2.2.2/json.html Gerrit
> documentation], "any field may be missing in the JSON messages, so
> consumers of this JSON stream should deal with that appropriately".

New description:

 I got:

 {{{
 Traceback (most recent call last):
   File "/home/tinuviel/buildbot/master/gerritchangesource.py", line 125,
 in eventReceived_patchset_created
     author="%s <%s>" % (change["owner"]["name"],
 change["owner"]["email"]),
 exceptions.KeyError: 'email'
 }}}

 In general, as you can read at [http://gerrit-
 documentation.googlecode.com/svn/Documentation/2.2.2/json.html Gerrit
 documentation], "any field may be missing in the JSON messages, so
 consumers of this JSON stream should deal with that appropriately".

--

Comment:

 #2918 has a patch, so I close this ticket as a duplicate.

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