Open Bug 1387418 Opened 7 years ago Updated 5 years ago

Output a clearer error if the pushlog entry referenced by pulse does not exist

Categories

(Tree Management :: Treeherder: Data Ingestion, defect, P3)

Product:
Tree Management
Component:
Treeherder: Data Ingestion
Type:
defect

Tracking

(Not tracked)

People

(Reporter: emorley, Unassigned)

References

Details

In bug 1387407 we were seeing exceptions on New Relic of form:

```
File "/app/treeherder/etl/tasks/pulse_tasks.py", line 30, in store_pulse_resultsets
File "/app/treeherder/etl/resultset_loader.py", line 42, in process
File "/app/treeherder/etl/resultset_loader.py", line 237, in transform
File "/app/treeherder/etl/resultset_loader.py", line 244, in fetch_resultset
exceptions:IndexError: list index out of range
```

Which corresponds to:
https://github.com/mozilla/treeherder/blob/3648340ab4155a24747a68ddde3f5b8a0eb24f99/treeherder/etl/resultset_loader.py#L244

This was due to a problem with hg.mozilla.org, however it would be great to make the exception/error clearer here, by either:
* Raising a specific new exception type (or at least one with a sensible message)
* Logging to papertrail

Cameron, I don't suppose you'd mind taking this since you've worked more with the pulse ingestion parts? :-)
Assignee: cdawson → nobody

Armen: Would you be up for taking this now? :)

Flags: needinfo?(armenzg)
Priority: P2 → P3

Answered in meeting.

Flags: needinfo?(armenzg)
You need to log in before you can comment on or make changes to this bug.