Closed Bug 1569588 Opened 6 years ago Closed 5 years ago

Analyze how Treeherder retriggers selected jobs

Categories

(Tree Management :: Perfherder, task, P3)

Tracking

(Not tracked)

RESOLVED FIXED

People

(Reporter: igoldan, Assigned: igoldan)

References

Details

Provide here an overview of this mechanism, so we now how to integrate it in Perfherder as well.

Blocks: 1569586
No longer blocks: 1202718
Blocks: 1570225
Assignee: nobody → igoldan
Priority: -- → P1

Looks like JobModel.retrigger(jobs, repoName, notify, times) is the main API we have to use.
Basically, we only need to know the perf jobs we want to retrigger & their repository.
This findings should suffice reusing this functionality on FE.

One weird thing I see: it appears repoName is not used inside this static method. :camd why is that? Could I just remove this parameter from this function signature?

Flags: needinfo?(cdawson)

Yeah, it's a bit of a vestigial artifact. We don't need it anymore, but some of the endpoints require it. I have been trying to remove the need for it as we go. And, yes, you're right. That call doesn't even use it now. :) So please feel free to remove it from retrigger as well as the other functions where it's no longer used.

I should have removed that before, but somehow forgot to do so...

Flags: needinfo?(cdawson)
See Also: → 1570874
See Also: → 1571643
Status: NEW → ASSIGNED
Assignee: igoldan → nobody
Status: ASSIGNED → NEW
Priority: P1 → P3
Status: NEW → RESOLVED
Closed: 5 years ago
Resolution: --- → FIXED
Assignee: nobody → igoldan
You need to log in before you can comment on or make changes to this bug.