github/containerd
2
0
Fork 0
Code Issues Pull Requests Packages Projects Releases 2 Wiki Activity

Don't log AlreadyExists errors

Browse Source 
Signed-off-by: Michael Crosby <crosbymichael@gmail.com>
This commit is contained in:
Michael Crosby 2017-06-16 16:39:52 -07:00
parent 7e3b7dead6
commit 60c08fa386
1 changed files with 9 additions and 7 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

16
services/content/service.go
View File

@ -238,13 +238,15 @@ func (s *Service) Write(session api.Content_WriteServer) (err error) {
defer func(msg *api.WriteResponse) { defer func(msg *api.WriteResponse) {
// pump through the last message if no error was encountered // pump through the last message if no error was encountered
if err != nil { if err != nil {
// TODO(stevvooe): Really need a log line here to track which if grpc.Code(err) != codes.AlreadyExists {
// errors are actually causing failure on the server side. May want // TODO(stevvooe): Really need a log line here to track which
// to configure the service with an interceptor to make this work // errors are actually causing failure on the server side. May want
// identically across all GRPC methods. // to configure the service with an interceptor to make this work
// // identically across all GRPC methods.
// This is pretty noisy, so we can remove it but leave it for now. //
log.G(ctx).WithError(err).Error("(*Service).Write failed") // This is pretty noisy, so we can remove it but leave it for now.
log.G(ctx).WithError(err).Error("(*Service).Write failed")
}
return return
} }