geerlingguy / ansible-role-redis

Ansible Role - Redis
https://galaxy.ansible.com/geerlingguy/redis/
MIT License
254 stars 197 forks source link

Managing systems features #13

Closed wanix closed 8 years ago

wanix commented 8 years ago

Hi,

I add to your roles the followings features:

If not managed, Redis emit a warning

I add the creation of the data directory and the log directory if inexistent too

Thanks for your role

geerlingguy commented 8 years ago

I'd rather split the logfile/log dir part out into a separate PR, as that looks like a change that I may put in (after a little extra testing).

For the other changes, since those can be more OS/server-specific, I'd rather not include those changes in this role—they should really be playbook-level, or a custom role for your particular environment that runs before this role.