1317469 - SHA512SUMS of Firefox for Android does not contain apk's

Status: VERIFIED FIXED

(Release Engineering :: Release Automation, defect)

Description

[Iceberg]

User Agent: Mozilla/5.0 (X11; Linux i686; rv:38.0) Gecko/20100101 Firefox/38.0
Build ID: 20160924153504

Steps to reproduce:

Open the page https://ftp.mozilla.org/pub/mobile/releases/49.0.2/


Actual results:

The file SHA512SUMS does not report the values of the apk downloadable files in the folders android-api-15 and android-x86


Expected results:

The file SHA512SUMS must report the values of all the apk files.

Comment 1

[Kevin Brosnan [Ex-Mozilla]]

Ben would you check that I've moved this to an appropriate component.

Comment 2

[Nick Thomas [:nthomas] (UTC+12)]

The most recent release (https://ftp.mozilla.org/pub/mobile/releases/50.0/SHA512SUMS) contains only the source tarball and bundle.

Comment 3

[Nick Thomas [:nthomas] (UTC+12)]

Attachment: [gecko] Include fennec.*\.apk

jlund, this should do the trick to pick up the fennec apk, while ignoring bouncer.apk & robocop.apk.

mconnor, this will also stop pushing bouncer.apk to locations like https://archive.mozilla.org/pub/mobile/releases/50.0/android-api-15/multi/bouncer.apk. Does that fit OK with our workflow for pre-installs ?

Comment 4

[Jordan Lund (:jlund)]

Comment on attachment 8811105 [details] [diff] [review]
[gecko] Include fennec.*\.apk

Review of attachment 8811105 [details] [diff] [review]:
-----------------------------------------------------------------

huh, looks like we haven't been providing a checksum for at least the last few releases. I suppose this hasn't been caught since this is a copy of the apk we ship via play store.

Comment 5

[Mike Kaply [:mkaply]]

Bouncer isn't really being used right now, so I don't think that's an issue.

Comment 6

[Pulsebot]

Pushed by [e-mail]:
https://hg.mozilla.org/integration/mozilla-inbound/rev/f3f76d129bfe
SHA512SUMS of Firefox for Android doesn't contain APKs, r=jlund DONTBUILD

Comment 7

[Nick Thomas [:nthomas] (UTC+12)]

Comment on attachment 8811105 [details] [diff] [review]
[gecko] Include fennec.*\.apk

https://hg.mozilla.org/integration/mozilla-inbound/rev/f3f76d129bfe5afcb9da08299c57686c7ee907dc
https://hg.mozilla.org/releases/mozilla-aurora/rev/96645fb4a91875e99b1037dfad6fab9079cbbfbd
https://hg.mozilla.org/releases/mozilla-beta/rev/da741a2c9f76e6d24857fdbfbbc9566376675d8b
https://hg.mozilla.org/releases/mozilla-release/rev/ee755a8e7cce6a39816d9844126c0729d01f13da

Should be fixed in 51.0b2, and 51.0.1 or 51.1.0 (whichever comes next).

Comment 8

[Nick Thomas [:nthomas] (UTC+12)]

(In reply to Nick Thomas [:nthomas] from comment #7)
> Should be fixed in 51.0b2, and 51.0.1 or 51.1.0 (whichever comes next).

I mean 51.0b2, and 50.0.1 or 50.1.0 (whichever comes next).

Comment 9

[Carsten Book [:Tomcat]]

https://hg.mozilla.org/mozilla-central/rev/f3f76d129bfe

Comment 10

[Nick Thomas [:nthomas] (UTC+12)]

Worked for 51.0b2 - https://archive.mozilla.org/pub/mobile/releases/51.0b2/SHA512SUMS
Missed 50.0.1 because the branch for that was cut earlier than ee755a8e7cce, but it'll be fixed for 50.1.0.