Closed
Bug 1319546
Opened 8 years ago
Closed 8 years ago
TC Android x86 nightlies as Tier-2
Categories
(Firefox Build System :: Task Configuration, task)
Firefox Build System
Task Configuration
Tracking
(Not tracked)
RESOLVED
FIXED
People
(Reporter: coop, Assigned: kmoir)
References
Details
Attachments
(4 files, 1 obsolete file)
We're missing Fennec x86 builds as nightlies in TC, although we have the builds running at tier 2 already.
Kim has volunteered to take this on.
Assignee | ||
Comment 1•8 years ago
|
||
Try run here
https://treeherder.mozilla.org/#/jobs?repo=try&revision=9958454e5841dd54c4df8ec11adcb160d3677487
(I fixed some lint errors in patch too)
Assignee | ||
Comment 2•8 years ago
|
||
Comment on attachment 8813765 [details] [diff] [review]
bug1319546.patch
iirc android x86 doesn't have l10n
Attachment #8813765 -
Flags: review?(jlund)
Reporter | ||
Updated•8 years ago
|
Reporter | ||
Updated•8 years ago
|
Summary: Fennec x86 nightlies in TC as tier 2 → TC Android x86 nightlies as Tier-2
Comment 3•8 years ago
|
||
Comment on attachment 8813765 [details] [diff] [review]
bug1319546.patch
Review of attachment 8813765 [details] [diff] [review]:
-----------------------------------------------------------------
\o/ two in-line comments before landing.
::: taskcluster/ci/build/android.yml
@@ +56,5 @@
> + job-name: android-x86-nightly-opt
> + treeherder:
> + platform: android-4-2-x86/opt
> + symbol: tc(B)
> + tier: 1
s/1/2
@@ +67,5 @@
> + actions: [get-secrets build multi-l10n update]
> + config:
> + - builds/releng_base_android_64_builds.py
> + - disable_signing.py
> + - platform_supports_post_upload_to_latest.py
I think we will need something like:
- taskcluster_nightly.py
here to tell mozharness to make a nightly
Attachment #8813765 -
Flags: review?(jlund) → review-
Assignee | ||
Comment 4•8 years ago
|
||
Attachment #8813765 -
Attachment is obsolete: true
Attachment #8813923 -
Flags: review?(jlund)
Updated•8 years ago
|
Attachment #8813923 -
Flags: review?(jlund) → review+
Assignee | ||
Comment 5•8 years ago
|
||
Assignee | ||
Comment 6•8 years ago
|
||
git hub PR, not sure if additional changes are needed
Assignee | ||
Comment 7•8 years ago
|
||
So I deployed the new beetmover script yesterday but it still moves the arm artifacts within the x86 task. Not sure why this is happening, debugging
https://tools.taskcluster.net/task-group-inspector/#/OuMsoGv2RwCzBY9VeH_tCA/cyYsVTaBTiWeF4PIqfdPDg?_k=0pl10c
Assignee | ||
Comment 8•8 years ago
|
||
aki> i have an idea, digging to see if there's anything to it
12:02 PM these template hardcode arm https://github.com/mozilla-releng/beetmoverscript/blob/master/beetmoverscript/templates/fennec_nightly_en_us_multi_signed.yml
12:02 PM s
12:03 PM
<kmoir> Kim Moir right but I made new templates for x86
12:03 PM I'm not sure where the call to the specific template is
12:03 PM
<Callek> kmoir: could it be related to https://dxr.mozilla.org/build-central/source/puppet/modules/beetmover_scriptworker/templates/script_config.json.erb#5
12:04 PM kmoir: related: https://github.com/mozilla-releng/beetmoverscript/blob/d75bdd87b60d49049e4c5ae7cc6bab9fafad4075/beetmoverscript/utils.py#L64
12:04 PM
<kmoir> Kim Moir ah okay I didn't see that puppet part, thanks aki
12:05 PM
<aki> itym callek :)
12:05 PM
<Callek> kmoir: but yea, note the template_key piece as well, since I believe its finding the non x86 version because of that
12:05 PM
<kmoir> Kim Moir okay
12:06 PM
<aki> i'm thinking once we get this working, it might be nice to allow for beetmoving multiple locales in one pass
12:07 PM
<Callek> kmoir: my current thinking, for way forward --- don't change the puppet config -- do something like `{% set locale_prefix = 'en-US/' if locale == 'en-US' else '' %}` in the existing fennec templates, to switch between android-arm and android-i386, based on platform...
Assignee | ||
Comment 9•8 years ago
|
||
Assignee | ||
Comment 10•8 years ago
|
||
Attachment #8815853 -
Flags: review?(bugspam.Callek)
Assignee | ||
Updated•8 years ago
|
Attachment #8815852 -
Flags: review?(bugspam.Callek)
Updated•8 years ago
|
Attachment #8815852 -
Flags: review?(bugspam.Callek) → review+
Updated•8 years ago
|
Attachment #8815853 -
Flags: review?(bugspam.Callek) → review+
Assignee | ||
Comment 11•8 years ago
|
||
verified that last beetmover changes work and move the x86 artifacts
Status: NEW → RESOLVED
Closed: 8 years ago
Resolution: --- → FIXED
Updated•7 years ago
|
Product: TaskCluster → Firefox Build System
You need to log in
before you can comment on or make changes to this bug.
Description
•