[raptor] Create a parent raptor decision task for gaia

RESOLVED FIXED in 2.2 S12 (15may)

Status

Firefox OS
Gaia::PerformanceTest
RESOLVED FIXED
3 years ago
3 years ago

People

(Reporter: lightsofapollo, Assigned: rwood)

Tracking

unspecified
2.2 S12 (15may)
ARM
Gonk (Firefox OS)
Dependency tree / graph

Firefox Tracking Flags

(Not tracked)

Details

Attachments

(2 attachments)

(Reporter)

Description

3 years ago
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

3 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
(Assignee)

Updated

3 years ago
Blocks: 1149266
Status: NEW → ASSIGNED
Created attachment 8605477 [details] [review]
[gaia] rwood-moz:bug1160589 > mozilla-b2g:master
(Assignee)

Comment 2

3 years ago
Created attachment 8605827 [details] [review]
https://github.com/mozilla-b2g/gaia/pull/30052

- 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)
(Assignee)

Updated

3 years ago
Blocks: 1162212

Comment 3

3 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

3 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
Just commenting to see if Autolander is alive. (Maybe it didn't get poked from this bug?)
(Assignee)

Updated

3 years ago
Component: Gaia → Gaia::PerformanceTest
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
Last Resolved: 3 years ago
Resolution: --- → FIXED
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.