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
Trap close error on shutdown #1501
Conversation
Codecov Report
@@ Coverage Diff @@
## master #1501 +/- ##
=======================================
Coverage 40.82% 40.82%
=======================================
Files 23 23
Lines 2920 2920
=======================================
Hits 1192 1192
Misses 1451 1451
Partials 277 277 Continue to review 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
One small nit..given the change from GracefulStop
to Stop
we might want to change the method comment above it as Stop
sounds like it cancels any open gRPC connections (unlike GracefulStop
which according to its GoDoc: blocks until all the pending RPCs are finished)
957ce3d
to
8192544
Compare
Fixes containerd#1499 This traps any Accept() errors on server shutdown so that it exits cleanly. It also changes gracefulstop to stop on the grpc server as the daemon will not shutdown if there are connected clients. Signed-off-by: Michael Crosby <crosbymichael@gmail.com>
8192544
to
03b43ac
Compare
@estesp updated the comment |
LGTM |
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
…-cgroups-controller Tolerate missing HugeTLB cgroups controller
Fixes #1499
This traps any Accept() errors on server shutdown so that it exits
cleanly.
It also changes gracefulstop to stop on the grpc server as the daemon
will not shutdown if there are connected clients.
Signed-off-by: Michael Crosby crosbymichael@gmail.com