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

Metricbeat modules: rename dashboards and visualizations #4979

Merged
merged 1 commit into from
Aug 24, 2017

Conversation

monicasarbu
Copy link
Contributor

This is an effort to apply common naming conventions for dashboards, visualizations and searches.

Similar with #4952

@monicasarbu monicasarbu added :Dashboards in progress Pull request is currently in progress. labels Aug 23, 2017
@tsg tsg added needs_backport PR is waiting to be backported to other branches. v6.0.0-beta2 labels Aug 24, 2017
@tsg
Copy link
Contributor

tsg commented Aug 24, 2017

@monicasarbu This one needs a rebase. I might have changed the whitespace on the module.yml file, I think that's why it's failing.

@monicasarbu monicasarbu force-pushed the update_metricbeat_dashboards branch from 45b30df to e709228 Compare August 24, 2017 09:19
@monicasarbu monicasarbu force-pushed the update_metricbeat_dashboards branch from b30e801 to 3f31757 Compare August 24, 2017 09:24
@monicasarbu monicasarbu added review and removed in progress Pull request is currently in progress. labels Aug 24, 2017
@monicasarbu monicasarbu mentioned this pull request Aug 24, 2017
5 tasks
@tsg
Copy link
Contributor

tsg commented Aug 24, 2017

Tested manually, looks good.

@tsg tsg merged commit 30badf6 into elastic:master Aug 24, 2017
tsg pushed a commit to tsg/beats that referenced this pull request Aug 24, 2017
@tsg tsg removed the needs_backport PR is waiting to be backported to other branches. label Aug 24, 2017
kvch pushed a commit that referenced this pull request Aug 24, 2017
leweafan pushed a commit to leweafan/beats that referenced this pull request Apr 28, 2023
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