-
Notifications
You must be signed in to change notification settings - Fork 1.1k
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
Enhance gRPC errors #1608
Enhance gRPC errors #1608
Conversation
main.go
Outdated
@@ -190,7 +190,11 @@ func setupGlobals() { | |||
} | |||
|
|||
if config.Global.CoProcessOptions.EnableCoProcess { | |||
CoProcessInit() | |||
if err := CoProcessInit(); err != nil { | |||
log.WithFields(logrus.Fields{ |
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.
Can just use log.WithField("prefix", "coprocess").Error(err)
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.
Updated
ddb8496
to
f9aa340
Compare
This is a very small change, to test this, set this into your "coprocess_options": {
"enable_coprocess": true,
"coprocess_grpc_server": ""
}, Without the patch you should see the recurring dialer errors. |
f9aa340
to
71fddbf
Compare
Just fixed conflicts on this PR |
c0d51fb
to
46533c4
Compare
46533c4
to
f269457
Compare
* main: display CP init errors * coprocess: check for empty gRPC URLs before calling the dialer * coprocess: fix dummy driver * coprocess: fix config access
Enhancement for #991
The dialer already does an URL check (parsing) but this is a simpler thing to check during init