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

Replace missing field names from status Metricset in MongoDB Dashboard #9795

Merged
merged 3 commits into from
Jan 8, 2019

Conversation

sayden
Copy link
Contributor

@sayden sayden commented Dec 26, 2018

According to issue #9715 it seems that some field names within the MongoDB dashboard were incorrect.

It seems that this commit changed the field names 5544838 5 months ago.

In this PR, the correct field names are set. I'm not very experienced with Kibana so I have just modified current dashboard JSON file instead of replacing it with the exported one using this method.

Working in Kibana 6.5.4

@sayden sayden requested review from jsoriano and ruflin December 26, 2018 16:10
@sayden sayden changed the title Replace missing field names in MongoDB Dashboard Replace missing field names from status Metricset in MongoDB Dashboard Dec 26, 2018
@sayden sayden requested a review from a team as a code owner December 26, 2018 16:14
Copy link
Contributor

@ruflin ruflin left a comment

Choose a reason for hiding this comment

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

Did you test these dashboards on your end? I think the host.name part should be revert but the other changes LGTM. To which versions should we backport this?

@@ -37,7 +37,7 @@
"enabled": true,
"id": "2",
"params": {
"field": "metricset.host",
"field": "host.name",
Copy link
Contributor

Choose a reason for hiding this comment

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

metricset.host and host.name are not the same thing. host.name is then name from the host on which the beat runs, metricset.host is what address of the service it connects to.

These fields should not have changed in the PR your linked. Same below.

Copy link
Contributor Author

Choose a reason for hiding this comment

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

Ok, I have reverted both ocurrences of host.name to their original values metricset.host

@ruflin ruflin added the needs_backport PR is waiting to be backported to other branches. label Dec 27, 2018
@sayden sayden force-pushed the bugfix/mb/mongodb-dashboard branch from b1b7eac to c2eba63 Compare December 27, 2018 10:07
@sayden sayden force-pushed the bugfix/mb/mongodb-dashboard branch from c2eba63 to 3b4da99 Compare December 27, 2018 10:22
@sayden sayden self-assigned this Dec 27, 2018
@sayden sayden added the bug label Dec 27, 2018
Copy link
Contributor

@ruflin ruflin left a comment

Choose a reason for hiding this comment

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

LGTM.

Did you manage to test this locally and confirm the change / fix?

@sayden
Copy link
Contributor Author

sayden commented Jan 2, 2019

Yes, of course. That's how I realized that host.name was missing in the dashboard. So maybe there's something else that is wrong?

I'll wait for your answer before merging.

@ruflin
Copy link
Contributor

ruflin commented Jan 7, 2019

@sayden All good. Please merge and backport.

@sayden sayden merged commit fba0680 into elastic:master Jan 8, 2019
sayden added a commit to sayden/beats that referenced this pull request Jan 8, 2019
@sayden sayden added v6.7.0 and removed needs_backport PR is waiting to be backported to other branches. labels Jan 8, 2019
sayden added a commit that referenced this pull request Jan 8, 2019
…ricset in MongoDB Dashboard (#9945)

Replace missing field names from status Metricset in MongoDB Dashboard (#9795)
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants