Closed Bug 1075607 Opened 5 years ago Closed 5 years ago

create mozilla-b2g34_v2_1 branch

Categories

(Release Engineering :: General, defect, P2)

defect

Tracking

(Not tracked)

RESOLVED FIXED

People

(Reporter: rail, Assigned: rail)

References

Details

Attachments

(9 files, 2 obsolete files)

+++ This bug was initially created as a clone of Bug #1037107 +++
Attached patch bg2_tagSplinter Review
This will be used to tag all live b2g branches. These tags are usually used by the security team.
Attachment #8498230 - Flags: review?(bhearsum)
Attachment #8498233 - Flags: review?(bhearsum)
Attached patch b2g34-tools.diffSplinter Review
Attachment #8498241 - Flags: review?(bhearsum)
Attached patch b2g43-tbpl.diffSplinter Review
Ryan, can I land and deploy this in advance, or should I wait until we get some build data?
Attachment #8498246 - Flags: review?(ryanvm)
Depends on: 1075629
Comment on attachment 8498246 [details] [diff] [review]
b2g43-tbpl.diff

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

No, you can land it whenever. Having it in the list isn't going to hurt anything.

Please file a Treeherder bug as well, though.
Attachment #8498246 - Flags: review?(ryanvm) → review+
Depends on: 1075639
Attachment #8498259 - Flags: review?(bhearsum)
Depends on: 1075650
Summary: create mozilla-b2g32_v2_1 branch → create mozilla-b2g34_v2_1 branch
Attached patch b2g34-buildbot-configs.diff (obsolete) — Splinter Review
Attachment #8498324 - Flags: review?(bhearsum)
Depends on: 1075708
Attachment #8498331 - Flags: review?(bhearsum)
Attached patch in-tree-l10n.diff (obsolete) — Splinter Review
Attachment #8498338 - Flags: review?(bhearsum)
With B2G_UPDATE_CHANNEL updated
Attachment #8498338 - Attachment is obsolete: true
Attachment #8498338 - Flags: review?(bhearsum)
Attachment #8498343 - Flags: review?(bhearsum)
Attachment #8498348 - Flags: review?(bhearsum)
Depends on: 1075743
Status: NEW → ASSIGNED
Attachment #8498230 - Flags: review?(bhearsum) → review+
Attachment #8498233 - Flags: review?(bhearsum) → review+
Attachment #8498241 - Flags: review?(bhearsum) → review+
Comment on attachment 8498259 [details] [diff] [review]
b2g43-graphs.diff

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

whyyyyyyyyyy does this still exist?
Attachment #8498259 - Flags: review?(bhearsum) → review+
Attachment #8498348 - Flags: review?(bhearsum) → review+
Attachment #8498331 - Flags: review?(bhearsum) → review+
Attachment #8498343 - Flags: review?(bhearsum) → review+
Comment on attachment 8498324 [details] [diff] [review]
b2g34-buildbot-configs.diff

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

There's a few typos to fix below.

::: mozilla/b2g_config.py
@@ +1605,5 @@
>  BRANCHES['mozilla-aurora']['platforms']['emulator-kk-debug']['enable_nightly'] = True
>  BRANCHES['mozilla-aurora']['platforms']['dolphin']['enable_nightly'] = True
>  BRANCHES['mozilla-aurora']['platforms']['dolphin_eng']['enable_nightly'] = True
>  
> +######## mozilla-aurora

please remove or fix this comment

@@ +1611,5 @@
> +# HGURL + repo_path should be a valid repository
> +BRANCHES['mozilla-b2g34_v2_1']['repo_path'] = 'releases/mozilla-b2g34_v2_1'
> +BRANCHES['mozilla-b2g34_v2_1']['gaia_l10n_root'] = 'https://hg.mozilla.org/releases/gaia-l10n/v2_1/'
> +BRANCHES['mozilla-b2g34_v2_1']['gecko_l10n_root'] = 'https://hg.mozilla.org/releases/l10n/mozilla-beta'
> +BRANCHES['mozilla-b2g34_v2_1']['start_hour'] = [0, 16]

We have a lot of branches with start hour 16 already...maybe we should start staggering a bit? With this, we'll have 4 things start at 16:02 - and that time of day is already very busy. We should probably stagger the start minutes at the very least.

::: mozilla/config.py
@@ +1816,5 @@
>      },
> +    'mozilla-b2g34_v2_1': {
> +        'branch_projects': [],
> +        'lock_platforms': True,
> +        'gecko_version': 32,

need to fix gecko version here.

@@ +2277,5 @@
>  BRANCHES['mozilla-b2g32_v2_0']['enable_valgrind'] = False
>  BRANCHES['mozilla-b2g32_v2_0']['enabled_products'] = ['firefox', 'mobile']
>  
> +######## mozilla-b2g34_v2_1
> +BRANCHES['mozilla-b2g34_v2_1']['repo_path'] = 'releases/mozilla-b2g32_v2_0'

repo name is wrong

@@ +2278,5 @@
>  BRANCHES['mozilla-b2g32_v2_0']['enabled_products'] = ['firefox', 'mobile']
>  
> +######## mozilla-b2g34_v2_1
> +BRANCHES['mozilla-b2g34_v2_1']['repo_path'] = 'releases/mozilla-b2g32_v2_0'
> +BRANCHES['mozilla-b2g34_v2_1']['update_channel'] = 'nightly-b2g34'

Make sure you rceate this in balrog after the first nightlies!
Attachment #8498324 - Flags: review?(bhearsum) → review-
Something here landed in production
Comment on attachment 8498259 [details] [diff] [review]
b2g43-graphs.diff

http://hg.mozilla.org/graphs/rev/50f94b442dae
Inserted into the DB as well
Attachment #8498259 - Flags: checked-in+
Interdiff: https://gist.github.com/rail/2991980b92c21536dead
Attachment #8498324 - Attachment is obsolete: true
Attachment #8500449 - Flags: review?(bhearsum)
Depends on: 1078439
Attachment #8500449 - Flags: review?(bhearsum) → review+
Remaining 3 patches will be landed on the merge day after pulling from aurora.
The automation looks working fine. Some of the builds failed due to out of date l10n repos, but this will be fixed on Monday.
Status: ASSIGNED → RESOLVED
Closed: 5 years ago
Resolution: --- → FIXED
No longer blocks: 1082322
Component: General Automation → General
You need to log in before you can comment on or make changes to this bug.