brentpabst / newrelic-rabbitmq-plugin

A New Relic plugin for monitoring RabbitMQ http://newrelic.com/plugins
MIT License
4 stars 3 forks source link

Changed internal agent configuration from static, enabling multi-agent config #5

Closed espenwa closed 7 years ago

espenwa commented 7 years ago

Nice plugin @brentpabst - has worked fine with one agent/ElasticSearch server configured. However it fails when adding multiple agent-configurations; it will push the same data into all instances in New Relic.

This is due to the fact the host config is static, making the last agent-config being the one used for all instances.

This tiny fix should remedy that. Hope you'll be able to accept and publish an update. Since the host-config is internal, I didn't add any unit-tests.

Regards, Espen Wang Andreassen

brentpabst commented 7 years ago

Builds for this merge are failing due to repo renames we did a while back and AppVeyor integrations. I'm manually publishing v1.0.16 for now.