-
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 autodiscover docs after adding container
input
#12795
Comments
Hi, You are right, we should update docs to make use of the new input. I saw you are using both static input and autodiscover at the same time, that's not a good idea as they will try to gather the same logs. The other issue is that you are using a wildcard ( I would expect this to work:
|
@exekias damn sorry! I'm absolutely didn't get that I can't mix static and autodiscover cases 😨 |
Let's use this issue to clarify docs on this regard 👍 |
Just for clarification: proposed config fix the issue with |
container
input
My Config is as follows -
In the following condition if I change the
|
ey @dnmahendra, |
@exekias are you sure that the implementation of #12162 is finished? I try to use
container
as input forautodiscover
Docker provider but the setup is not working:But in documentation
docker
input is used forautodiscover
Docker provider still and it works:but in this case I see deprecation warnings in logs:
{"level":"warn","timestamp":"2019-07-04T18:22:45.776Z","logger":"cfgwarn","caller":"docker/input.go:49","message":"DEPRECATED: 'docker' input deprecated. Use 'container' input instead. Will be removed in version: 8.0.0"}
Full config:
(no one of these templates of autodiscover are work)
The text was updated successfully, but these errors were encountered: