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)

task
Not set
normal

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.
Attached patch bug1319546.patch (obsolete) (deleted) — Splinter Review
Comment on attachment 8813765 [details] [diff] [review] bug1319546.patch iirc android x86 doesn't have l10n
Attachment #8813765 - Flags: review?(jlund)
Blocks: 1267430
No longer blocks: 1267422
No longer blocks: 1267430
Summary: Fennec x86 nightlies in TC as tier 2 → TC Android x86 nightlies as Tier-2
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-
Attached patch bug1319546-2.patch (deleted) — Splinter Review
Attachment #8813765 - Attachment is obsolete: true
Attachment #8813923 - Flags: review?(jlund)
Attachment #8813923 - Flags: review?(jlund) → review+
git hub PR, not sure if additional changes are needed
Blocks: 1321006
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
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...
Attached patch bug1319546puppet.patch (deleted) — Splinter Review
Attached file pull request for beetmover (deleted) —
Attachment #8815853 - Flags: review?(bugspam.Callek)
Attachment #8815852 - Flags: review?(bugspam.Callek)
Attachment #8815852 - Flags: review?(bugspam.Callek) → review+
Attachment #8815853 - Flags: review?(bugspam.Callek) → review+
verified that last beetmover changes work and move the x86 artifacts
Status: NEW → RESOLVED
Closed: 8 years ago
Resolution: --- → FIXED
Product: TaskCluster → Firefox Build System
You need to log in before you can comment on or make changes to this bug.

Attachment

General

Created:
Updated:
Size: