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

[DOCS] Update monitoring diagrams with metricbeat #119

Merged
merged 3 commits into from
Sep 11, 2018

Conversation

lcawl
Copy link
Contributor

@lcawl lcawl commented Sep 10, 2018

Related to elastic/beats#7035

This PR replaces the architectural diagrams with one that includes Metricbeat. It also adds text to describe the beta behaviour.


If you have at least a Gold license, you can route data from multiple production
clusters to a single monitoring cluster:
If you are not using {metricbeat} to collect the {kib} data, you might want to
Copy link
Contributor

Choose a reason for hiding this comment

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

I wouldn't say there is a causality between the usage of metricbeat and having a dedicated kibana instance. I think the previous wording is more accurate over here:

If Kibana is a regular part of your stack, you might want to create a dedicated Kibana instance for monitoring, rather than using a single Kibana instance to access both your production cluster and monitoring cluster:

Copy link
Contributor Author

Choose a reason for hiding this comment

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

Though when you're using Kibana with Metricbeat, you're not accessing the production cluster, are you?

Copy link
Contributor

@ycombinator ycombinator Sep 11, 2018

Choose a reason for hiding this comment

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

In a setup where Metricbeat is being used to monitor Kibana:

  • Kibana has two connections: one to the production ES cluster (to visualize data in the cluster and administer the cluster) and another to the Monitoring ES cluster (to display the Monitoring UI). Of course, the two clusters might actually be the same, single cluster.
  • Metricbeat has one connection: to the monitoring ES cluster.

Does this answer your question or did I just make it more confusing :)?

Copy link
Contributor Author

Choose a reason for hiding this comment

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

Thanks, I've updated that paragraph, since it hadn't been clear to me what was meant by being "a regular part of the stack".

Copy link
Contributor

Choose a reason for hiding this comment

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

Brilliant, this explanation is much clearer IMO. Thanks!

activities from impacting the performance of your production cluster.
activities from impacting the performance of your production cluster.

beta[] In 6.5 and later, you can use {metricbeat} to collect and ship data about
Copy link
Contributor

Choose a reason for hiding this comment

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

I like that we are flagging this as a "beta" section, including the diagram that follows it. What do you think about also keeping the original non-beta way of doing things, including the diagram? Do you think it might be too much information and could lead to confusion?

Copy link
Contributor Author

@lcawl lcawl Sep 11, 2018

Choose a reason for hiding this comment

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

I have added back the first simple diagram and refreshed the preview. I think the diagram with two clusters is unnecessary.

If {kib} is a regular part of your stack, you might want to create a dedicated
{kib} instance for monitoring, rather than using a single {kib} instance to
If {kib} is a regular part of your stack, you might want to create a dedicated
{kib} instance for monitoring, rather than using a single {kib} instance to
access both your production cluster and monitoring cluster:

image::monitoring/images/architecture3.jpg[A monitoring environment with separate {kib} instances]
Copy link
Contributor

Choose a reason for hiding this comment

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

Looking at the preview URL, this image only shows one Kibana instance. Maybe this ought to reference a different image file?

Copy link
Contributor Author

Choose a reason for hiding this comment

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

The last image has the write and read coming from separate "instances" within the Kibana box, but if that's not clear, I can add a second Kibana box.

Copy link
Contributor

Choose a reason for hiding this comment

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

Ugh, indeed it does. Sorry about that, I just missed the second instance. No need to change anything here, sorry about the noise again.

Copy link
Contributor

@ycombinator ycombinator left a comment

Choose a reason for hiding this comment

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

LGTM.

@lcawl lcawl merged commit 72cf3ee into elastic:master Sep 11, 2018
@lcawl lcawl deleted the lcawley-beats-monitoring branch September 11, 2018 19:09
@lcawl lcawl added the v6.4.2 label Oct 1, 2018
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