etcd3-py icon indicating copy to clipboard operation
etcd3-py copied to clipboard

Are you aware of https://github.com/dims/etcd3-gateway ?

Open nelljerram opened this issue 4 years ago • 9 comments

How would you say etcd3-py compares against etcd3-gateway? Could you work together instead of apparently duplicating effort?

nelljerram avatar Oct 29 '19 19:10 nelljerram

Issue-Label Bot is automatically applying the label question to this issue, with a confidence of 0.84. Please mark this comment with :thumbsup: or :thumbsdown: to give our bot feedback!

Links: app homepage, dashboard and code for this bot.

issue-label-bot[bot] avatar Oct 29 '19 19:10 issue-label-bot[bot]

this repo supports python2 and python3 with asyncio

and stateful utils with human friendly api

Revolution1 avatar Oct 30 '19 03:10 Revolution1

Thanks. I think etcd3-gateway also supports python2 and python3, but maybe not asyncio.

nelljerram avatar Oct 30 '19 15:10 nelljerram

yeah, it does support python2

Revolution1 avatar Oct 30 '19 15:10 Revolution1

etcd3-gateway does not support user/pass authentication. We are replacing with etcd3-py

heitorPB avatar Apr 27 '22 20:04 heitorPB

FWIW, we implemented user/pass authentication by subclassing the Etcd3Client class, as you can see here: https://github.com/projectcalico/calico/blob/master/networking-calico/networking_calico/etcdv3.py#L388-L472

nelljerram avatar Apr 28 '22 08:04 nelljerram

@neiljerram that is a simple solution, I liked it. Is there a reason it was not upstreamed?

heitorPB avatar Apr 28 '22 12:04 heitorPB

@heitorPB Ah, good question!

I don't recall thinking about this carefully at the time, but I guess the thought in my mind was that the solution might not work for all possible uses of Etcd3Client. As the commenting says, I was comfortable that it works in all the cases that Calico (my project) needs - but it's possible that there are other cases not covered.

Nevertheless, I think it's right to try upstreaming, and then the discussion can discuss that - so will plan to do this shortly.

nelljerram avatar May 06 '22 08:05 nelljerram

I'd be happy to help you if you need! I created an issue on their bugtracker with this request: https://bugs.launchpad.net/python-etcd3gw/+bug/1970750

heitorPB avatar May 06 '22 14:05 heitorPB