Closed
Bug 1290993
Opened 5 years ago
Closed 5 years ago
Android mochitests and reftests all failing on Beta because of unsigned addons | remoteautomation.py | application timed out after 330 seconds with no output
Categories
(Testing :: General, defect)
Testing
General
Tracking
(Not tracked)
RESOLVED
FIXED
People
(Reporter: KWierso, Assigned: kmoir)
References
Details
(Keywords: intermittent-failure)
Attachments
(1 file)
695 bytes,
patch
|
mshal
:
review+
|
Details | Diff | Splinter Review |
After the merge of 49 to beta, Android tests started failing en masse. In the logs, I spotted these lines: 08:33:57 INFO - 08-01 08:26:42.836 I/Gecko ( 2211): 1470065202838 addons.xpi WARN Refusing to install staged add-on special-powers@mozilla.org with signed state 0 08:33:57 INFO - 08-01 08:26:43.026 I/Gecko ( 2211): 1470065203025 addons.xpi WARN Refusing to install staged add-on mochikit@mozilla.org with signed state 0 http://archive.mozilla.org/pub/mobile/tinderbox-builds/mozilla-beta-android-api-15/1470060053/mozilla-beta_ubuntu64_vm_armv7_large_test-mochitest-1-bm123-tests1-linux64-build1.txt.gz Are we requiring signed addons for Fennec now?
Reporter | ||
Updated•5 years ago
|
Keywords: intermittent-failure
Comment 2•5 years ago
|
||
Andy is on PTO this week but we have always required signed addons for all release versions, the big recent change was bug 1186522. Prior to that change, signing could be disabled in automation, but after that change, it can not be disabled. There was try run on the beta branch about a month ago to test that change, but is it possible that something changed with the way specialpowers gets loaded on Android since then?
Flags: needinfo?(amckay)
Reporter | ||
Comment 3•5 years ago
|
||
Beta is closed until this can get sorted out, for the record.
Comment 4•5 years ago
|
||
Regular desktop mochitests load special powers via Marionette using this code: https://dxr.mozilla.org/mozilla-central/source/testing/marionette/client/marionette_driver/addons.py#88 The reference to "staged add-on" in the log message above suggests that Fennec uses a different technique, one that is broken by bug 1186522. But here is an example of a try run that succeeded in the last release cycle: https://treeherder.mozilla.org/#/jobs?repo=mozilla-beta&revision=35e0d7dd7cd7bff301b75756361b36b275102404 Since that run succeeded, that suggests that something changed with the way tests are run (or specifically, with the way test extensions are loaded) on fennec between 48 and 49? I have to admit I'm pretty ignorant about how tests work on Fennec, not even sure who to ni? who could help...
Comment 5•5 years ago
|
||
I should have read more closely, those try runs weren't actually requiring signing on fennec and given comments on bug 1233199 I guess it is known that there is remaining work to be done before requiring signing on fennec. Reading through various patches on bug 1186522 I got myself thoroughly confused but I think the changes to mobile/android/config/mozconfigs/common.override from https://hg.mozilla.org/releases/mozilla-beta/rev/35e0d7dd7cd7bff301b75756361b36b275102404 ought to actually be checked in everywhere? (ie, central, aurora, and beta)
Flags: needinfo?(kmoir)
Comment 6•5 years ago
|
||
ahal did some of the initial work on moving Firefox over to use Marionette to inject addons to circumvent the addon signing tyranny.
Assignee | ||
Comment 7•5 years ago
|
||
Android addons should not be signed on any branch. It looks like that patch that prevented this on beta on fennec was overwritten in the merge. I'll attach a patch to fix.
Flags: needinfo?(kmoir)
Assignee | ||
Comment 8•5 years ago
|
||
I'll look at the merge script too to see what this happened on uplift.
Attachment #8776929 -
Flags: review?(mshal)
Assignee | ||
Comment 9•5 years ago
|
||
Looking at the merge script, it only does replacements. So I think we should land the previous patch on all other branches until we are ready to enable signing on Fennec, at which point we can update the merge script.
Comment 11•5 years ago
|
||
Comment on attachment 8776929 [details] [diff] [review] bug1290993.patch It looks like build/mozconfig.common now sets MOZ_REQUIRE_SIGNING to 1 by default on beta. Is that the line that's generated by the merge script?
Attachment #8776929 -
Flags: review?(mshal) → review+
Assignee | ||
Comment 12•5 years ago
|
||
Yes, we want that in the merge script. The merge script doesn't set the preference on beta in mobile/android/config/mozconfigs/common.override to this MOZ_REQUIRE_SIGNING=0 which caused the test failures. So I'll land this patch on trunk and then request permission to land on m-b and m-a. We don't have a date set for when we will enable the preference on beta for Android.
Comment 13•5 years ago
|
||
Pushed by kmoir@mozilla.com: https://hg.mozilla.org/integration/mozilla-inbound/rev/8921d7e94257 Android mochitests and reftests all failing on Beta because of unsigned addons | remoteautomation.py | application timed out after 330 seconds with no output r=mshal DONTBUILD
Assignee | ||
Comment 14•5 years ago
|
||
Comment on attachment 8776929 [details] [diff] [review] bug1290993.patch Approval Request Comment [Feature/regressing bug #]:1290993 [User impact if declined]: test failures will continue on android [Describe test coverage new/current, TreeHerder]: currently android tests are failing because the preference is enabled for addons on android, no new tests needed [Risks and why]: if this preference is not enabled, android tests will continue to fail [String/UUID change made/needed]:n/a
Attachment #8776929 -
Flags: approval-mozilla-beta?
Assignee | ||
Comment 15•5 years ago
|
||
Comment on attachment 8776929 [details] [diff] [review] bug1290993.patch Approval Request Comment [Feature/regressing bug #]:1290993 [User impact if declined]: test failures will continue on android [Describe test coverage new/current, TreeHerder]: currently android tests are failing because the preference is enabled for addons on android, no new tests needed [Risks and why]: if this preference is not enabled, android tests will continue to fail [String/UUID change made/needed]:n/a
Attachment #8776929 -
Flags: approval-mozilla-aurora?
Assignee | ||
Updated•5 years ago
|
Assignee: nobody → kmoir
Assignee | ||
Comment 16•5 years ago
|
||
Comment on attachment 8776929 [details] [diff] [review] bug1290993.patch uploaded to m-a and m-b since they are only test issues Looks like the test failures are fixed on on beta
Attachment #8776929 -
Flags: approval-mozilla-beta?
Attachment #8776929 -
Flags: approval-mozilla-aurora?
Assignee | ||
Updated•5 years ago
|
Status: NEW → RESOLVED
Closed: 5 years ago
Resolution: --- → FIXED
Comment 17•5 years ago
|
||
bugherder |
https://hg.mozilla.org/mozilla-central/rev/8921d7e94257
You need to log in
before you can comment on or make changes to this bug.
Description
•