Closed
Bug 1366404
Opened 8 years ago
Closed 8 years ago
Add AArch64 Nightly builds
Categories
(Firefox Build System :: Android Studio and Gradle Integration, enhancement)
Tracking
(firefox55 fixed)
RESOLVED
FIXED
mozilla55
Tracking | Status | |
---|---|---|
firefox55 | --- | fixed |
People
(Reporter: jchen, Assigned: jchen)
References
Details
Attachments
(2 files, 1 obsolete file)
No description provided.
Comment hidden (mozreview-request) |
Assignee | ||
Updated•8 years ago
|
Assignee: nobody → nchen
Status: NEW → ASSIGNED
Comment 2•8 years ago
|
||
mozreview-review |
Comment on attachment 8870100 [details]
Bug 1366404 - Add AArch64 Nightly build;
https://reviewboard.mozilla.org/r/141582/#review145578
OK, this looks fine by me, but let's get Dustin's additonal review (or redirect), since this touches more things than I am expert in (`taskcluster/taskgraph/*` in particular).
I would like you to explicitly state why this is not `-api-21` as well. I see that you've cloned `android-x86`, but are you really confident that we'll not want to split `aarch64` by API if and when we do the split? (It's quite likely we _wouldn't_ split `x86` since it's basically a dead architecture in the hardware device market.) I'm happy with this for now -- the work to tag by API is roughly the same now as it is later -- but it's worth stating explicitly that you understand this and have a reason for the choice.
Please file follow-up to ensure artifact builds get filled. Thanks!
::: commit-message-c800b:5
(Diff revision 1)
> +Bug 1366404 - Add AArch64 Nightly build; r?nalexander
> +
> +Add configurations for building and uploading AArch64 Nightly builds.
> +Right now the builds are in tier 1, but I'm not sure if they should be
> +in tier 2.
I think you and your team get the "right of first refusal": if you think that patches that burn the aarch64 build should be backed out with prejudice, you should be able to start with that policy until it proves onerous.
::: commit-message-c800b:7
(Diff revision 1)
> +
> +Add configurations for building and uploading AArch64 Nightly builds.
> +Right now the builds are in tier 1, but I'm not sure if they should be
> +in tier 2.
> +
> +I also didn't want to worry about Artifact builds for this patch, but I
Nah, happy to have it as follow-up. I reckon you'll just need to bump `artifacts.py` to know about your architecture.
Attachment #8870100 -
Flags: review?(nalexander) → review+
Updated•8 years ago
|
Attachment #8870100 -
Flags: review?(dustin)
Assignee | ||
Comment 3•8 years ago
|
||
(In reply to Nick Alexander :nalexander from comment #2)
> Comment on attachment 8870100 [details]
> Bug 1366404 - Add AArch64 Nightly build;
>
> https://reviewboard.mozilla.org/r/141582/#review145578
>
> I would like you to explicitly state why this is not `-api-21` as well. I
> see that you've cloned `android-x86`, but are you really confident that
> we'll not want to split `aarch64` by API if and when we do the split? (It's
> quite likely we _wouldn't_ split `x86` since it's basically a dead
> architecture in the hardware device market.) I'm happy with this for now --
> the work to tag by API is roughly the same now as it is later -- but it's
> worth stating explicitly that you understand this and have a reason for the
> choice.
Ok. I don't really think we will split AArch64 the way we split ARMv7 before. Originally, we split into API 9 and API 11+ because of lots of "constrained" devices that were stuck with API 9. We made an API 9 APK in order to lower our footprint on those devices. That probably will not be a problem for AArch64, because devices with API 21+ and AArch64 support are usually more than capable for running Fennec.
Secondly, it was a big change for Android going from API 9 to API 11+, so we saved quite a bit of code/resources when we stripped out API 11+. I don't see such drastic changes going from API 21 to upcoming versions, so even if we did split, I don't think it'll get us much benefit.
Comment 4•8 years ago
|
||
mozreview-review |
Comment on attachment 8870100 [details]
Bug 1366404 - Add AArch64 Nightly build;
https://reviewboard.mozilla.org/r/141582/#review146134
Attachment #8870100 -
Flags: review?(dustin) → review+
Pushed by nchen@mozilla.com:
https://hg.mozilla.org/integration/mozilla-inbound/rev/a62c6d515e19
Add AArch64 Nightly build; r=nalexander r=dustin
Comment 6•8 years ago
|
||
bugherder |
Status: ASSIGNED → RESOLVED
Closed: 8 years ago
status-firefox55:
--- → fixed
Resolution: --- → FIXED
Target Milestone: --- → Firefox 55
Comment 7•8 years ago
|
||
This is causing beetmover bustage; we probably need to copy this comment [1] above every single artifact constant, as well as land a beetmoverscript patch.
[1] https://hg.mozilla.org/mozilla-central/file/a62c6d515e19/taskcluster/taskgraph/transforms/beetmover.py#l18
Comment 8•8 years ago
|
||
https://hg.mozilla.org/integration/mozilla-inbound/rev/d9530f3895709efb48ee662fb2561bda2640ce09
bug 1366404 - add comments before every beetmover artifact list. r=comment DONTBUILD
Comment 9•8 years ago
|
||
https://hg.mozilla.org/build/puppet/rev/196c31048845a6bb90b157bab81b0ca7bb951c5c
bug 1366404 - update to beetmoverscript 0.5.4 for aarch64. r=callek
Comment 10•8 years ago
|
||
Attachment #8871919 -
Flags: review?(mtabara)
Comment 11•8 years ago
|
||
Comment on attachment 8871919 [details] [diff] [review]
aarch64_puppet.diff
Review of attachment 8871919 [details] [diff] [review]:
-----------------------------------------------------------------
r+ with following comments:
::: modules/beetmover_scriptworker/templates/script_config.json.erb
@@ +18,5 @@
> "firefox_nightly_repacks": "<%= scope.lookupvar("beetmover_scriptworker::settings::root") %>/lib/python3.5/site-packages/beetmoverscript/templates/firefox_nightly_repacks.yml",
> "fennec_nightly": "<%= scope.lookupvar("beetmover_scriptworker::settings::root") %>/lib/python3.5/site-packages/beetmoverscript/templates/fennec_nightly.yml",
> "fennec_nightly_repacks": "<%= scope.lookupvar("beetmover_scriptworker::settings::root") %>/lib/python3.5/site-packages/beetmoverscript/templates/fennec_nightly_repacks.yml",
> "fennecx86_nightly": "<%= scope.lookupvar("beetmover_scriptworker::settings::root") %>/lib/python3.5/site-packages/beetmoverscript/templates/fennecx86_nightly.yml"
> + "fennecaarch64_nightly": "<%= scope.lookupvar("beetmover_scriptworker::settings::root") %>/lib/python3.5/site-packages/beetmoverscript/templates/fennecarch64_nightly.yml"
Might be a syntax error here "fennecarch64" instead of "fennecaarch64". I don't see any "fennecarch64" under https://github.com/mozilla-releng/beetmoverscript/pull/65/files.
@@ +28,5 @@
> "firefox_nightly_repacks": "<%= scope.lookupvar("beetmover_scriptworker::settings::root") %>/lib/python3.5/site-packages/beetmoverscript/templates/firefox_nightly_repacks.yml",
> "fennec_nightly": "<%= scope.lookupvar("beetmover_scriptworker::settings::root") %>/lib/python3.5/site-packages/beetmoverscript/templates/fennec_nightly.yml",
> "fennec_nightly_repacks": "<%= scope.lookupvar("beetmover_scriptworker::settings::root") %>/lib/python3.5/site-packages/beetmoverscript/templates/fennec_nightly_repacks.yml",
> "fennecx86_nightly": "<%= scope.lookupvar("beetmover_scriptworker::settings::root") %>/lib/python3.5/site-packages/beetmoverscript/templates/fennecx86_nightly.yml"
> + "fennecarch64_nightly": "<%= scope.lookupvar("beetmover_scriptworker::settings::root") %>/lib/python3.5/site-packages/beetmoverscript/templates/fennecaarch64_nightly.yml"
Same as above.
Attachment #8871919 -
Flags: review?(mtabara) → review+
Comment 12•8 years ago
|
||
Attachment #8871919 -
Attachment is obsolete: true
Attachment #8871926 -
Flags: review?(mtabara)
Comment 13•8 years ago
|
||
Comment on attachment 8871926 [details] [diff] [review]
aarch64_puppet.diff with typo fixes
Carrying forward r+
Attachment #8871926 -
Flags: review?(mtabara) → review+
Comment 14•8 years ago
|
||
https://hg.mozilla.org/build/puppet/rev/40307b8b89a41f4a47825f22e22e563c51566f32
bug 1366404 - add fennecaarch64 templates for beetmover. r=mtabara
Comment 15•8 years ago
|
||
https://hg.mozilla.org/build/puppet/rev/cbdb0eadc8444d6a8d3aa415ed00b2acd50422bd
bug 1366404 - add commas. r=bustage
Comment 16•8 years ago
|
||
Comment 17•8 years ago
|
||
bugherder |
Updated•6 years ago
|
Updated•6 years ago
|
Updated•6 years ago
|
Product: Firefox for Android → Firefox Build System
Target Milestone: Firefox 55 → mozilla55
You need to log in
before you can comment on or make changes to this bug.
Description
•