-
Notifications
You must be signed in to change notification settings - Fork 39.3k
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
Upgrade etcd libraries to v3.5.13 #124469
Conversation
/assign @dims |
cb9f614
to
f761daa
Compare
We are adding one more dependency, it's transitive, so should be ok:
|
most of the new updates to vendored libraries look great. otel is usually a pain and it looks like we had to update a test case. Let's please get it reviewed by responsible folks in the area @serathius thanks! |
// TODO: Skip the use of strings.ToLower() to index the map after v1.59 | ||
// is released to switch to case sensitive balancer registry. Also, | ||
// remove this warning and update the docstrings for Register and Get. | ||
logger.Warningf("Balancer registered with name %q. grpc-go will be switching to case sensitive balancer registries soon", b.Name()) |
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.
non-blocking for this PR, but do we need to do anything to warn users about things we use grpc for (non-lower-case etcd addresses, etc)?
Add otelgrpc.WithMessageEvents(otelgrpc.ReceivedEvents, otelgrpc.SentEvents) to tracing options due to open-telemetry/opentelemetry-go-contrib#3964
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.
/lgtm
LGTM label has been added. Git tree hash: 9f635831c0d6cf9b5df0769490791a8385b196ba
|
/lgtm |
/triage accepted |
/kind bug
Unblocks testing of consistent reads from cache as we use embedded etcd server in tests. Upgrade comes with a fix to etcd-io/etcd#17507
/assign @liggitt