Closed Bug 474297 Opened 13 years ago Closed 13 years ago

use for en-US dep/nightly builds


(Release Engineering :: General, defect, P2)



(Not tracked)



(Reporter: bhearsum, Assigned: bhearsum)



(3 files, 1 obsolete file)

We landed 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 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 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 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] 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/ - buildbotcustom patch

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