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

[feature request]add LogPath field to the output of pouch inspect #2283

Closed
flyer103 opened this issue Sep 29, 2018 · 1 comment · Fixed by #2286
Closed

[feature request]add LogPath field to the output of pouch inspect #2283

flyer103 opened this issue Sep 29, 2018 · 1 comment · Fixed by #2286
Assignees
Labels
areas/api areas/log kind/feature kind/feature-request This is a feature request from community for PouchContainer priority/P1 this is high priority that all maintainers should stop to handle this issue

Comments

@flyer103
Copy link
Contributor

Ⅰ. Issue Description

Aliyun SLS could use logtail to collect docker containers' stdout/stderr logs.
logtail uses docker inspect to get the path of stdout/stderr log through LogPath field and then collects logs on the host machine.
For pouch container, pouch inspect doesn't offer the LogPath field like docker. This results in logtail failing to collect pouch containers' stdout/stderr logs.

Ⅱ. Describe what happened

logtail couldn't collect pouch containers' stdout/stderr logs.

Ⅲ. Describe what you expected to happen

pouch inspect offers the LogPath field to indicate the path of stdout/stderr logs.
And then logtail could collect pouch containers' stdout/stderr logs.

Ⅳ. How to reproduce it (as minimally and precisely as possible)

  1. Deploy logtail container to collect pouch containers' stdout/stderr logs following the doc
@pouchrobot pouchrobot added areas/log kind/feature kind/feature-request This is a feature request from community for PouchContainer labels Sep 29, 2018
@allencloud allencloud added priority/P1 this is high priority that all maintainers should stop to handle this issue areas/api labels Sep 29, 2018
@pouchrobot
Copy link
Collaborator

Thanks for your report, @flyer103
😱 This is a priority/P1 issue which is highest.
Seems to be severe enough.
ping @alibaba/pouch , PTAL.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
areas/api areas/log kind/feature kind/feature-request This is a feature request from community for PouchContainer priority/P1 this is high priority that all maintainers should stop to handle this issue
Projects
None yet
Development

Successfully merging a pull request may close this issue.

4 participants