CentOS 6: /var/run/redis directory not created

[echocat]

CentOS 6: /var/run/redis directory not created

[TiagoTT]

I have the same problem on Debian 8 and 9.
The workaround of having Puppet create the runtime directory is not an option for me because I need Redis service to start on boot and therefore before Puppet runs.

I think the problem is in the SystemD unit file:
https://github.com/echocat/puppet-redis/blob/master/templates/systemd/redis.service.erb#L8

The commands given in the ExecStartPre option will run as the User and Group selected, which can't create a directory inside /var/run because that is owned by root.
The creation of the redis_run_dir in an ExecStartPre only works if the service is ran as root.

According to the documentation:
https://www.freedesktop.org/software/systemd/man/systemd.exec.html#RuntimeDirectory=
The recommended way to manage runtime directories with SystemD is to use RuntimeDirectory option, which will create the directory and set ownership.

So I think the following block:
https://github.com/echocat/puppet-redis/blob/master/templates/systemd/redis.service.erb#L6-L12
Could simply be replaced with:

RuntimeDirectory=redis

[TiagoTT]

Sorry, I had not noticed I was running an older version of this module which did not include the PermissionsStartOnly=true setting yet.
But still, I guess that using the RuntimeDirectory= setting would be cleaner.

[dwerder]

If you want , send me a PR.