-
Notifications
You must be signed in to change notification settings - Fork 4.9k
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
Simplify Beats security docs #11329
Simplify Beats security docs #11329
Conversation
@dedemorton add the two roles required. |
libbeat/docs/security/users.asciidoc
Outdated
// For example, on 6.7, I get the following | ||
// error when I run Filebeat (with the system module enabled) | ||
// when the user doesn't have the `manage_ingest_pipelines` cluster | ||
// privilege (this happens even if I load the pipeline in advance): |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Yeah, this is why we should encourage users to run filebeat setup
with another user.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I did set up filebeat in advance (with a different user), but the manage_ingest_pipelines
privilege was still required. I will retest to see if I can reproduce the behavior in 7.0 as well as 6.7.
This is ready for review again. I moved the section about built-in roles because I worked them into the sections about privileges. @urso I am still setting that
|
@@ -39,7 +39,11 @@ configured {es} output, specify the following minimal configuration: | |||
-- | |||
[source, yml] | |||
-------------------- | |||
xpack.monitoring.enabled: true | |||
xpack.monitoring: |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
@ycombinator When I remove xpack
from this, I no longer see Beats monitoring data in Kibana 7.0.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Hi @dedemorton, that would be expected as the new monitoring.*
settings were only introduced in 7.1.0: #9260.
Just to avoid confusion, the xpack.monitoring.*
settings will continue to be available in 7.1.0 and above. However they will be deprecated in favor of the monitoring.*
settings.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Ok, I need to backport this to 7.0, so I will use xpack.monitoring in this PR and then change it back after I backport to 7.1.
Note that I've updated the docs here to reflect the latest changes: https://filebeatsecurityupdates.firebaseapp.com/securing-beats.html |
@dedemorton Filebeat always tries to load pipelines on startup if at least one module is configured. There is no way to skip it right now. So |
* Simplify Beats security docs * Add changes from review
* Simplify Beats security docs * Add changes from review
* Simplify Beats security docs * Add changes from review
The website is still up and it appeared in a google search for me just now. Can we kill it? Or at least ensure it’s not being indexed, for example using robots.txt? |
Yup. I’ll remove the project tomorrow |
…#12290) * Simplify Beats security docs (elastic#11329) * Simplify Beats security docs * Add changes from review * Fix monitoring example * Add attribute subsitution to resolve attribute * Remove to-do.
…#12287) * Simplify Beats security docs (elastic#11329) * Simplify Beats security docs * Add changes from review * Apply suggestions from code review Add attribute substitution to resolve attribute. * Remove to-do.
…#12288) * Simplify Beats security docs (elastic#11329) * Simplify Beats security docs * Add changes from review * Add attribute subsitution to resolve attribute * Remove to-do.
Closes #10241
Documents the privileges required by Beats
6.77.0 users. I've rewritten the content to provide info that is easier to reference, and removed the instructions to create specific roles and users. The idea is to provide users with the ability to create roles that are more suitable for their needs (rather than proscribing specific users and roles).You can preview the changes here: https://filebeatsecurityupdates.firebaseapp.com/securing-beats.html
Please read and respond to questions tagged for REVIEWERS in the review. I have a lot of questions because the combination of possible features makes things more complicated. This was a very time consuming project considering the output, but I hope that means I've suffered so that our users don't have to anymore. :-)
@LeeDr I've tested the combinations, but would like a second pair of eyes, if you have time. I'm trying to boil things down to essentials, but not sure I am 100% there.
@ph Could you check out the beats CM privileges?
@urso Can we get someone on the security team to take a look at this, too? I feel like our security info has been pretty confusing to users, so I'd like to get this fixed before 6.7, if we can.
@karenzone @bmorelli25 FYI because APM and LS either inherit or reuse the same general style of instructions. Would like your feedback, too, if you have time.
Remaining todo items:
Test updates against 6.7 then backport.