-
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
The parameter is called "hosts" and not "endpoint" #23644
Conversation
💚 Build Succeeded
Expand to view the summary
Build stats
❕ Flaky test reportNo test was executed to be analysed. |
Pinging @elastic/integrations (Team:Integrations) |
This was fixed here a few days ago: #23628 |
However, looks like that change never made it to the actual readme.md file. |
jenkins, test this |
/test |
Sorry, @DracoBlue , can you add an entry to |
Done! |
See
beats/x-pack/dockerlogbeat/pipelinemanager/config.go
Line 37 in 621c9e2
What does this PR do?
Fixes the documentation stating endpoint as parameter, even though it must be hosts.
Why is it important?
Otherwise it's missleading and people are confused.
Checklist
How to test this PR locally
Check the docs in readme.md