Skip to content
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

Allows to pass inlined pipeline stages to the docker driver. #2076

Merged
merged 2 commits into from
May 14, 2020

Conversation

cyriltovena
Copy link
Contributor

Fixes #1296

Long overdue, this will help to use pipeline stages with the docker driver. Turns out it's even better when using yaml compose file.

A bit more complicated when using bash.

Signed-off-by: Cyril Tovena cyril.tovena@gmail.com

Fixes grafana#1296

Signed-off-by: Cyril Tovena <cyril.tovena@gmail.com>
Signed-off-by: Cyril Tovena <cyril.tovena@gmail.com>
Copy link
Collaborator

@slim-bean slim-bean left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM!

@slim-bean slim-bean merged commit c2f100c into grafana:master May 14, 2020
@iabughosh
Copy link

Great news! When this going to be available or which version?

@cyriltovena
Copy link
Contributor Author

As soon as one build pass on master with it, you can use latest or the tag it will spit out.

cyriltovena pushed a commit to cyriltovena/loki that referenced this pull request Jun 11, 2021
See the conversation that led to this: https://cloud-native.slack.com/archives/CCYDASBLP/p1580342552117900

I personally think that 600ms is quite short, and bumping it shouldn't
have a bad effect.

Signed-off-by: Goutham Veeramachaneni <gouthamve@gmail.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Support for inlined pipeline config for docker logger
3 participants