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

Add CockroachDB Metricbeat module to Kibana Home #49672

Merged
merged 3 commits into from
Oct 30, 2019
Merged

Add CockroachDB Metricbeat module to Kibana Home #49672

merged 3 commits into from
Oct 30, 2019

Conversation

jsoriano
Copy link
Member

Summary

Add CockroachDB metricbeat module to the Kibana add data UI.

Captura de pantalla de 2019-10-29 20-49-36

Checklist

Use strikethroughs to remove checklist items you don't feel are applicable to this PR.

For maintainers

@jsoriano jsoriano added v8.0.0 release_note:skip Skip the PR/issue when compiling release notes v7.6.0 labels Oct 29, 2019
@jsoriano jsoriano requested a review from exekias October 29, 2019 19:58
@jsoriano jsoriano self-assigned this Oct 29, 2019
@elasticmachine
Copy link
Contributor

💚 Build Succeeded

@@ -46,7 +46,7 @@ export function corednsLogsSpecProvider(server, context) {
learnMoreLink: '{config.docs.beats.filebeat}/filebeat-module-coredns.html',
},
}),
//TODO: euiIconType: 'logoCoredns',
euiIconType: '/plugins/kibana/home/tutorial_resources/logos/coredns.svg',
Copy link
Contributor

Choose a reason for hiding this comment

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

I understand this is intended, isn't it?

Copy link
Member Author

Choose a reason for hiding this comment

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

Yep, I found that the logo was added for metrics but was not being used for logs.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
release_note:skip Skip the PR/issue when compiling release notes v7.6.0 v8.0.0
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants