Closed
Bug 515165
Opened 16 years ago
Closed 16 years ago
Buildbot should be managed by puppet on Linux slaves
Categories
(Release Engineering :: General, defect)
Tracking
(Not tracked)
RESOLVED
FIXED
People
(Reporter: bhearsum, Assigned: bhearsum)
Details
Attachments
(1 file, 1 obsolete file)
|
8.57 KB,
patch
|
catlee
:
review+
bhearsum
:
checked-in+
|
Details | Diff | Splinter Review |
We're going to be pulling in a couple of upstream changes very soon (in the bug this one blocks). One of them requires slaveside code changes. We need to get Buildbot managed by Puppet to make deployment feasible.
| Assignee | ||
Comment 1•16 years ago
|
||
This patch looks more complicated than it is due to the whitespace changes (to make the indentation not suck). The actual changes here are as follows:
* Add buildbot to the install_devtools list
* Add /tools/buildbot to the symlink list, with force enabled so it removes the existing directory.
I'd like to deploy this as is once it's ready, and then modify it when we pull in bug 514242. This way, we have an easy way to backout those changes if necessary.
Attachment #399217 -
Flags: review?(catlee)
Updated•16 years ago
|
Attachment #399217 -
Flags: review?(catlee) → review+
Comment 2•16 years ago
|
||
Comment on attachment 399217 [details] [diff] [review]
manage Linux buildbot install with Puppet
we'll need a way to delete old versions eventually, but this is ok for now.
| Assignee | ||
Comment 3•16 years ago
|
||
No functional change here.
Attachment #399217 -
Attachment is obsolete: true
Attachment #402444 -
Flags: review?(catlee)
Updated•16 years ago
|
Attachment #402444 -
Flags: review?(catlee) → review+
| Assignee | ||
Comment 4•16 years ago
|
||
Comment on attachment 402444 [details] [diff] [review]
unbitrotted version
Landed with version set as 5f43578cba2b.
Attachment #402444 -
Flags: checked-in+
| Assignee | ||
Comment 5•16 years ago
|
||
This worked fine.
Status: ASSIGNED → RESOLVED
Closed: 16 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
•