search

containerd / nerdctl

contaiNERD CTL - Docker-compatible CLI for containerd, with support for Compose, Rootless, eStargz, OCIcrypt, IPFS, ...
Apache License 2.0
8.22k stars 612 forks source link

Support read logs from fluentd/journal log driver #1374

Open Zheaoli opened 2 years ago

Zheaoli commented 2 years ago

What is the problem you're trying to solve

For now, we just support one-way write process for fluentd/journal log driver, but in some circumstances, the user may need to use nerdctl logs command to fetch the log.

So we can write to the driver destination and a local JSON file for the log driver simultaneously, and the nerdctl logs command can fetch the log from the local log file

Describe the solution you'd like

Additional context

No response

yuchanns commented 2 years ago

count me in

fahedouch commented 2 years ago

I started a PR around this https://github.com/containerd/nerdctl/pull/1115/files a few days ago. Unfortunately, I didn't have time to finish it. @yuchanns this may inspire you :)

t1anz0ng commented 2 years ago

I can take the journald driver job😃

manugupt1 commented 2 years ago

@Zheaoli Should this be fluentbit and not fluentd? Ref: https://github.com/containerd/nerdctl/commit/8bf043a52b5d64a9a7a38103427c884e4a542271