Open Bug 1560596 Opened 6 years ago Updated 3 years ago

Clean up some of the code for ingesting Taskcluster tasks

Categories

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

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

Tracking

(Not tracked)

People

(Reporter: armenzg, Unassigned)

References

Details

In bug 1395254 we're going to switch from consuming Pulse messages produced by taskcluster-treeherder to Taskcluster canonical exchanges.

In order to make the code changes as close as possible to the original JS codebase I decided to not make code improvements as I passed by to reduce the possibility of introducing bugs.

However, there are various changes that we can make to make the code more readable and for the data structures to make more sense.

I will be taking this as soon as the other bug lands and sticks.

Depends on: 1395254
See Also: → 1395254

Setting this as P3 until you're ready to take this, Armen.

Priority: -- → P3
Type: defect → enhancement
Summary: Improve some of the code for ingesting Taskcluster tasks → Clean up some of the code for ingesting Taskcluster tasks

Review this comment when tackling this work:
https://github.com/mozilla/treeherder/pull/5042#discussion_r305470356

Make sure to write tests for this:
https://github.com/mozilla/treeherder/pull/5213

Another test to write:
https://github.com/mozilla/treeherder/pull/5215

Assignee: nobody → armenzg
Status: NEW → ASSIGNED
Priority: P3 → P2
Assignee: armenzg → nobody
Status: ASSIGNED → NEW
Priority: P2 → P3
You need to log in before you can comment on or make changes to this bug.