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

Break-up Logstash docs + Add Centralized Pipeline Management documentation #8471

Open
wants to merge 1 commit into
base: main
Choose a base branch
from

Conversation

TimBosman
Copy link
Contributor

@TimBosman TimBosman commented Feb 11, 2025

I did three things in this PR:

  • Go from one asciidoc file for Logstash documentation to a asciidoc file for every page
  • Replaced some product names with attributes (e.g. Logstash -> {ls})
  • Added a documentation page for Centralized Pipeline Management

Only thing I'm not completely sure the following section is recommended:

xpack.management.elasticsearch.username: "${PRODUCTION_ES_USER}"
xpack.management.elasticsearch.password: "${PRODUCTION_ES_PASSWORD}"

If I recall correctly this will setup a connection with the elastic user, which would be an escalation of permissions.

EDIT: It won't setup using the elastic user but with a different user (See the docs). This could still be an escalation of permissions.

Related issues:

@TimBosman TimBosman added the >docs Documentation label Feb 11, 2025
@pebrc
Copy link
Collaborator

pebrc commented Feb 26, 2025

The documentation content is being moved to https://github.com/elastic/docs-content starting with the next release ECK 3.0. I wonder if t would make sense to make these changes there instead.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
>docs Documentation
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants