| opendevreview | OpenStack Proposal Bot proposed openstack/requirements master: update constraint for virtualbmc to new release 3.3.0 https://review.opendev.org/c/openstack/requirements/+/977938 | 10:38 |
|---|---|---|
| opendevreview | OpenStack Proposal Bot proposed openstack/requirements master: update constraint for python-zaqarclient to new release 4.4.0 https://review.opendev.org/c/openstack/requirements/+/977942 | 10:59 |
| opendevreview | OpenStack Proposal Bot proposed openstack/requirements master: update constraint for python-neutronclient to new release 11.8.0 https://review.opendev.org/c/openstack/requirements/+/977943 | 11:06 |
| opendevreview | OpenStack Proposal Bot proposed openstack/requirements master: update constraint for python-ironicclient to new release 6.0.0 https://review.opendev.org/c/openstack/requirements/+/977944 | 11:15 |
| opendevreview | Merged openstack/requirements master: Update testtools https://review.opendev.org/c/openstack/requirements/+/977082 | 11:25 |
| opendevreview | Eduardo Olivares proposed openstack/requirements master: Bump virtualenv version to 20.39.0 https://review.opendev.org/c/openstack/requirements/+/977947 | 11:47 |
| opendevreview | Merged openstack/requirements master: update constraint for virtualbmc to new release 3.3.0 https://review.opendev.org/c/openstack/requirements/+/977938 | 13:02 |
| tkajinam | frickler, so I now see failure caused by new tox requiring new virtualenv and we actually need https://review.opendev.org/c/openstack/requirements/+/977947 or pin tox ... | 13:45 |
| tkajinam | that's blocking python-heatclient change we want to include for the new release... | 13:47 |
| frickler | IMO we should pin tox now if possible and next cycle discuss whether we want to move away from it completely. but that's likely a larger discussion for the whole community. cc tonyb prometheanfire mnasiadka elodilles ^^ | 14:06 |
| frickler | tkajinam: do you have a link to the failure? I'm wondering why not everyone seems to be broken. or is this just a case of "not yet"? | 14:07 |
| tkajinam | frickler, https://zuul.opendev.org/t/openstack/build/151eb32740ae4663a84b11a7d34243e7 | 14:13 |
| tkajinam | tox 4.46 is installed by | 14:15 |
| tkajinam | lib/neutron_plugins/ovn_agent:install_ovn:405 : pip_install tox | 14:15 |
| tkajinam | so I think this is not specific to heat and likely affect the other jobs later | 14:15 |
| frickler | oh, so that's not some openstack-tox- | 14:16 |
| frickler | eh .. openstack-tox-* job, but devstack running tox | 14:16 |
| tkajinam | yeah that's the job deploying devstack and run tests via tox | 14:17 |
| tkajinam | I think this is a common pattern for client libs | 14:17 |
| tkajinam | and tox is updated unexpectedly by devstack | 14:18 |
| tkajinam | however I'm pretty sure that even normal openstack-tox-* jobs are affected once new CI image is built and new tox is pulled into it | 14:18 |
| tkajinam | (these jobs are using tox in the image, IIUC | 14:19 |
| tkajinam | so it's also affecting functional tests for cinderclient ... which follows the same pattern | 14:26 |
| elodilles | tkajinam: quick question, does 4.46.1 fix the issue (the failure came with tox 4.46.0) | 14:31 |
| elodilles | ? | 14:31 |
| tkajinam | checking | 14:31 |
| tkajinam | elodilles, I think no, according to quick diff | 14:32 |
| elodilles | :/ | 14:32 |
| elodilles | hmm, and there will come a 4.46.2 soon if i see it right | 14:33 |
| tkajinam | 4.46.2 touches different things | 14:36 |
| elodilles | damn :/ | 14:36 |
| tkajinam | this is basically caused by the fact that we pin virtualenv but does not pin tox | 14:36 |
| tkajinam | removing pip_install tox from devstack may resolve the failure now but we later face problems in same functional job once new CI image is built with new tox, because I suspect that virutalenv may be downgraded according to u-c | 14:38 |
| tkajinam | https://review.opendev.org/c/openstack/devstack/+/977970 might unblock the failure now but we might find failure in different jobs later once new tox is pulled to CI image | 14:47 |
| tkajinam | oh, wait | 14:53 |
| tkajinam | I think they didn't bump the lower bound | 14:53 |
| rpittau | hi all o/ I'm wondering if there's already a discussion open to evaluate the use of libpass in openstack as replacement for the unmaintained passlib | 15:12 |
| tkajinam | rpittau, I think we had discussion about its usage in keystone and we remove it a few cycles ago but I'm not aware of any discussions about libpass IIRC | 15:16 |
| tkajinam | https://review.opendev.org/q/topic:%22passlib%22 | 15:16 |
| rpittau | tkajinam: thanks, I guess we need to look into removing it also in ironic then | 15:16 |
| rpittau | it's mainly because of ansible | 15:17 |
| tkajinam | yeah | 15:18 |
| tkajinam | I think it's no longer compatible with some of the dependencies we have (bcrypt, I guess) | 15:18 |
| rpittau | exactly | 15:18 |
| rpittau | oh well, I'll look into it | 15:19 |
| tkajinam | though the impact may be dependent on the implementations actually used | 15:19 |
| rpittau | it's for the basic http auithentication | 15:19 |
| tkajinam | ah, ok | 15:20 |
| tkajinam | to create the hash in the auth file I guess | 15:20 |
| rpittau | actually I think ironic does not use passlib anymore | 15:20 |
| rpittau | so maybe only bifrost needs updating | 15:20 |
| tkajinam | seems so | 15:21 |
| rpittau | ok, thanks, I guess we'll be fine :D | 15:21 |
| mnasiadka | frickler: any reason to not just bump virtualenv? | 15:54 |
| frickler | mnasiadka: well no, nothing in particular other than to protest against tox devs giving zilch interest to stability | 16:00 |
| mnasiadka | Ok, that one I understand - but changing tox to anything else is probably going to be an ecosystem level job | 16:03 |
| frickler | yeah, likely a saner first step would be pinning tox, like at the beginning of each cycle. which we'll still need to do retrospectively now to fix stable branches I think | 16:32 |
| tkajinam | I do not object to adding pin but as we are close to the client lib freeze we really need at least short term workaround now | 16:36 |
| tkajinam | oops. I posted the above before I noticed frickler approved that req bump.... thanks for that | 16:37 |
| tkajinam | I also confirmed that https://review.opendev.org/c/openstack/devstack/+/977970 unblocks heatclient. We can probably merge it now to reduce possibility of tox being updated unexpectedly | 16:38 |
| tkajinam | fyi: we got 4.46.3 released which corrected the min version of virtualenv... however the previous versions are still available so this may not directly fix the issue | 16:54 |
| stephenfin | tkajinam: Hmm, can you sanity check this for me? https://paste.opendev.org/show/bvZzudhGGahYBp1NfTqw/ | 17:06 |
| stephenfin | tkajinam: nvm: the pip output confused me. I have my reproducer | 17:09 |
| tkajinam | stephenfin, which version of tox installed ? | 17:09 |
| tkajinam | is installed ? | 17:10 |
| tkajinam | I suspect 4.46.2 | 17:10 |
| stephenfin | https://paste.opendev.org/show/bQ1mKouwbusPiN1rQkLg/ | 17:10 |
| stephenfin | Yeah 4.46.2 but it tried 4.46.3 first. I didn't scan down far enough 😅 | 17:10 |
| tkajinam | stephenfin, thanks for posting the comment to ask for yank. I was about to ask the same | 17:15 |
| stephenfin | nw | 17:15 |
| tkajinam | https://zuul.opendev.org/t/openstack/build/5a511431d79440549c11694e0201072b | 17:28 |
| tkajinam | hmm I'm afraid that gabbi bump we merged recently might kill multiple jobs https://review.opendev.org/c/openstack/requirements/+/972462 | 17:28 |
| tkajinam | my bad that I kept https://github.com/cdent/gabbi/pull/344 out of my memory ... | 17:31 |
| tkajinam | this hit gnocchi earlier but we eventually decided to pin gabbi IIRC... | 17:33 |
| opendevreview | Merged openstack/requirements master: update constraint for python-ironicclient to new release 6.0.0 https://review.opendev.org/c/openstack/requirements/+/977944 | 17:47 |
| opendevreview | Merged openstack/requirements master: update constraint for python-neutronclient to new release 11.8.0 https://review.opendev.org/c/openstack/requirements/+/977943 | 17:48 |
| opendevreview | Merged openstack/requirements master: update constraint for python-zaqarclient to new release 4.4.0 https://review.opendev.org/c/openstack/requirements/+/977942 | 17:48 |
| stephenfin | tkajinam: fwiw this is how I fixed it in placement https://review.opendev.org/c/openstack/placement/+/972708 | 18:10 |
| opendevreview | Merged openstack/requirements master: Bump virtualenv version to 20.39.0 https://review.opendev.org/c/openstack/requirements/+/977947 | 18:13 |
| opendevreview | OpenStack Proposal Bot proposed openstack/requirements master: update constraint for python-swiftclient to new release 4.10.0 https://review.opendev.org/c/openstack/requirements/+/978028 | 21:22 |
Generated by irclog2html.py 4.0.0 by Marius Gedminas - find it at https://mg.pov.lt/irclog2html/!