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.

Sign up for GitHub

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

Jump to bottom

[DOCS] Adds apm_system user and role #120

Merged
merged 3 commits into from
Sep 12, 2018
Merged

[DOCS] Adds apm_system user and role #120

merged 3 commits into from
Sep 12, 2018

Conversation

lcawl
Copy link
Contributor

@lcawl lcawl commented Sep 10, 2018 •
edited
Loading

Related to elastic/elasticsearch#32515

This PR adds information about the apm_system builtin user and role to the Stack Overview.

@lcawl lcawl requested a review from ycombinator September 10, 2018 21:33
@ycombinator
Copy link
Contributor

To be determined: Does the APM configuration file (apm-server.yml) need to be updated to use the built-in user (similar to steps for the other products here: elastic.co/guide/en/elastic-stack-overview/master/built-in-users.html#add-built-in-user-passwords)?

@graphaelli maybe you know?

@graphaelli
Copy link
Member

Yes, we'll need the same treatment that beats currently documents.

xpack.monitoring.elasticsearch.username: apm_server_system
xpack.monitoring.elasticsearch.password: apmserverpassword

@lcawl
Copy link
Contributor Author

lcawl commented Sep 11, 2018

Thanks @graphaelli, I've added that information. Will we need to add a page about "Configuring monitoring" in the APM Server Guide (e.g. in this section: https://www.elastic.co/guide/en/apm/server/master/configuring-howto-apm-server.html) too?

@graphaelli
Copy link
Member

Those changes look great, thank you. Yes, we'll need a section in the APM docs as well. Opened elastic/apm-server#1378 for tracking.

ycombinator
ycombinator reviewed Sep 12, 2018
View reviewed changes
docs/en/stack/security/authentication/built-in-users.asciidoc Outdated
@@ -11,6 +11,7 @@ passwords have been set. The `elastic` user can be used to
`kibana`:: The user Kibana uses to connect and communicate with Elasticsearch.
`logstash_system`:: The user Logstash uses when storing monitoring information in Elasticsearch.
`beats_system`:: The user the Beats use when storing monitoring information in Elasticsearch.
`apm_system`:: The user APM uses when storing monitoring information in {es}.
Copy link
Contributor

Choose a reason for hiding this comment

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

Should we clarify that this is the user the APM server uses?

Copy link
Contributor Author

Choose a reason for hiding this comment

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

Done!

ycombinator
ycombinator approved these changes Sep 12, 2018
View reviewed changes
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! Thanks for making these updates.

@lcawl lcawl merged commit 43d4a65 into elastic:master Sep 12, 2018
@lcawl lcawl deleted the lcawley-builtin-users branch September 12, 2018 18:47
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@graphaelli graphaelli graphaelli approved these changes

@ycombinator ycombinator ycombinator approved these changes

Assignees
No one assigned
Labels
v6.5.0 v7.0.0
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
3 participants
@lcawl @ycombinator @graphaelli