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 docs about loading heartbeat dashboard #3804

Merged
merged 1 commit into from
Mar 28, 2017

Conversation

dedemorton
Copy link
Contributor

@urso You might want to add a more interesting screen capture of Heartbeat data.

BTW, I use "dashboards" plural in case we add more later.

@dedemorton
Copy link
Contributor Author

Adds dashboard content for item tracked in #3539 meta issue.

@tsg
Copy link
Contributor

tsg commented Mar 24, 2017

You can find a somehow more colorful dashboard screenshot here: #3531

Also, the new screenshot for kibana-created-indexes.png seems wrong to me. Perhaps you replaced by mistake?

@dedemorton dedemorton force-pushed the add_heartbeat_dashboards branch from 22a20db to e432404 Compare March 24, 2017 16:33
@dedemorton
Copy link
Contributor Author

dedemorton commented Mar 24, 2017

@tsg I've replaced the Heartbeat dashboard. There were a few minor changes to the Kibana UI after you added the dashboard, but I think that having realistic looking data is more important than having the UI match up exactly (it's not too far off).

The other screen capture, kibana-created-indexes.png, probably looks wrong because I replaced an old screenshot (which showed how to set an index as the default) with a screenshot of the Discover page. The old screen shot was leftover from me copying files to use as a template. We don't tell users to set an index as the default anymore because that happens automatically. So the new screenshot is the right one...it just looks weird when you compare it to the old one.

@tsg tsg merged commit 419d46f into elastic:master Mar 28, 2017
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