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

Cannot create /etc/sensu/conf.d/redis.json without "password" #503

Closed
paramite opened this issue May 26, 2016 · 3 comments
Closed

Cannot create /etc/sensu/conf.d/redis.json without "password" #503

paramite opened this issue May 26, 2016 · 3 comments

Comments

@paramite
Copy link
Contributor

For cases where Redis instance is not password protected, it is impossible to setup Sensu, with this Puppet module, to connect to it because even when 'redis_password' is set to undef, the resulting config file contains "password": "".

@jaxxstorm
Copy link
Contributor

I can't reproduce this with the latest version of the module, my redis.json

{
  "redis": {
    "port": 6379,
    "host": "sensuhost",
    "reconnect_on_error": true,
    "db": 0,
    "auto_reconnect": true
  }
}

Can you show the puppet code/hiera data you're using?

@paramite
Copy link
Contributor Author

Back then I was not using latest version of the module. I will try to reproduce this with latest version and let you know.

@paramite
Copy link
Contributor Author

Indeed, with latest version of the module, redis.json is created correctly. I'm closing the bug. Sorry for the noise.

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

No branches or pull requests

2 participants