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

Recommend cluster scope for Metricbeat #70865

Conversation

DaveCTurner
Copy link
Contributor

@DaveCTurner DaveCTurner commented Mar 25, 2021

Today we mention Metricbeat's scope parameter but offer no guidance
about how it should be used. This commit adds guidance to use scope: cluster, especially on clusters with dedicated master-eligible nodes.

Closes #70853

Today we mention Metricbeat's `scope` parameter but offer no guidance
about how it should be used. This commit adds guidance to use `scope:
cluster`, especially on clusters with dedicated master-eligible nodes.
@elasticmachine
Copy link
Collaborator

Pinging @elastic/es-docs (Team:Docs)

@elasticmachine
Copy link
Collaborator

Pinging @elastic/es-core-features (Team:Core/Features)

@DaveCTurner
Copy link
Contributor Author

@elasticmachine update branch

@elasticmachine
Copy link
Collaborator

merge conflict between base and head

@gwbrown gwbrown added v7.12.2 and removed v7.12.1 labels Apr 14, 2021
Copy link
Contributor

@jakommo jakommo left a comment

Choose a reason for hiding this comment

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

LGTM!

Copy link
Contributor

@henningandersen henningandersen left a comment

Choose a reason for hiding this comment

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

Left a comment, looking good otherwise.

Comment on lines 49 to 51
the cluster. If this is not possible, and the cluster does not use dedicated
master-eligible nodes, then install one {metricbeat} instance for each {es}
node in the production cluster and use the default `scope: node`. When
Copy link
Contributor

Choose a reason for hiding this comment

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

I find the and the cluster does not use dedicated master-eligible nodes a bit too hard/prescriptive. I would prefer to add a separate sentence that this is not recommended with dedicated master-eligible nodes and requires beefier master nodes than when using scope: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.

ok, see faa0147

Copy link
Contributor

@henningandersen henningandersen left a comment

Choose a reason for hiding this comment

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

LGTM.

@DaveCTurner DaveCTurner merged commit f379216 into elastic:master May 10, 2021
@DaveCTurner
Copy link
Contributor Author

Thanks all!

@DaveCTurner DaveCTurner deleted the 2021-03-25-recommend-cluster-scope-for-metricbeat branch May 10, 2021 13:16
DaveCTurner added a commit that referenced this pull request May 10, 2021
Today we mention Metricbeat's `scope` parameter but offer no guidance
about how it should be used. This commit adds guidance to use `scope:
cluster`, especially on clusters with dedicated master-eligible nodes.
DaveCTurner added a commit that referenced this pull request May 10, 2021
Today we mention Metricbeat's `scope` parameter but offer no guidance
about how it should be used. This commit adds guidance to use `scope:
cluster`, especially on clusters with dedicated master-eligible nodes.
DaveCTurner added a commit that referenced this pull request May 10, 2021
Today we mention Metricbeat's `scope` parameter but offer no guidance
about how it should be used. This commit adds guidance to use `scope:
cluster`, especially on clusters with dedicated master-eligible nodes.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
:Data Management/Monitoring >docs General docs changes Team:Data Management Meta label for data/management team Team:Docs Meta label for docs team v7.11.3 v7.12.2 v7.13.0 v8.0.0-alpha1
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Elasticsearch monitoring docs recommend installing Metricbeat on every node regardless of cluster config
8 participants