Skip to content
This repository has been archived by the owner on Jan 1, 2020. It is now read-only.

Init script start does not sleep long enough to ensure successful start #353

Closed
promisedlandt opened this issue Aug 17, 2012 · 5 comments
Closed

Comments

@promisedlandt
Copy link

In the init script start function, there's a section:

sleep 1

# make sure it's still running. some errors occur only after startup
status &> /dev/null
if [ $? -ne 0 ]; then
    echo_fail
    exit 1
fi

However, errors like not being able to connect to RabbitMQ occur on my low-power system only after about 4 seconds.
On the other hand, increasing the sleep time would be an inconvenience when everythign works fine, so I'm not sure what a good solution would be here.

@portertech
Copy link
Contributor

@promisedlandt Seems that this issue is related to https://github.com/sensu/sensu-build

I believe we are still planning to ship an embedded runit with the Sensu packages, killing the need for these init scripts.

@joemiller Thoughts? Should we attack runit now?

@promisedlandt
Copy link
Author

@portertech Not sure how that's related, but apparently runit can run under sysv-init, so I'm cool with that. It would solve #354 as well I believe, as runit doesn't need pidfiles?

Off topic: I got some days off next month, is there anywhere in Sensu I can help out with mediocre programming and low sysadmin skills?

@joemiller
Copy link
Contributor

Yeah, might be a good time to try to embed runit now. We'll have to take a
look at chef-omnibus and see what techniques they used to get it going on
each platform.

On Thu, Aug 23, 2012 at 1:47 AM, Nils Landt notifications@github.comwrote:

@portertech https://github.com/portertech Not sure how that's related,
but apparently runit can run under sysv-init, so I'm cool with that. It
would solve #354 #354 as well I
believe, as runit doesn't need pidfiles?

Off topic: I got some days off next month, is there anywhere in Sensu I
can help out with mediocre programming and low sysadmin skills?


Reply to this email directly or view it on GitHubhttps://github.com//issues/353#issuecomment-7963490.

@nstielau
Copy link
Contributor

@promisedlandt Not sure about where to code right now, but a blog post, docs or being helpful in #sensu are great ways to make sensu more successful.

More plugins/handlers are always great too ;)

@portertech
Copy link
Contributor

sensu-0.9.12.beta.5-5 has an embedded runit w/ some tooling.

/opt/sensu/bin/sensu-ctl configure
/opt/sensu/bin/sensu-ctl sensu-client enable

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

No branches or pull requests

4 participants