Skip to content
This repository has been archived by the owner on Nov 8, 2022. It is now read-only.

grpc based plugins create noise in snapteld's log when they are stopped #1438

Closed
jcooklin opened this issue Dec 13, 2016 · 4 comments
Closed

Comments

@jcooklin
Copy link
Collaborator

The call from snapteld stopping (killing) the plugin results in the following log message (from grpc) to be printed.

2016/12/13 10:36:50 grpc: addrConn.resetTransport failed to create client transport: connection error: desc = "transport: dial tcp 127.0.0.1:55081: operation was canceled"; Reconnecting to {"127.0.0.1:55081" <nil>}
2016/12/13 10:36:50 grpc: addrConn.transportMonitor exits due to: grpc: the connection is closing

To reproduce simply load any grpc based plugin.

@rashmigottipati
Copy link
Contributor

rashmigottipati commented Dec 16, 2016

Reproduced the issue by running snap and loading apache collector and graphite publisher

Error Log -
016/12/15 14:56:52 transport: http2Client.notifyError got notified that the client transport was broken EOF.
Plugin loaded
Name: apache
Version: 3
Type: collector
Signed: false
Loaded Time: Thu, 15 Dec 2016 14:56:52 PST

2016/12/15 14:56:52 grpc: addrConn.resetTransport failed to create client transport: connection error: desc = "transport: dial tcp 127.0.0.1:51151: operation was canceled"; Reconnecting to {"127.0.0.1:51151" }
2016/12/15 14:56:52 grpc: addrConn.transportMonitor exits due to: grpc: the connection is closing

@katarzyna-z
Copy link
Contributor

Noisy logs come from grpc logger, see:

I don't think that we can do something with these logs. Do you have any ideas? Should we close this issue?

@IRCody
Copy link
Contributor

IRCody commented Mar 1, 2017

We can use SetLogger to set a logger for grpc -- either to make it match our log format or to ignore certain kinds of messages except when logs are set to debug.

@katarzyna-z
Copy link
Contributor

@IRCody Thanks, I'll try to use SetLogger.

katarzyna-z added a commit to katarzyna-z/snap that referenced this issue Mar 6, 2017
katarzyna-z added a commit to katarzyna-z/snap that referenced this issue Mar 6, 2017
katarzyna-z added a commit to katarzyna-z/snap that referenced this issue Mar 6, 2017
katarzyna-z added a commit that referenced this issue Mar 7, 2017
Fixes #1438, set standard logger as logger for grpc
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Projects
None yet
Development

No branches or pull requests

5 participants