| rm_work[m] | So, apparently the python consul lib keeps getting passed around like a hot-potato T_T | 07:52 |
|---|---|---|
| rm_work[m] | this is for the new one: https://review.opendev.org/c/openstack/requirements/+/997160 | 07:52 |
| rm_work[m] | I am worried that the lack of packaging in distros is going to be a problem -- the old (now unmaintained) lib is still the one that is packaged, but tooz is dropping drivers that use it specifically because it's unmaintained, so the path seems to be to switch to this one | 07:53 |
| rm_work[m] | any thoughts/advice on how to handle this? | 07:53 |
| mnasiadka | rm_work[m]: well, it’s Consul, no open source alternative of the software itself, can’t be packaged by deployers, so I’m not surprised that it’s a hot potato ;) | 08:05 |
| opendevreview | Takashi Kajinami proposed openstack/requirements master: Add third party libraries for manila share drivers https://review.opendev.org/c/openstack/requirements/+/997489 | 08:13 |
| opendevreview | Takashi Kajinami proposed openstack/requirements master: Add third party libraries for manila share drivers https://review.opendev.org/c/openstack/requirements/+/997489 | 08:14 |
| rm_work[m] | mnasiadka: yeah, technically it isn't even what we use (we use k8s in tooz), but I'm collaborating with LINE/Yahoo folks and trying to make my proposed direction support it since they need it | 08:33 |
| rm_work[m] | if I can get this in, it will simplify things a whole lot for HTTP-RPC I believe (and make it a lot more reviewable / upstream friendly) | 08:34 |
| rm_work[m] | just wondering if there's anything to be done about the fact that it'd be the third variant in global-reqs for essentially the same lib | 08:34 |
| rm_work[m] | like, is that a good argument that it needs to be added? because we supported the old two and this one is the path forward? or does it make it more difficult? I really don't know T_T | 08:35 |
| mnasiadka | Supported sounds like a past thing, should those old ones be removed from master requirements? | 08:40 |
| rm_work[m] | I'm not sure TBH | 08:43 |
| rm_work[m] | I don't know exactly how we can remove stuff from g-r | 08:44 |
| rm_work[m] | projects we don't even know about could be using it? and we'd break them? | 08:44 |
| opendevreview | OpenStack Proposal Bot proposed openstack/requirements master: update constraint for neutron-lib to new release 4.2.0 https://review.opendev.org/c/openstack/requirements/+/997493 | 09:04 |
| opendevreview | OpenStack Proposal Bot proposed openstack/requirements master: update constraint for oslo.concurrency to new release 7.6.1 https://review.opendev.org/c/openstack/requirements/+/997495 | 09:10 |
| tkajinam | rm_work[m], we can drop libraries which are not used by anything in the openstack namespace | 09:32 |
| tkajinam | more precisely speaking "anything active and is compliant with global requirements management' | 09:33 |
| tkajinam | afair tooz was the one using python-consul2 but we drop that dependency so it can be technically removed from global u-c | 09:33 |
| tkajinam | I didn't because I've heard that LY was hoping to get that http driver thing in the current shape first and they will replace it soon | 09:34 |
| tkajinam | though I've not seen no actual progress about that replacement plan... | 09:34 |
| opendevreview | Takashi Kajinami proposed openstack/requirements master: Add third party libraries for manila share drivers https://review.opendev.org/c/openstack/requirements/+/997489 | 10:26 |
| rm_work[m] | that's what i'm working on | 11:00 |
| rm_work[m] | I'm proposing to them a reworked version using the newer lib | 11:00 |
| rm_work[m] | still getting the chain in order | 11:01 |
| rm_work[m] | I've moved their Consul driver into tooz | 11:01 |
| rm_work[m] | (the old tooz consul driver was lock-only, this one is full group membership matching their implementation) | 11:01 |
| opendevreview | OpenStack Proposal Bot proposed openstack/requirements master: update constraint for oslo.config to new release 10.6.0 https://review.opendev.org/c/openstack/requirements/+/997527 | 11:33 |
| opendevreview | Merged openstack/requirements master: update constraint for oslo.concurrency to new release 7.6.1 https://review.opendev.org/c/openstack/requirements/+/997495 | 12:05 |
| opendevreview | Merged openstack/requirements master: update constraint for neutron-lib to new release 4.2.0 https://review.opendev.org/c/openstack/requirements/+/997493 | 12:56 |
| *** efoley_ is now known as efoley | 13:24 | |
| *** efoley_ is now known as efoley | 14:15 | |
| gmaan | frickler: elodilles this is ready, finally https://review.opendev.org/c/openstack/requirements/+/996511 | 16:35 |
| frickler | +3, thx | 17:14 |
Generated by irclog2html.py 4.1.0 by Marius Gedminas - find it at https://mg.pov.lt/irclog2html/!