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
don't fail logs
when driver:none is set
#10107
Conversation
5ed4bfc
to
b6e4920
Compare
Codecov ReportBase: 76.98% // Head: 75.79% // Decreases project coverage by
Additional details and impacted files@@ Coverage Diff @@
## v2 #10107 +/- ##
==========================================
- Coverage 76.98% 75.79% -1.20%
==========================================
Files 2 2
Lines 252 252
==========================================
- Hits 194 191 -3
- Misses 51 53 +2
- Partials 7 8 +1
Help us with your feedback. Take ten seconds to tell us how you rate us. Have a feature suggestion? Share it here. ☔ View full report at Codecov. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LGTM
c1c162e
to
7b4d94a
Compare
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LGTM
Signed-off-by: Nicolas De Loof <nicolas.deloof@gmail.com>
7b4d94a
to
22d2e83
Compare
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Nice!
What I did
detect engine response requesting container logs as
ErrNotImplemented
when logging driver is configured tonone
. This should not fail the command, just show a warningbonus:
logContainers
to collect logs after last watch event, so we don't get previous logs repeated after a container restart.Restarting
attribute (not sure why we didn't used this before?)logs
can now detect container will not restart and then exits--follow
Related issue
fixes: #9030
(not mandatory) A picture of a cute animal, if possible in relation to what you did