Closed
Bug 840556
Opened 12 years ago
Closed 12 years ago
shorten spidermonkey build dir names
Categories
(Release Engineering :: General, defect)
Release Engineering
General
Tracking
(Not tracked)
RESOLVED
FIXED
People
(Reporter: bhearsum, Assigned: bhearsum)
Details
Attachments
(1 file)
1.13 KB,
patch
|
rail
:
review+
bhearsum
:
checked-in+
|
Details | Diff | Splinter Review |
Specifically, it looks like some b2g builds (device ones were what I saw), fuzzing, and spidermonkey aren't. Suspiciously, all of these things use ScriptFactory. Might be other jobs too, not sure.
Assignee | ||
Comment 1•12 years ago
|
||
OK, so fuzzing and nanojit definitely aren't using normalizeName (https://github.com/mozilla/build-buildbotcustom/blob/041c7f18a8dd9bd7fb894b6efbb8dcd417b92578/misc.py#L2276, https://github.com/mozilla/build-buildbotcustom/blob/041c7f18a8dd9bd7fb894b6efbb8dcd417b92578/misc.py#L2324) -- but that's fine because they're nothing like the builds that we were trying to sync up.
Spidermonkey is using it, but has a max higher than the min (https://github.com/mozilla/build-buildbotcustom/blob/041c7f18a8dd9bd7fb894b6efbb8dcd417b92578/misc.py#L2428). We should fix that because those builds are run on various branches and could hit max dir/path length problems.
The B2G ones are surprising to me. AFAIK the code that creates them is here: https://github.com/mozilla/build-buildbotcustom/blob/041c7f18a8dd9bd7fb894b6efbb8dcd417b92578/misc.py#L998 -- and those are both using normalizeName without overriding min or max.
Assignee | ||
Comment 2•12 years ago
|
||
Here's a patch for the Spidermonkey part. Still investigating b2g.
Assignee | ||
Comment 3•12 years ago
|
||
Oh, and because of the problems with changing slave build dirs in a reconfig, should probably plan to do a graceful restart when landing this.
Assignee | ||
Comment 4•12 years ago
|
||
oooookay. Now I'm seeing all of the b2g jobs using padded directories -- which makes sense. I was probably looking at older jobs than I thought I was earlier. So to recap: the only thing to do here is adjust the spidermonkey names.
Updated•12 years ago
|
Attachment #712983 -
Flags: review?(rail) → review+
Assignee | ||
Updated•12 years ago
|
Attachment #712983 -
Flags: checked-in+
Assignee | ||
Updated•12 years ago
|
Summary: not all jobs using build dir padding from bug 827306 → shorten spidermonkey build dir names
Assignee | ||
Comment 5•12 years ago
|
||
I see this in a build I just triggered:
hg clone http://hg.mozilla.org/build/tools scripts
in dir /builds/slave/m-in_l64-d_sm-warnaserrdebug-0/. (timeout 1320 secs)
Status: ASSIGNED → RESOLVED
Closed: 12 years ago
Resolution: --- → FIXED
Updated•12 years ago
|
Product: mozilla.org → Release Engineering
Updated•7 years ago
|
Component: General Automation → General
You need to log in
before you can comment on or make changes to this bug.
Description
•