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

Comments

Projects
None yet
5 participants
@runningman84
Contributor

runningman84 commented Mar 25, 2015

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

This comment has been minimized.

Show comment
Hide comment
@runningman84

runningman84 Apr 8, 2015

Contributor

A redis installation can be a sample as this:

package "redis-server" do
  action :install
end

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

runningman84 commented Apr 8, 2015

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

This comment has been minimized.

Show comment
Hide comment
@portertech

portertech May 22, 2015

Member

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

Member

portertech commented May 22, 2015

@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

This comment has been minimized.

Show comment
Hide comment
@spuder

spuder Oct 27, 2015

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

spuder commented Oct 27, 2015

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

@cwjohnston

This comment has been minimized.

Show comment
Hide comment
@cwjohnston

cwjohnston Feb 3, 2016

Member

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

Member

cwjohnston commented Feb 3, 2016

@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 cwjohnston added the question label Feb 3, 2016

@cwjohnston

This comment has been minimized.

Show comment
Hide comment
@cwjohnston

cwjohnston Apr 19, 2016

Member

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.

Member

cwjohnston commented Apr 19, 2016

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.

@cwjohnston cwjohnston closed this Apr 19, 2016

@cwjohnston cwjohnston added wontfix and removed question labels Apr 19, 2016

@majormoses majormoses added Status: Won't Fix and removed wontfix labels Mar 2, 2018

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