New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Remove deprecated MongoDB metrics reporter #2443

Merged
merged 2 commits into from Jul 1, 2016

Conversation

Projects
None yet
2 participants
@joschi
Contributor

joschi commented Jul 1, 2016

The integrated MongoDB metrics reporter has been deprecated for quite some time now and should be removed.

If users really depend on functionality like this, they can use https://github.com/Graylog2/graylog-plugin-metrics-reporter/tree/master/metrics-reporter-mongodb as replacement.

@joschi joschi added this to the 2.1.0 milestone Jul 1, 2016

@bernd

This comment has been minimized.

Member

bernd commented Jul 1, 2016

👍 - Please mention the metrics reporter plugin in UPGRADING.

@bernd bernd self-assigned this Jul 1, 2016

@joschi joschi force-pushed the remove-mongodb-metrics-reporter branch from f44f31a to 27d877f Jul 1, 2016

@bernd

This comment has been minimized.

Member

bernd commented Jul 1, 2016

Sorry, please rebase again. 😄

@joschi

This comment has been minimized.

Contributor

joschi commented Jul 1, 2016

Sorry, please rebase again. 😄

You're doing this on purpose, aren't you? 😉

@bernd

This comment has been minimized.

Member

bernd commented Jul 1, 2016

¯\_(ツ)_/¯

joschi added some commits Jul 1, 2016

@joschi joschi force-pushed the remove-mongodb-metrics-reporter branch from 27d877f to 6681fe2 Jul 1, 2016

@bernd bernd merged commit 9d4bb80 into master Jul 1, 2016

2 checks passed

ci-server-integration Jenkins build graylog2-server-integration-pr 1050 has succeeded
Details
ci-web-linter Jenkins build graylog-pr-linter-check 536 has succeeded
Details

@bernd bernd deleted the remove-mongodb-metrics-reporter branch Jul 1, 2016

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment