If you think a bug might affect users in the 57 release, please set the correct tracking and status flags for Release Management.

Split Android crashtests from 2 chunks to 3 chunks

RESOLVED FIXED

Status

Release Engineering
General
P3
major
RESOLVED FIXED
6 years ago
4 years ago

People

(Reporter: philor, Assigned: philor)

Tracking

Firefox Tracking Flags

(Not tracked)

Details

(Whiteboard: [android])

Attachments

(1 attachment)

(Assignee)

Description

6 years ago
Based on my retriggering on try, about 20-30% of XUL Fennec and 50% of native Fennec crashtest-1 runs time out. Despite the timeout hitting at around the same test every time, disabling the last few tests, or even every test in that directory, just pushes the timeout a few tests later.

It would pay to sanity check whether it helps in staging first, but that probably means it's just more work than a Tegra can do without taking a nap, and it's time to split it in 3.

Updated

6 years ago
Assignee: nobody → bear
Whiteboard: [buildduty][android]

Updated

6 years ago
Priority: -- → P3

Comment 1

6 years ago
I should have bounced this back to the queue earlier
Assignee: bear → nobody
Whiteboard: [buildduty][android] → [android]
(Assignee)

Comment 2

6 years ago
Created attachment 594380 [details] [diff] [review]
fix

Conveniently, there's really no need for staging. I got fed up with C1, and hid it on m-i and m-c, and I'll hide it anywhere else I have to look as I have to look, so there's zero risk to taking an untested patch, because there's nowhere to go but up now.
Assignee: nobody → philringnalda
Status: NEW → ASSIGNED
Attachment #594380 - Flags: review?(bear)

Updated

6 years ago
Attachment #594380 - Flags: review?(bear) → review+
(Assignee)

Comment 3

6 years ago
Comment on attachment 594380 [details] [diff] [review]
fix

http://hg.mozilla.org/build/buildbot-configs/rev/48d40e93397d
Attachment #594380 - Flags: checked-in+
(Assignee)

Comment 4

6 years ago
hidden: m-i, m-c, m-a, s-c, fx-team
I don't see C3 on https://tbpl.mozilla.org/?noignore=1, just c1 and c2.  Do we need a reconfig for this?
(Assignee)

Comment 6

6 years ago
Yep
(Assignee)

Comment 7

6 years ago
Two, rather than just one, actually.
(Assignee)

Comment 8

6 years ago
Not the most profound of effects, but fixed nonetheless.
Status: ASSIGNED → RESOLVED
Last Resolved: 6 years ago
Resolution: --- → FIXED
Product: mozilla.org → Release Engineering
You need to log in before you can comment on or make changes to this bug.