Skip to content

[Metricbeat] Migrate Ceph osd_df to use ReporterV2 interface#11013

Merged
sayden merged 5 commits intoelastic:masterfrom
sayden:migration/mb/reporterv2/ceph/osd_df
Mar 7, 2019
Merged

[Metricbeat] Migrate Ceph osd_df to use ReporterV2 interface#11013
sayden merged 5 commits intoelastic:masterfrom
sayden:migration/mb/reporterv2/ceph/osd_df

Conversation

@sayden
Copy link
Copy Markdown
Contributor

@sayden sayden commented Mar 1, 2019

Refer to #10774 for more info

@sayden sayden added Metricbeat Metricbeat Team:Integrations Label for the Integrations team labels Mar 1, 2019
@sayden sayden self-assigned this Mar 1, 2019
@sayden sayden requested a review from a team as a code owner March 1, 2019 10:42
@sayden sayden force-pushed the migration/mb/reporterv2/ceph/osd_df branch from a4e1adc to 855db7b Compare March 1, 2019 12:36
@ruflin
Copy link
Copy Markdown
Contributor

ruflin commented Mar 5, 2019

Failure seems to be related.

@sayden sayden force-pushed the migration/mb/reporterv2/ceph/osd_df branch from b91e55e to e8c5c04 Compare March 6, 2019 15:11
if err != nil {
logp.Err("Error: %+v", err)
return nil, err
return nil, errors.Wrap(err, "error getting data for OSD_DF")
Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Thanks for improving the error 👍

@sayden
Copy link
Copy Markdown
Contributor Author

sayden commented Mar 7, 2019

Error in RabbitMQ seems unrelated. Merging.

@sayden sayden merged commit fd27e46 into elastic:master Mar 7, 2019
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Metricbeat Metricbeat Team:Integrations Label for the Integrations team

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants