Closed
Bug 1197239
Opened 9 years ago
Closed 9 years ago
Support to trigger dependent tasks once a mid point is reached
Categories
(Taskcluster :: Services, defect)
Taskcluster
Services
Tracking
(Not tracked)
RESOLVED
WONTFIX
mozilla46
People
(Reporter: armenzg, Unassigned)
Details
The way that buildbot works is that once the artifacts of a build are uploaded, we do a sendchange step which triggers the dependent tasks.
This is because all that the test jobs need is ready.
The build job continues past the sendchange step because it runs that make check step which can take 10-20 minutes depending on the task.
As-is, this would delay the starting of all of our tests.
This is probably not super urgent to fix since there are initiatives to get rid of the "make check" step after and upload, however, I wanted to file it just in case.
Adding ted since he might be aware of the "make check" removal work.
Comment 1•9 years ago
|
||
Getting rid of "make check" is bug 992323. We're getting pretty close, once chmanchester gets gtest running as a separate job we can probably mop up the rest easily.
Comment 2•9 years ago
|
||
Well, we could make the build task manually schedule the dependent task.
It's the sort of hack that can be justified if this a corner case and gains are huge.
But building a feature specifically for triggering dependents based on some artifact sounds dangerous.
There is something very simple and super powerful about having strict boundaries for when a task
triggers another task. Giving up simplicity of architecture can only be justified for HUGE gains.
Reporter | ||
Comment 3•9 years ago
|
||
Fair enough. I'm OK if we ended up making the build task schedule the dependent task.
I think getting the "make check" work done will help us here
Status: NEW → RESOLVED
Closed: 9 years ago
Resolution: --- → WONTFIX
Comment 4•9 years ago
|
||
Moving closed bugs across to new Bugzilla product "TaskCluster".
Component: TaskCluster → Integration
Product: Testing → Taskcluster
Target Milestone: --- → mozilla46
Assignee | ||
Updated•6 years ago
|
Component: Integration → Services
You need to log in
before you can comment on or make changes to this bug.
Description
•