Skip to content

Commit

Permalink
server: ignore server closed error
Browse files Browse the repository at this point in the history
no harm in ignoring this error, it comes from the underlying kube
stream server and we have no power on it as we don't own the listener
(containerd/cri also ignores it)

Signed-off-by: Antonio Murdaca <runcom@redhat.com>
  • Loading branch information
runcom committed Sep 14, 2018
1 parent 2fdd6db commit 207ee60
Showing 1 changed file with 1 addition and 1 deletion.
2 changes: 1 addition & 1 deletion server/server.go
Original file line number Diff line number Diff line change
Expand Up @@ -383,7 +383,7 @@ func New(ctx context.Context, config *Config) (*Server, error) {
s.stream.streamServerCloseCh = make(chan struct{})
go func() {
defer close(s.stream.streamServerCloseCh)
if err := s.stream.streamServer.Start(true); err != nil {
if err := s.stream.streamServer.Start(true); err != nil && err != http.ErrServerClosed {
logrus.Errorf("Failed to start streaming server: %v", err)
}
}()
Expand Down

0 comments on commit 207ee60

Please sign in to comment.