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)

ARM
Gonk (Firefox OS)
defect
Not set
normal

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: 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
Blocks: 1149266
Status: NEW → ASSIGNED
- 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)
Blocks: 1162212
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+
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?)
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
Closed: 9 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.

Attachment

General

Created:
Updated:
Size: