Closed Bug 1061861 Opened 12 years ago Closed 11 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
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+
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: 11 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: