Closed
Bug 1160589
Opened 9 years ago
Closed 9 years ago
[raptor] Create a parent raptor decision task for gaia
Categories
(Firefox OS Graveyard :: Gaia::PerformanceTest, defect)
Tracking
(Not tracked)
RESOLVED
FIXED
2.2 S12 (15may)
People
(Reporter: jlal, Assigned: rwood)
References
Details
Attachments
(2 files)
Right now it's impossible to retrigger the entire raptor performance suite (and get new results) without retriggering the whole graph. A quick hack around this would be to add a task which the other raptor tasks depend on. Graph wise it looks like this: D -> R(Decision task for raptor) -> se1/se2 -> se
Assignee | ||
Updated•9 years ago
|
Assignee: nobody → rwood
Summary: Trigger raptor performance tests from their own decision task so it's easier to retrigger the entire suite. → [raptor] Create a parent raptor decision task for gaia
Comment 1•9 years ago
|
||
Assignee | ||
Comment 2•9 years ago
|
||
- New raptor parent/decision task - Decision task provides appropriate gaia rev to raptor test tasks; the base task always runs on base/master; the patch task runs on base + PR, if not testing on PR (gaia-master) the patch task will run against base -1 commit - Decision task grabs latest successful emulator build from TC and provides that same URL for all subsequent raptor tasks - Use new raptor worker type; faster aws instance - Increase test runs from 15 to 30
Attachment #8605827 -
Flags: review?(garndt)
Comment 3•9 years ago
|
||
Comment on attachment 8605827 [details] [review] https://github.com/mozilla-b2g/gaia/pull/30052 r+ with my comments on GH. I don't recall any of my comments being anything absolutely critical to fix, but please feel free to respond to them if the comments don't make sense or if you don't see the value in making the changes. Looking at the TH link, it's looks like the decision task and the launch tasks work. Curious to see what this looks like when a push happens, but I'm not sure how that could be tested without this code being committed.
Attachment #8605827 -
Flags: review?(garndt) → review+
Assignee | ||
Comment 4•9 years ago
|
||
Thanks :garndt, updates made (tests are green; first time the gaia decision task failed but it was running the old code, it passed when I retriggered, I guess because I changed the gaia decision code itself). Carrying-over the R+, using auto-lander to land
Keywords: checkin-needed
Comment 5•9 years ago
|
||
Just commenting to see if Autolander is alive. (Maybe it didn't get poked from this bug?)
Assignee | ||
Updated•9 years ago
|
Component: Gaia → Gaia::PerformanceTest
Comment 6•9 years ago
|
||
Tests seem good, and autolander might be a bit tipsy since the tree closure - have some things to work out there. Let's land manually for now: https://github.com/mozilla-b2g/gaia/commit/2bbdce1ea10b731b587c915a466a746ee84164a4
Status: ASSIGNED → RESOLVED
Closed: 9 years ago
Resolution: --- → FIXED
Updated•9 years ago
|
Keywords: checkin-needed
Target Milestone: --- → 2.2 S12 (15may)
You need to log in
before you can comment on or make changes to this bug.
Description
•