Closed
Bug 496491
Opened 15 years ago
Closed 15 years ago
[Bootstrap] Move update verify bumping to Updates step
Categories
(Release Engineering :: General, defect, P2)
Tracking
(Not tracked)
RESOLVED
FIXED
People
(Reporter: nthomas, Assigned: nthomas)
Details
Attachments
(1 file)
9.95 KB,
patch
|
bhearsum
:
review+
nthomas
:
checked-in+
|
Details | Diff | Splinter Review |
This is the Bootstrap (Tb2 & Fx3) equivalent of bug 486454. Basically, doing the config bump for update verify on each platform doesn't work so well when the config lives in a hg repository.
Assignee | ||
Updated•15 years ago
|
Assignee: nobody → nthomas
Status: NEW → ASSIGNED
Priority: -- → P2
Assignee | ||
Comment 1•15 years ago
|
||
Only passes perl -c at this point. Will test in staging.
Comment 2•15 years ago
|
||
tripped over this during TB2.0.0.22 release.
Assignee | ||
Comment 3•15 years ago
|
||
(In reply to comment #2) > tripped over this during TB2.0.0.22 release. Yes, this is a problem in RELEASE_AUTOMATION_M13 and M14.
Assignee | ||
Comment 4•15 years ago
|
||
Comment on attachment 382438 [details] [diff] [review] WIP, tested I tested this on staging-1.9, to make sure that it could reproduce the bump from 3.0.11 build1 to build2 bump. It did that, modulo the hostname changes that come with staging. Had to jump through a bunch of hoops to setup staging but I'm pretty confident this will work fine in production. Patch also fixes pulling shipped-locales from the relbranch rather than the release tag, when updating the patcher config.
Attachment #382438 -
Attachment description: WIP → WIP, tested
Attachment #382438 -
Flags: review?(bhearsum)
Assignee | ||
Comment 5•15 years ago
|
||
(In reply to comment #4) > (From update of attachment 382438 [details] [diff] [review]) > I tested this on staging-1.9, to make sure that it could reproduce the bump > from 3.0.11 build1 to build2 ... for both the patcher and update verify configs.
Assignee | ||
Comment 6•15 years ago
|
||
Do we want to have this for Fx3.0.12 ?
Comment 7•15 years ago
|
||
Comment on attachment 382438 [details] [diff] [review] WIP, tested Sorry this sat for so long, looks good though.
Attachment #382438 -
Flags: review?(bhearsum) → review+
Assignee | ||
Comment 8•15 years ago
|
||
Comment on attachment 382438 [details] [diff] [review] WIP, tested Checking in Bootstrap/Step/PatcherConfig.pm; /cvsroot/mozilla/tools/release/Bootstrap/Step/PatcherConfig.pm,v <-- PatcherConfig.pm new revision: 1.24; previous revision: 1.23 done Checking in Bootstrap/Step/Updates.pm; /cvsroot/mozilla/tools/release/Bootstrap/Step/Updates.pm,v <-- Updates.pm new revision: 1.49; previous revision: 1.48 done And tagged as RELEASE_AUTOMATION_M15.
Attachment #382438 -
Flags: checked‑in+
Assignee | ||
Updated•15 years ago
|
Status: ASSIGNED → RESOLVED
Closed: 15 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
•