-
Notifications
You must be signed in to change notification settings - Fork 278
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 from source #340
Comments
A redis installation can be a sample as this:
|
@runningman84 Redis from source is for consistency and it ensures a recent version installed that works with Sensu and the Redis configuration can be successfully managed by the cookbook (new options etc). We can certainly have another pass at it and see if things have changed. |
Agreed, the version of redis in ubuntu 14.04 and centos 6 is very out of date. |
@portertech any thoughts on this as of now? I believe the instructions on sensuapp.org specify to install Redis (>= 1.3.14) via distro packages. |
I'm closing this as something we won't fix. The |
Right now this cookbook uses a redis recipe which builds redis from source. Why is that?
Every modern linux distribution like ubuntu, centos contains a redis package. Using the default package would allow to use the security patches of the distribution and reduce the install complexity.
This was already posted here but belongs to sensu-chef more than to chef-monitor...
portertech/chef-monitor#33
The text was updated successfully, but these errors were encountered: