Closed
Bug 667253
Opened 13 years ago
Closed 13 years ago
purge_builds can't clean up on-change builds from mozilla-release
Categories
(Release Engineering :: General, defect, P3)
Tracking
(Not tracked)
RESOLVED
FIXED
People
(Reporter: nthomas, Assigned: bhearsum)
Details
Attachments
(1 file, 1 obsolete file)
728 bytes,
patch
|
bhearsum
:
checked-in+
|
Details | Diff | Splinter Review |
From bug 667249. There was a rel-lnx-dbg dir on that slave, from an on-change build of the mozilla-releases/ repo. When we clear enough disk space for the new build we call this purge_builds.py -s 6 -n info -n 'rel-*' .. /scratchbox/users/cltbld/home/cltbld/build so it's not allowed to remove that on-change build. It should be. The release builds are in rel-rel-... dirs.
Assignee | ||
Comment 2•13 years ago
|
||
The simple way to fix this is not shortening "mozilla" to nothing. This will give us a one time hit in that anything with "mozilla" in its name will end up being a full build. I'll also have to go around and delete the existing "rel" builddirs by hand once this lands.
Attachment #542481 -
Flags: review?(nrthomas)
Reporter | ||
Comment 3•13 years ago
|
||
Comment on attachment 542481 [details] [diff] [review] don't shorten "mozilla" to nothing Shame about the clobber, but perhaps lets go for broke and add some mappings for alder/holly/larch/system/accessibility/inbound/devtools/services/private-browsing/any other branches created since this was setup. Just watch out for name collisions up to the first '-'.
Attachment #542481 -
Flags: review?(nrthomas) → review+
Assignee | ||
Comment 4•13 years ago
|
||
(In reply to comment #3) > Comment on attachment 542481 [details] [diff] [review] [review] > don't shorten "mozilla" to nothing > > Shame about the clobber, but perhaps lets go for broke and add some mappings > for > alder/holly/larch/system/accessibility/inbound/devtools/services/private- > browsing/any other branches created since this was setup. Just watch out for > name collisions up to the first '-'. Will do.
Assignee | ||
Comment 5•13 years ago
|
||
Here's the version I landed, with all the branches you mentioned added to it.
Attachment #542481 -
Attachment is obsolete: true
Attachment #542789 -
Flags: checked-in+
Assignee | ||
Comment 6•13 years ago
|
||
This made it to production today. It ended up burning any running builds (but not tests) for any branches affected by the name changes, because they took effect mid-build instead of in the next one. That was 20-30 builds total, all of which I manually retriggered. I also went around to all of the build machines and deleted rel-{{lnx,osx}{64,},w32}.
Status: NEW → RESOLVED
Closed: 13 years ago
Resolution: --- → FIXED
Updated•11 years ago
|
Product: mozilla.org → Release Engineering
You need to log in
before you can comment on or make changes to this bug.
Description
•