-
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
Update docs #7164
Update docs #7164
Conversation
@ruflin could you please verify that adding the |
Not this is not correct client authentication is only valid in the tcp
input. I will make a PR to clarify the doc.
On Wed, May 23, 2018 at 5:18 AM Silvia Mitter ***@***.***> wrote:
@ruflin <https://github.com/ruflin> could you please verify that adding
the output.elasticsearch.ssl.client_authentication to the reference.yml
files is ok. When I updated the docs in apm-server I realized that this
option was added to the official docs but not to the reference yml files.
—
You are receiving this because you are subscribed to this thread.
Reply to this email directly, view it on GitHub
<#7164 (comment)>, or mute
the thread
<https://github.com/notifications/unsubscribe-auth/AAACgFbeZ34Oeuo8CINTHDZ-ZcTzdRIgks5t1SlcgaJpZM4UKEir>
.
--
ph
|
@ph could you please ping me on the PR, so I can also update for the APM Server. |
@ruflin could you have a look at this PR please. |
ifeval::["{beatname_lc}"!="auditbeat"] | ||
For rpm and deb, you'll find the configuration file at +/etc/{beatname_lc}/{beatname_lc}.yml+. | ||
For mac and win, look in the archive that you just extracted. | ||
Under Docker, it's located at +/usr/share/{beatname_lc}/{beatname_lc}.yml+. |
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.
If I get this right the only difference for auditbeat is that it does not include the docker part? Could we have only an not auditbeat if clause
for the docker part?
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 tried that, but then it renders the docker part in a new paragraph, which I think looks weird.
@simitt LGTM. Left a minor comment. I think the logic is ok as you did it but it could be simplified. |
beat.reference.yml
in the official docs for the beats that actually have areference.yml
.output.elasticsearch.ssl.client_authentication
toreference.yml
files that has been documented already on the official beats docs pages.