-
Notifications
You must be signed in to change notification settings - Fork 3.5k
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
Support for inlined pipeline config for docker logger #1296
Comments
Unfortunately docker options are only string value. But we are aware of the problem, on Linux it seems that you can have the file added to the plugin using a specific plugin path, there's some issues explaining this. |
@cyriltovena yes, and now we using this case - copy file to plugin |
see #1374 (comment) |
I do exactly the same, but that looks weird. If that's all, thanks anyway |
Would it be more helpful if the docker plugin forward log to a promtail container instance ? This way the you can easily configure promtail by mounting a host volume or a docker config ? The downside is that you have to run a promtail container instance. This is something we've been thinking about. |
So there’s a propagated mount config, I’ll try it. |
This issue has been automatically marked as stale because it has not had any activity in the past 30 days. It will be closed in 7 days if no further activity occurs. Thank you for your contributions. |
tagging this to keepalive as I still think there is a gap in configuring the pipeline for the docker logging driver. |
So, is there any possibility for mounting config? |
Yes we're explaining this here: |
@cyriltovena can you test this with the Yaml multilline string marker:
Don't know if that syntax is correct? |
1 similar comment
@cyriltovena can you test this with the Yaml multilline string marker:
Don't know if that syntax is correct? |
i would like this solution very much.
|
Dears, I am using the latest loki-docker-driver plugin and my docker version is 19.03.8 and my OS is MacOs. Note : Also documentation doesn't contain any steps for what is being discussed here. |
So, what about any updates? |
|
Ok, but I tried to make a graph for loki source in grafana. And without lables it's not possible to retrieve such data as |
That's something we're solving very soon with logQL, actually before june ! I'll run a demo during the GrafanaCon Loki future talk. |
Thanks for the update. Please inform us here if it is fixed. I really liked it! |
LogQL will help you, but what we are talking here is allowing inline config for pipeline. I'm still working on that the config improvement even though LogQL will help. |
I was able to filter with LogQl. But as @wapmorgan mentioned, we need pipelines for other purposes. |
Fixes grafana#1296 Signed-off-by: Cyril Tovena <cyril.tovena@gmail.com>
Let me know what you think about the PR. |
Using in line stages is also awesome and works for me! But as I understood from the PR, you can't parse files unless it is inside the driver, You can't parse an external one, am I right? |
* Allows to pass inlined pipeline stages to the docker driver. Fixes #1296 Signed-off-by: Cyril Tovena <cyril.tovena@gmail.com> * doc. Signed-off-by: Cyril Tovena <cyril.tovena@gmail.com>
Fixes grafana#1296 Signed-off-by: Cyril Tovena <cyril.tovena@gmail.com>
Fixes grafana#1296 Signed-off-by: Cyril Tovena <cyril.tovena@gmail.com>
* Allows to pass inlined pipeline stages to the docker driver. Fixes #1296 Signed-off-by: Cyril Tovena <cyril.tovena@gmail.com> * Add validation before using storage config for the logcli. Signed-off-by: Cyril Tovena <cyril.tovena@gmail.com>
Is your feature request related to a problem? Please describe.
When there's need to rebuild plugin or to copy file with pipeline to pluigin directory, it's big problem.
The problem is /ag/app.yaml is should be inside plugin container, not in root fs!
Describe the solution you'd like
Allow to inline pipeline stage config in config parameter. Like this:
Describe alternatives you've considered
Only injecting (copying) pipeline file to plugin directory.
The text was updated successfully, but these errors were encountered: