Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Redis service fails to start on Docker containers #2166

Closed
svenfuchs opened this issue Apr 15, 2014 · 6 comments
Closed

Redis service fails to start on Docker containers #2166

svenfuchs opened this issue Apr 15, 2014 · 6 comments

Comments

@svenfuchs
Copy link
Contributor

The Redis package that we currently install uses an init file like this:

root@089fae39196f:~# cat /etc/init/redis-server.conf
# redis-server - Redis Datastore Server
#
# Redis is a key value in memory persistent datastore

#start on (local-filesystems and runlevel [2345])
stop on runlevel [016]
respawn
expect fork
limit nofile 20000 65000
pre-start script
mkdir -p /var/run/redis
chown redis:redis /var/run/redis
end script
exec start-stop-daemon --start --chuid redis:redis --pidfile /var/run/redis/redis.pid --umask 007 --exec /usr/bin/redis-server -- /etc/redis/redis.conf

The limit nofile stanza fails on Docker containers (apparently ulimit can't be changed unless when run in -privileged model moby/moby#1900 (comment)).

Can this be removed?

@roidrage
Copy link
Contributor

roidrage commented May 5, 2014

I think this could probably be removed. From what I can tell it sets ulimits on the open files?

@roidrage
Copy link
Contributor

@svenfuchs is this still an issue?

@svenfuchs
Copy link
Contributor Author

Yeah, I think this is still open.

@roidrage
Copy link
Contributor

Can you look into fixing the cookbooks or our Redis configuration template?

@joshk
Copy link
Contributor

joshk commented Jun 21, 2014

making it configurable would be ideal

@roidrage
Copy link
Contributor

Closing this issue, as this seems to work as expected now.

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

No branches or pull requests

5 participants