newrelic / infra-integrations-sdk

New Relic Infrastructure Integrations SDK
Apache License 2.0
44 stars 23 forks source link

jmx: wrap nrjmx errors so error details are not lost #272

Closed roobre closed 2 years ago

roobre commented 3 years ago

Motivated by the following line in the agent log:

[ERR] Connection error for catalogone-kafka-prod3-east.vpc.verizon.com:9999 : jmx endpoint connection error\n

Which doesn't really help to know the cause of the connection error.

If you agree with the change, I can port the commit to the v3 branch as well.

cristianciutea commented 2 years ago

LGTM, thanks! I'll take care of putting this into v3