Closed Bug 1017923 Opened 10 years ago Closed 10 years ago

Tracking bug for 21-jul-2014 migration work

Categories

(Release Engineering :: General, defect)

defect
Not set
normal

Tracking

(Not tracked)

RESOLVED FIXED

People

(Reporter: mozilla, Assigned: mozilla)

References

Details

Attachments

(13 files, 1 obsolete file)

(deleted), patch
hwine
: review+
mozilla
: checked-in+
Details | Diff | Splinter Review
(deleted), patch
rail
: review+
mozilla
: checked-in+
Details | Diff | Splinter Review
(deleted), patch
rail
: review+
mozilla
: checked-in+
Details | Diff | Splinter Review
(deleted), patch
rail
: review+
mozilla
: checked-in+
Details | Diff | Splinter Review
(deleted), patch
pmoore
: review+
mozilla
: checked-in+
Details | Diff | Splinter Review
(deleted), patch
hwine
: review+
mozilla
: checked-in+
Details | Diff | Splinter Review
(deleted), patch
hwine
: review+
mozilla
: checked-in+
Details | Diff | Splinter Review
(deleted), patch
rail
: review+
mozilla
: checked-in+
Details | Diff | Splinter Review
(deleted), patch
rail
: review+
mozilla
: checked-in+
Details | Diff | Splinter Review
(deleted), patch
rail
: review+
mozilla
: checked-in+
Details | Diff | Splinter Review
(deleted), patch
rail
: review+
mozilla
: checked-in+
Details | Diff | Splinter Review
(deleted), patch
hwine
: review+
mozilla
: checked-in+
Details | Diff | Splinter Review
(deleted), patch
hwine
: review+
mozilla
: checked-in+
Details | Diff | Splinter Review
No description provided.
Depends on: 1022909
Assignee: nobody → aki
Depends on: 1022907
Attached patch bump_release_version (deleted) — Splinter Review
Attachment #8453972 - Flags: review?(hwine)
Depends on: 1037107
Depends on: 1037124
Depends on: 1037144
Attachment #8453972 - Flags: review?(hwine) → review+
Depends on: 1038282
#r-d convo that resulted in bug 1038282: [10:43] <aki> Sylvestre: sw is shipping in 31 ? [10:43] <aki> Pike: ^^ [10:44] <Sylvestre> aki, I haven't seen information about that :/ [10:45] <aki> Sylvestre: https://bugzilla.mozilla.org/show_bug.cgi?id=1019151 turned it off for 30; unclear whether it's ready to ship [10:47] <Sylvestre> I guess it is the same as in 30 [10:47] <aki> "142 missing, 148 obsolete" seems bad, but i also see "1 error(s), 2701 missing, 74 obsolete" https://l10n.mozilla.org/shipping/about-milestone/fx31_beta_b9 [10:48] <aki> ok, i'll try without sw [10:51] <Pike> yeah, not [10:51] <Pike> aki: want a bug on that? [10:51] <aki> Pike: i'm fine with irc. hsb is shipping? [10:52] <aki> also, is there a way for relman to know this ahead of merge day? [10:53] <Pike> just filed bug 1038282 for paper work either way [10:54] <aki> ty [10:54] <Pike> I think in general this should the "Final locale check with :Pike before we build RC candidate next week" last wednesday on the release calendar [10:54] <Pike> that process didn't work out well, though Keeping the "Get the list of l10n locales to remove from release management." line in https://wiki.mozilla.org/ReleaseEngineering/Merge_Duty/Steps#Perform_mozilla-beta_-.3E_mozilla-release_migration
Attached patch migration_script_release (deleted) — Splinter Review
Found some changes I needed to make in gecko_migration.py for the beta->release migration. Should be a one-time patch.
Attachment #8455498 - Flags: review?(rail)
Attachment #8455498 - Flags: review?(rail) → review+
Attached patch beta_tag_fix (deleted) — Splinter Review
Another one-time fix. Basically, change the http://hg.mozilla.org/releases/mozilla-beta/rev/6befadcaa685 comment to only say "Tagging mozilla-beta ..." instead of "Tagging /src/.../mozilla-beta ..."
Attachment #8455532 - Flags: review?(rail)
Attachment #8455532 - Flags: review?(rail) → review+
Attached patch disable-aurora (deleted) — Splinter Review
Not sure what I think of the build_localconfig.py bit. In a way it was a pain to remember, and in a way this patch is a little ugly.
Attachment #8458460 - Flags: review?(rail)
Attached patch legacy_vcs-sync_l10n (deleted) — Splinter Review
Attachment #8458511 - Flags: review?(hwine)
Attachment #8458460 - Flags: review?(rail) → review+
Comment on attachment 8458475 [details] [diff] [review] new_vcs-sync_l10n Review of attachment 8458475 [details] [diff] [review]: ----------------------------------------------------------------- r+ with comments removed (and if v2_1 does not yet need to be added) ::: configs/vcs_sync/l10n.py @@ +89,2 @@ > # MERGE DAY switch to the below hg_url on 2014-07-21 > # 'hg_url': 'https://hg.mozilla.org/releases/gaia-l10n/v2_0/%(locale)s', The updated hg_url is fine, but the comments in lines 89 and 90 need to be removed. I haven't yet checked the merge day docs - but I guess adding v2_1 comes later, and then we point v2_1 to https://hg.mozilla.org/gaia-l10n/%(locale)s - or does that need to be done at the same time as this change?
Attachment #8458475 - Flags: review?(pmoore) → review+
(In reply to Pete Moore [:pete][:pmoore] from comment #12) > ::: configs/vcs_sync/l10n.py > @@ +89,2 @@ > > # MERGE DAY switch to the below hg_url on 2014-07-21 > > # 'hg_url': 'https://hg.mozilla.org/releases/gaia-l10n/v2_0/%(locale)s', > > The updated hg_url is fine, but the comments in lines 89 and 90 need to be > removed. Good catch. > I haven't yet checked the merge day docs - but I guess adding v2_1 comes > later, and then we point v2_1 to https://hg.mozilla.org/gaia-l10n/%(locale)s > - or does that need to be done at the same time as this change? 2.1 is on master, so it's called 'master' til 6 weeks from now. At that point gecko goes to mozilla-aurora and we create a v2_1 from gaia-l10n, and then 12 weeks from now we pull v2_1 from releases/gaia-l10n/v2_1, unless the process changes again.
Comment on attachment 8458511 [details] [diff] [review] legacy_vcs-sync_l10n lgtm, and applying the patch seems to be consistent: = config diff = == before applying == * number of l10n hgrc files r!ls *l10n*/hgrc | wc 129 129 3977 == after applying == * number of l10n hgrc files r!ls *l10n*/hgrc | wc 129 129 3977 * number of files in patch (2x # repos): r! grep -c ^diff /tmp/legacy_vcs-sync_l10n.txt 148 * number of l10n config files with a renameremotev20defaultbranchto line: r! grep -l renameremotev20defaultbranchto *l10n*/config | wc 74 74 2199 * number of l10n config files with v2_0 in remotes_to_pull r! grep remotes_to_pull *l10n*/hgrc | grep -cw v2_0 74
Attachment #8458511 - Flags: review?(hwine) → review+
(In reply to Aki Sasaki [:aki] from comment #15) > Comment on attachment 8458475 [details] [diff] [review] > new_vcs-sync_l10n > > https://hg.mozilla.org/build/mozharness/rev/fe63e782cbe2 New vcs-sync l10n isn't live yet, so not merging atm. In later merge days the timing on this will be more important, but not critical until we start pulling l10n from git in our b2g builds.
In production.
Attached patch pause_gecko_git_legacy (deleted) — Splinter Review
Attachment #8459702 - Flags: review?(hwine)
Attached patch pause_new_vcs-sync (deleted) — Splinter Review
This isn't live yet, but it's good practice and may help avoid breaking pmoore's staging.
Attachment #8459704 - Flags: review?(rail)
Attachment #8459704 - Flags: review?(rail) → review+
Comment on attachment 8459702 [details] [diff] [review] pause_gecko_git_legacy Review of attachment 8459702 [details] [diff] [review]: ----------------------------------------------------------------- lgtm
Attachment #8459702 - Flags: review?(hwine) → review+
Attached patch fix_migration (obsolete) (deleted) — Splinter Review
Attachment #8459757 - Flags: review?(rail)
Attached patch fix_migration (deleted) — Splinter Review
With less hardcode and more branches for release_to_esr
Attachment #8459757 - Attachment is obsolete: true
Attachment #8459757 - Flags: review?(rail)
Attachment #8459758 - Flags: review?(rail)
Comment on attachment 8459758 [details] [diff] [review] fix_migration maybe pull_by_revision() and make revision mandatory?
Attachment #8459758 - Flags: review?(rail) → review+
Comment on attachment 8459758 [details] [diff] [review] fix_migration (In reply to Rail Aliiev [:rail] from comment #28) > Comment on attachment 8459758 [details] [diff] [review] > fix_migration > > maybe pull_by_revision() and make revision mandatory? Per IRC, making revision mandatory would break beta_to_release and release_to_esr. http://hg.mozilla.org/build/mozharness/rev/09e438ee911a
Attachment #8459758 - Flags: checked-in+
Attached patch get_merge_day_working (deleted) — Splinter Review
Needed to actually get the migration working from aurora->beta.
Attachment #8459798 - Flags: review?(rail)
Attachment #8459798 - Flags: review?(rail) → review+
Landed gaia_l10n.sh in braindump for getting the gaia-l10n v2_0 repos in sync. https://hg.mozilla.org/build/braindump/rev/2e7c16ba5c9c I got "WARNING releases/gaia-l10n/v2_0/es is newer!" for https://hg.mozilla.org/releases/gaia-l10n/v2_0/es/rev/895d5487bc41 ; the comment says it's 2.0 specific so no worries there. Updating docs...
Attached patch enable_new_vcs-sync (deleted) — Splinter Review
Attachment #8459878 - Flags: review?(rail)
Attached patch legacy_2.0_tools (deleted) — Splinter Review
Attachment #8459879 - Flags: review?(hwine)
Attached patch legacy_2.0_configs (deleted) — Splinter Review
Attachment #8459880 - Flags: review?(hwine)
Comment on attachment 8459879 [details] [diff] [review] legacy_2.0_tools Review of attachment 8459879 [details] [diff] [review]: ----------------------------------------------------------------- lgtm
Attachment #8459879 - Flags: review?(hwine) → review+
Comment on attachment 8459880 [details] [diff] [review] legacy_2.0_configs Review of attachment 8459880 [details] [diff] [review]: ----------------------------------------------------------------- lgtm
Attachment #8459880 - Flags: review?(hwine) → review+
Attachment #8459878 - Flags: review?(rail) → review+
Status: NEW → RESOLVED
Closed: 10 years ago
Resolution: --- → FIXED
We used to have RELEASE_BASE_* after merge day but we don't have RELEASE_BASE_20140721. Is that expected? Thanks
Yes, the naming has changed to match aurora and beta. https://hg.mozilla.org/releases/mozilla-beta/rev/1772e55568e4 FIREFOX_RELEASE_31_BASE
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: