| *** bauzas9 is now known as bauzas | 01:40 | |
| *** mhen_ is now known as mhen | 01:41 | |
| opendevreview | Yaguang Tang proposed openstack/cinder stable/2024.1: RBD: Fix upload volume with different format https://review.opendev.org/c/openstack/cinder/+/946101 | 03:23 |
|---|---|---|
| opendevreview | Fernando Ferraz proposed openstack/cinder master: NVMe-oF: Fix incorrect check to iscsi initiator data https://review.opendev.org/c/openstack/cinder/+/946103 | 04:11 |
| opendevreview | Fernando Ferraz proposed openstack/cinder master: NVMe-oF: Fix incorrect check for connector initiator data https://review.opendev.org/c/openstack/cinder/+/946103 | 04:12 |
| jssfr | So this is strange, and I'm wondering if we're doing something horribly wrong. We have cinder and glance deployed with an rbd backend. We came from zed and upgraded to 2024.1. after the upgrade, we noticed that the tempest test id-d8f1ca95-3d5b-44a3-b8ca-909691c9532d is failing | 07:44 |
| jssfr | we looked into it and it's reproducible with a simple `openstack volume create --size 10 foo` followed by `openstack image create --volume foo` (substituting IDs and names as appropriate) | 07:44 |
| jssfr | the cause is cinder-volume failing to do the upload with: https://paste.debian.net/hidden/26eddca7/ | 07:45 |
| jssfr | looking at the corresponding code in requests (<https://github.com/psf/requests/blob/main/src/requests/utils.py#L154>) it has been using that `except` clause for several years already. | 07:46 |
| jssfr | and that doesn't catch OSError | 07:46 |
| jssfr | (nor IOError) | 07:46 |
| jssfr | so I'm wondering: how did that ever work *or* is that never tested in CI? | 07:46 |
| jssfr | I'm wondering that, because it might also be some kind of exotic configuration thing which I'd then like to rectify. | 07:46 |
| jssfr | Currently, I'm making a patch to make Rbd.fileno() in os_brick raise io.UnsupportedOperation instead | 07:46 |
| opendevreview | Jonas Schäfer proposed openstack/os-brick master: Make exception type raised by Rbd.fileno() more specific https://review.opendev.org/c/openstack/os-brick/+/946117 | 08:18 |
| opendevreview | Jonas Schäfer proposed openstack/os-brick master: Make exception raised by Rbd.fileno() more specific https://review.opendev.org/c/openstack/os-brick/+/946117 | 08:18 |
| opendevreview | Jonas Schäfer proposed openstack/os-brick master: Make exception raised by Rbd.fileno() more specific https://review.opendev.org/c/openstack/os-brick/+/946117 | 08:19 |
| opendevreview | Jonas Schäfer proposed openstack/os-brick master: Make exception raised by Rbd.fileno() more specific https://review.opendev.org/c/openstack/os-brick/+/946117 | 10:20 |
| opendevreview | Jonas Schäfer proposed openstack/os-brick master: Make exception raised by Rbd.fileno() more specific https://review.opendev.org/c/openstack/os-brick/+/946117 | 12:44 |
| opendevreview | Fernando Ferraz proposed openstack/cinder master: NVMe-oF: Fix incorrect check for initiator in connector properties https://review.opendev.org/c/openstack/cinder/+/946103 | 12:46 |
| jbernard | jungleboyj rosmaita smcginnis tosky whoami-rajat m5z e0ne geguileo eharney jbernard hemna fabiooliveira yuval tobias-urdin adiare happystacker dosaboy hillpd msaravan sp-bmilanov Luzi sfernand simondodsley: meeting in #openstack-meeting-alt | 14:01 |
| Uggla | Hi jbernard, is the cross team glance/cinder/nova session on Wednesday 14h --> 15h UTC is it ok for you ? | 14:08 |
| opendevreview | Goutham Pacha Ravi proposed openstack/devstack-plugin-ceph stable/2025.1: Disable Ingress https://review.opendev.org/c/openstack/devstack-plugin-ceph/+/946156 | 14:08 |
| jbernard | Uggla: about to bring it up in our meeting | 14:08 |
| jbernard | Uggla: will get back to you shortly | 14:09 |
| Uggla | jbernard, there is also the one with neutron/cinder/nova on Thursday 13h --> 14h UTC. Please confirm as soon as possible if this is ok. | 14:12 |
| jbernard | Uggla: do we need to choose between those two? | 14:13 |
| jbernard | Uggla: or we are needing both slots? | 14:14 |
| jbernard | Uggla: tell me: what works best for you? | 14:14 |
| Uggla | jbernard, no it is 2 slots 1h each. | 14:14 |
| jbernard | do you feel those slots will be sufficient to discuss the needed topic? | 14:14 |
| Uggla | one with glance and one with neutron | 14:14 |
| Uggla | I think so from what I heard from neutron ptl and glance ptl | 14:15 |
| Uggla | We will have ~30mn per topic. | 14:16 |
| jbernard | Uggla: no conflicts raised, i think those slots are okay | 14:21 |
| Uggla | jbernard, \o/ thx | 14:24 |
| lanson | Hello, could someone have a look a this https://review.opendev.org/c/openstack/cinder/+/930682 I believe it should be ready to merge. | 15:52 |
Generated by irclog2html.py 2.17.3 by Marius Gedminas - find it at https://mg.pov.lt/irclog2html/!