Skip to content

Proposal: Add Logging driver for fluentd#12540

Closed
tagomoris wants to merge 1 commit intomoby:masterfrom
tagomoris:proposal-logger-driver-fluentd
Closed

Proposal: Add Logging driver for fluentd#12540
tagomoris wants to merge 1 commit intomoby:masterfrom
tagomoris:proposal-logger-driver-fluentd

Conversation

@tagomoris
Copy link
Copy Markdown
Contributor

I think that some more Logging driver make users happy!
This is the first for more logging drivers for Fluentd.

See the files for more details.

@tagomoris tagomoris force-pushed the proposal-logger-driver-fluentd branch from bc3fe0a to 502b746 Compare April 20, 2015 10:34
@kysnm
Copy link
Copy Markdown

kysnm commented Apr 22, 2015

👍

8 similar comments
@ijin
Copy link
Copy Markdown

ijin commented Apr 22, 2015

👍

@katutoshi
Copy link
Copy Markdown

👍

@zchee
Copy link
Copy Markdown
Contributor

zchee commented Apr 22, 2015

👍

@kazegusuri
Copy link
Copy Markdown

👍

@ixixi
Copy link
Copy Markdown

ixixi commented Apr 22, 2015

👍

@saisse
Copy link
Copy Markdown

saisse commented Apr 22, 2015

👍

@otaki-ryuta
Copy link
Copy Markdown

👍

@tmtk75
Copy link
Copy Markdown

tmtk75 commented Apr 22, 2015

👍

@icecrime icecrime removed the dco/yes label Apr 23, 2015
@jessfraz
Copy link
Copy Markdown
Contributor

hey all we get it you want this, but can you not +1, because it makes feedback hard to read :), we promise we will do our best to review this as soon as possible

@jessfraz
Copy link
Copy Markdown
Contributor

adding this seems fine to me, will wait for others to approve then we can move into code-review if so :)

@kzk
Copy link
Copy Markdown

kzk commented Apr 29, 2015

@jfrazelle Just a background. @tagomoris is a maintainer of Fluentd (http://www.fluentd.org/), and I'm from Treasure Data, Inc. sponsoring Fluentd project.

Docker + Fluentd adopted by Google & Kubernetes

We recently started seeing more people adopting Docker + Fluentd. For example. Google's Kubernetes adopted Fluentd as a default log collector. But Kubernetes uses Fluentd's 'tail' input plugin for log files. It's not performant, reliable, and config files become relatively messy...

FYI: Google adopted Fluentd as a default log collector for the entire cloud platform (packaging repo, bigquery documentation).

Docker + Fluentd by Digital Ocean

Digital Ocean wrote a documentation about how to centralize Docker logs into one place, using Fluentd. But as I mentioned above, it tails log files as well.

Push logs directly from Docker to Fluentd (Code is available)

We'd like to have a way to directly push the logs from Docker to Fluentd, which is more performant, reliable, and simple. Just in case, @tagomoris already wrote a code so that you can review.

Logging is the essential part of the immutable infrastructure, so hope Docker has a flexible way to centralized logs using the solution like Fluentd.

@Zaki-XL
Copy link
Copy Markdown

Zaki-XL commented Apr 29, 2015

👍

@cpuguy83
Copy link
Copy Markdown
Member

Definitely +1 as long as we can ping for issues. @tagomoris @kzk do you plan to contribute code for this?
Also, proposal doc is probably longer than the actual implementation will be :)

@tagomoris
Copy link
Copy Markdown
Contributor Author

I already have a complete code for this feature.
So I've just created a pull request #12876.

@jessfraz
Copy link
Copy Markdown
Contributor

can you then combine this with that one so there are not two threads for the same feature and close this one

@kzk
Copy link
Copy Markdown

kzk commented Apr 29, 2015

@cpuguy83 I can give you our point of contacts for you. Please reach out to me at [email protected]. Of course just mentioning on github works too.

@LK4D4
Copy link
Copy Markdown
Contributor

LK4D4 commented May 12, 2015

I think we can close this, because now it is real PR #12876

@LK4D4 LK4D4 closed this May 12, 2015
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.