Build Thunderbird 17.0.4 esr with mock

RESOLVED WONTFIX

Status

RESOLVED WONTFIX
6 years ago
5 years ago

People

(Reporter: bhearsum, Assigned: bhearsum)

Tracking

Firefox Tracking Flags

(Not tracked)

Details

Attachments

(3 attachments, 4 obsolete attachments)

(Assignee)

Description

6 years ago
Created attachment 697056 [details] [diff] [review]
enable mock for tb 17.0.1 esr
Attachment #697056 - Flags: review?(catlee)

Updated

6 years ago
Attachment #697056 - Flags: review?(catlee) → review+
(Assignee)

Updated

6 years ago
Attachment #697056 - Flags: checked-in+
(Assignee)

Comment 1

6 years ago
in production, forcing some builds to verify it
(Assignee)

Comment 2

6 years ago
The builds that I triggered went fine.
Status: ASSIGNED → RESOLVED
Last Resolved: 6 years ago
Resolution: --- → FIXED
(Assignee)

Comment 3

6 years ago
Comment on attachment 697056 [details] [diff] [review]
enable mock for tb 17.0.1 esr

Backed out due to bug 826567.
Attachment #697056 - Flags: checked-in+ → checked-in-
(Assignee)

Comment 4

6 years ago
need to try for this for the next esr
Assignee: bhearsum → nobody
Blocks: 825317
No longer blocks: 809320
Status: RESOLVED → REOPENED
Resolution: FIXED → ---
Summary: build thunderbird 17.0.1 esr with mock → build thunderbird 17.0.3 esr with mock
(Assignee)

Updated

6 years ago
Component: Release Engineering: Automation (Release Automation) → Release Engineering: Releases
Depends on: 827354
(Assignee)

Updated

6 years ago
Assignee: nobody → coop
(Assignee)

Comment 5

6 years ago
Warning: the comments in config.py are in the wrong place. You need to delete from here on: https://github.com/mozilla/build-buildbot-configs/blob/e6215b879206a46f9a6a9013be2efbbc0307f636/mozilla/thunderbird_config.py#L934
Moving this to 17.0.4 ESR because it's too close to 17.0.3 for comfort.
Blocks: 837126
No longer blocks: 825317
Summary: build thunderbird 17.0.3 esr with mock → Build Thunderbird 17.0.4 esr with mock
(Assignee)

Comment 8

6 years ago
I'm going to make sure this uplift happens right after we ship 17.0.3esr.
Assignee: coop → bhearsum
Created attachment 713021 [details] [diff] [review]
[buildbot-configs] updated patch that uses 'mock' hgSshKey
Attachment #697056 - Attachment is obsolete: true
Attachment #713021 - Flags: review?(bhearsum)
(Assignee)

Comment 10

6 years ago
Comment on attachment 713021 [details] [diff] [review]
[buildbot-configs] updated patch that uses 'mock' hgSshKey

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

I think you still need the l10n mozconfig changes like in https://hg.mozilla.org/users/bhearsum_mozilla.com/buildbot-configs/rev/f22f3cc894c0
Attachment #713021 - Flags: review?(bhearsum) → review-
(Assignee)

Comment 11

6 years ago
Created attachment 716535 [details] [diff] [review]
fix gcc in l10n mozconfigs too
Attachment #713021 - Attachment is obsolete: true
Attachment #716535 - Flags: review?(jhopkins)
(Assignee)

Comment 12

6 years ago
Comment on attachment 716535 [details] [diff] [review]
fix gcc in l10n mozconfigs too

wrong version of gcc - need to fix this.
Attachment #716535 - Attachment is obsolete: true
Attachment #716535 - Flags: review?(jhopkins)
(Assignee)

Comment 13

6 years ago
Created attachment 716547 [details] [diff] [review]
use 0moz4 gcc
(Assignee)

Comment 14

6 years ago
Created attachment 716556 [details] [diff] [review]
bump in repo file
(Assignee)

Comment 15

6 years ago
Verified:
[root@dev-stage01 tb]# tar -jvxf /pub/mozilla.org/thunderbird/tinderbox-builds/comm-esr17-linux/1361468088/thunderbird-17.0.3.en-US.linux-i686.tar.bz2
[root@dev-stage01 thunderbird]# readelf -D -s *.so | grep UNIQUE
[root@dev-stage01 thunderbird]#
(Assignee)

Updated

6 years ago
Attachment #716547 - Flags: review?(jhopkins)
(Assignee)

Updated

6 years ago
Attachment #716556 - Flags: review?(jhopkins)
Attachment #716556 - Flags: review?(jhopkins) → review+
Comment on attachment 716547 [details] [diff] [review]
use 0moz4 gcc


The name of the SSH key file changed in the staging configs:

-releaseConfig['hgSshKey']            = '~cltbld/.ssh/ffxbld_dsa'
+releaseConfig['hgSshKey']            = '/home/mock_mozilla/.ssh/tbirdbld_dsa'

r+ once you have the key name fixed up.
Attachment #716547 - Flags: review?(jhopkins) → review-
(Assignee)

Comment 17

6 years ago
Created attachment 716700 [details] [diff] [review]
fix ssh keys for staging configs
Attachment #716547 - Attachment is obsolete: true
Attachment #716700 - Flags: review?(jhopkins)
Comment on attachment 716556 [details] [diff] [review]
bump in repo file

a=Standard8 for landing this on comm-esr17
(Assignee)

Comment 19

6 years ago
Comment on attachment 716556 [details] [diff] [review]
bump in repo file

15:02 <@bhearsum> Standard8: is there a comm esr17 approval flag i need before landing 
                  https://bugzilla.mozilla.org/attachment.cgi?id=716556&action=edit 
15:07 -!- catlee is now known as catlee-away
15:07 < Standard8> bhearsum: releng component doesn't have one, but a=me.
Attachment #716700 - Flags: review?(jhopkins) → review+
(Assignee)

Comment 20

6 years ago
Comment on attachment 716700 [details] [diff] [review]
fix ssh keys for staging configs

Going to reconfig for this sometime this morning. I'll push the comm-esr17 patch at that time, which will trigger builds to verify everything.
Attachment #716700 - Flags: checked-in+
(Assignee)

Updated

6 years ago
Attachment #716556 - Flags: checked-in+
(Assignee)

Comment 21

6 years ago
Created attachment 717164 [details] [diff] [review]
fix compiler for debug builds

Also fixes non-esr.
Attachment #717164 - Flags: review?(jhopkins)
(Assignee)

Comment 22

6 years ago
Comment on attachment 717164 [details] [diff] [review]
fix compiler for debug builds

Catlee r+'ed this on IRC.
Attachment #717164 - Flags: review?(jhopkins) → review+
(Assignee)

Updated

6 years ago
Attachment #717164 - Flags: checked-in+
In production
(Assignee)

Comment 24

6 years ago
The comm-esr17 patch caused us to fail a test:
TEST-UNEXPECTED-FAIL | check-sync-dirs.py | build file copies are not in sync
TEST-INFO | check-sync-dirs.py | file(s) found in:               /builds/slave/tb-c-esr17-lx/build/build
TEST-INFO | check-sync-dirs.py | differ from their originals in: /builds/slave/tb-c-esr17-lx/build/mozilla/build
TEST-INFO | check-sync-dirs.py | differing file:                 ./unix/mozconfig.linux
In general, the files in '/builds/slave/tb-c-esr17-lx/build/build' should
always be exact copies of originals in
'/builds/slave/tb-c-esr17-lx/build/mozilla/build'.  A change made to one
should also be made to the other.  See 'check-sync-dirs.py' for more
details.

That will be fixed when bug 813613 lands, which is currently waiting on approval. In the meantime I've starred the builds.
(Assignee)

Comment 25

6 years ago
Other than known intermittent test failures, things have greened up.
Status: REOPENED → RESOLVED
Last Resolved: 6 years ago6 years ago
Resolution: --- → FIXED

Updated

6 years ago
Blocks: 837128
(Assignee)

Comment 26

6 years ago
Comment on attachment 716556 [details] [diff] [review]
bump in repo file

This was backed out due to bustage in the chemspill 17.0.4esr release. Will be relanded once all the chemspills are done. Also need to land the mozilla-esr17 patch on the relbranches.
Attachment #716556 - Flags: checked-in+ → checked-in-
(Assignee)

Updated

6 years ago
Attachment #716700 - Flags: checked-in+ → checked-in-
(Assignee)

Updated

6 years ago
Attachment #717164 - Flags: checked-in+ → checked-in-
(Assignee)

Comment 27

6 years ago
This was backed out for chemspills. Planning to reland after they ship today.
Status: RESOLVED → REOPENED
Resolution: FIXED → ---
(Assignee)

Comment 28

6 years ago
Comment on attachment 716700 [details] [diff] [review]
fix ssh keys for staging configs

Relanded now that chemspills are over. I'll be triggering some builds to verify once a reconfig puts these patches in production again.
Attachment #716700 - Flags: checked-in- → checked-in+
(Assignee)

Updated

6 years ago
Attachment #716556 - Flags: checked-in- → checked-in+
(Assignee)

Updated

6 years ago
Attachment #717164 - Flags: checked-in- → checked-in+

Updated

6 years ago
No longer blocks: 837126
(Assignee)

Comment 29

6 years ago
Builds are looking good.
Status: REOPENED → RESOLVED
Last Resolved: 6 years ago6 years ago
Resolution: --- → FIXED
(Assignee)

Updated

6 years ago
Blocks: 849352
(Assignee)

Comment 30

6 years ago
This still broke shit. It's getting backed out again. See https://bugzilla.mozilla.org/show_bug.cgi?id=813613#c38 for details.
Status: RESOLVED → REOPENED
Resolution: FIXED → ---
(Assignee)

Updated

6 years ago
Attachment #716700 - Flags: checked-in+ → checked-in-
(Assignee)

Updated

6 years ago
Attachment #717164 - Flags: checked-in+ → checked-in-
(Assignee)

Updated

6 years ago
Attachment #716556 - Flags: checked-in+ → checked-in-
(Assignee)

Comment 31

6 years ago
Given all of the issues we've had with trying to do this, I don't think it's worthwhile investing any more time into. We'll just have to prop up the old build platform until esr17 dies.
Status: REOPENED → RESOLVED
Last Resolved: 6 years ago6 years ago
Resolution: --- → WONTFIX
Product: mozilla.org → Release Engineering
You need to log in before you can comment on or make changes to this bug.