Closed
Bug 842864
Opened 12 years ago
Closed 11 years ago
Tracking bug for 13-may-2013 migration work
Categories
(Release Engineering :: General, defect, P3)
Tracking
(Not tracked)
RESOLVED
FIXED
People
(Reporter: joduinn, Assigned: rail)
References
Details
Attachments
(3 files, 4 obsolete files)
(deleted),
patch
|
armenzg
:
review+
jhopkins
:
review+
rail
:
checked-in+
|
Details | Diff | Splinter Review |
(deleted),
patch
|
mozilla
:
review+
Callek
:
review+
armenzg
:
review+
rail
:
checked-in+
|
Details | Diff | Splinter Review |
(deleted),
patch
|
rail
:
review+
rail
:
checked-in+
|
Details | Diff | Splinter Review |
Instructions: https://wiki.mozilla.org/ReleaseEngineering:MergeDuty
Assignee | ||
Updated•12 years ago
|
Assignee: nobody → rail
Assignee | ||
Updated•12 years ago
|
Priority: -- → P3
Comment 1•12 years ago
|
||
# MERGE DAY - pulseaudio-libs-devel package rides the trains (bug 662417)
# MERGE DAY - Remove branches as FF21 reaches them
Depends on: 662417
Comment 2•12 years ago
|
||
# MERGE DAY NOTE: remove v21 based branches from the list below NON_UBUNTU_BRANCHES = ("mozilla-beta", "mozilla-release", "mozilla-esr17",
"mozilla-b2g18", "mozilla-b2g18_v1_0_0", "mozilla-b2g18_v1_0_1")
Depends on: 835660
Comment 4•12 years ago
|
||
Merge is scheduled for the 13th, release on the 14th, per
https://wiki.mozilla.org/RapidRelease/Calendar
Summary: Tracking bug for 14-may-2013 migration work → Tracking bug for 13-may-2013 migration work
Assignee | ||
Comment 5•11 years ago
|
||
Assignee | ||
Comment 6•11 years ago
|
||
Assignee | ||
Comment 7•11 years ago
|
||
All patches are incremental and will be re-checked next week.
Assignee | ||
Comment 8•11 years ago
|
||
I think this can be landed anytime.
Attachment #744836 -
Attachment is obsolete: true
Attachment #745856 -
Flags: review?(bhearsum)
Comment 9•11 years ago
|
||
Comment on attachment 745856 [details] [diff] [review]
beta to release
Review of attachment 745856 [details] [diff] [review]:
-----------------------------------------------------------------
I thought these things were supposed to be taken care of in the individual bugs, by their owners? I don't have context on most of these, so my review isn't very useful I think...
::: mozilla-tests/config.py
@@ +1228,5 @@
> loadDefaultValues(BRANCHES, projectBranch, branchConfig)
> loadCustomTalosSuites(BRANCHES, SUITES, projectBranch, branchConfig)
>
> # MERGE DAY NOTE: remove v21 based branches from the list below
> +NON_UBUNTU_BRANCHES = ("mozilla-esr17", "mozilla-b2g18", "mozilla-b2g18_v1_0_1")
Does this change cover uplifting mozharness unit tests (bug 843408)?
Attachment #745856 -
Flags: review?(bhearsum)
Assignee | ||
Comment 10•11 years ago
|
||
(In reply to Ben Hearsum [:bhearsum] from comment #9)
> Comment on attachment 745856 [details] [diff] [review]
> beta to release
>
> Review of attachment 745856 [details] [diff] [review]:
> -----------------------------------------------------------------
>
> I thought these things were supposed to be taken care of in the individual
> bugs, by their owners? I don't have context on most of these, so my review
> isn't very useful I think...
I prepared the patch myself by expecting the dep bugs and skimming the code. It looks safer to me, easier to catch missed uplifts like https://bugzilla.mozilla.org/show_bug.cgi?id=842863#c6.
> ::: mozilla-tests/config.py
> @@ +1228,5 @@
> > loadDefaultValues(BRANCHES, projectBranch, branchConfig)
> > loadCustomTalosSuites(BRANCHES, SUITES, projectBranch, branchConfig)
> >
> > # MERGE DAY NOTE: remove v21 based branches from the list below
> > +NON_UBUNTU_BRANCHES = ("mozilla-esr17", "mozilla-b2g18", "mozilla-b2g18_v1_0_1")
>
> Does this change cover uplifting mozharness unit tests (bug 843408)?
No. That will be for aurora->beta uplift (for 22.0b1).
Assignee | ||
Comment 11•11 years ago
|
||
Comment on attachment 745856 [details] [diff] [review]
beta to release
Review of attachment 745856 [details] [diff] [review]:
-----------------------------------------------------------------
::: mozilla-tests/config.py
@@ +1211,5 @@
>
> # Let's load jetpack for the following branches:
> # MERGE DAY once FF21 merges into a branch remove it from this list
> for branch in BRANCHES.keys():
> + if branch not in ('mozilla-esr17', 'mozilla-b2g18', 'mozilla-b2g18_v1_0_1'):
Armen, any stoppers for this uplift?
::: mozilla/config.py
@@ +1964,5 @@
> del BRANCHES[b]['platforms'][p]
>
> # MERGE DAY - pulseaudio-libs-devel package rides the trains (bug 662417)
> # MERGE DAY - Remove branches as FF21 reaches them
> +for b in ['mozilla-esr17', 'mozilla-b2g18', 'mozilla-b2g18_v1_0_1']:
jhopkins, any stoppers here?
Attachment #745856 -
Flags: review?(jhopkins)
Attachment #745856 -
Flags: review?(armenzg)
Comment 12•11 years ago
|
||
Comment on attachment 745856 [details] [diff] [review]
beta to release
Review of attachment 745856 [details] [diff] [review]:
-----------------------------------------------------------------
All good from my side.
Attachment #745856 -
Flags: review?(armenzg) → review+
Comment 13•11 years ago
|
||
Comment on attachment 745856 [details] [diff] [review]
beta to release
Looks good. As Rail mentioned in IRC, Thunderbird tracks FF17 so thunderbird_config.py is not included in this patch.
Attachment #745856 -
Flags: review?(jhopkins) → review+
Assignee | ||
Comment 14•11 years ago
|
||
Comment on attachment 745856 [details] [diff] [review]
beta to release
https://hg.mozilla.org/build/buildbot-configs/rev/968af0be175b
Attachment #745856 -
Flags: checked-in+
Comment 15•11 years ago
|
||
in production
Assignee | ||
Comment 16•11 years ago
|
||
Attachment #744838 -
Attachment is obsolete: true
Assignee | ||
Comment 17•11 years ago
|
||
Comment on attachment 747976 [details] [diff] [review]
aurora to beta
Review of attachment 747976 [details] [diff] [review]:
-----------------------------------------------------------------
::: mozilla-tests/config.py
@@ -1169,5 @@
> BRANCHES['mozilla-beta']['platforms']['win32']['talos_slave_platforms'] = ['xp', 'win7', 'win8']
> # End MERGE DAY remove the above when Firefox 23 merges in
> -# MERGE DAY remove the below when Firefox 22 merges in
> -BRANCHES['mozilla-beta']['platforms']['linux']['fedora']['opt_unittest_suites'] = BUILDBOT_UNITTEST_SUITES['opt_with_ipc'][:]
> -BRANCHES['mozilla-beta']['platforms']['linux']['fedora']['debug_unittest_suites'] = BUILDBOT_UNITTEST_SUITES['debug'][:]
aki, per http://hg.mozilla.org/build/buildbot-configs/annotate/0300777067f5/mozilla-tests/config.py#l1171 you added this code. Are we safe to uplift the changes in this hunk?
::: mozilla-tests/mobile_config.py
@@ +577,5 @@
> # MERGE DAY, drop trees from branch list as Firefox 22 rides forward.
> for branch in BRANCHES.keys():
> # Loop removes it from any branch that gets beyond here
> + if branch not in ('mozilla-release', 'mozilla-esr17', 'mozilla-b2g18',
> + 'mozilla-b2g18_v1_0_1'):
Callek, are we safe to uplift this hunk?
::: mozilla/config.py
@@ +1996,1 @@
> BRANCHES[b]["run_make_alive_tests"] = False
Armen, are we safe to uplift this hunk?
Attachment #747976 -
Flags: review?(bugspam.Callek)
Attachment #747976 -
Flags: review?(armenzg)
Attachment #747976 -
Flags: review?(aki)
Assignee | ||
Comment 18•11 years ago
|
||
Attachment #744840 -
Attachment is obsolete: true
Assignee | ||
Comment 19•11 years ago
|
||
Comment on attachment 747982 [details] [diff] [review]
central to aurora
Review of attachment 747982 [details] [diff] [review]:
-----------------------------------------------------------------
::: mozilla-tests/config.py
@@ -1174,5 @@
> BRANCHES['mozilla-aurora']['pgo_strategy'] = 'per-checkin'
> -# MERGE DAY remove the below when Firefox 23 merges in
> -del BRANCHES['mozilla-aurora']['platforms']['win32']['win7-ix']
> -BRANCHES['mozilla-aurora']['platforms']['win32']['talos_slave_platforms'] = ['xp', 'win7', 'win8']
> -# End MERGE DAY remove the above when Firefox 23 merges in
Armen, are we safe to uplift this?
@@ -1355,5 @@
>
> # MERGE DAY: remove branches when Firefox 23 merges in
> -NON_UBUNTU_TALOS_BRANCHES = ("mozilla-aurora", "mozilla-beta",
> - "mozilla-release", "mozilla-esr17",
> - "mozilla-b2g18", "mozilla-b2g18_v1_0_1")
r+ from me :)
::: mozilla-tests/mobile_config.py
@@ +596,5 @@
> # MERGE DAY, drop trees from branch list as Firefox 23 rides forward.
> for branch in BRANCHES:
> # Loop removes it from any branch that gets beyond here
> + if branch not in ('mozilla-beta', 'mozilla-release', 'mozilla-esr17',
> + 'mozilla-b2g18', 'mozilla-b2g18_v1_0_1'):
Callek, how about this uplift?
Attachment #747982 -
Flags: review?(bugspam.Callek)
Attachment #747982 -
Flags: review?(armenzg)
Updated•11 years ago
|
Attachment #747976 -
Flags: review?(bugspam.Callek) → review+
Updated•11 years ago
|
Attachment #747982 -
Flags: review?(bugspam.Callek) → review+
Updated•11 years ago
|
Attachment #747976 -
Flags: review?(aki) → review+
Updated•11 years ago
|
Attachment #747976 -
Flags: review?(armenzg) → review+
Comment 20•11 years ago
|
||
Comment on attachment 747982 [details] [diff] [review]
central to aurora
Review of attachment 747982 [details] [diff] [review]:
-----------------------------------------------------------------
::: mozilla-tests/config.py
@@ -1174,5 @@
> BRANCHES['mozilla-aurora']['pgo_strategy'] = 'per-checkin'
> -# MERGE DAY remove the below when Firefox 23 merges in
> -del BRANCHES['mozilla-aurora']['platforms']['win32']['win7-ix']
> -BRANCHES['mozilla-aurora']['platforms']['win32']['talos_slave_platforms'] = ['xp', 'win7', 'win8']
> -# End MERGE DAY remove the above when Firefox 23 merges in
Can I take care of this one? I hope to have more win7 machines next week to handle more load than just Cedar but I'm not 100% if they will be ready by Monday.
Attachment #747982 -
Flags: review?(armenzg) → review+
Assignee | ||
Comment 21•11 years ago
|
||
(In reply to Armen Zambrano G. [:armenzg] (Release Enginerring) from comment #20)
> Comment on attachment 747982 [details] [diff] [review]
> central to aurora
>
> Review of attachment 747982 [details] [diff] [review]:
> -----------------------------------------------------------------
>
> ::: mozilla-tests/config.py
> @@ -1174,5 @@
> > BRANCHES['mozilla-aurora']['pgo_strategy'] = 'per-checkin'
> > -# MERGE DAY remove the below when Firefox 23 merges in
> > -del BRANCHES['mozilla-aurora']['platforms']['win32']['win7-ix']
> > -BRANCHES['mozilla-aurora']['platforms']['win32']['talos_slave_platforms'] = ['xp', 'win7', 'win8']
> > -# End MERGE DAY remove the above when Firefox 23 merges in
>
> Can I take care of this one? I hope to have more win7 machines next week to
> handle more load than just Cedar but I'm not 100% if they will be ready by
> Monday.
Sure. I removed that hunk. Carry over r+.
Attachment #747982 -
Attachment is obsolete: true
Attachment #748050 -
Flags: review+
Assignee | ||
Comment 22•11 years ago
|
||
Comment on attachment 747976 [details] [diff] [review]
aurora to beta
https://hg.mozilla.org/build/buildbot-configs/rev/dfa24ec34795
Attachment #747976 -
Flags: checked-in+
Assignee | ||
Comment 23•11 years ago
|
||
Comment on attachment 748050 [details] [diff] [review]
central to aurora
https://hg.mozilla.org/build/buildbot-configs/rev/fdd520dd4d89
Attachment #748050 -
Flags: checked-in+
Assignee | ||
Comment 24•11 years ago
|
||
In production
Assignee | ||
Comment 25•11 years ago
|
||
All done here.
Status: NEW → RESOLVED
Closed: 11 years ago
Resolution: --- → FIXED
Updated•11 years ago
|
Product: mozilla.org → Release Engineering
You need to log in
before you can comment on or make changes to this bug.
Description
•