| *** dviroel|rover is now known as dviroel|out | 00:10 | |
| sfernand | whoami-rajat: just realized I’d forgotten to add the link to the patch I mentioned earlier https://review.opendev.org/c/openstack/cinder/+/802955 | 02:01 |
|---|---|---|
| sfernand | :| | 02:01 |
| opendevreview | Oleg proposed openstack/cinder master: PowerStore driver - Request data validation fix https://review.opendev.org/c/openstack/cinder/+/849626 | 08:08 |
| opendevreview | Oleg proposed openstack/cinder master: PowerStore driver - Request data validation fix https://review.opendev.org/c/openstack/cinder/+/849626 | 08:12 |
| opendevreview | Oleg proposed openstack/cinder master: PowerStore driver - Request data validation fix https://review.opendev.org/c/openstack/cinder/+/849626 | 08:14 |
| opendevreview | Alexander Malashenko proposed openstack/cinder master: Add Cinder NFS driver for Dell PowerStore https://review.opendev.org/c/openstack/cinder/+/797608 | 09:15 |
| *** dviroel is now known as dviroel|rover | 12:12 | |
| whoami-rajat | Cinder meeting in #openstack-meeting-alt at 1400 UTC | 13:58 |
| whoami-rajat | jungleboyj rosmaita smcginnis tosky whoami-rajat m5z e0ne geguileo eharney walshh_ jbernard sfernand enriquetaso hemna fabiooliveira yuval tobias-urdin | 13:58 |
| rosmaita | whoami-rajat: ty | 13:58 |
| whoami-rajat | np | 13:59 |
| geguileo | whoami-rajat: thanks | 13:59 |
| *** senrique__ is now known as enriquetaso | 14:05 | |
| opendevreview | Simon Dodsley proposed openstack/cinder master: Add additional transport type constants https://review.opendev.org/c/openstack/cinder/+/849690 | 14:30 |
| opendevreview | Oleg proposed openstack/cinder master: PowerStore driver - Request data validation fix https://review.opendev.org/c/openstack/cinder/+/849626 | 14:39 |
| opendevreview | Simon Dodsley proposed openstack/cinder master: Add Pure Storage NVMe-RoCE driver https://review.opendev.org/c/openstack/cinder/+/799871 | 14:44 |
| -opendevstatus- NOTICE: Due to an incident in our hosting provider, the tarballs.opendev.org site (and possibly other sites served from static.opendev.org) is offline while we attempt recovery | 14:52 | |
| *** dviroel|rover is now known as dviroel|rover|biab | 14:53 | |
| geguileo | rosmaita: os_brick/initiator/connectors/rbd.py | 15:06 |
| rosmaita | is that the only one? | 15:06 |
| geguileo | rosmaita: that's the relevant file to make the ceph job vote | 15:06 |
| rosmaita | ok, cool, that's easy | 15:06 |
| geguileo | anything else is commong to other connectors as well | 15:06 |
| eharney | i would say utils.py, too | 15:06 |
| whoami-rajat | geguileo, how about https://github.com/openstack/os-brick/blob/master/os_brick/initiator/linuxrbd.py ? | 15:07 |
| rosmaita | eharney: don't get greedy! | 15:07 |
| geguileo | eharney: +1 | 15:07 |
| geguileo | rosmaita: and even os_brick/initiator/connector.py | 15:07 |
| rosmaita | ok, so connector, utils, linuxrbd | 15:07 |
| rosmaita | and base connector | 15:07 |
| eharney | well utils.py is what just broke us | 15:07 |
| geguileo | whoami-rajat: +1 | 15:07 |
| geguileo | eharney: but it broke us with a change in rbd.py as well | 15:08 |
| eharney | ok | 15:08 |
| rosmaita | ok, i will start with those and we can fight it out on the patch | 15:08 |
| geguileo | so those 4 files | 15:08 |
| eharney | i'll +2 whatever | 15:08 |
| geguileo | eharney: lol that's the spirit!!! | 15:08 |
| enriquetaso | I'll start the bug meeting then :) | 15:09 |
| enriquetaso | #startmeeting cinder_bs | 15:09 |
| opendevmeet | Meeting started Wed Jul 13 15:09:23 2022 UTC and is due to finish in 60 minutes. The chair is enriquetaso. Information about MeetBot at http://wiki.debian.org/MeetBot. | 15:09 |
| opendevmeet | Useful Commands: #action #agreed #help #info #idea #link #topic #startvote. | 15:09 |
| opendevmeet | The meeting name has been set to 'cinder_bs' | 15:09 |
| geguileo | enriquetaso: yes, sorry | 15:09 |
| enriquetaso | Today we have six new bugs. | 15:09 |
| enriquetaso | Full email report | 15:09 |
| enriquetaso | #link https://lists.openstack.org/pipermail/openstack-discuss/2022-July/029521.html | 15:09 |
| enriquetaso | #topic RBD disconnect fails with AttributeError for startswith | 15:09 |
| enriquetaso | #link https://bugs.launchpad.net/os-brick/+bug/1981455 | 15:10 |
| enriquetaso | As Gorka mentioned at the main meeting https://meetings.opendev.org/meetings/cinder/2022/cinder.2022-07-13-14.00.log.html#l-66 | 15:10 |
| enriquetaso | There are 4 high important patches to review and this is one of those. We need those os-brick changes released for NVMe drivers to work. | 15:10 |
| enriquetaso | Fix proposed to master: | 15:11 |
| enriquetaso | #link https://review.opendev.org/c/openstack/os-brick/+/849542 | 15:11 |
| enriquetaso | The patch is waiting for some replies but it's the right way to go. Please review asap :) | 15:11 |
| eharney | geguileo said he was going to go reply to a question there | 15:11 |
| enriquetaso | Cool, thanks :) We also need those patches for the next os-brick release. | 15:12 |
| enriquetaso | #topic Dell PowerStore and Dell PowerMax | 15:14 |
| enriquetaso | #link https://bugs.launchpad.net/cinder/+bug/1981068 | 15:14 |
| enriquetaso | #link https://bugs.launchpad.net/cinder/+bug/1981420 | 15:14 |
| enriquetaso | #link https://bugs.launchpad.net/cinder/+bug/1980870 | 15:14 |
| enriquetaso | There are some basic volume functionalities that are not working properly. | 15:14 |
| enriquetaso | Do we have anyone from the Dell team that would like to add anything? :) | 15:14 |
| enriquetaso | Please feel free to review them ^ | 15:14 |
| eharney | i'm adding "drivers" tags to these bugs | 15:14 |
| enriquetaso | oops my bad | 15:15 |
| eharney | always helpful to delineate bugs that are purely driver bugs from core cinder issues | 15:15 |
| amalashenko | sure, about first bug, oleg working with that bug https://review.opendev.org/c/openstack/cinder/+/849626 | 15:16 |
| geguileo | eharney: I just did | 15:16 |
| enriquetaso | thanks amalashenko, i read you're new, do you have any question for the team? | 15:17 |
| * enriquetaso forgot about the "drivers" tag but she'll be more careful | 15:18 | |
| amalashenko | yes, i have one question, about this bug https://bugs.launchpad.net/cinder/+bug/1981420 | 15:18 |
| geguileo | amalashenko: what's the question? | 15:20 |
| amalashenko | when we are trying to resize instance nova raises error, because cinder shows qcow2 format for volume but qemu-img info shows that volume has raw format, so, i can try to fix it | 15:20 |
| amalashenko | is this bug or no? | 15:20 |
| geguileo | amalashenko: where does Cinder say it's a qcow2 file? | 15:21 |
| geguileo | because it may be referring to the image metadata | 15:21 |
| geguileo | which is different from the volume itself | 15:21 |
| amalashenko | got it, i see | 15:22 |
| geguileo | I'm not sure that's the case though | 15:22 |
| amalashenko | them maybe there is a problem with nova? | 15:23 |
| amalashenko | then | 15:23 |
| geguileo | amalashenko: you would have to check where cinder is saying that it's qcow2 | 15:25 |
| enriquetaso | I thinks we are going to need more info from you to check if this is a bug or not amalashenko (i.e steps to reproduce the problem, the c-vol logs with errors) | 15:25 |
| amalashenko | volume_image_metadata field with qcow2 format | 15:25 |
| geguileo | that only refers to the source of the image | 15:26 |
| geguileo | afaik the volume doesn't have to match that | 15:26 |
| geguileo | eharney: whoami-rajat rosmaita enriquetaso is that correct? | 15:26 |
| eharney | IIRC volume_image_metadata show info about the image used, if it was converted when being written to the volume, then yeah, the volume would be raw and that image would be recorded as qcow2 | 15:27 |
| eharney | i'm unclear how this is linked to bug 1981420 | 15:28 |
| amalashenko | i thought, when i create volume type with enable qcow2 format and try to write image on volume, volume will has raw format, but nova expected qcow2 format | 15:28 |
| eharney | nova will expect the format provided in the connection info for the volume, which is typically raw for iscsi/fc drivers (like i think we are discussing here) | 15:29 |
| rosmaita | amalashenko: which driver are you seeing this with? | 15:30 |
| amalashenko | generic-nfs driver | 15:31 |
| amalashenko | i wrote in the bug launchpad | 15:31 |
| rosmaita | i must be looking at the wrong bug | 15:32 |
| enriquetaso | i'm confused | 15:32 |
| enriquetaso | are we talking about https://bugs.launchpad.net/cinder/+bug/1981420 ? | 15:33 |
| amalashenko | sorry, my mistake | 15:33 |
| amalashenko | https://bugs.launchpad.net/cinder/+bug/1981562 | 15:33 |
| whoami-rajat | geguileo, yes, that's image format and not volume format, we've added a 'format' field in admin metadata for filesystem type drivers that can be seen with attachment show response | 15:33 |
| enriquetaso | #link https://bugs.launchpad.net/cinder/+bug/1981562 | 15:34 |
| enriquetaso | looks like a bug | 15:38 |
| whoami-rajat | i can take a look | 15:41 |
| enriquetaso | thanks for bringing this up amalashenko. whoami-rajat will keep an eye on it :) thanks Rajat! and I'll add it on next week report | 15:42 |
| amalashenko | i can work with it too, but after when NFS driver will merged :) | 15:42 |
| enriquetaso | sounds good, moving on, because I have 2 bug left and no time | 15:42 |
| enriquetaso | #topic Infinidat Cinder driver does not return all iSCSI portals for multipath storage | 15:42 |
| enriquetaso | #link https://bugs.launchpad.net/cinder/+bug/1981354 | 15:42 |
| enriquetaso | Fix proposed to master | 15:42 |
| enriquetaso | #link https://review.opendev.org/c/openstack/cinder/+/849022 | 15:42 |
| enriquetaso | feel free to review it! ^ | 15:43 |
| enriquetaso | #topic [stable/yoga] Update attachment failed for attachment | 15:43 |
| enriquetaso | #link https://bugs.launchpad.net/cinder/+bug/1981211 | 15:43 |
| enriquetaso | The bug is incomplete and I left some questions on it. However, should I assign it to Nova? | 15:43 |
| enriquetaso | Summary "Error 500: Unable to update attachment." I guess the volume it's attached to an instance | 15:45 |
| enriquetaso | well, i guess i'll link nova to the bug as well but it's a cinder bug | 15:45 |
| enriquetaso | OK, i have no more time | 15:46 |
| enriquetaso | Thank you for attending the bug meeting! | 15:46 |
| enriquetaso | #endmeeting | 15:46 |
| opendevmeet | Meeting ended Wed Jul 13 15:46:48 2022 UTC. Information about MeetBot at http://wiki.debian.org/MeetBot . (v 0.1.4) | 15:46 |
| opendevmeet | Minutes: https://meetings.opendev.org/meetings/cinder_bs/2022/cinder_bs.2022-07-13-15.09.html | 15:46 |
| opendevmeet | Minutes (text): https://meetings.opendev.org/meetings/cinder_bs/2022/cinder_bs.2022-07-13-15.09.txt | 15:46 |
| opendevmeet | Log: https://meetings.opendev.org/meetings/cinder_bs/2022/cinder_bs.2022-07-13-15.09.log.html | 15:46 |
| *** dviroel|rover|biab is now known as dviroel|rover | 16:11 | |
| opendevreview | Gorka Eguileor proposed openstack/os-brick master: Support independent file lock path https://review.opendev.org/c/openstack/os-brick/+/849324 | 16:23 |
| geguileo | rosmaita: ^ that includes the failsafe you requested | 16:23 |
| rosmaita | geguileo: ty! | 16:23 |
| geguileo | np | 16:23 |
| opendevreview | Gorka Eguileor proposed openstack/os-brick stable/yoga: RBD: Fix disconnect_volume for encrypted volumes https://review.opendev.org/c/openstack/os-brick/+/849718 | 16:27 |
| geguileo | eharney: jungleboyj ^ the backport of the patch you +2 on master and that's on its way to get merged | 16:28 |
| opendevreview | Merged openstack/cinder stable/xena: PowerMax Driver - QoS should not be set on parent storage group https://review.opendev.org/c/openstack/cinder/+/821745 | 16:40 |
| *** dviroel__ is now known as dviroel|rover|biab | 16:42 | |
| opendevreview | Merged openstack/cinder stable/xena: PowerMax Driver - Update unsupported retype combinations https://review.opendev.org/c/openstack/cinder/+/821746 | 16:45 |
| *** dviroel|rover|biab is now known as dviroel|rover | 17:21 | |
| opendevreview | Merged openstack/os-brick master: RBD: Fix disconnect_volume for encrypted volumes https://review.opendev.org/c/openstack/os-brick/+/849542 | 17:24 |
| opendevreview | Simon Dodsley proposed openstack/cinder master: Add Pure Storage NVMe-RoCE driver https://review.opendev.org/c/openstack/cinder/+/799871 | 21:24 |
| opendevreview | Simon Dodsley proposed openstack/cinder master: Add Pure Storage NVMe-RoCE driver https://review.opendev.org/c/openstack/cinder/+/799871 | 22:22 |
| opendevreview | Brian Rosmaita proposed openstack/os-brick master: WIP: make ceph job voting (sometimes) https://review.opendev.org/c/openstack/os-brick/+/849759 | 22:30 |
| opendevreview | Brian Rosmaita proposed openstack/os-brick master: WIP: make ceph job voting (sometimes) https://review.opendev.org/c/openstack/os-brick/+/849759 | 22:47 |
| opendevreview | Brian Rosmaita proposed openstack/os-brick master: WIP: make ceph job voting (sometimes) https://review.opendev.org/c/openstack/os-brick/+/849759 | 22:57 |
| *** dviroel|rover is now known as dviroel|rover|Afk | 22:58 | |
| *** dviroel|rover|Afk is now known as dviroel|rover | 23:57 | |
Generated by irclog2html.py 2.17.3 by Marius Gedminas - find it at https://mg.pov.lt/irclog2html/!