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: Fix monitoring privileges #14307

Merged
merged 3 commits into from
Nov 1, 2019
Merged

Conversation

cwurm
Copy link
Contributor

@cwurm cwurm commented Oct 29, 2019

I noticed that the monitoring role on https://www.elastic.co/guide/en/beats/filebeat/master/feature-roles.html#privileges-to-publish-monitoring only contains the monitor cluster privilege. This is the lowest level cluster privilege that gives access to basic cluster APIs to retrieve things like cluster health and version, but does not allow writing data.

This PR adds create_index and create_doc on .monitoring-beats-* as required privileges to the monitoring role.

Copy link
Contributor

@dedemorton dedemorton left a comment

Choose a reason for hiding this comment

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

Thanks for fixing this! The changes look good. Full disclosure: I didn't test these privileges. Assuming they passed your testing, LGTM.

@cwurm cwurm force-pushed the monitoring_privileges branch from 9426270 to 44a91d9 Compare November 1, 2019 21:01
@cwurm
Copy link
Contributor Author

cwurm commented Nov 1, 2019

With #14313 merged, create_doc is now enough. I've updated this PR accordingly.

@cwurm cwurm merged commit 53954d8 into elastic:master Nov 1, 2019
@cwurm cwurm deleted the monitoring_privileges branch November 1, 2019 23:05
jorgemarey pushed a commit to jorgemarey/beats that referenced this pull request Jun 8, 2020
Adds `create_index` and `create_doc` on `.monitoring-beats-*` as required privileges to the monitoring role.
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