| opendevreview | Hervé Beraud proposed openstack/oslo.utils master: fix strutils password regex https://review.opendev.org/c/openstack/oslo.utils/+/832791 | 07:41 |
|---|---|---|
| opendevreview | Hervé Beraud proposed openstack/oslo.utils master: fix strutils password regex https://review.opendev.org/c/openstack/oslo.utils/+/832791 | 07:49 |
| hberaud | stephenfin, damani: o/ Sorry to harass you with this topic, please can you review this patch https://review.opendev.org/c/openstack/oslo.utils/+/832791 | 09:57 |
| damani | hberaud, ok | 10:45 |
| hberaud | stephenfin damani: Thanks guys | 10:48 |
| damani | hberaud, i have validate the workflow | 10:51 |
| damani | that will be merge soon | 10:51 |
| hberaud | thanks | 10:51 |
| opendevreview | Hervé Beraud proposed openstack/oslo.utils stable/yoga: fix strutils password regex https://review.opendev.org/c/openstack/oslo.utils/+/832897 | 10:53 |
| tobias-urdin | hberaud: when you have some seconds can you check https://review.opendev.org/c/openstack/tooz/+/829412 and let me know how I could proceed, thanks! | 14:22 |
| * hberaud look | 14:25 | |
| hberaud | tobias-urdin: isn't simply that the version of tenacity waited by python-etcd3 (https://github.com/kragniz/python-etcd3/blob/8ae4f2aa18b4d592d4b6c444c1d2b0261e77ef43/requirements/test.txt#L159) isn't compatible with your pin? http://pastebin.test.redhat.com/1036052 | 14:34 |
| hberaud | tobias-urdin: apparently not https://github.com/kragniz/python-etcd3/blob/833a7335150c08e864dd4a0cdda0034017ccd3b0/requirements/base.txt#L11 | 14:36 |
| hberaud | tooz seems to pull the latest version of etcd3 (https://opendev.org/openstack/tooz/src/branch/master/setup.cfg#L52), that is the version 0.12.0 | 14:37 |
| hberaud | and they still use tenacity in the version 6.1.0 - strict version | 14:38 |
| hberaud | so your patch should be compatible | 14:39 |
| tobias-urdin | i want to unblock the tooz pin for tenacity, but etcd3 (which seems dead btw) is blocking me from doing that - so not sure what i can do for the dead etcd3 library blocking with it's tenacity pin | 14:46 |
| tobias-urdin | tooz in turn is blocking gnocchi which is what I want to solve (https://github.com/gnocchixyz/gnocchi/pull/1187) | 14:46 |
| tobias-urdin | etcd3 in master code has removed tenacity dependency but there is no new release (no maintainer i guess so project is stale), they don't have any stable branches either so can't propose any fixes for an older version | 14:47 |
| tobias-urdin | i guess we can't really just deprecate etcd3 driver in tooz on-the-fly either | 14:48 |
| hberaud | tobias-urdin: if no one complain it WFM. So I'd suggest first propose to propose a patch to deprecate this driver and disabling the related job, and then sending an email to the ML to check the trend before removing this driver. | 14:52 |
| hberaud | you could rebase your patch on the top of the "deprecate" patch | 14:53 |
| hberaud | and so unlock your path | 14:53 |
| tobias-urdin | ack, we do have the etcd3gw driver still that we actually has people maintaining in opendev | 14:53 |
| hberaud | yes this is also an oslo deliverable | 14:54 |
| hberaud | in third time it could be the right moment to see if we can drop etcd from our global reqirements | 14:55 |
| opendevreview | Merged openstack/oslo.utils master: fix strutils password regex https://review.opendev.org/c/openstack/oslo.utils/+/832791 | 14:55 |
| hberaud | damani: FYI ^. Starting the backport chain | 14:56 |
| opendevreview | Hervé Beraud proposed openstack/oslo.utils stable/yoga: fix strutils password regex https://review.opendev.org/c/openstack/oslo.utils/+/832897 | 14:59 |
| hberaud | damani: here we go to Yoga :) ^ | 15:00 |
| damani | ok | 15:06 |
| damani | hberaud, i waiting the ci and merge it | 15:07 |
| hberaud | thanks | 15:07 |
| opendevreview | Tobias Urdin proposed openstack/tooz master: Deprecate the etcd3 driver https://review.opendev.org/c/openstack/tooz/+/833107 | 15:16 |
| opendevreview | Tobias Urdin proposed openstack/tooz master: Support later tenacity versions https://review.opendev.org/c/openstack/tooz/+/829412 | 15:19 |
| tobias-urdin | hberaud: thanks for your feedback, I've proposed above and sent email to discuss ML | 15:19 |
| hberaud | ack | 15:20 |
| opendevreview | Merged openstack/oslo.utils stable/yoga: fix strutils password regex https://review.opendev.org/c/openstack/oslo.utils/+/832897 | 15:26 |
| opendevreview | Hervé Beraud proposed openstack/oslo.utils stable/xena: fix strutils password regex https://review.opendev.org/c/openstack/oslo.utils/+/833109 | 15:30 |
| hberaud | damani: xena ^ | 15:32 |
| damani | ok | 15:34 |
| *** gmann is now known as gmann_afk | 16:13 | |
| *** gmann_afk is now known as gmann | 16:29 | |
| opendevreview | Merged openstack/oslo.utils stable/xena: fix strutils password regex https://review.opendev.org/c/openstack/oslo.utils/+/833109 | 18:24 |
| opendevreview | Hervé Beraud proposed openstack/oslo.utils stable/wallaby: fix strutils password regex https://review.opendev.org/c/openstack/oslo.utils/+/832907 | 20:48 |
| opendevreview | Goutham Pacha Ravi proposed openstack/oslo.log master: Add Zed in versionutils https://review.opendev.org/c/openstack/oslo.log/+/833181 | 21:45 |
Generated by irclog2html.py 2.17.3 by Marius Gedminas - find it at https://mg.pov.lt/irclog2html/!