opendevreview | Goutham Pacha Ravi proposed openstack/openstacksdk master: Remove "feature/r1" override from manila job https://review.opendev.org/c/openstack/openstacksdk/+/874046 | 00:58 |
---|---|---|
opendevreview | Reynaldo proposed openstack/openstacksdk master: Add share snapshot instance resource https://review.opendev.org/c/openstack/openstacksdk/+/798464 | 06:23 |
opendevreview | Rajat Dhasmana proposed openstack/python-openstackclient master: Add volume revert command https://review.opendev.org/c/openstack/python-openstackclient/+/869654 | 09:01 |
opendevreview | Rajat Dhasmana proposed openstack/python-openstackclient master: Add block storage log level {list, set} commands https://review.opendev.org/c/openstack/python-openstackclient/+/869811 | 09:01 |
opendevreview | Rajat Dhasmana proposed openstack/python-openstackclient master: Add block storage cleanup command https://review.opendev.org/c/openstack/python-openstackclient/+/872390 | 09:01 |
opendevreview | Rajat Dhasmana proposed openstack/python-openstackclient master: Add block storage manageable list commands https://review.opendev.org/c/openstack/python-openstackclient/+/872692 | 09:01 |
opendevreview | Rajat Dhasmana proposed openstack/python-openstackclient master: Add options to create volume group from source https://review.opendev.org/c/openstack/python-openstackclient/+/872798 | 09:01 |
whoami-rajat | stephenfin, gtema updated all the patches ^ | 09:12 |
opendevreview | Stephen Finucane proposed openstack/osc-lib master: tox 4 compatibility https://review.opendev.org/c/openstack/osc-lib/+/874075 | 10:27 |
frickler | gtema: stephenfin: would one of you have time to look into https://zuul.opendev.org/t/openstack/build/284499a2edc041f78d6b94dbee1e1e07 ? this is blocking getting sdk 1.0.1 into u-c (not the only blocker, but kuryr at least has a fix proposed) | 11:00 |
gtema | frickler, ack, looking | 11:01 |
gtema | but I am not sure whether this is OSC issue or octaviaclient | 11:02 |
gtema | frickler - I think I found a direction. python-openstackclient is not installing python-octaviaclient through requirements.txt. It is done in doc/requirements.txt | 11:30 |
gtema | but I am not sure this is respected in the requirements job, cause according to the log python-octaviaclient is not visible in the packages installed into the tox venv | 11:31 |
gtema | at least it does not look like it is there and if I remove it locally in the docs venv of OSC I get the same error | 11:33 |
frickler | gtema: ah, so I'll check the job definition in the reqs repo, thx for looking | 12:05 |
gtema | frickler, since this was running before I assume this might be something new in toxv4 as well. Logs show octaviaclient is installed (into some env), but during the build it is just not there where expected | 12:09 |
frickler | yes, this is weird. the job is only failing for the sdk bump, not for other updates, so there must be some connection. let me try with a dep on osc | 12:46 |
frickler | I can reproduce the error locally if I manually update openstacksdk in the venv or point to the updated u-c file. but I have no idea why this is happening. also not related to tox4, can repro with tox3, too | 12:57 |
opendevreview | Dr. Jens Harbott proposed openstack/python-openstackclient master: DNM: Test sdk update https://review.opendev.org/c/openstack/python-openstackclient/+/874095 | 12:58 |
gtema | Weird | 13:01 |
gtema | What we had figured out for Octaviaclient is that in SDK 1.0 we dropped munch from dependencies | 13:02 |
gtema | I added it i to requirements there and it was merged | 13:02 |
frickler | ah, maybe the bump for octaviaclient needs to be merged first, then | 13:03 |
gtema | Maybe. But that is weird that in this case it causes such a hidden failure | 13:04 |
gtema | Can you try where you reporoduced the issue to manually install munch and check whether it works then or not | 13:05 |
gtema | It may be the pip issue and not tox | 13:05 |
frickler | I just had the same idea, with much installed into the docs venv, the issue goes away | 13:07 |
frickler | so we could add that to doc/reqs.txt in osc. or try to find out why the transitive requirement doesn't seem to work | 13:08 |
frickler | I'll wait for test results in my patch above and then try that | 13:09 |
frickler | s/much/munch/ above | 13:09 |
opendevreview | Dr. Jens Harbott proposed openstack/python-openstackclient master: Add munch to doc requirements https://review.opendev.org/c/openstack/python-openstackclient/+/874095 | 15:23 |
opendevreview | Merged openstack/osc-lib master: tox 4 compatibility https://review.opendev.org/c/openstack/osc-lib/+/874075 | 17:10 |
opendevreview | Stephen Finucane proposed openstack/python-openstackclient master: WIP: Follow-up for I87482a5dd43c519dfdcf981635aa879914a70a5c https://review.opendev.org/c/openstack/python-openstackclient/+/874151 | 19:12 |
Generated by irclog2html.py 2.17.3 by Marius Gedminas - find it at https://mg.pov.lt/irclog2html/!