Closed
Bug 1244720
Opened 9 years ago
Closed 9 years ago
Add last missing jobs, fix any wrong settings and enable more green jobs
Categories
(Testing :: General, defect)
Testing
General
Tracking
(firefox47 fixed)
RESOLVED
FIXED
mozilla47
Tracking | Status | |
---|---|---|
firefox47 | --- | fixed |
People
(Reporter: armenzg, Assigned: armenzg)
References
Details
Attachments
(8 files, 1 obsolete file)
(deleted),
patch
|
jmaher
:
review+
|
Details | Diff | Splinter Review |
(deleted),
patch
|
jmaher
:
review+
|
Details | Diff | Splinter Review |
(deleted),
patch
|
jmaher
:
review+
|
Details | Diff | Splinter Review |
(deleted),
patch
|
jmaher
:
review+
|
Details | Diff | Splinter Review |
(deleted),
patch
|
jmaher
:
review+
|
Details | Diff | Splinter Review |
(deleted),
patch
|
dustin
:
review+
|
Details | Diff | Splinter Review |
(deleted),
patch
|
dustin
:
review+
|
Details | Diff | Splinter Review |
(deleted),
patch
|
jmaher
:
review+
|
Details | Diff | Splinter Review |
No description provided.
Assignee | ||
Comment 1•9 years ago
|
||
Notes from Joel. Time to verify and fix!
````````````````````````````````````````
do we have Mn for taskcluster? also R-e10s(J) ?
we run jetpack on buildbot, and not on tc!
missing in TC:
Mn
wpt-e10s(wr-e10s)
m(oth)
and we are running tc-m-e10s(*) without the --e10s flag! I was looking into the dt jobs and realized that the plain, bc, and dt are all opt.
wpt(3) is failing in tc, but passing great in bb
wpt(4) is missing in both tc and bb
wpt-e10s(wr-e10s) is missing in tc
wpt-e10s(3 4) are missing in both tc bb (well 3 is failing bad and 4 is missing)
we are also running m-e10s(dt*) in tc, but I don't see them in the buildbot jobs on your try push...moar coverage :)
Assignee | ||
Comment 2•9 years ago
|
||
We don't run jetpack on debug builds
Added (in my next patch):
* Mn
* wr-e10s
* m-oth
Wrt to e10s jobs not using --e10s is very unfortunate. Even though --e10s is specified in the task definition!
I will investigate.
What do you mean by "opt"?
Wrt to m-e10s, that is fine. They had been forgotten for Buildbot. See bug 1242986.
Comment 3•9 years ago
|
||
I have jetpack definitions from buildbot on your try push:
https://treeherder.mozilla.org/#/jobs?repo=try&author=armenzg@mozilla.com&filter-searchStr=jetpack&exclusion_profile=false
We should turn off the jetpack scheduling or add it to taskcluster. No point in turning it off accidentally a few weeks from now.
I mis-spoke on opt, I meant non-e10s, I usually refer to talos jobs as opt or e10s. Looking forward to how these new jobs work out.
Assignee | ||
Comment 4•9 years ago
|
||
Wrt to --e10s:
https://public-artifacts.taskcluster.net/QPzXhw2rQvmYeVpFStRq8w/0/public/graph.json
wpt, reftests and crashtest uses it.
The problem is that one the same definition we have 'payload' a second time (woohoo!):
https://dxr.mozilla.org/mozilla-central/source/testing/taskcluster/tasks/tests/fx_linux64_mochitest_plain_e10s.yml#14
Assignee | ||
Comment 5•9 years ago
|
||
Attachment #8714375 -
Flags: review?(jmaher)
Comment 6•9 years ago
|
||
Comment on attachment 8714375 [details] [diff] [review]
fix mochitest e10s jobs + add jetpack definition
Review of attachment 8714375 [details] [diff] [review]:
-----------------------------------------------------------------
good find on the double payload definitions! yay for in-tree configs to shoot you in the foot :)
Attachment #8714375 -
Flags: review?(jmaher) → review+
Assignee | ||
Comment 7•9 years ago
|
||
This will make it load this flavour:
https://dxr.mozilla.org/mozilla-central/source/testing/mozharness/configs/unittests/linux_unittest.py#203
Assignee | ||
Comment 8•9 years ago
|
||
Assignee | ||
Updated•9 years ago
|
Attachment #8714393 -
Flags: review?(jmaher)
Assignee | ||
Comment 9•9 years ago
|
||
Comment 10•9 years ago
|
||
Comment on attachment 8714393 [details] [diff] [review]
Devtools are running without --chunk-by-runtime
Review of attachment 8714393 [details] [diff] [review]:
-----------------------------------------------------------------
nice!
Attachment #8714393 -
Flags: review?(jmaher) → review+
Assignee | ||
Comment 11•9 years ago
|
||
https://hg.mozilla.org/integration/mozilla-inbound/rev/6554c3c4f0ea0c27f58b356938ae67764cdb44c6
Bug 1244720 - TaskCluster Linux64 mochitest devtools are running without --chunk-by-runtime. DONTBUILD. r=jmaher
Assignee | ||
Comment 12•9 years ago
|
||
This has to go with attachement 8714375 (since mochitest e10s is running side by side on production).
Attachment #8714473 -
Flags: review?(jmaher)
Comment 13•9 years ago
|
||
Comment on attachment 8714473 [details] [diff] [review]
Disable mochitest e10s, increase Marionette and m-other runtime and fix jetpack
Review of attachment 8714473 [details] [diff] [review]:
-----------------------------------------------------------------
lots of little changes, thanks for making this smooth
Attachment #8714473 -
Flags: review?(jmaher) → review+
Assignee | ||
Comment 14•9 years ago
|
||
https://hg.mozilla.org/integration/mozilla-inbound/rev/1c6457541936f6df1ea797bfd545b906c738fcdf
Bug 1244720 - Modifications to tier 2 TaskCluster jobs. DONTBUILD. r=jmaher
Blocks: 1244936
Comment 15•9 years ago
|
||
bugherder |
https://hg.mozilla.org/mozilla-central/rev/6554c3c4f0ea
https://hg.mozilla.org/mozilla-central/rev/1c6457541936
Status: NEW → RESOLVED
Closed: 9 years ago
status-firefox47:
--- → fixed
Resolution: --- → FIXED
Target Milestone: --- → mozilla47
Assignee | ||
Comment 16•9 years ago
|
||
It's full of intermittent issues:
https://treeherder.mozilla.org/#/jobs?repo=mozilla-inbound&filter-searchStr=tc%20bc%20e10s&group_state=expanded&fromchange=650c2be3cbf3
Attachment #8714772 -
Flags: review?(jmaher)
Comment 17•9 years ago
|
||
Comment on attachment 8714772 [details] [diff] [review]
disable_bc.diff
Review of attachment 8714772 [details] [diff] [review]:
-----------------------------------------------------------------
sold
Attachment #8714772 -
Flags: review?(jmaher) → review+
Assignee | ||
Comment 18•9 years ago
|
||
https://hg.mozilla.org/integration/mozilla-inbound/rev/c3de1fe6ce6c1dcc5bbbd40c65ded15fc5a8c17f
Bug 1244720 - Disable TaskCluster tier2 e10s mochitest-browser-chrome. DONTBUILD r=jmaher
Comment 19•9 years ago
|
||
bugherder |
Assignee | ||
Updated•9 years ago
|
Assignee | ||
Comment 20•9 years ago
|
||
Attachment #8715847 -
Flags: review?(jmaher)
Assignee | ||
Comment 21•9 years ago
|
||
Comment 22•9 years ago
|
||
Comment on attachment 8715847 [details] [diff] [review]
Allow running Wr e10s
Review of attachment 8715847 [details] [diff] [review]:
-----------------------------------------------------------------
good find!
Attachment #8715847 -
Flags: review?(jmaher) → review+
Assignee | ||
Comment 23•9 years ago
|
||
https://hg.mozilla.org/integration/mozilla-inbound/rev/a7f2b93dd5645b22e82e7fa5dfef6a926e46ea83
Bug 1244720 - Allow scheduling on try Wr-e10s jobs for TaskCluster Linux64 debug builds. tier-2. DONTBUILD. r=jmaher
Comment 24•9 years ago
|
||
bugherder |
Assignee | ||
Comment 25•9 years ago
|
||
Because they're borderlining 60 minutes.
Attachment #8717096 -
Flags: review?(dustin)
Updated•9 years ago
|
Attachment #8717096 -
Flags: review?(dustin) → review+
Assignee | ||
Comment 26•9 years ago
|
||
Since dt6 and dt8 are bordering 60 minutes.
Attachment #8717155 -
Flags: review?(dustin)
Updated•9 years ago
|
Attachment #8717155 -
Flags: review?(dustin) → review+
Assignee | ||
Comment 27•9 years ago
|
||
Comment on attachment 8717155 [details] [diff] [review]
Bump devtools timeout from 60 to 90 minutes
It seems this landed with https://reviewboard.mozilla.org/r/33831/diff/2#index_header
I was based on m-c.
I decided we don't want to go from 10 to 8 chunks.
Attachment #8717155 -
Attachment is obsolete: true
Assignee | ||
Comment 28•9 years ago
|
||
https://hg.mozilla.org/integration/mozilla-inbound/rev/1e5acf7ffffc538147ec6c56663f6ffce32f2a77
Bug 1244720 - TaskCluster tier2 jobs: bump wpt reftests from 60 to 90 minutes time out. DONTBUILD. r=dustin
Assignee | ||
Comment 29•9 years ago
|
||
Comment 30•9 years ago
|
||
bugherder |
Assignee | ||
Comment 31•9 years ago
|
||
Assignee | ||
Comment 32•9 years ago
|
||
https://treeherder.mozilla.org/#/jobs?repo=try&revision=88d1caed025e&group_state=expanded&selectedJob=16559429
Wr has recently become very intermittent.
I've a patch to fix it.
Attachment #8718044 -
Flags: review?(dustin)
Updated•9 years ago
|
Attachment #8718044 -
Flags: review?(dustin) → review+
Assignee | ||
Comment 33•9 years ago
|
||
https://hg.mozilla.org/integration/mozilla-inbound/rev/ab6e685f82b0b38abb888e127755d62493c3e799
Bug 1244720 - Enable more green TaskCluster tier-2 jobs. r=dustin
Comment 34•9 years ago
|
||
bugherder |
Assignee | ||
Comment 35•9 years ago
|
||
We can wait to land once we have the last two issues greened up.
Attachment #8720288 -
Flags: review?(jmaher)
Comment 36•9 years ago
|
||
Comment on attachment 8720288 [details] [diff] [review]
Enable mochitest-plain-e10s and mochitest-browser-chrome-e10s
Review of attachment 8720288 [details] [diff] [review]:
-----------------------------------------------------------------
yes, once we are green- we might need to disable tests at the same time.
Attachment #8720288 -
Flags: review?(jmaher) → review+
Assignee | ||
Comment 37•9 years ago
|
||
https://hg.mozilla.org/integration/mozilla-inbound/rev/d8f9f159cee2b704177973576215c4f7d83d0a90
Bug 1244720 - Enable mochitest plain and mochitest browser-chrome for e10s. tier-2 jobs. r=jmaher
Assignee | ||
Updated•9 years ago
|
Status: REOPENED → RESOLVED
Closed: 9 years ago → 9 years ago
Resolution: --- → FIXED
Comment 38•9 years ago
|
||
bugherder |
Comment 39•7 years ago
|
||
Removing leave-open keyword from resolved bugs, per :sylvestre.
Keywords: leave-open
You need to log in
before you can comment on or make changes to this bug.
Description
•