-
Notifications
You must be signed in to change notification settings - Fork 77
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
[bugfix] gRPC pool connection health check for DNS Addr may fail during VIP member disconnection #2277
[bugfix] gRPC pool connection health check for DNS Addr may fail during VIP member disconnection #2277
Conversation
[CHATOPS:HELP] ChatOps commands.
|
Codecov ReportAttention:
Additional details and impacted files@@ Coverage Diff @@
## main #2277 +/- ##
==========================================
- Coverage 30.02% 29.99% -0.04%
==========================================
Files 371 371
Lines 36089 36115 +26
==========================================
- Hits 10837 10832 -5
- Misses 24738 24770 +32
+ Partials 514 513 -1 ☔ View full report in Codecov by Sentry. |
d8ca6ff
to
c40b91a
Compare
Deploying with Cloudflare Pages
|
4c391b4
to
2c374b3
Compare
…ng VIP member disconnection fix Signed-off-by: kpango <kpango@vdaas.org>
2c374b3
to
bd15979
Compare
if !p.isIP && p.resolveDNS && hash != nil && *hash != "" { | ||
ips, err := p.lookupIPAddr(ctx) | ||
if err != nil { | ||
return p, nil |
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.
🚫 [golangci] reported by reviewdog 🐶
error is not nil (line 392) but it returns nil (nilerr)
Profile Report
|
…ailure-during-target-vip-member-restarts
Description:
SSIA
Related Issue:
Versions:
Checklist:
Special notes for your reviewer: