| gksk | Hi Team can some one review the below patch | 04:43 |
|---|---|---|
| gksk | https://review.opendev.org/c/openstack/cinder/+/869704 | 04:43 |
| opendevreview | Aneesh Pachilangottil proposed openstack/cinder master: NVMe-TCP volume driver for Fungible Storage https://review.opendev.org/c/openstack/cinder/+/849143 | 04:46 |
| opendevreview | Gorka Eguileor proposed openstack/cinder-tempest-plugin master: WIP: zuul: Test LVM/nvmet additional job https://review.opendev.org/c/openstack/cinder-tempest-plugin/+/870889 | 09:27 |
| geguileo | tosky: ^ Trying a new job for nvmeof | 09:27 |
| geguileo | tosky: this one exercises complete different code paths in LVM+nvmet and os-brick | 09:27 |
| geguileo | and of course, syntax error | 09:28 |
| opendevreview | Gorka Eguileor proposed openstack/cinder-tempest-plugin master: WIP: zuul: Test LVM/nvmet additional job https://review.opendev.org/c/openstack/cinder-tempest-plugin/+/870889 | 09:29 |
| tosky | geguileo: oh, I didn't know about that additional code path, thanks | 09:35 |
| geguileo | tosky: yes, all the Cinder patches in LVM+nvmet are not only to fix bugs, but also to ensure we can configure LVM+nvmet to exercise almost all the os-brick code paths | 09:36 |
| opendevreview | Gorka Eguileor proposed openstack/cinderlib master: WIP: Figure out what's wrong in the gate https://review.opendev.org/c/openstack/cinderlib/+/870092 | 10:00 |
| gksk | Hi Team | 10:52 |
| gksk | can some one review my patch | 10:53 |
| gksk | https://review.opendev.org/c/openstack/cinder/+/869704 | 10:53 |
| *** dviroel|afk is now known as dviroel | 11:19 | |
| ierdem__ | Hi everyone, is there any way to boot a signed image from volume? I am testing image validation, I can create VM by using signed images on ephemeral disks but when I try boot from volume, it throws an excepiton (https://paste.openstack.org/show/blZen5ID7OIbi47TN8ib/). I am currently working on OpenStack Ussuri, and image backend is Ceph | 11:37 |
| opendevreview | Gorka Eguileor proposed openstack/cinderlib master: WIP: Figure out what's wrong in the gate https://review.opendev.org/c/openstack/cinderlib/+/870092 | 11:37 |
| *** dviroel|afk is now known as dviroel | 11:54 | |
| opendevreview | Merged openstack/cinder stable/yoga: Serialize message_* properties of RequestContext https://review.opendev.org/c/openstack/cinder/+/867023 | 12:13 |
| geguileo | tosky: whoami-rajat the latest NVMe-oF test job has detected an issue with patch https://review.opendev.org/c/openstack/cinder/+/836074/6 | 12:22 |
| geguileo | test job == CI job https://review.opendev.org/c/openstack/cinder-tempest-plugin/+/870889/ | 12:22 |
| whoami-rajat | looking | 12:22 |
| geguileo | whoami-rajat: no, no, don't look | 12:23 |
| geguileo | I was just saying that the patch in the series has a bug | 12:23 |
| geguileo | but it hasn't merged yet, so it's fine | 12:23 |
| whoami-rajat | ok | 12:23 |
| geguileo | the problem is with force detach where the connector info is not provided | 12:23 |
| whoami-rajat | is that an issue on the brick code or the cinder code? | 12:23 |
| geguileo | so that test fails with a 500 error from cinder | 12:23 |
| geguileo | unmerged cinder code | 12:23 |
| geguileo | so it's all good | 12:24 |
| tosky | nice, the power of tests | 12:24 |
| whoami-rajat | oh ok, then it should be fine | 12:24 |
| geguileo | tosky: yup | 12:24 |
| whoami-rajat | yep, glad we got that job working | 12:24 |
| whoami-rajat | (assuming geguileo will update the patch with the fix) | 12:25 |
| geguileo | yeah, thanks to tosky insisting in having a patch there we have found this issue before merging a broken feature for LVM+nvmet | 12:25 |
| geguileo | whoami-rajat: yes, I'll be working on fixing it | 12:25 |
| whoami-rajat | geguileo, would you like to emphasize on the nvme efforts in the midcycle? | 12:25 |
| whoami-rajat | geguileo, great, thanks | 12:25 |
| geguileo | whoami-rajat: not really, because this work is only for the lvm+nvme | 12:25 |
| geguileo | the os-brick code should be fine | 12:25 |
| whoami-rajat | ok, i was referring to nvme in general since some people might not have the whole context of nvme changes made | 12:26 |
| whoami-rajat | like which protocols are supported with nvme now etc | 12:26 |
| whoami-rajat | but maybe it's just me | 12:26 |
| whoami-rajat | and needs to update myself | 12:27 |
| geguileo | I don't think it should be necessary to give an update, though I could be wrong | 12:28 |
| whoami-rajat | no, it sounds good | 12:30 |
| whoami-rajat | i see we don't have connector == None check in the lvm driver https://review.opendev.org/c/openstack/cinder/+/836074/6/cinder/volume/drivers/lvm.py#857 | 12:30 |
| whoami-rajat | i thought we had it done for all drivers but probably not | 12:30 |
| happystacker | hey guys! does anyone of you can explain why do we do a qemu-img convert operation when fetching an image to a volume? | 12:55 |
| happystacker | I'm looking particularly into this bug https://bugs.launchpad.net/cinder/+bug/2002535 | 12:55 |
| senrique | happystacker, on generic-nfs we fetch the image to raw and then convert it to qcow2 in order to perform snapshots or other operations | 13:55 |
| whoami-rajat | rosmaita, hey, can you take a look at this, recently passed gate after all unrealted errors and we need to merge it at some point https://review.opendev.org/c/openstack/cinder/+/865429 | 13:55 |
| whoami-rajat | Cinder Midcycle 1400-1600 UTC at https://bluejeans.com/556681290 | 13:58 |
| whoami-rajat | jungleboyj rosmaita smcginnis tosky whoami-rajat m5z e0ne geguileo eharney walshh_ jbernard sfernand enriquetaso hemna fabiooliveira yuval tobias-urdin adiare | 13:58 |
| whoami-rajat | etherpad: https://etherpad.opendev.org/p/cinder-antelope-midcycles | 13:59 |
| jungleboyj | I have had some other issues come up today. :-( I will be on and off when I can join. | 14:00 |
| whoami-rajat | sure jungleboyj no problem | 14:00 |
| senrique | happystacker, im not sure why nfs coverts it back to raw, i need to debug that a big more, there's some other issues regarding cache, i wonder if that could be the issue | 14:05 |
| *** senrique is now known as enriquetaso | 14:05 | |
| opendevreview | Felipe Rodrigues proposed openstack/cinder master: NetApp: Add NVMe/TCP driver https://review.opendev.org/c/openstack/cinder/+/870004 | 14:07 |
| opendevreview | Gorka Eguileor proposed openstack/cinderlib master: Fix tox.ini https://review.opendev.org/c/openstack/cinderlib/+/870668 | 14:14 |
| opendevreview | Gorka Eguileor proposed openstack/cinderlib master: Fix DBMS storage plugin https://review.opendev.org/c/openstack/cinderlib/+/870092 | 14:14 |
| happystacker | thank you senrique | 14:27 |
| happystacker | senrique: we have observed this bug in our powerstore environment, does generic-nfs apply too? i thought everything was handled by our driver | 14:30 |
| opendevreview | Felipe Rodrigues proposed openstack/cinder master: NetApp: Add NVMe/TCP driver https://review.opendev.org/c/openstack/cinder/+/870004 | 14:32 |
| opendevreview | Eric Harney proposed openstack/cinder master: Remove six from cinder.tests.unit.volume.drivers https://review.opendev.org/c/openstack/cinder/+/780759 | 15:13 |
| opendevreview | Eric Harney proposed openstack/cinder master: remove six from cinder.transfer https://review.opendev.org/c/openstack/cinder/+/781024 | 15:21 |
| *** dviroel is now known as dviroel|lunch | 15:32 | |
| enriquetaso | #startmeeting cinder_bs | 15:41 |
| opendevmeet | Meeting started Wed Jan 18 15:41:47 2023 UTC and is due to finish in 60 minutes. The chair is enriquetaso. Information about MeetBot at http://wiki.debian.org/MeetBot. | 15:41 |
| opendevmeet | Useful Commands: #action #agreed #help #info #idea #link #topic #startvote. | 15:41 |
| opendevmeet | The meeting name has been set to 'cinder_bs' | 15:41 |
| enriquetaso | #topic [NFS] Server resize failed when image volume cache enabled | 15:42 |
| enriquetaso | #link https://bugs.launchpad.net/cinder/+bug/2002535 | 15:42 |
| enriquetaso | #topic storpool driver | 15:44 |
| enriquetaso | #link https://bugs.launchpad.net/cinder/+bug/2002996 | 15:44 |
| enriquetaso | #link https://bugs.launchpad.net/cinder/+bug/2002995 | 15:44 |
| enriquetaso | #topic Dell PowerFlex: password appears in plain text when creating a volume from an image. | 15:45 |
| enriquetaso | #link https://bugs.launchpad.net/cinder/+bug/2003179 | 15:45 |
| enriquetaso | #topic open discussion | 15:49 |
| enriquetaso | thanks! | 15:51 |
| enriquetaso | #endmeeting | 15:51 |
| opendevmeet | Meeting ended Wed Jan 18 15:51:21 2023 UTC. Information about MeetBot at http://wiki.debian.org/MeetBot . (v 0.1.4) | 15:51 |
| opendevmeet | Minutes: https://meetings.opendev.org/meetings/cinder_bs/2023/cinder_bs.2023-01-18-15.41.html | 15:51 |
| opendevmeet | Minutes (text): https://meetings.opendev.org/meetings/cinder_bs/2023/cinder_bs.2023-01-18-15.41.txt | 15:51 |
| opendevmeet | Log: https://meetings.opendev.org/meetings/cinder_bs/2023/cinder_bs.2023-01-18-15.41.log.html | 15:51 |
| enriquetaso | zaitcev, i think you may be interested in https://bugs.launchpad.net/cinder/+bug/2002535 | 15:51 |
| zaitcev | enriquetaso: thanks | 15:51 |
| whoami-rajat | enriquetaso, did you forget to end the meeting? | 15:54 |
| whoami-rajat | oh sorry i didn't scroll | 15:54 |
| whoami-rajat | it's already ended | 15:54 |
| geguileo | whoami-rajat: rosmaita eharney enriquetaso abishop I can't find sathya online, but this is the idea I was referring to for the retype with migration: https://paste.openstack.org/show/818338/ | 16:03 |
| geguileo | I wrote it in the patch review | 16:03 |
| geguileo | and in the etherpad | 16:03 |
| enriquetaso | woah | 16:03 |
| enriquetaso | ++ | 16:03 |
| rosmaita | geguileo: thanks , that should be helpful, and sathya should be able to find it easily | 16:04 |
| geguileo | it should be a better explanation that what I tried to do talking in the meeting rofl rofl | 16:04 |
| opendevreview | Eric Harney proposed openstack/cinder master: Tests: Fix double mock of Popen (test_backup_ceph) https://review.opendev.org/c/openstack/cinder/+/867824 | 16:07 |
| rosmaita | geguileo: i thought your explanation was very clear, but this will be even more clear | 16:07 |
| geguileo | rosmaita: thanks, sometimes I think I explain myself as a closed book (like we say in Spain) :-) | 16:08 |
| geguileo | s/as/like | 16:08 |
| rosmaita | geguileo: i had not heard that expression before :D | 16:08 |
| opendevreview | Eric Harney proposed openstack/cinder master: Remove six from cinder.tests.unit.volume.drivers https://review.opendev.org/c/openstack/cinder/+/780759 | 16:11 |
| *** dviroel|lunch is now known as dviroel | 16:23 | |
| opendevreview | Brian Rosmaita proposed openstack/python-cinderclient master: Continue using tox 3 https://review.opendev.org/c/openstack/python-cinderclient/+/870971 | 16:28 |
| whoami-rajat | geguileo, looks good, i will let him know if i talk to him, thanks! | 16:48 |
| geguileo | whoami-rajat: thanks | 16:48 |
| *** gmann is now known as gmann_afk | 17:29 | |
| *** gmann_afk is now known as gmann | 17:41 | |
| *** gmann is now known as gmann_afk | 18:06 | |
| *** gmann_afk is now known as gmann | 18:56 | |
| opendevreview | Merged openstack/cinder stable/yoga: Fix and unify capacity calculations https://review.opendev.org/c/openstack/cinder/+/867994 | 21:01 |
| *** dviroel is now known as dviroel|afk | 21:09 | |
Generated by irclog2html.py 2.17.3 by Marius Gedminas - find it at https://mg.pov.lt/irclog2html/!