Closed
Bug 1390375
Opened 7 years ago
Closed 7 years ago
Windows DevEdition tests on Beta are running on incorrectly-branded builds
Categories
(Release Engineering :: General, enhancement, P1)
Tracking
(firefox56 verified, firefox57 fixed)
RESOLVED
FIXED
People
(Reporter: RyanVM, Assigned: catlee)
Details
Attachments
(1 file)
I noticed recently when looking at test failure screenshots that the Windows DevEdition runs didn't appear to have proper branding compared to Linux/OSX. I dug a bit deeper and found out that this is because we're running both tc(B) and tc(N) jobs, of which tc(B) is incorrectly branded and tc(N) is branded as expected. Unfortunately, we're running tests on the tc(B) builds instead of the tc(N) ones, so we aren't actually testing what we're shipping. Looking at other platforms, it appears that for Linux and OSX, we in fact *only* create & test tc(N) builds, so it seems like the right fix here is to remove the Windows DevEdition tc(B) builds & tests and make sure that tests get scheduled off tc(N) builds instead.
Reporter | ||
Comment 1•7 years ago
|
||
On a side note, it seems that in general, we run both tc(B)/tc(N) on most platforms on Beta and I'm wondering if there's more room for optimizing which builds we create there to save some resources. Fodder for a follow-up maybe :)
Assignee | ||
Updated•7 years ago
|
Severity: normal → major
Component: Scheduler → General Automation
Priority: -- → P1
Product: Taskcluster → Release Engineering
QA Contact: catlee
Comment hidden (mozreview-request) |
Comment 3•7 years ago
|
||
mozreview-review |
Comment on attachment 8897518 [details] Bug 1390375: Stop extra DE builds on Beta https://reviewboard.mozilla.org/r/168786/#review174146
Attachment #8897518 -
Flags: review?(bugspam.Callek) → review+
Assignee | ||
Updated•7 years ago
|
Assignee: nobody → catlee
Pushed by catlee@mozilla.com: https://hg.mozilla.org/integration/autoland/rev/b76e179f89c8 Stop extra DE builds on Beta r=Callek
Reporter | ||
Comment 5•7 years ago
|
||
bugherder uplift |
https://hg.mozilla.org/releases/mozilla-beta/rev/c9ec6aa5e7d2
status-firefox56:
--- → fixed
Reporter | ||
Comment 6•7 years ago
|
||
DevEdition runs look perfect on Beta now. Thanks for the patch, catlee!
Comment 7•7 years ago
|
||
bugherder |
https://hg.mozilla.org/mozilla-central/rev/b76e179f89c8
Updated•6 years ago
|
Component: General Automation → General
You need to log in
before you can comment on or make changes to this bug.
Description
•