Sync from gerrit is failing

[Guilherme Salgado]

With the following traceback:

/srv/patches.linaro.org/apps/patchmetrics/bin/sync-gerrit-changes.py:23: DeprecationWarning: BaseException.message has been deprecated as of Python 2.6
  if '%s not found' % person.email in e.message:
Traceback (most recent call last):
  File "/srv/patches.linaro.org/apps/patchmetrics/bin/sync-gerrit-changes.py", line 21, in <module>
    person.email, start_after=start_after)
  File "/srv/patches.linaro.org/apps/patchmetrics/gerrit.py", line 56, in get_aosp_changes_from
    'allQueryNext', [query, start_after, page_size])
  File "/srv/patches.linaro.org/apps/patchmetrics/gerrit.py", line 33, in jsonrpc_request
    raise GerritError(response['error']['message'])
patchmetrics.gerrit.GerritError: Invalid query: owner: (address hidden)

line 1:7 no viable alternative at input '('

[Guilherme Salgado (salgado)]

It turns out this was caused because there was a Person in the patchwork DB with "(address hidden)" as its email address. That, in turn, was caused by the parsing of http://patches.linaro.org/5668/. I asked Mike on IRC whether or not we should expect to see more of those, but the answer was not; the address mangling was not intentional:

  <mturquette> i pulled the baseline patch from russell's patch tracker... so maybe the email address mangling was done there...

I've deleted the bogus Person entry so the script should run fine now

[Guilherme Salgado (salgado)]

We could try and do something smart when there's a hidden email address in a patch but it's probably not worth it.