Closed Bug 1119236 Opened 7 years ago Closed 7 years ago

tracking bug for build and release of Firefox and Fennec 36.0

Categories

(Release Engineering :: Release Requests, defect)

defect
Not set
normal

Tracking

(Not tracked)

RESOLVED FIXED

People

(Reporter: rail, Assigned: rail)

References

Details

Attachments

(4 files, 3 obsolete files)

This includes betas and release
I'm doing 36.0b1, and rail has the rest.
Assignee: nobody → rail
The aim is to add all the info for 35.0build3 to the beta patcher config, so that we can specify that in ship-it and have partials generated and tested automatically. Ideally we'd have 35.0build2 as well, based on ADI, but the machinery doesn't know how to deal with two builds with the same version. Anyway, most people should have updated build2 -> build3 by the time we ship 36.0b1.

I was able to do a bump from this to 36.0b1 without hitting any errors (faked out the 36.0b1 buildID), and the linux64 update verify config bumped fine too.
Attachment #8547788 - Flags: review?(bhearsum)
Comment on attachment 8547788 [details] [diff] [review]
[tools] Set up patcher config for 35.0build3 --> 36.0b1 partial

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

Looks pretty straightforward to me.
Attachment #8547788 - Flags: review?(bhearsum) → review+
Comment on attachment 8547788 [details] [diff] [review]
[tools] Set up patcher config for 35.0build3 --> 36.0b1 partial

https://hg.mozilla.org/build/tools/rev/84f3e5e3ee6d
Attachment #8547788 - Flags: checked-in+
Depends on: 1122101
We had some update verification errors like this:

Retrieving 'http://stage.mozilla.org/pub/mozilla.org//firefox/candidates/36.0b2-candidates/build1/win32/de/Firefox Setup 36.0b2.exe' from cache...
PATCH DIRECTORY ../../update
INSTALLATION DIRECTORY .
WORKING DIRECTORY .
failed: 22
calling QuitProgressUI
FAIL: update status was not succeeded: failed: 22
FAIL: check_updates returned failure for WINNT_x86-msvc downloads/Firefox Setup 35.0.exe vs. downloads/Firefox Setup 36.0b2.exe: 1


Error 22 means 
#define MAR_CHANNEL_MISMATCH_ERROR 22

Sounds like we need to override the update channels before we try to update.

The patch should solve the issue. The second hunk is generated by:

python tools/scripts/updates/create-update-verify-configs.py \
 -c tools/release/patcher-configs/mozBeta-branch-patcher2.cfg \
 --platform win32 --output tools/release/updates/mozBeta-firefox-win32.cfg \
 --release-config-file mozilla/release-firefox-mozilla-beta.py \
 -b https://hg.mozilla.org/build/buildbot-configs --channel beta-localtest \
 -t FIREFOX_36_0b2_RELEASE
Attachment #8552468 - Flags: review?(bhearsum)
Comment on attachment 8552468 [details] [diff] [review]
add_mar-channel-ids.diff

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

Looks sensible to me. Make sure you retag with _RELEASE_RUNTIME!
Attachment #8552468 - Flags: review?(bhearsum) → review+
Depends on: 1126794
Attached patch WNP.diff (obsolete) — Splinter Review
Attachment #8564797 - Flags: review?(nthomas)
Attached patch RC.diff (obsolete) — Splinter Review
Attachment #8564798 - Flags: review?(nthomas)
Attachment #8564797 - Attachment is obsolete: true
Attachment #8564797 - Flags: review?(nthomas)
Comment on attachment 8564798 [details] [diff] [review]
RC.diff

I'll update it with WNP
Attachment #8564798 - Flags: review?(nthomas)
Attached patch RC+WNP.diff (obsolete) — Splinter Review
Attachment #8564798 - Attachment is obsolete: true
Attachment #8564800 - Flags: review?(nthomas)
Attachment #8564800 - Flags: review?(nthomas) → review+
Attached patch RC+WNP.diffSplinter Review
We have beta 10
Attachment #8564800 - Attachment is obsolete: true
Attachment #8565616 - Flags: review+
Comment on attachment 8565616 [details] [diff] [review]
RC+WNP.diff

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

::: mozilla/release-firefox-mozilla-release.py.template
@@ +46,3 @@
>  }
> +# What's New Page for Hello TODO: remove on request
> +releaseConfig['openURL'] = 'https://www.mozilla.org/%locale%/firefox/{{ appVersion }}/whatsnew/?oldversion=%OLD_VERSION%'

The lower case %locale% caused bug 1136143. Need to use upper case next time!
Attached patch fix_locale.diffSplinter Review
Just to follow up bug 1136143
Attachment #8568538 - Flags: review?(bhearsum)
Depends on: 1136143
Attachment #8568538 - Flags: review?(bhearsum) → review+
Status: NEW → RESOLVED
Closed: 7 years ago
Resolution: --- → FIXED
Blocks: 1138403
You need to log in before you can comment on or make changes to this bug.