Fixed problem in the init script #4

Merged
merged 1 commit into from Jan 14, 2013

Conversation

Projects
None yet
2 participants
Contributor

iconara commented Jan 14, 2013

This is travis-ci/travis-cookbooks#134

My master branch of this repo contains lots of other changes, I've basically rewritten almost all of it. Partly because I needed to understand things, partly because it didn't work with C* 1.2 and partly because I wanted it to work in CentOS. Perhaps we can make something together that would work in both Ubuntu and CentOS? I'm not a chef expert.

@iconara iconara Fixed problem in the init script
Updated the Cassandra init script so that the `cassandra` service starts properly and shuts down cleanly.

The current script repeats most of what's in `/usr/local/cassandra/bin/cassandra`, but not enough, there's a lot of the environment Cassandra needs to start that is missing. This patch changes it so that it simply runs `cassandra -p $PIDFILE` and daemonizes it with `start-stop-daemon` (by way of `start_daemon`). The stop and status tasks had to change too.
e68ed7e

@michaelklishin michaelklishin pushed a commit that referenced this pull request Jan 14, 2013

Michael Klishin Merge pull request #4 from iconara/fix_for_c_1.2.0
Fixed problem in the init script
36c464d

@michaelklishin michaelklishin merged commit 36c464d into michaelklishin:master Jan 14, 2013

Owner

michaelklishin commented Jan 14, 2013

Sure, we can join forces on this

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment