-
Notifications
You must be signed in to change notification settings - Fork 184
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
python-etcd3 is incompatible with grpcio versions newer than 1.44.0 (1.45+) #1876
Comments
same issue here |
If I'm not mistaken, grpcio now requires that the etcd server uses TLS when you need authentication. |
I've checked the code of python-etcd3 library itself, and it seems that secure_channel is used for HTTPS connections only. However, if someone else stumbles upon this issue might find this useful. I'll close the issue, since it seems to be working just fine. |
We got the issue by trying to auth agaist a local etcd. We fixed it by using our own Etcd3Client, with that in the def __init__(self, host='localhost', port=2379, timeout=None, user=None,
password=None):
if user and password:
credentials = grpc.local_channel_credentials()
self.uses_secure_channel = True
else:
credentials = None
self.uses_secure_channel = False
# Step 2: create Endpoint
ep = Endpoint(host, port, secure=self.uses_secure_channel,
creds=credentials)
super(Etcd3Client, self).__init__(endpoints=[ep], timeout=timeout,
user=user, password=password) (I removed extra unused args) Not sure how / if I can create a PR with a proper fix :) |
@the-glu could you provide the complete code you used to conenct etcd and then to get and put keys |
What version of gRPC and what language are you using?
grpc v1.45.0
What operating system (Linux, Windows,...) and version?
Ubuntu 20.04
What runtime / compiler are you using (e.g. python version or version of gcc)
python 3.8.10
What did you do?
Update grpc from 1.44.0 to 1.45.0 and run it against etcd v3 server with authentication enabled using python-etcd3 module.
What did you expect to see?
No errors.
What did you see instead?
The text was updated successfully, but these errors were encountered: