Closed Bug 474297 Opened 13 years ago Closed 13 years ago

use post_upload.py for en-US dep/nightly builds

Categories

(Release Engineering :: General, defect, P2)

x86
macOS
defect

Tracking

(Not tracked)

RESOLVED FIXED

People

(Reporter: bhearsum, Assigned: bhearsum)

Details

Attachments

(3 files, 1 obsolete file)

We landed post_upload.py for l10n and release builds in bug 455578. We should flip over en-US dep/nightly builds in order to be consistent, and get some better testing. There's already been some bustage on the l10n nightlies that would've been caught earlier if en-US builds were using the same system.
Priority: -- → P3
Priority: P3 → P2
NightlyBuildFactory is the most important stuff here. I've added logic that generates the proper post_upload.py command, and then calls out to the upload target. This has been tested for both dep and nightly builds and works fine.

I moved productName out of ReleaseBuildFactory since Nightly builds depend on it now, too.

This gets rid of all of our usage of MozillaStageUpload, which is great. I'd like to get rid of SetMozillaBuildProperties, too, but I think that's going to need some Makefile hacking to get a simple way to print out the BuildID. I filed bug 475382 on this.
Attachment #358883 - Flags: review?(ccooper)
Dead simple patch. Just adds productName to the configuration dict and passes it to NightlyBuildFactory.
Attachment #358884 - Flags: review?(ccooper)
This patch fixes a few bugs that weren't caught in the original testing:
* try/finally statement around symlink generation. Can't use try/except/finally because post_upload.py is called with Python 2.3 on staging-stage. All I'd do is re-raise the exception anyways, so no big deal.
* Reject MAR files for tinderbox builds directories. It's not the greatest thing to do this here, but it's better than adding a second upload target or otherwise mucking around to not even pass them post_upload.py IMHO.
Attachment #358885 - Flags: review?(ted.mielczarek)
Attachment #358884 - Flags: review?(ccooper) → review+
Attachment #358883 - Flags: review?(ccooper) → review+
Attachment #358885 - Flags: review?(ted.mielczarek) → review+
Comment on attachment 358885 [details] [diff] [review]
post_upload.py fixes for en-US nightly uploading

changeset:   38:a16c40591252

I updated the checkouts on staging-stage and stage.m.o, too.
Attachment #358885 - Flags: checked‑in+ checked‑in+
Attachment #358884 - Attachment is obsolete: true
Attachment #359600 - Flags: review?(ccooper)
Attachment #359600 - Flags: review?(ccooper) → review+
Comment on attachment 359600 [details] [diff] [review]
post-upload master side, staging+production

changeset:   710:b8b8019d7ace
Attachment #359600 - Flags: checked‑in+ checked‑in+
Comment on attachment 358883 [details] [diff] [review]
move en-US nightlies to upload target/post_upload.py - buildbotcustom patch

changeset:   181:baf292596907
Attachment #358883 - Flags: checked‑in+ checked‑in+
This changed landed without issue.
Status: ASSIGNED → RESOLVED
Closed: 13 years ago
Resolution: --- → FIXED
Product: mozilla.org → Release Engineering
You need to log in before you can comment on or make changes to this bug.