1336310 - (SM2.48b1) Tracking bug for build and release of SeaMonkey 2.48b1

Status: RESOLVED FIXED

(SeaMonkey :: Release Engineering, defect, P1)

Description

[Edmund Wong (:ewong)]

+++ 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.

Comment 1

[Frank-Rainer Grahl (:frg)]

Attachment: 1336310-disableExtensions.patch

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.

Comment 2

[Edmund Wong (:ewong)]

Comment on attachment 8839183 [details] [diff] [review]
1336310-disableExtensions.patch

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

lgtm

Comment 3

[Ian Neal]

Comment on attachment 8839183 [details] [diff] [review]
1336310-disableExtensions.patch

[Triage Comment]
r/a=me for c-b

Comment 4

[Edmund Wong (:ewong)]

(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.

Comment 5

[Frank-Rainer Grahl (:frg)]

https://hg.mozilla.org/releases/comm-beta/rev/c4daf50288efe9f4062b5e83185641bdb80b38dc

Comment 6

[Edmund Wong (:ewong)]

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).

Comment 7

[Edmund Wong (:ewong)]

(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.

Comment 8

[Edmund Wong (:ewong)]

There is a chance we'll hit bug 1342383

Comment 9

[Edmund Wong (:ewong)]

Attachment: [puppet] do not clobber beta-related dirs in linux hosts.

This is mainly to help me debug any possible bustages.  Will remove these
whitelists post-2.48b1.

Comment 10

[Edmund Wong (:ewong)]

Attachment: [configs] Update configs for 2.48b1. (build1) (v1)

Comment 11

[Frank-Rainer Grahl (:frg)]

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

Comment 12

[Edmund Wong (:ewong)]

Attachment: [configs] Update configs for 2.48b1 (build1) v2

Changed the reporevisions for cZ and inspector (even if they aren't
going to be built).

Comment 13

[Edmund Wong (:ewong)]

Attachment: [configs] source step needs to a different mozconfig

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

Comment 14

[Edmund Wong (:ewong)]

Attachment: [mozconfigs] source mozconfig for release source step

this patch needs to be pushed to c-c, c-a, c-b proper and c-r(2.48 one)

Comment 15

[Edmund Wong (:ewong)]

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

Comment 16

[Edmund Wong (:ewong)]

Attachment: [mozconfigs] source mozconfig for release source step

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

Comment 17

[Edmund Wong (:ewong)]

Pushed to relbranch:
https://hg.mozilla.org/releases/comm-beta/rev/7696fb72bd36

Comment 18

[Edmund Wong (:ewong)]

Attachment: [configs] Update configs for 2.48b1 build 2

Comment 19

[Edmund Wong (:ewong)]

Attachment: [configs] Update configs for 2.48b1 build 2

Comment 20

[Ian Neal]

Comment on attachment 8843211 [details] [diff] [review]
[mozconfigs] source mozconfig for release source step

a=me

Comment 21

[Edmund Wong (:ewong)]

Attachment: [configs] Update configs for 2.48b1 build 2(updated)

updated m-b's patch to fix the linux build issues.

Comment 22

[Edmund Wong (:ewong)]

Attachment: [configs] Update configs for 2.48b1 build 2(updated)

Comment 23

[Edmund Wong (:ewong)]

For future references.. relbranch should be named:

SEA_COMM510_20170301_RELBRANCH

So:

SEA_COMM<gecko version>_<datestamp:yyyymmdd>_RELBRANCH

Comment 24

[Edmund Wong (:ewong)]

Attachment: [configs] Update configs for 2.48b1 build 2(updated w/ fixes)

used the wrong relbranch override for cZ, domi and l10n.

Comment 25

[Edmund Wong (:ewong)]

Attachment: [configs] fix for source step. [fix]

Comment 26

[Edmund Wong (:ewong)]

Attachment: [configs] Update configs for 2.48b1 build 3

Comment 27

[Edmund Wong (:ewong)]

I disabled osx64 mini -1 due to bug 1348756.

Comment 28

[Edmund Wong (:ewong)]

Attachment: [configs] Update configs for 2.48b1 build 4

Comment 29

[Frank-Rainer Grahl (:frg)]

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

Comment 30

[Edmund Wong (:ewong)]

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..

Comment 31

[Edmund Wong (:ewong)]

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.

Comment 32

[Edmund Wong (:ewong)]

Attachment: [configs] Update config for 2.48b1 - build 5

for post land review

Comment 33

[Edmund Wong (:ewong)]

Attachment: [configs] Update config for 2.48b1 - build 6

Comment 34

[Edmund Wong (:ewong)]

Attachment: remove be from {all,shipped}-locales

Comment 35

[Edmund Wong (:ewong)]

Attachment: [configs] Update configs for 2.48b1 (build 7)

Comment 36

[Frank-Rainer Grahl (:frg)]

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.

Comment 37

[Edmund Wong (:ewong)]

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

[Ian Neal]

Comment on attachment 8852340 [details] [diff] [review]
remove be from {all,shipped}-locales

As mentioned by frg, covered in another bug.

Comment 39

[Edmund Wong (:ewong)]

Attachment: [configs] Update configs for 2.48b1 (build 7) (v2)

Comment 40

[Edmund Wong (:ewong)]

Attachment: [configs] fix stagingServer for beta release

Comment 41

[Edmund Wong (:ewong)]

Attachment: [configs] Use candidates and not nightly for releases' candidatesPath

Comment 42

[Edmund Wong (:ewong)]

Attachment: [configs] Use candidates and not nightly for releases' candidatesPath