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

Trailing comma issue in config #492

Closed
nux opened this issue Apr 6, 2016 · 3 comments
Closed

Trailing comma issue in config #492

nux opened this issue Apr 6, 2016 · 3 comments

Comments

@nux
Copy link

nux commented Apr 6, 2016

When debugging an issue with uchiwa not connecting to API, I found out that the lack of a trailing slash in redis.json on the last item caused the connection to redis to fail.

{
"redis": {
"port": 6379,
"host": "localhost",
"reconnect_on_error": false,
"db": 0,
"auto_reconnect": "true"
}
}
vs
{
"redis": {
"port": 6379,
"host": "localhost",
"reconnect_on_error": false,
"db": 0,
"auto_reconnect": "true",
}
}

This happened after updating packages on Debian (Jessie). Debian was updated from 8.3 to 8.4. Not sure if it's related.

ii ruby2.1 2.1.5-2+deb8u2 amd64 Interpreter of object-oriented scripting language Ruby
ii sensu 0.23.0-1 amd64 A monitoring framework that aims to be simple, malleable, and scalable.
ii uchiwa 1:0.14.0-1 amd64 Uchiwa is a simple dashboard for the Sensu monitoring framework.

@GhostLyrics
Copy link

This is not an issue in sensu-puppet. With a trailing comma the content of the file is not valid JSON and as such not accepted by sensu, causing it to ignore the configuration and go with the default settings. (grep for "ignored" in /var/log/sensu/sensu-server.log)

To solve your issue, the host needs to be "127.0.0.1" since resolution is broken in 0.23.0 for IPv6.

@GhostLyrics
Copy link

Please check sensu/sensu#1209

@nux
Copy link
Author

nux commented Apr 7, 2016

Thanks!

@nux nux closed this as completed Apr 7, 2016
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