Closed Bug 1097328 Opened 10 years ago Closed 9 years ago

job_guid for coalesced jobs isn't generated correctly

Categories

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

defect

Tracking

(Not tracked)

RESOLVED DUPLICATE of bug 1093743

People

(Reporter: camd, Unassigned)

Details

We use the first element in the request_ids to generate the job_guid for running jobs, but we use a comma-separated concatenated string of all the request_ids for complete jobs.  Whichever is the right one, these should probably match.

See line 455 of buildapi.py
Here's commit for a potential fix: https://github.com/mozilla/treeherder-service/compare/coalesced-job-guid

Mauro: maybe we could chat about whether this was intentional, or if it should match this line:
https://github.com/mozilla/treeherder-service/blob/6b7c16a6ffc4bce44b85d33aabe7115a22d8767b/treeherder/etl/buildapi.py#L65
Flags: needinfo?(mdoglio)
Could this be the cause of bug 1097310?

I would think that the correct method would be to use just the last request_id for completed jobs (similar to bug 1093743), since if there was another (later) job, it would have to have had a newer request, so wouldn't result in a job guid collision.
Otherwise pending jobs (which will only have a single request_id) will get a different job guid to their later running/completed job, if coalescing occurred, surely?
Priority: -- → P1
:camd I think :edmorley is right, we should probably just use the last element from request_ids and the corresponding value in request_time_list to create the job_guid.
Flags: needinfo?(mdoglio)
Component: Treeherder → Treeherder: Data Ingestion
Taking this because the fix for this is also in https://github.com/mozilla/treeherder-service/pull/359
Assignee: nobody → cdawson
Status: NEW → ASSIGNED
Bug 1093743 should fix this :-)
Assignee: cdawson → nobody
Status: ASSIGNED → RESOLVED
Closed: 9 years ago
Resolution: --- → DUPLICATE
You need to log in before you can comment on or make changes to this bug.