Page MenuHomePhabricator

ReleaseTaggerBot broke on May 24
Closed, ResolvedPublic

Description

@ReleaseTaggerBot hasn't made any changes to Phabricator since May 24.

Related Objects

Event Timeline

Catrope triaged this task as Unbreak Now! priority.May 31 2016, 5:26 PM
Traceback (most recent call last):
  File "/data/project/forrestbot/forrestbot/forrestbot.py", line 223, in <module>
    for task, acts in itertools.groupby(sorted(actions, key=key), key=key):
  File "/data/project/forrestbot/forrestbot/forrestbot.py", line 221, in key
    return int(x['task'].strip()[1:])
ValueError: invalid literal for int() with base 10: '51256, T110143, T78456'

We don't support multi-bug commits at all at the moment (if there are multiple Bug: entries, the earlier ones get overwritten). I think the best 'right now' solution is to change the code to find the first 'Txxxx' entry (regex), and leaving the multi-task support for later.

Change 291965 had a related patch set uploaded (by Merlijn van Deen):
Make Bug: parsing more lenient

https://gerrit.wikimedia.org/r/291965

Change 291965 abandoned by Merlijn van Deen:
Make Bug: parsing more lenient

https://gerrit.wikimedia.org/r/291965

Change 291966 had a related patch set uploaded (by Merlijn van Deen):
Make Bug: parsing more lenient

https://gerrit.wikimedia.org/r/291966

Change 291966 merged by jenkins-bot:
Make Bug: parsing more lenient

https://gerrit.wikimedia.org/r/291966

valhallasw claimed this task.