Closed
Bug 1058362
Opened 10 years ago
Closed 10 years ago
Perma-failures on b-i, ime_uninstallation_test.js, uninstallation_test.js
Categories
(Firefox OS Graveyard :: Gaia, defect)
Tracking
(Not tracked)
RESOLVED
FIXED
2.1 S3 (29aug)
People
(Reporter: kgrandon, Assigned: tedders1)
References
Details
(Whiteboard: [systemsfe])
Attachments
(1 file, 1 obsolete file)
Suspecting this is caused by bug 1042797. https://tbpl.mozilla.org/php/getParsedLog.php?id=46702680&tree=B2g-Inbound
Reporter | ||
Comment 1•10 years ago
|
||
Ted - can you try running these tests locally to verify that we can still uninstall keyboards? The failing tests are: apps/keyboard/test/marionette/uninstallation_test.js apps/settings/test/marionette/tests/ime_uninstallation_test.js
Flags: needinfo?(tclancy)
Reporter | ||
Comment 2•10 years ago
|
||
And it appears the whole world started failing like crazy on the latest b2g-inbound: https://tbpl.mozilla.org/php/getParsedLog.php?id=46741568&tree=B2g-Inbound
Comment 3•10 years ago
|
||
It seems some test fixing patches were not landed correctly, see bug 1000315 comment 65. This is a patch to fix those tests, and could run locally. Let's see how it runs on Gaia-Try.
Attachment #8478793 -
Flags: review?(kgrandon)
Reporter | ||
Comment 4•10 years ago
|
||
Comment on attachment 8478793 [details] [review] Patch V1 Argh, R+ assuming we get a green try run. Thanks so much for tracking this down.
Attachment #8478793 -
Flags: review?(kgrandon) → review+
Reporter | ||
Comment 5•10 years ago
|
||
Just an update that Ted thinks this patch won't fix it due to some changes last week. If we don't get a green try run from it, and Ted can't fix it, then we will backout the regressing patches.
Reporter | ||
Comment 6•10 years ago
|
||
If we backout, we would backout all of these patches: https://tbpl.mozilla.org/?tree=B2g-Inbound&showall=1&rev=4b37cddaed77 As well as gaia changes here: https://hg.mozilla.org/integration/b2g-inbound/rev/a840609deb46
Assignee | ||
Comment 7•10 years ago
|
||
Assignee: nobody → tclancy
Attachment #8478793 -
Attachment is obsolete: true
Status: NEW → ASSIGNED
Attachment #8478857 -
Flags: review?(timdream)
Flags: needinfo?(tclancy)
Updated•10 years ago
|
Attachment #8478857 -
Flags: review?(timdream) → review+
Comment 8•10 years ago
|
||
(In reply to Kevin Grandon :kgrandon from comment #5) > Just an update that Ted thinks this patch won't fix it due to some changes > last week. If we don't get a green try run from it, and Ted can't fix it, > then we will backout the regressing patches. It turned out Rudy's patch turned green (not really green, just failed on some other stuff and not the one to be fixed.) but after discussion with Rudy he agreed we should land Ted's fix as apparently it's formatted in a better way. So, this is the Try run we are now waiting on: https://tbpl.mozilla.org/?rev=27ba765e270d58138fbb5f2134944566a24b0925&tree=Gaia-Try
Comment 9•10 years ago
|
||
master: https://github.com/mozilla-b2g/gaia/commit/c710c7fac2987bc0f3ea76a2361fe16e995468a0
Status: ASSIGNED → RESOLVED
Closed: 10 years ago
Resolution: --- → FIXED
Updated•10 years ago
|
Whiteboard: [systemsfe]
Target Milestone: --- → 2.1 S3 (29aug)
You need to log in
before you can comment on or make changes to this bug.
Description
•