Closed
Bug 1255078
Opened 9 years ago
Closed 9 years ago
Android Beta tests broken: Found installer url http://archive.mozilla.org/pub/mobile/tinderbox-builds/mozilla-beta-android-api-15-debug/1457532253/bouncer.apk .
Categories
(Release Engineering :: General, defect)
Release Engineering
General
Tracking
(Not tracked)
RESOLVED
FIXED
People
(Reporter: nalexander, Assigned: aselagea)
References
Details
Attachments
(1 file)
(deleted),
patch
|
catlee
:
review+
aselagea
:
checked-in+
|
Details | Diff | Splinter Review |
This isn't right:
Found installer url http://archive.mozilla.org/pub/mobile/tinderbox-builds/mozilla-beta-android-api-15-debug/1457532253/bouncer.apk.
that shouldn't be picking up the new bouncer APK, it should be picking up the fennec-* APK, like always.
Reporter | ||
Comment 2•9 years ago
|
||
catlee: can you direct me to a person who can fix the URL-determining logic in beta? This is different between {m-c, m-a}, and m-b (and release?).
Flags: needinfo?(catlee)
Comment 3•9 years ago
|
||
Are the artifacts that are uploaded different between m-a and m-b?
Flags: needinfo?(catlee)
Reporter | ||
Comment 4•9 years ago
|
||
(In reply to Chris AtLee [:catlee] from comment #3)
> Are the artifacts that are uploaded different between m-a and m-b?
I believe the artifacts are the same between m-a and m-b. Both should be uploading an additional bouncer.apk. I've seen bad handling of APKs before -- maybe we take alphabetically first, or whatever the filesystem lists first.
Flags: needinfo?(catlee)
Comment 5•9 years ago
|
||
I'm pretty sure this is due to https://dxr.mozilla.org/build-central/source/buildbotcustom/process/factory.py#197
m-b, m-r and esr are still using this code to do android builds, whereas m-c, m-a are using mozharness scripts.
Can we ignore bouncer.apk for the purposes of figuring out the package URL?
Flags: needinfo?(catlee)
Reporter | ||
Comment 6•9 years ago
|
||
(In reply to Chris AtLee [:catlee] from comment #5)
> I'm pretty sure this is due to
> https://dxr.mozilla.org/build-central/source/buildbotcustom/process/factory.
> py#197
>
> m-b, m-r and esr are still using this code to do android builds, whereas
> m-c, m-a are using mozharness scripts.
>
> Can we ignore bouncer.apk for the purposes of figuring out the package URL?
Are you asking me? If so, yes -- new APKs shouldn't change anything in automation. I can't do this work, though, since I can't schedule buildbot reconfigures. Can you assign this to a releng person?
Flags: needinfo?(catlee)
Comment 7•9 years ago
|
||
Yeah, I'm asking if the location of bouncer.apk is important to know for tests. It sounds like it's not.
Flags: needinfo?(catlee)
Assignee | ||
Updated•9 years ago
|
Assignee: nobody → alin.selagea
Assignee | ||
Comment 8•9 years ago
|
||
Attachment #8729019 -
Flags: review?(catlee)
Updated•9 years ago
|
Attachment #8729019 -
Flags: review?(catlee) → review+
Updated•9 years ago
|
Component: Build Config & IDE Support → General Automation
Product: Firefox for Android → Release Engineering
QA Contact: catlee
Reporter | ||
Comment 9•9 years ago
|
||
Comment on attachment 8729019 [details] [diff] [review]
bug_1255078.patch
If this is all this takes, can we try to be a little more future-proof? Why not actually try to find APKs that look like Fennec? For example, things that start with {fennec,firefox} and look like they have version numbers in them. This is going to break when we upload some other APK (for example, a robocop-debug.apk, which I tried before).
Comment 10•9 years ago
|
||
Because we hope this code is going away in the next few months.
Assignee | ||
Comment 11•9 years ago
|
||
@Nick: I think "robocop-debug.apk" would be handled fine. According to https://dxr.mozilla.org/build-central/source/buildbotcustom/process/factory.py#215, it should assign it to robocopApkUrl.
That being said, can I go on and land the patch? Or we should try to find some workaround for this?
Thanks.
Flags: needinfo?(catlee)
Reporter | ||
Comment 12•9 years ago
|
||
(In reply to Alin Selagea [:aselagea][:buildduty] from comment #11)
> @Nick: I think "robocop-debug.apk" would be handled fine. According to
> https://dxr.mozilla.org/build-central/source/buildbotcustom/process/factory.
> py#215, it should assign it to robocopApkUrl.
>
> That being said, can I go on and land the patch? Or we should try to find
> some workaround for this?
Thumbs up to land for me. If this cruft is going away (in favour of whatever we do on trunk), roll on.
Assignee | ||
Updated•9 years ago
|
Attachment #8729019 -
Flags: checked-in+
Comment 13•9 years ago
|
||
In production: https://hg.mozilla.org/build/buildbotcustom/rev/b4c6ed50c14c
Assignee | ||
Updated•9 years ago
|
Status: NEW → RESOLVED
Closed: 9 years ago
Flags: needinfo?(catlee)
Resolution: --- → FIXED
Updated•7 years ago
|
Component: General Automation → General
You need to log in
before you can comment on or make changes to this bug.
Description
•