Closed
Bug 785052
Opened 12 years ago
Closed 12 years ago
/usr/local/bin in missing from buildslave_path on 10.8 puppetagain test slaves
Categories
(Infrastructure & Operations Graveyard :: CIDuty, task)
Tracking
(Not tracked)
RESOLVED
FIXED
People
(Reporter: kmoir, Assigned: kmoir)
References
Details
Attachments
(2 files)
|
(deleted),
patch
|
dustin
:
review+
|
Details | Diff | Splinter Review |
|
(deleted),
patch
|
kmoir
:
checked-in+
|
Details | Diff | Splinter Review |
This causes the jetpack builds to fail because they clone scripts via hg which is stored in /usr/local/bin
|
Assignee | |
Updated•12 years ago
|
Assignee: nobody → kmoir
|
|
Assignee | |
Comment 1•12 years ago
|
||
I've tested it and it works - apply the puppet config and login as logout as the build user via vnc to rerun the launch agent.
Attachment #654756 -
Flags: review?(dustin)
|
||
Comment 2•12 years ago
|
||
Comment on attachment 654756 [details] [diff] [review]
patch
Ah, on Linux the PATH is inherited from the bash config, I think, but not on OS X. So the PATH is definitely required here!
I don't think we need CVS_RSH or PYTHONPATH, do we? In particular, we shouldn't be sending things using the packages::mozilla::python* into /Library/Python/2.5/site-packages! And I don't think we use CVS at all in the automation at this point.
r+ with those two removed (no need for a second r?), or some comments indicating why they are needed.
Attachment #654756 -
Flags: review?(dustin) → review+
|
|
Assignee | |
Comment 3•12 years ago
|
||
Attachment #654777 -
Flags: checked-in+
|
|
Assignee | |
Comment 4•12 years ago
|
||
Rebooted all the slaves, ran a test build and verified that this issue is resolved.
Status: NEW → RESOLVED
Closed: 12 years ago
Resolution: --- → FIXED
|
||
Updated•11 years ago
|
Product: mozilla.org → Release Engineering
|
|
||
Updated•6 years ago
|
Component: Platform Support → Buildduty
Product: Release Engineering → Infrastructure & Operations
|
||
Updated•5 years ago
|
Product: Infrastructure & Operations → Infrastructure & Operations Graveyard
You need to log in
before you can comment on or make changes to this bug.
Description
•