newrelic / nri-jmx

New Relic Infrastructure JMX Integration
Apache License 2.0
6 stars 16 forks source link

Ignoring nrjmx output after a warning message #64

Closed josemore closed 3 years ago

josemore commented 3 years ago

It seems that the sdk is ignoring the rest of nrjmx reponse after nrjmx logs a warning message: https://github.com/newrelic/infra-integrations-sdk/blob/master/jmx/jmx.go#L370

But nrjmx code looks like its expected to continue the data fetching: https://github.com/newrelic/nrjmx/blob/c9b0ae557bf3ab1a938bb63519546de79189a373/src/main/java/org/newrelic/nrjmx/JMXFetcher.java#L352

Steps to reproduce:

See: https://newrelic.atlassian.net/browse/IHOST-3341

josemore commented 3 years ago

Fixed in nri-jmx 2.4.5