You signed in with another tab or window. Reload to refresh your session.You signed out in another tab or window. Reload to refresh your session.You switched accounts on another tab or window. Reload to refresh your session.Dismiss alert
The nginx code follows a pattern of catching all exceptions when attempting to submit a metric, which makes sense due to wanting to continue without error. Unfortunately, it additionally eats the real error message and displays a super generic message in the logs. This makes it very difficult to actually debug issues.
I ended up manually modifying the code on my local install to figure out that my nginx.yaml had a syntax error in it. It would have been more obvious if the true exception (unhashable type: 'dict') had been printed out. (Additionally, there should probably be error checking when tags are loaded instead of reported.)
I'm willing to fix either or both of these issues, but would like advice on how you would like to see them done (e.g. log at a different level? Log to stderr?). Thanks!
The text was updated successfully, but these errors were encountered:
The nginx code follows a pattern of catching all exceptions when attempting to submit a metric, which makes sense due to wanting to continue without error. Unfortunately, it additionally eats the real error message and displays a super generic message in the logs. This makes it very difficult to actually debug issues.
I ended up manually modifying the code on my local install to figure out that my
nginx.yaml
had a syntax error in it. It would have been more obvious if the true exception (unhashable type: 'dict'
) had been printed out. (Additionally, there should probably be error checking when tags are loaded instead of reported.)I'm willing to fix either or both of these issues, but would like advice on how you would like to see them done (e.g. log at a different level? Log to stderr?). Thanks!
The text was updated successfully, but these errors were encountered: