Closed
Bug 1005984
Opened 10 years ago
Closed 10 years ago
Don't generate APKs for apps incompatible with Android
Categories
(Marketplace Graveyard :: Integration, defect, P3)
Tracking
(Not tracked)
RESOLVED
FIXED
2014-05-27
People
(Reporter: mat, Assigned: kumar)
Details
(Whiteboard: [qa-])
STR: - Upload an app compatible with Firefox OS or Firefox Desktop only, *not* Android. - Observe in the logs that an APK is generated for it even though it's useless in mkt/webapps/models.py pre_generate_apk(), we should check for compatible devices before calling the actual task.
Assignee | ||
Updated•10 years ago
|
Assignee: nobody → kumar.mcmillan
Assignee | ||
Updated•10 years ago
|
Priority: -- → P3
Assignee | ||
Comment 1•10 years ago
|
||
https://github.com/mozilla/zamboni/pull/2076
Assignee | ||
Comment 2•10 years ago
|
||
For QA, it would be nice if you could also help verify that APKs are pre-generated. That is, in addition to the STR above: - upload a new Android app - install it on Android If the APK was pre-generated you will see it install very quickly. Otherwise, it will take a few seconds to install.
Status: NEW → RESOLVED
Closed: 10 years ago
Resolution: --- → FIXED
Target Milestone: --- → 2014-05-27
Comment 3•10 years ago
|
||
I have uploaded a packaged app on https://marketplace-dev.allizom.org/ and then installed it on my Android 4.2.1 device. It took like 20 seconds from the press of the "Free" button to the Install window. https://marketplace-dev.allizom.org/app/droid-packed/
Updated•10 years ago
|
Flags: needinfo?(kumar.mcmillan)
Assignee | ||
Comment 4•10 years ago
|
||
Could you try uploading a new app again? It looks like there was a problem but now it's fixed. When you uploaded the app, the manifest could not be fetched which was preventing the APK from being pre-generated. The manifest URL was returning a 400: PreGenAPKError: Error pre-generating APK for app 490148: Dev at https://marketplace-dev.allizom.org/app/f34d3c22-3efe-47ca-803d-6c740da1a851/manifest.webapp; generator=http://dapk.net/application.apk (SSL cert ok?); HTTPError: 400 Client Error: Bad Request Details: http://sentry.dmz.phx1.mozilla.com/marketplace-dev/marketplace-dev/group/19083/ However, I just tried the manifest URL and it's a 200 now: curl -v 'https://marketplace-dev.allizom.org/app/f34d3c22-3efe-47ca-803d-6c740da1a851/manifest.webapp'
Flags: needinfo?(kumar.mcmillan)
Comment 5•10 years ago
|
||
I have uploaded a new app but I have encountered the same issue again https://marketplace-dev.allizom.org/app/papple-packed/ It still takes about 20 seconds until the Install window is displayed.
Status: RESOLVED → REOPENED
Resolution: FIXED → ---
Assignee | ||
Comment 6•10 years ago
|
||
Thanks for helping us find this new issue. Since this may not be related to my changes, I opened bug 1017098 to track the specific problem where manifests can't be retrieved. It appears that the pre-generation task is indeed running so I'm closing this. Since it can't be verified until the 400 issue is fixed, I'll mark it qa-
Status: REOPENED → RESOLVED
Closed: 10 years ago → 10 years ago
Resolution: --- → FIXED
Whiteboard: [qa-]
You need to log in
before you can comment on or make changes to this bug.
Description
•