Upstart script fails to create missing pid directory

[sous-chefs]

Upstart script fails to create missing pid directory

[brianbianco]

Oh, the joy i've felt working with upstart. The TL;DR is that when using upstart as a job control, i will have to tell redis not to daemonize itself, then i can run it via upstart. In order to allow the creation of the pid directory in the upstart script I can't use setuid as it is global to the script. That means I have to run the redis server binary like so

su -s /bin/sh -c exec "$0" "$@" redis /usr/local/bin/redis-server /etc/redis/6379.conf

As using sudo causes issues with upstart as it forks.

The bright side here is this actually seems to work, and work on reboot without any issue.

I also have started to depend on the "ulimit" cookbook to handle the setting of the file descriptor limit for redis. I had resisted this change in the past as I like keeping requirements thin, but i've buckled on this one.

[brianbianco]

This is finally fixed. Is released as 1.4.1

[lock]

This thread has been automatically locked since there has not been any recent activity after it was closed. Please open a new issue for related bugs.