Skip to content
This repository was archived by the owner on Nov 23, 2017. It is now read-only.

CHEF-154 runit: add a :down parameter to down services by default#161

Closed
philips wants to merge 1 commit into
chef-boneyard:masterfrom
philips:runit-down-param
Closed

CHEF-154 runit: add a :down parameter to down services by default#161
philips wants to merge 1 commit into
chef-boneyard:masterfrom
philips:runit-down-param

Conversation

@philips

@philips philips commented Jul 6, 2011

Copy link
Copy Markdown
Contributor

Setting down to true will set the service to non-running by default. To
start it up again run:

sv start <servicename>

This just adds a down file. See the runsv manual:

"If the file service/down exists, runsv does not start ./run immediately."
http://smarden.org/runit/runsv.8.html

Setting down to true will set the service to non-running by default. To
start it up again run:

	sv start <servicename>

This just adds a down file. See the runsv manual:

"If the file service/down exists, runsv does not start ./run immediately."
http://smarden.org/runit/runsv.8.html
@jtimberman

Copy link
Copy Markdown

Longer term we're looking to implement runit as a LWRP, rather than a definition.

http://tickets.opscode.com/browse/CHEF-154

My branch needs to be updated, but it will be the basis of the code for the implementation.

@philips

philips commented Dec 19, 2011

Copy link
Copy Markdown
Contributor Author

How long term are we talking? Is it reasonable to merge this now?

@jtimberman

Copy link
Copy Markdown

I commented on CHEF-154, we'll incorporate this change in the LWRP.

@jtimberman

Copy link
Copy Markdown

Hello,

This is implemented in my branch for CHEF-154:

I'm closing this pull request based on that.

Thank you.

@jtimberman jtimberman closed this Apr 3, 2012
clkao pushed a commit to clkao-cookbooks/runit that referenced this pull request Nov 4, 2013
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants