Closed
Bug 1151699
Opened 9 years ago
Closed 9 years ago
Do something about the number of builds and tests run on Cypress
Categories
(Release Engineering :: General, defect)
Release Engineering
General
Tracking
(Not tracked)
RESOLVED
FIXED
People
(Reporter: philor, Assigned: mshal)
Details
(Whiteboard: [capacity])
Attachments
(2 files)
1.92 KB,
patch
|
coop
:
review+
mshal
:
checked-in+
|
Details | Diff | Splinter Review |
1.88 KB,
patch
|
catlee
:
review+
mshal
:
checked-in+
|
Details | Diff | Splinter Review |
Dunno what it actually needs, since it's just a stew of codenames, but I'm pretty sure it doesn't need what it has, ten or eleven full sets of desktop and Android Firefox builds and tests per day, when the only thing changing between them is a gaia commit, and the Gecko that they are rebuilding and retesting is nine days old. Could be that it wants per-product builds, or whatever it is we actually call only building b2g on commits which only touch b2g/ while building everything on commits that touch Gecko, or it could be that it only wants b2g builds and never wants Android or Firefox, or it could be that it only wants some subset of b2g builds.
Comment 1•9 years ago
|
||
We only needs b2g device builds, and there's no interesting tests to run on cypress that we don't run on m-c. It's even very possible that some are broken on cypress because I landed hacks on this "experimental" branch without updating patches until I land back on m-c.
Comment 2•9 years ago
|
||
We could just disable these tests entirely, really. There's not a lot happening on cypress, and there are only a few more minor patches that may or may not land on it only.
Reporter | ||
Comment 3•9 years ago
|
||
I closed Cypress, because I can't spend all day every day killing builds to keep it from running thousands of unwanted jobs. If it was a contributor pushing to try like that who kept going after being warned, we would remove his push access.
Severity: major → blocker
Assignee | ||
Comment 4•9 years ago
|
||
(In reply to Fabrice Desré [:fabrice] from comment #1) > We only needs b2g device builds Does this include the emulator builds? Or just nexus/flame/dolphin device builds?
Flags: needinfo?(fabrice)
Assignee | ||
Comment 5•9 years ago
|
||
Actually, dolphin appears to be limited to non-twig branches, so it's not currently running on cypress anyway.
Comment 6•9 years ago
|
||
(In reply to Michael Shal [:mshal] from comment #4) > (In reply to Fabrice Desré [:fabrice] from comment #1) > > We only needs b2g device builds > > Does this include the emulator builds? Or just nexus/flame/dolphin device > builds? I'm fine with doing only the nexus/flame builds.
Flags: needinfo?(fabrice)
Updated•9 years ago
|
Assignee: nobody → mshal
Assignee | ||
Comment 7•9 years ago
|
||
I just checked this with builder_lists.sh and it appears to be correct, but I'd appreciate it if you could double-check it.
Attachment #8590399 -
Flags: review?(coop)
Comment 8•9 years ago
|
||
Comment on attachment 8590399 [details] [diff] [review] disable-cypress Review of attachment 8590399 [details] [diff] [review]: ----------------------------------------------------------------- LGTM (using larch as the example for comparison)
Attachment #8590399 -
Flags: review?(coop) → review+
Assignee | ||
Comment 9•9 years ago
|
||
Comment on attachment 8590399 [details] [diff] [review] disable-cypress https://hg.mozilla.org/build/buildbot-configs/rev/9fef2e7bb9cf
Attachment #8590399 -
Flags: checked-in+
Assignee | ||
Comment 10•9 years ago
|
||
We should be able to turn cypress back on after a reconfig.
Assignee | ||
Comment 11•9 years ago
|
||
Seems we need to explicitly disable tests too, otherwise the travis checks aren't happy.
Updated•9 years ago
|
Attachment #8591043 -
Flags: review+
Assignee | ||
Comment 12•9 years ago
|
||
Comment on attachment 8591043 [details] [diff] [review] disable-cypress-tests https://hg.mozilla.org/build/buildbot-configs/rev/9ccdc8a458ec
Attachment #8591043 -
Flags: checked-in+
Comment 13•9 years ago
|
||
In production: https://hg.mozilla.org/build/buildbot-configs/rev/9fef2e7bb9cf
Comment 14•9 years ago
|
||
In production: https://hg.mozilla.org/build/buildbot-configs/rev/9ccdc8a458ec
Reporter | ||
Updated•9 years ago
|
Status: NEW → RESOLVED
Closed: 9 years ago
Resolution: --- → FIXED
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
•