-
Notifications
You must be signed in to change notification settings - Fork 6
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
Renew cert fix #166
Renew cert fix #166
Conversation
cb28fff
to
1153248
Compare
smclient/smclient.go
Outdated
|
||
if client.stream != nil { | ||
if err := client.stream.CloseSend(); err != nil { | ||
log.WithField("err", err).Error("SMClient failed send close") |
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.
SM client
smclient/smclient.go
Outdated
@@ -332,7 +362,7 @@ func (client *SMClient) processMessages() (err error) { | |||
if err != nil { | |||
if code, ok := status.FromError(err); ok { | |||
if code.Code() == codes.Canceled { | |||
log.Debug("SM client connection closed") | |||
log.Debug("CM client connection closed") |
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.
SM
1153248
to
f7038c4
Compare
Codecov ReportAttention: Patch coverage is
Additional details and impacted files@@ Coverage Diff @@
## develop #166 +/- ##
===========================================
- Coverage 67.73% 66.87% -0.86%
===========================================
Files 23 21 -2
Lines 4128 4091 -37
===========================================
- Hits 2796 2736 -60
- Misses 840 857 +17
- Partials 492 498 +6 ☔ View full report in Codecov by Sentry. |
d1b5567
to
05deeb8
Compare
Signed-off-by: Mykola Kobets <mykola_kobets@epam.com>
Signed-off-by: Mykola Kobets <mykola_kobets@epam.com>
Signed-off-by: Mykola Kobets <mykola_kobets@epam.com>
05deeb8
to
9ea867d
Compare
|
No description provided.