If you think a bug might affect users in the 57 release, please set the correct tracking and status flags for Release Management.

[Bootstrap] Move update verify bumping to Updates step

RESOLVED FIXED

Status

Release Engineering
General
P2
normal
RESOLVED FIXED
9 years ago
4 years ago

People

(Reporter: nthomas, Assigned: nthomas)

Tracking

Firefox Tracking Flags

(Not tracked)

Details

Attachments

(1 attachment)

(Assignee)

Description

9 years ago
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

9 years ago
Assignee: nobody → nthomas
Status: NEW → ASSIGNED
Priority: -- → P2
(Assignee)

Comment 1

9 years ago
Created attachment 382438 [details] [diff] [review]
WIP, tested

Only passes perl -c at this point. Will test in staging.
tripped over this during TB2.0.0.22 release.
(Assignee)

Comment 3

9 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

8 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

8 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

8 years ago
Do we want to have this for Fx3.0.12 ?
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

8 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

8 years ago
Status: ASSIGNED → RESOLVED
Last Resolved: 8 years ago
Resolution: --- → FIXED
Product: mozilla.org → Release Engineering
You need to log in before you can comment on or make changes to this bug.