-
Notifications
You must be signed in to change notification settings - Fork 490
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
failed to send alert data to VictorOps: failed to understand VictorOps response #232
Comments
@devilldog Which version are you using? There should be a more verbose error logged about how the communication with VictorOps failed. |
10.0-1 |
If you upgrade to v0.10.1-1 you will get a better error message and we can On Fri, Feb 12, 2016 at 1:44 PM, devilldog [email protected] wrote:
|
OK Will do thanks ! |
I upgraded to 0.10.1 but i get the same message for VictorOps as before and i enabled DEBUG in the logs |
quickly looking at the code for a language i do not know, it looks like the error is not being returned and it is not actually printing out the error just the proceeding line "failed to understand response" |
any thoughts ? |
@devilldog The relevant line in the code is https://github.com/influxdata/kapacitor/blob/master/services/victorops/service.go#L83
Can you post the logs you are seeing relating to the error? |
Yeah it is not including the actual response
|
rpm -qa|grep kapac |
@devilldog I think I have finally reproduced the issue! VictorOps will return a 404 with no content when the API key is not valid. I have a fix coming that will print out a better error message indicating the actual error. Can you double check your API key to make sure it is correct? |
yeah i came to the same conclusion, i will test and let you know thanks ! |
Here is my config.
enabled = true
api-key = "xxxxx"
routing-key = "everyone"
url = "https://alert.victorops.com/integrations/generic/20131114/alert"
global = false
The text was updated successfully, but these errors were encountered: