Closed kang-makes closed 1 year ago
Notice that (unrelated to your PR) the pre-release of MongoDB failed since the e2e action is run as well at release time and we forgot to fix it there as well.
Can you fix that as well? Or we can open a different PR :)
Tests are now passing. Also fixed the issue @paologallinaharbur pointed me out as it is simple enough to not pollute this review.
We finally decided to keep two versions of the mongo integration at the same time, the old one that uses an outdated library and this one that leverages the MongoDB exporter.
As the exporters break everything, not only use a completely different codebase but also send a different set of metrics, y break also dashboards and alerts.
Because of that, we are renaming this to
mongodb3
so that in a package manager you can installnri-mongodb
andnri-mongodb3
at the same time.