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 from source #340

Closed
runningman84 opened this issue Mar 25, 2015 · 5 comments
Closed

redis from source #340

runningman84 opened this issue Mar 25, 2015 · 5 comments

Comments

@runningman84
Copy link
Contributor

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

@runningman84
Copy link
Contributor Author

A redis installation can be a sample as this:

package "redis-server" do
  action :install
end

service "redis-server" do
  action [ :enable, :start ]
end

@portertech
Copy link
Contributor

@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.

@spuder
Copy link

spuder commented Oct 27, 2015

Agreed, the version of redis in ubuntu 14.04 and centos 6 is very out of date.

@cwjohnston
Copy link
Contributor

@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.

@cwjohnston
Copy link
Contributor

I'm closing this as something we won't fix. The redis recipe in this cookbook is provided as a convenience; those who wish to install Redis from packages will be better served by providing a simple recipe in their organization's wrapper cookbook.

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