Skip to content

Various Sentinel fixes#56

Open
prachetasp wants to merge 3 commits intofsalum:masterfrom
boomerdigital:psp/sentinel_fixes
Open

Various Sentinel fixes#56
prachetasp wants to merge 3 commits intofsalum:masterfrom
boomerdigital:psp/sentinel_fixes

Conversation

@prachetasp
Copy link

This PR includes:

  • The ability to make the Redis installation optional in the Sentinel setup (useful when you manage Redis installation in separate manifests/modules)
  • The ability to specify the bind parameter in the Sentinel config
  • Executes the Sentinel config file copy as root instead of redis. This caused subsequent Puppet runs to break since the redis user didn't have write privs on /etc/redis (this is advisable for security). Since we already specify the file should be owner => redis, group => redis specifying it again is unnecessary

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

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant