Closed
Bug 929122
Opened 11 years ago
Closed 11 years ago
Disable currently failing debug B2G mochitests so we can get a green run on cedar
Categories
(Testing :: Mochitest, defect)
Testing
Mochitest
Tracking
(Not tracked)
RESOLVED
FIXED
People
(Reporter: jgriffin, Unassigned)
References
(Depends on 2 open bugs, Blocks 1 open bug)
Details
(Whiteboard: [leave-open])
Attachments
(4 files, 1 obsolete file)
(deleted),
patch
|
mozilla
:
review+
|
Details | Diff | Splinter Review |
(deleted),
patch
|
ahal
:
review+
|
Details | Diff | Splinter Review |
(deleted),
patch
|
gwagner
:
review+
|
Details | Diff | Splinter Review |
(deleted),
patch
|
gwagner
:
review+
|
Details | Diff | Splinter Review |
According to Jonas, we want to green up the debug B2G mochitests ASAP, and roll them out everywhere. So, we won't wait for the mochitest manifest work, but rather will add a new instance of b2g.json that we can use to exclude failing tests.
Reporter | ||
Comment 1•11 years ago
|
||
Will disable failing tests in a separate patch.
Attachment #819923 -
Flags: review?(ahalberstadt)
Comment 2•11 years ago
|
||
Comment on attachment 819923 [details] [diff] [review]
Add b2g-debug.json,
Review of attachment 819923 [details] [diff] [review]:
-----------------------------------------------------------------
This could get ugly fast if we keep creating these things :p
Attachment #819923 -
Flags: review?(ahalberstadt) → review+
Reporter | ||
Comment 3•11 years ago
|
||
Agreed. Let's hope we move to a manifest before this becomes a big headache.
Reporter | ||
Comment 4•11 years ago
|
||
https://hg.mozilla.org/integration/mozilla-inbound/rev/358f2e245f45
Now I need to get the mozharness script to use it.
Whiteboard: [leave-open]
Reporter | ||
Comment 5•11 years ago
|
||
I'm not sure if this is the preferred mechanism for doing something like this. I didn't see a way to do this in buildbot-configs without adding additional logic.
Attachment #819992 -
Flags: review?(aki)
Reporter | ||
Updated•11 years ago
|
Attachment #819923 -
Attachment is obsolete: true
Comment 6•11 years ago
|
||
Comment on attachment 819992 [details] [diff] [review]
Use b2g-debug.json for debug mochitests,
Yeah, I wish we had a better way to do this too.
Pretty sure this will do the trick.
Attachment #819992 -
Flags: review?(aki) → review+
Reporter | ||
Comment 7•11 years ago
|
||
(In reply to Aki Sasaki [:aki] from comment #6)
> Comment on attachment 819992 [details] [diff] [review]
> Use b2g-debug.json for debug mochitests,
>
> Yeah, I wish we had a better way to do this too.
> Pretty sure this will do the trick.
https://hg.mozilla.org/build/mozharness/rev/18e3bd278a5f
Comment 9•11 years ago
|
||
merged to production mozharness
Reporter | ||
Comment 10•11 years ago
|
||
This disables all the currently failing debug mochitests on B2G; we may need to disable others after this, since some failures are likely hiding others. Per jonas' advice, I've not filed bugs for each of these, but all these are marked with 'debug-only failure' to make them easy to find.
Attachment #824111 -
Flags: review?(ahalberstadt)
Comment 11•11 years ago
|
||
Comment on attachment 824111 [details] [diff] [review]
Disable failing debug mochitests on B2G,
Review of attachment 824111 [details] [diff] [review]:
-----------------------------------------------------------------
Looks good to me, though if you are only landing these on cedar for now, I don't think you need to get my r+ each time. I've been keeping a master patch in my local patch queue that I'll get review on once everythings ready to land.
Attachment #824111 -
Flags: review?(ahalberstadt) → review+
Reporter | ||
Comment 12•11 years ago
|
||
(In reply to Andrew Halberstadt [:ahal] from comment #11)
> Comment on attachment 824111 [details] [diff] [review]
> Disable failing debug mochitests on B2G,
>
> Review of attachment 824111 [details] [diff] [review]:
> -----------------------------------------------------------------
>
> Looks good to me, though if you are only landing these on cedar for now, I
> don't think you need to get my r+ each time. I've been keeping a master
> patch in my local patch queue that I'll get review on once everythings ready
> to land.
Good point!
https://hg.mozilla.org/projects/cedar/rev/d691b456f44b
Reporter | ||
Comment 13•11 years ago
|
||
This updates b2g-debug.json from the version that's currently running on pine.
Attachment #8357926 -
Flags: review?(anygregor)
Updated•11 years ago
|
Attachment #8357926 -
Flags: review?(anygregor) → review+
Reporter | ||
Comment 14•11 years ago
|
||
Reporter | ||
Comment 15•11 years ago
|
||
This seems to clean things up on pine.
Attachment #8358543 -
Flags: review?(anygregor)
Comment 16•11 years ago
|
||
Comment on attachment 8358543 [details] [diff] [review]
Disable a couple more debug B2G mochitests, DONTBUILD because NPOTB
Review of attachment 8358543 [details] [diff] [review]:
-----------------------------------------------------------------
Thanks!
So much green \o/
Attachment #8358543 -
Flags: review?(anygregor) → review+
Reporter | ||
Comment 17•11 years ago
|
||
https://hg.mozilla.org/integration/b2g-inbound/rev/746cbddb0b76
Hopefully this does it, but I'll leave this bug open until the tests are running on trunks branches, just in case.
Comment 18•11 years ago
|
||
Comment 19•11 years ago
|
||
mochitest-5 in particular is hitting a lot of orange. See the new deps. Doesn't help that the runtime is ~80min. I've also added a few other failures that we're hitting in the debug tests primarily.
Reporter | ||
Comment 20•11 years ago
|
||
I'm good with disabling these tests if they appear more than occasionally:
> Depends on: 923329, 961107, 922975, 961021, 960922, 960762, 931672, 960488, 947869
Reporter | ||
Comment 21•11 years ago
|
||
The most egregious are 923329, 961107, and 947869. I'll file a patch for disabling just these.
Reporter | ||
Comment 22•11 years ago
|
||
(In reply to Jonathan Griffin (:jgriffin) from comment #21)
> The most egregious are 923329, 961107, and 947869. I'll file a patch for
> disabling just these.
Nvm, philor beat me to it for the first two, gwagner already did the last.
Comment 23•11 years ago
|
||
And now bug 961021; disabling that test.
Reporter | ||
Updated•11 years ago
|
Assignee: jgriffin → nobody
Reporter | ||
Comment 24•11 years ago
|
||
These are running everywhere now, so am closing this meta-bug.
Status: NEW → RESOLVED
Closed: 11 years ago
Resolution: --- → FIXED
You need to log in
before you can comment on or make changes to this bug.
Description
•