Closed
Bug 667253
Opened 14 years ago
Closed 14 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•14 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•14 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•14 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•14 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•14 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: 14 years ago
Resolution: --- → FIXED
Updated•12 years ago
|
Product: mozilla.org → Release Engineering
You need to log in
before you can comment on or make changes to this bug.
Description
•