remove DO_NOT_START check from runslave.py

RESOLVED FIXED

Status

RESOLVED FIXED
8 years ago
5 years ago

People

(Reporter: dustin, Assigned: dustin)

Tracking

Firefox Tracking Flags

(Not tracked)

Details

Attachments

(1 attachment)

Since we have a UI to disable slaves, there's no need to add an extra slave-side file to get that effect.
Created attachment 510348 [details] [diff] [review]
m632103-puppet-manifests-r1.patch

Remove code.
Attachment #510348 - Flags: review?(bhearsum)
What's the supported way to disable a slave if the allocator is down? Just move the tac file out of the way?
Yes.

If the slave allocator is down, we should be hard at work getting it back up, anyway, and probably disabling slaves is the least of our worries during that time.  Also, any previously-disabled slaves will stay disabled if the allocator is down.
Blocks: 622980
Comment on attachment 510348 [details] [diff] [review]
m632103-puppet-manifests-r1.patch

I think there's still a need to be able to disable slaves when the allocator is down, but creating this file is just as easy as moving the tac out the way, r+.
Attachment #510348 - Flags: review?(bhearsum) → review+
Comment on attachment 510348 [details] [diff] [review]
m632103-puppet-manifests-r1.patch

1099e90f1e1f

Deployed, and I've seen a slave start up on each of the masters, so I think we're good.
Attachment #510348 - Flags: checked-in+
Status: NEW → RESOLVED
Last Resolved: 8 years ago
Resolution: --- → FIXED
Product: mozilla.org → Release Engineering
You need to log in before you can comment on or make changes to this bug.