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
[DOC] precedence between environment variables and yml in docker #12198
Conversation
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.
Looks like there's an extra word. Otherwise, LGTM
docs/static/docker.asciidoc
Outdated
for variables that can be mapped to Logstash settings. Settings that are found | ||
in the environment are merged into `logstash.yml` as the container starts up. | ||
in the environment are override those in the `logstash.yml` as the container starts up. |
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.
Nice change. Looks like one more word needs to be deleted:
in the environment are override those in the logstash.yml
as the container starts up.
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.
@andsel Ha! Looks like we were reviewing the same time. :-)
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.
Just one nitpick, fixed that, LGTM
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.
There is an are
that doesn't fit as per comment https://github.com/elastic/logstash/pull/12198/files#r474645544
Co-authored-by: Andrea Selva <selva.andre@gmail.com>
📃 DOCS PREVIEW ✨ https://logstash_12198.docs-preview.app.elstc.co/diff |
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.
LGTM
@logstashmachine backport 8.1 |
@logstashmachine backport 7.17 |
) * [DOC] precedence between environment variables and yml in docker Co-authored-by: Andrea Selva <selva.andre@gmail.com> (cherry picked from commit 682f07b)
) * [DOC] precedence between environment variables and yml in docker Co-authored-by: Andrea Selva <selva.andre@gmail.com> (cherry picked from commit 682f07b)
No description provided.