Closed
Bug 1336310
(SM2.48b1)
Opened 8 years ago
Closed 7 years ago
Tracking bug for build and release of SeaMonkey 2.48b1
Categories
(SeaMonkey :: Release Engineering, defect, P1)
Tracking
(Not tracked)
RESOLVED
FIXED
People
(Reporter: ewong, Assigned: frg)
References
()
Details
Attachments
(15 files, 8 obsolete files)
+++ This bug was initially created as a clone of Bug #1294433 +++
This is a tracking bug for Build and Release of SeaMonkey 2.48b1
We expect an actual release on TBA.
Reporter | ||
Updated•8 years ago
|
Alias: SM2.48b1
Reporter | ||
Updated•8 years ago
|
Reporter | ||
Updated•8 years ago
|
Depends on: operation_ysnp
Assignee | ||
Updated•8 years ago
|
Depends on: operation_babelfish, 1312353
Assignee | ||
Comment 1•8 years ago
|
||
Patch to disables extensions as in 2.46. Because l10n Bug 1231349 and the Lightning upload is still not fixed this needs to be done.
Adapted from:
https://hg.mozilla.org/releases/comm-release/rev/2229823cdfa83745335eb45860571daf1f6fce93
and tested with 2.48 beta branch in c-b.
Attachment #8839183 -
Flags: review?(iann_bugzilla)
Attachment #8839183 -
Flags: feedback?(ewong)
Reporter | ||
Comment 2•8 years ago
|
||
Comment on attachment 8839183 [details] [diff] [review]
1336310-disableExtensions.patch
Review of attachment 8839183 [details] [diff] [review]:
-----------------------------------------------------------------
lgtm
Attachment #8839183 -
Flags: feedback?(ewong) → feedback+
Comment on attachment 8839183 [details] [diff] [review]
1336310-disableExtensions.patch
[Triage Comment]
r/a=me for c-b
Attachment #8839183 -
Flags: review?(iann_bugzilla)
Attachment #8839183 -
Flags: review+
Attachment #8839183 -
Flags: approval-comm-beta+
Reporter | ||
Comment 4•8 years ago
|
||
(In reply to Ian Neal from comment #3)
> Comment on attachment 8839183 [details] [diff] [review]
> 1336310-disableExtensions.patch
>
> [Triage Comment]
> r/a=me for c-b
Please push it to c-b's SEA248b1_2017021701_RELBRANCH.
Assignee | ||
Comment 5•8 years ago
|
||
Reporter | ||
Comment 6•8 years ago
|
||
Please note (for future references)
The buildbot-config patch that I'll attach to this bug will depend on:
bug 1342826 - Fix osx64 release-l10n entries/debug mozconfig
bug 1329379 - Drop OSX Universal Builds
Rationale:
The buildbot-config changes in bug 1329379 and bug 1342826 will need
to be entered first due the requirement that the osx64 release build
be using the 'right' mozconfig BUT also allowing c-c and c-a use the
new non-universal mozconfig.
Once the buildbot-config patch for this bug is attached, reviewed and
pushed, bug 1338960's buildbot-configs, tools AND buildbotcustom patches
(all to be attached, reviewed and pushed) will be needed.
Since Balrog is more or less new stuff for our infra, the balrog bug
will most likely see a few updated patches to add stuff to fix
balrog issues.
The above is just headsup for everyone (and a reminder for me as to
what the smeg I'm doing).
Reporter | ||
Updated•8 years ago
|
Reporter | ||
Comment 7•8 years ago
|
||
(In reply to Edmund Wong (:ewong) from comment #6)
> Once the buildbot-config patch for this bug is attached, reviewed and
> pushed, bug 1338960's buildbot-configs, tools AND buildbotcustom patches
> (all to be attached, reviewed and pushed) will be needed.
>
I forgot. The tracking bug for balrog migration is bug 1254401 which
depends on 1338960.
Reporter | ||
Comment 8•8 years ago
|
||
There is a chance we'll hit bug 1342383
Reporter | ||
Comment 9•8 years ago
|
||
This is mainly to help me debug any possible bustages. Will remove these
whitelists post-2.48b1.
Reporter | ||
Comment 10•8 years ago
|
||
Attachment #8841897 -
Flags: review?(frgrahl)
Assignee | ||
Comment 11•8 years ago
|
||
Comment on attachment 8841897 [details] [diff] [review]
[configs] Update configs for 2.48b1. (build1) (v1)
> releaseConfig['inspectorRepoRevision'] = 'SEA2_37_RELBRANCH'
Inspector branch need to be changed to DOMI_2_0_17 but it doesn't matter because it won't be build.
Likewise irc branch is now SEA2_48_RELBRANCH
Attachment #8841897 -
Flags: review?(frgrahl) → review+
Reporter | ||
Comment 12•8 years ago
|
||
Changed the reporevisions for cZ and inspector (even if they aren't
going to be built).
Attachment #8842246 -
Flags: review+
Reporter | ||
Comment 13•8 years ago
|
||
completely forgot that the linux64 source step requires a special
source mozconfig (i.e. https://bug1305912.bmoattachments.org/attachment.cgi?id=8796051)
so will need to copy that source to *all* the trees' suite/config/mozconfigs/linux64/source
Reporter | ||
Updated•8 years ago
|
Attachment #8843205 -
Flags: review?(frgrahl)
Reporter | ||
Comment 14•8 years ago
|
||
this patch needs to be pushed to c-c, c-a, c-b proper and c-r(2.48 one)
Attachment #8843207 -
Flags: review?(frgrahl)
Reporter | ||
Comment 15•8 years ago
|
||
Comment on attachment 8843207 [details] [diff] [review]
[mozconfigs] source mozconfig for release source step
[Approval Request Comment]
Regression caused by (bug #):
User impact if declined: Source step during release goes red.
Testing completed (on m-c, etc.):
Risk to taking this patch (and alternatives if risky):
String changes made by this patch: None
Attachment #8843207 -
Flags: approval-comm-release?
Attachment #8843207 -
Flags: approval-comm-beta?
Attachment #8843207 -
Flags: approval-comm-aurora?
Assignee | ||
Updated•8 years ago
|
Attachment #8843205 -
Flags: review?(frgrahl) → review+
Reporter | ||
Comment 16•8 years ago
|
||
Removed all unnecessary options from the source list.
[Approval Request Comment]
Regression caused by (bug #):
User impact if declined: release's source step fails
Testing completed (on m-c, etc.):
Risk to taking this patch (and alternatives if risky):
String changes made by this patch: None
Attachment #8843211 -
Flags: review?(frgrahl)
Attachment #8843211 -
Flags: approval-comm-release?
Attachment #8843211 -
Flags: approval-comm-beta?
Attachment #8843211 -
Flags: approval-comm-aurora?
Assignee | ||
Updated•8 years ago
|
Attachment #8843211 -
Flags: review?(frgrahl) → review+
Reporter | ||
Updated•8 years ago
|
Attachment #8843207 -
Attachment is obsolete: true
Attachment #8843207 -
Flags: review?(frgrahl)
Attachment #8843207 -
Flags: approval-comm-release?
Attachment #8843207 -
Flags: approval-comm-beta?
Attachment #8843207 -
Flags: approval-comm-aurora?
Reporter | ||
Comment 17•8 years ago
|
||
Pushed to relbranch:
https://hg.mozilla.org/releases/comm-beta/rev/7696fb72bd36
Reporter | ||
Comment 18•8 years ago
|
||
Attachment #8843223 -
Flags: review?(frgrahl)
Reporter | ||
Comment 19•8 years ago
|
||
Attachment #8843223 -
Attachment is obsolete: true
Attachment #8843223 -
Flags: review?(frgrahl)
Attachment #8843228 -
Flags: review?(frgrahl)
Assignee | ||
Updated•8 years ago
|
Attachment #8843228 -
Flags: review?(frgrahl) → review+
Comment 20•8 years ago
|
||
Comment on attachment 8843211 [details] [diff] [review]
[mozconfigs] source mozconfig for release source step
a=me
Attachment #8843211 -
Flags: approval-comm-release?
Attachment #8843211 -
Flags: approval-comm-release+
Attachment #8843211 -
Flags: approval-comm-beta?
Attachment #8843211 -
Flags: approval-comm-beta+
Attachment #8843211 -
Flags: approval-comm-aurora?
Attachment #8843211 -
Flags: approval-comm-aurora+
Reporter | ||
Comment 21•8 years ago
|
||
updated m-b's patch to fix the linux build issues.
Attachment #8845204 -
Flags: review?(frgrahl)
Reporter | ||
Comment 22•8 years ago
|
||
Attachment #8845204 -
Attachment is obsolete: true
Attachment #8845204 -
Flags: review?(frgrahl)
Attachment #8845211 -
Flags: review?(frgrahl)
Reporter | ||
Comment 23•8 years ago
|
||
For future references.. relbranch should be named:
SEA_COMM510_20170301_RELBRANCH
So:
SEA_COMM<gecko version>_<datestamp:yyyymmdd>_RELBRANCH
Reporter | ||
Comment 24•8 years ago
|
||
used the wrong relbranch override for cZ, domi and l10n.
Attachment #8845211 -
Attachment is obsolete: true
Attachment #8845211 -
Flags: review?(frgrahl)
Attachment #8845222 -
Flags: review?(frgrahl)
Reporter | ||
Comment 25•8 years ago
|
||
Assignee | ||
Updated•8 years ago
|
Attachment #8845222 -
Flags: review?(frgrahl) → review+
Reporter | ||
Comment 26•8 years ago
|
||
Attachment #8848991 -
Flags: review?(frgrahl)
Assignee | ||
Updated•8 years ago
|
Attachment #8848991 -
Flags: review?(frgrahl) → review+
Reporter | ||
Comment 27•8 years ago
|
||
I disabled osx64 mini -1 due to bug 1348756.
Reporter | ||
Comment 28•8 years ago
|
||
Attachment #8850280 -
Flags: review?(frgrahl)
Assignee | ||
Comment 29•8 years ago
|
||
Comment on attachment 8850280 [details] [diff] [review]
[configs] Update configs for 2.48b1 build 4
You missed mozilla-beta didn't you:
https://hg.mozilla.org/releases/mozilla-beta/file/631acd5e8736
r+ with this corrected
Attachment #8850280 -
Flags: review?(frgrahl) → review+
Reporter | ||
Comment 30•8 years ago
|
||
Comment on attachment 8843211 [details] [diff] [review]
[mozconfigs] source mozconfig for release source step
This needs to be pushed to c-c, c-a and the current c-b. This will
be needed for future releases..
Reporter | ||
Comment 31•8 years ago
|
||
well.. 2.46 was plagued by code-pain. 2.48b1 is plagued by infra-pain.
we need some form of infranol for the infra-pain but at the moment, the
some of the linux64 slaves are back up and not to tempt fate, I will
not touch the sea-puppet until I get 2.48b1 out.
I am going to do build5 ONLY because I want to avoid manually retriggering
macosx64-build as the csets properties are probably screwed up somehow.
Reporter | ||
Comment 32•8 years ago
|
||
for post land review
Attachment #8851878 -
Flags: review?(frgrahl)
Assignee | ||
Updated•8 years ago
|
Attachment #8851878 -
Flags: review?(frgrahl) → review+
Reporter | ||
Comment 33•8 years ago
|
||
Attachment #8852298 -
Flags: review?(frgrahl)
Reporter | ||
Comment 34•8 years ago
|
||
Attachment #8852340 -
Flags: review?(iann_bugzilla)
Reporter | ||
Comment 35•8 years ago
|
||
Attachment #8852298 -
Attachment is obsolete: true
Attachment #8852298 -
Flags: review?(frgrahl)
Attachment #8852342 -
Flags: review?(frgrahl)
Assignee | ||
Updated•8 years ago
|
Attachment #8852342 -
Flags: review?(frgrahl) → review+
Assignee | ||
Comment 36•8 years ago
|
||
Comment on attachment 8852340 [details] [diff] [review]
remove be from {all,shipped}-locales
Thie locale changehas already been reviewed in bug 1311209. I think only branch approvals including comm-esr52 and the beta branch are needed now there.
Attachment #8852340 -
Flags: feedback-
Reporter | ||
Comment 37•8 years ago
|
||
Comment on attachment 8852342 [details] [diff] [review]
[configs] Update configs for 2.48b1 (build 7)
Jumped the gun with this patch. Forgot I needed the c-b push's new
cset.
Comment 38•8 years ago
|
||
Comment on attachment 8852340 [details] [diff] [review]
remove be from {all,shipped}-locales
As mentioned by frg, covered in another bug.
Attachment #8852340 -
Flags: review?(iann_bugzilla) → review-
Reporter | ||
Comment 39•8 years ago
|
||
Attachment #8852342 -
Attachment is obsolete: true
Attachment #8852733 -
Flags: review?(frgrahl)
Reporter | ||
Updated•8 years ago
|
Attachment #8852340 -
Attachment is obsolete: true
Assignee | ||
Updated•8 years ago
|
Attachment #8852733 -
Flags: review?(frgrahl) → review+
Reporter | ||
Comment 40•8 years ago
|
||
Attachment #8853313 -
Flags: review?(frgrahl)
Assignee | ||
Updated•8 years ago
|
Attachment #8853313 -
Flags: review?(frgrahl) → review+
Reporter | ||
Comment 41•8 years ago
|
||
Attachment #8860252 -
Flags: review?(frgrahl)
Reporter | ||
Comment 42•8 years ago
|
||
Attachment #8860252 -
Attachment is obsolete: true
Attachment #8860252 -
Flags: review?(frgrahl)
Reporter | ||
Updated•8 years ago
|
Attachment #8860253 -
Flags: review?(frgrahl)
Assignee | ||
Updated•8 years ago
|
Attachment #8860253 -
Flags: review?(frgrahl) → review+
Reporter | ||
Updated•7 years ago
|
Status: ASSIGNED → RESOLVED
Closed: 7 years ago
Resolution: --- → FIXED
You need to log in
before you can comment on or make changes to this bug.
Description
•