Skip to content
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

Release the Ceph Metricbeat module as GA #7661

Closed
wants to merge 1 commit into from

Conversation

ruflin
Copy link
Contributor

@ruflin ruflin commented Jul 20, 2018

  • Add docs
  • Fix default config
  • Extend reference config

# - osd_tree
# - osd_df
# - pool_disk
metricsets:
Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I think this was actually a "bug" before as it meant the module is started twice with all default metricsets. Should we backport this fix to 6.3?

* Add docs
* Fix default config
* Extend reference config
@herver
Copy link
Contributor

herver commented Jul 20, 2018

Hi there,

Sorry to jump on this PR, but we're using Ceph here and it turns out that the ceph-rest-api is deprecated since the Luminous release and has been removed in Mimic (see http://docs.ceph.com/docs/mimic/releases/luminous/)

It has been replaced by a ceph-mgr plugin (http://docs.ceph.com/docs/luminous/mgr/restful/#).

@ruflin ruflin added in progress Pull request is currently in progress. and removed review labels Jul 20, 2018
@ruflin
Copy link
Contributor Author

ruflin commented Jul 20, 2018

@herver Thank you for the heads up. Will need to investigate more here.

Let's hold off merging this PR for now.

@ruflin
Copy link
Contributor Author

ruflin commented Jul 25, 2018

I'm closing this PR and created a follow up issue here to track the implementation of the new API here: #7723

@herver Appreciate a lot that you jumped in here and provided the above details.

@ruflin ruflin closed this Jul 25, 2018
@ruflin ruflin deleted the ceph-to-ga branch July 25, 2018 06:57
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
in progress Pull request is currently in progress. Metricbeat Metricbeat
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants