Closed Bug 774345 Opened 12 years ago Closed 12 years ago

ScriptFactory-based builders don't seem to clobber correctly

Categories

(Release Engineering :: Release Automation: Other, defect, P1)

defect

Tracking

(Not tracked)

RESOLVED FIXED

People

(Reporter: bhearsum, Assigned: rail)

References

Details

Attachments

(2 files)

Noticed this while working on 14.0.1; the SUMS files had 13.0 checksums in them, but those .checksums files didn't exist in the candidates directory. Looking at the slave that did the work, I see:
[cltbld@linux-ix-slave39 temp]$ ls  -l linux-i686/af
total 8
-rw-r--r-- 1 cltbld cltbld 2568 Jun  1 06:07 firefox-13.0.checksums
-rw-r--r-- 1 cltbld cltbld 2616 Jul 13 15:05 firefox-14.0.1.checksums

But in the builder only 14.0.1 was downloaded:
linux-i686/af/firefox-14.0.1.checksums

I think some of our scripts need to use always clobber:
https://github.com/mozilla/build-tools/blob/master/scripts/release/generate-sums.sh#L38
https://github.com/mozilla/build-tools/blob/master/scripts/release/push-to-mirrors.sh
https://github.com/mozilla/build-tools/blob/master/scripts/release/tagging.sh

There might be others, too.
Assignee: nobody → rail
Severity: normal → critical
Priority: -- → P1
Attached patch fix propertiesSplinter Review
Bad properties are passed to the scripts :/
Attachment #642982 - Flags: review?(bhearsum)
Attachment #642983 - Flags: review?(bhearsum)
Comment on attachment 642982 [details] [diff] [review]
fix properties

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

Thanks for fixing this!
Attachment #642982 - Flags: review?(bhearsum) → review+
Attachment #642983 - Flags: review?(bhearsum) → review+
Status: NEW → RESOLVED
Closed: 12 years ago
Resolution: --- → FIXED
Blocks: 777230
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: