Closed Bug 825959 Opened 12 years ago Closed 11 years ago

Build Thunderbird 17.0.4 esr with mock

Categories

(Release Engineering :: Release Requests, defect)

defect
Not set
normal

Tracking

(Not tracked)

RESOLVED WONTFIX

People

(Reporter: bhearsum, Assigned: bhearsum)

References

Details

Attachments

(3 files, 4 obsolete files)

Attached patch enable mock for tb 17.0.1 esr (obsolete) (deleted) — Splinter Review
      No description provided.
Attachment #697056 - Flags: review?(catlee)
Attachment #697056 - Flags: review?(catlee) → review+
Attachment #697056 - Flags: checked-in+
in production, forcing some builds to verify it
The builds that I triggered went fine.
Status: ASSIGNED → RESOLVED
Closed: 12 years ago
Resolution: --- → FIXED
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-
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
Component: Release Engineering: Automation (Release Automation) → Release Engineering: Releases
Depends on: 827354
Assignee: nobody → coop
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
I'm going to make sure this uplift happens right after we ship 17.0.3esr.
Assignee: coop → bhearsum
Attachment #697056 - Attachment is obsolete: true
Attachment #713021 - Flags: review?(bhearsum)
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-
Attached patch fix gcc in l10n mozconfigs too (obsolete) (deleted) — Splinter Review
Attachment #713021 - Attachment is obsolete: true
Attachment #716535 - Flags: review?(jhopkins)
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)
Attached patch use 0moz4 gcc (obsolete) (deleted) — Splinter Review
Attached patch bump in repo file (deleted) — Splinter Review
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]#
Attachment #716547 - Flags: review?(jhopkins)
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-
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
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+
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+
Attachment #716556 - Flags: checked-in+
Attached patch fix compiler for debug builds (deleted) — Splinter Review
Also fixes non-esr.
Attachment #717164 - Flags: review?(jhopkins)
Comment on attachment 717164 [details] [diff] [review]
fix compiler for debug builds

Catlee r+'ed this on IRC.
Attachment #717164 - Flags: review?(jhopkins) → review+
Attachment #717164 - Flags: checked-in+
In production
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.
Other than known intermittent test failures, things have greened up.
Status: REOPENED → RESOLVED
Closed: 12 years ago12 years ago
Resolution: --- → FIXED
Blocks: 837128
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-
Attachment #716700 - Flags: checked-in+ → checked-in-
Attachment #717164 - Flags: checked-in+ → checked-in-
This was backed out for chemspills. Planning to reland after they ship today.
Status: RESOLVED → REOPENED
Resolution: FIXED → ---
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+
Attachment #716556 - Flags: checked-in- → checked-in+
Attachment #717164 - Flags: checked-in- → checked-in+
No longer blocks: 837126
Builds are looking good.
Status: REOPENED → RESOLVED
Closed: 12 years ago12 years ago
Resolution: --- → FIXED
Blocks: 849352
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 → ---
Attachment #716700 - Flags: checked-in+ → checked-in-
Attachment #717164 - Flags: checked-in+ → checked-in-
Attachment #716556 - Flags: checked-in+ → checked-in-
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
Closed: 12 years ago11 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.

Attachment

General

Created:
Updated:
Size: