Closed Bug 1061861 Opened 10 years ago Closed 10 years ago

Tracking bug for 13-oct-2014 migration work

Categories

(Release Engineering :: General, defect)

defect
Not set
normal

Tracking

(Not tracked)

RESOLVED FIXED

People

(Reporter: rail, Assigned: rail)

References

Details

Attachments

(12 files, 4 obsolete files)

934 bytes, patch
bhearsum
: review+
Details | Diff | Splinter Review
2.16 KB, patch
hwine
: review+
Details | Diff | Splinter Review
2.17 KB, patch
hwine
: review+
Details | Diff | Splinter Review
577 bytes, patch
hwine
: review+
Details | Diff | Splinter Review
516 bytes, patch
hwine
: review+
Details | Diff | Splinter Review
4.81 KB, patch
hwine
: review+
Details | Diff | Splinter Review
4.82 KB, patch
hwine
: review+
Details | Diff | Splinter Review
76.17 KB, patch
hwine
: review+
Details | Diff | Splinter Review
1.07 KB, patch
hwine
: review+
Details | Diff | Splinter Review
5.94 KB, patch
hwine
: review+
Details | Diff | Splinter Review
8.68 KB, patch
hwine
: review+
Details | Diff | Splinter Review
1.83 KB, patch
pmoore
: review+
Details | Diff | Splinter Review
+++ This bug was initially created as a clone of Bug #1029851 +++
Assignee: nobody → rail
Depends on: 1075600
Bump mozilla-release version for the release merge day.
Attachment #8498221 - Flags: review?(bhearsum)
Depends on: 1075607
Attachment #8498221 - Flags: review?(bhearsum) → review+
Depends on: 1077060
In production
Depends on: 1078249
Comment on attachment 8498221 [details] [diff] [review]
merge-release-buildbot-configs.diff

http://hg.mozilla.org/build/buildbot-configs/rev/6119b9881c88
Attachment #8498221 - Flags: checked-in+
Attached patch pause-gecko.git-legacy.diff (obsolete) — Splinter Review
Attached patch restart-gecko.git-legacy.diff (obsolete) — Splinter Review
Depends on: 1078439
Attached patch gaia_l10n_legacy.diff (obsolete) — Splinter Review
Attachment #8500501 - Flags: review?(hwine)
Attachment #8500502 - Flags: review?(hwine)
Attachment #8500503 - Flags: review?(hwine)
Attachment #8500510 - Flags: review?(hwine)
Attachment #8500511 - Flags: review?(hwine)
Attachment #8500603 - Flags: review?(hwine)
Attached patch bumper-switch-to-b2g34.diff (obsolete) — Splinter Review
Attachment #8500618 - Flags: review?(hwine)
Comment on attachment 8500500 [details] [diff] [review]
pause-gecko.git-legacy.diff

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

okay as is, but would be "safer" if 'aurora' is removed from for loop.
Attachment #8500500 - Flags: review?(hwine) → review+
Attachment #8500501 - Flags: review?(hwine) → review+
Comment on attachment 8500502 [details] [diff] [review]
restart-gecko.git-legacy.diff

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

::: keep_updated
@@ +19,5 @@
>  v13t_branch=b2g28_v1_3t
>  v14_branch=b2g30_v1_4
>  v20_branch=b2g32_v2_0
>  v20m_branch=b2g32_v2_0m
> +v21_branch=b2g34_v2_1

the hg repo needs to be added to the for loop
Attachment #8500502 - Flags: review?(hwine) → review-
Attachment #8500503 - Flags: review?(hwine) → review+
Attachment #8500510 - Flags: review?(hwine) → review+
Attachment #8500511 - Flags: review?(hwine) → review+
removed aurora from the loop
Attachment #8500500 - Attachment is obsolete: true
Attachment #8500745 - Flags: review?(hwine)
Attachment #8500502 - Attachment is obsolete: true
Attachment #8500750 - Flags: review?(hwine)
Comment on attachment 8500603 [details] [diff] [review]
gaia_l10n_legacy.diff

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

glad this was scripted, r-
 - 2 locales are new this time, and need the sections added as well -- br & pt-PT
 - the 'remotes_to_pull' value needs to be replaced (not just a new line added)

fwiw, I spotted an issue quickly using the following greps:
 - grep -c ^diff # total # files, must be even (150)
 - egrep -c '^diff.*hgrc$' # = #hgrc 1/2 of total (75)
 - egrep -c '^diff.*config$' # = #config 1/2 (75)
 - grep -c '^+v2_1' # paths added - s/b 1/2 of #hgrc (73)
 - grep -c '^+remotes_to_pull' - s/b 1/2 of #hgrc (73)
 - grep -c '^-remotes_to_pull' - s/b #hgrc - #new locales (0)
 - egrep -c '^\+\s+renameremote' # git renames - s/b #config (75)

I used vim to find the new locales, but the following gives a more direct pointer:
 $ egrep '^(diff.*hgrc)|(\+v2_1)' gaia_l10n_legacy.diff | awk 'last == $1 {print "new locale before " $0} {last=$1}'
new locale before diff --git a/releases-l10n-bs-gaia/hgrc b/releases-l10n-bs-gaia/hgrc
new locale before diff --git a/releases-l10n-ro-gaia/hgrc b/releases-l10n-ro-gaia/hgrc

::: releases-l10n-br-gaia/config
@@ +6,5 @@
>  	url = git+ssh://git.m.o/releases/l10n/br/gaia.git
>  	mirror = true
>  [gc]
>  	auto = 0
> +	renameremotev21defaultbranchto = v2.1

first time locale included -- needs section header

::: releases-l10n-br-gaia/hgrc
@@ +3,5 @@
>  [extensions]
>  hggit=
>  [git]
>  intree=1
> +

first time locale included -- needs path & added section header

::: releases-l10n-pt-PT-gaia/config
@@ +6,5 @@
>  	url = git+ssh://git.m.o/releases/l10n/pt-PT/gaia.git
>  	mirror = true
>  [gc]
>  	auto = 0
> +	renameremotev21defaultbranchto = v2.1

first time locale added - see above

::: releases-l10n-pt-PT-gaia/hgrc
@@ +3,5 @@
>  [extensions]
>  hggit=
>  [git]
>  intree=1
> +

first time locale added - see above
Attachment #8500603 - Flags: review?(hwine) → review-
Attachment #8500618 - Flags: review?(hwine) → review+
Attachment #8500745 - Flags: review?(hwine) → review+
Attachment #8500750 - Flags: review?(hwine) → review+
Thanks for catching the new locales issue. I also found another one (multiple remotes_to_pull lines) and fixed in this patch.
Attachment #8500603 - Attachment is obsolete: true
Attachment #8501233 - Flags: review?(hwine)
Comment on attachment 8501233 [details] [diff] [review]
gaia_l10n_legacy.diff

yay! lgtm
Attachment #8501233 - Flags: review?(hwine) → review+
Ooops. Forgot to change gecko_local_dir
Attachment #8500618 - Attachment is obsolete: true
Attachment #8501733 - Flags: review?(hwine)
Comment on attachment 8501733 [details] [diff] [review]
bumper-switch-to-b2g34.diff

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

oops - thanks for catching that; r+
Attachment #8501733 - Flags: review?(hwine) → review+
Comment on attachment 8501734 [details] [diff] [review]
b2g34-builds-mozharness.diff

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

got all files/instances
Attachment #8501734 - Flags: review?(hwine) → review+
Comment on attachment 8501874 [details] [diff] [review]
b2g34-bump-and-kill-aurora-buildbot-configs.diff

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

one question on dolphin builds - as long as that answered, lgtm

::: mozilla/b2g_config.py
@@ -1645,5 @@
> -BRANCHES['mozilla-aurora']['platforms']['emulator-kk']['enable_nightly'] = True
> -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 is also specified for dolphin builds around line 1815 -- does that need to change?
Attachment #8501874 - Flags: review?(hwine) → review+
(In reply to Hal Wine [:hwine] (use needinfo) from comment #25)
 > mozilla-aurora is also specified for dolphin builds around line 1815 -- does
> that need to change?

I didn't touch that part just in case we decide to continue using our automation.
Comment on attachment 8504051 [details] [diff] [review]
gaia_l10n_modern.diff

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

Looks good to me.
Attachment #8504051 - Flags: review?(hwine) → review+
Comment on attachment 8501874 [details] [diff] [review]
b2g34-bump-and-kill-aurora-buildbot-configs.diff

https://hg.mozilla.org/build/buildbot-configs/rev/e5c8d5450d09
Attachment #8501874 - Flags: checked-in+
Depends on: 1081935
Depends on: 1082005
No longer blocks: 1082322
Checked in code landed in production
All done here. The docs have been updated as well.
Status: NEW → RESOLVED
Closed: 10 years ago
Resolution: --- → FIXED
Component: General Automation → General
You need to log in before you can comment on or make changes to this bug.

Attachment

General

Created:
Updated:
Size: