Closed Bug 812509 Opened 12 years ago Closed 12 years ago

merge day work for February 18th, 2013 and kill esr10

Categories

(Release Engineering :: General, defect, P2)

x86
macOS
defect

Tracking

(Not tracked)

RESOLVED FIXED

People

(Reporter: armenzg, Assigned: armenzg)

References

Details

Attachments

(2 files, 2 obsolete files)

      No description provided.
Depends on: 812512
Depends on: 832050
Assignee: nobody → armenzg
Instructions: https://wiki.mozilla.org/ReleaseEngineering:MergeDuty

TODO:
* re-purpose Leopards rev3 10.5 machines
Priority: -- → P2
No longer depends on: 832050
Summary: merge day work for February 18th, 2012 → merge day work for February 18th, 2012 and kill esr10
Attached patch remove esr10 (tools) (deleted) — Splinter Review
Attachment #715528 - Flags: review?(bhearsum)
Attached patch remove esr10 (configs) (obsolete) (deleted) — Splinter Review
Attachment #715529 - Flags: review?(bhearsum)
Attachment #715528 - Attachment description: remove esr10 → remove esr10 (tools)
Comment on attachment 715529 [details] [diff] [review]
remove esr10 (configs)

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

::: mozilla/thunderbird_config.py
@@ -1090,5 @@
> -# This block can be removed at that time.
> -for branch in BRANCHES:
> -    if branch not in ('comm-release', 'comm-esr10', 'comm-esr17'):
> -        del BRANCHES[branch]['platforms']['macosx-debug']
> -

The merge day comment and the code don't match here. comm-release and comm-esr17 still exist. Are you sure this block should be removed? I see you and Rail on the blamelist for these lines. Please figure out what should be done here.
Comment on attachment 715528 [details] [diff] [review]
remove esr10 (tools)

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

r=me with the manage_buildslave.sh and setup_foopy.py changes removed.

::: buildfarm/mobile/manage_buildslave.sh
@@ -1,2 @@
>  #!/bin/bash
> -if [ -d "/tools/buildbot/bin" ]; then

This doesn't look like it's related to killing esr10.

::: buildfarm/mobile/setup_foopy.py
@@ -6,1 @@
>  

Nor does this.
Attachment #715528 - Flags: review?(bhearsum) → review+
Comment on attachment 715528 [details] [diff] [review]
remove esr10 (tools)

Those 2 files were not related. Thanks for the catch.
Attachment #715528 - Flags: checked-in+
Attached patch [wip] remove esr10 (configs) (obsolete) (deleted) — Splinter Review
I'm working on the patch to make sure that we are really removing only what is needed.

I know that the build side is working with this WIP but I still have to adjust the test side.
Attachment #715529 - Attachment is obsolete: true
Attachment #715529 - Flags: review?(bhearsum)
Depends on: 842638
Summary: merge day work for February 18th, 2012 and kill esr10 → merge day work for February 18th, 2013 and kill esr10
Bundle generation has been triggered as specified in the notes.
Attached patch remove esr10 (configs) (deleted) — Splinter Review
Attachment #715647 - Attachment is obsolete: true
Attachment #715771 - Flags: review?(aki)
Comment on attachment 715771 [details] [diff] [review]
remove esr10 (configs)

* as discussed in irc, we still need a patch for leopard
* could you remove these files? (hg rm)

mozilla/preproduction_release_overrides-esr10.py
mozilla/release-firefox-mozilla-esr10.py
mozilla/release-firefox-mozilla-esr10.py.template
mozilla/release-thunderbird-comm-esr10.py
mozilla/release-thunderbird-comm-esr10.py.template
mozilla/staging_release-firefox-mozilla-esr10.py
mozilla/staging_release-firefox-mozilla-esr10.py.template
mozilla/staging_release-thunderbird-comm-esr10.py
mozilla/staging_release-thunderbird-comm-esr10.py.template
Attachment #715771 - Flags: review?(aki) → review+
Attachment #715771 - Flags: checked-in+
Live in production.
Status: NEW → RESOLVED
Closed: 12 years ago
Resolution: --- → FIXED
Product: mozilla.org → Release Engineering
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: