Wednesday, 2022-01-12

opendevreviewTakashi Kajinami proposed openstack/cinderlib master: Migrate jobs from CentOS 8 to CentOS 8 Stream  https://review.opendev.org/c/openstack/cinderlib/+/82426100:48
opendevreviewTakashi Kajinami proposed openstack/cinderlib master: Migrate jobs from CentOS 8 to CentOS 8 Stream  https://review.opendev.org/c/openstack/cinderlib/+/82426100:50
*** _pewp__ is now known as _pewp_07:38
*** hemna0 is now known as hemna07:38
opendevreviewyuval proposed openstack/cinder master: Lightbits LightOS driver  https://review.opendev.org/c/openstack/cinder/+/82160209:21
*** elodille1 is now known as elodilles09:29
*** dviroel_ is now known as dviroel11:21
rosmaitaCourtesy reminder: Cinder meeting in #openstack-meeting-alt at 1400 UTC13:58
rosmaitajungleboyj rosmaita smcginnis tosky whoami-rajat m5z e0ne geguileo eharney walshh_ jbernard sfernand enriquetaso hemna fabiooliveira yuval ^^13:58
raghavendrathi reviewers, this is regarding backport patch: https://review.opendev.org/c/openstack/cinder/+/82062814:29
raghavendratIt would be great if someone can share feedback. Thanks.14:30
opendevreviewyuval proposed openstack/cinder master: Lightbits LightOS driver  https://review.opendev.org/c/openstack/cinder/+/82160214:51
enriquetasohey, cinder bug meeting now15:03
enriquetaso#startmeeting cinder_bs15:03
opendevmeetMeeting started Wed Jan 12 15:03:39 2022 UTC and is due to finish in 60 minutes.  The chair is enriquetaso. Information about MeetBot at http://wiki.debian.org/MeetBot.15:03
opendevmeetUseful Commands: #action #agreed #help #info #idea #link #topic #startvote.15:03
opendevmeetThe meeting name has been set to 'cinder_bs'15:03
enriquetasoThree bugs reported this period. Full list of bugs:15:03
enriquetaso#link http://lists.openstack.org/pipermail/openstack-discuss/2022-January/026631.html15:03
enriquetasoOne of them is incomplete, we are going to discuss the rest:15:03
rosmaitao/15:04
enriquetasohey Brian :P15:04
enriquetaso#topic rbd: snapshot can't be deleted/unmanaged if its source volume is deleted from backend15:04
enriquetaso#link https://bugs.launchpad.net/cinder/+bug/195707315:04
enriquetasoThere's already a fix proposed to master that looks right, so feel free to review it in your spare time:15:04
enriquetaso#link https://review.opendev.org/c/openstack/cinder/+/82414915:04
enriquetasoeharney ^ we've discussed this bug yesterday 15:04
enriquetasoso, you might be interested in the fix proposed 15:05
eharneyyeah we should review this one, i'll take a look15:05
enriquetasomoving on15:06
enriquetasolast bug:15:06
enriquetaso#topic LVM backed ISCSI device not reporting the same size to nova15:06
enriquetaso#link https://bugs.launchpad.net/cinder/+bug/195688715:06
enriquetasoSummary: The user creates a 20GB volume and nova shows 4 Terabyte. The VM disk size is bigger than the real disk size on the iscsi-backed lvm volume.15:06
enriquetasoI just ask you if this is a well known issue related to os-brick because I think I heard this a while ago but I'm not sure. 15:06
enriquetasoI wonder if* 15:07
enriquetasoOK.. need to check old bugs but the bug looks legit 15:08
geguileoenriquetaso: that's probably one of those backends that increase in 4TB chunks15:08
geguileorofl15:08
enriquetasoit's LVM15:08
geguileooh, missed that, sorry15:08
eharneyyeah, this probably needs some investigation15:09
geguileothen that doesn't make any sense...15:09
enriquetasoyes, it doesn't, looks crazy. OK, moving on15:09
enriquetaso#topic open discussion15:09
enriquetasoFeel free to share bugs here (if we have any)15:10
geguileothe LVM one is kind of scary...15:12
sfernandeharney: gouthamr if you guys have some time please take a look at Felipe's patch to address the online extend issue15:12
sfernandhttps://review.opendev.org/c/openstack/cinder/+/73907915:12
rosmaitageguileo: i agree15:12
rosmaitabut it sounds like although fdisk is reporting the wrong size, the vm isn't being allowed to write into the "extra" space?15:13
*** dviroel is now known as dviroel|lunch15:13
enriquetasorosmaita, i'm not sure about that from the bug description15:14
rosmaitaok, and the significance of 20GB volume isn't clear either, whether it doesn't happen for <20G or whether the reporter only tried with 20G15:17
enriquetasoOkay, I'll ask the reporter if  the vm isn't being allowed to write into the "extra" space. And if this happen for <20G or whether the reporter only tried with 20G15:19
rosmaitaenriquetaso: i guess read and write15:19
enriquetasosure15:20
geguileolooks like it by the kernel log message mentioned15:20
enriquetaso#action (enriquetaso): Ask  the reporter if  the vm isn't being allowed to read/write into the "extra" space. And if this happen for <20G or whether the reporter only tried with 20G15:20
rosmaitageguileo: looks like it's *not* being allowed?15:21
rosmaita(that was my reading, but i have only had one coffee this morning)15:21
geguileorosmaita: not sure where the failure is coming from15:22
geguileoI assume the LIO end15:22
enriquetasommh15:25
enriquetasoWhat should I check with the reporter then?15:26
enriquetasowe're running out of time15:31
enriquetasoI'll left some question15:31
rosmaitasorry, i was reading through the bug again15:32
enriquetasosure, don't worry, feel free to add question on the bug report or anything15:32
enriquetaso#endmeeting15:32
opendevmeetMeeting ended Wed Jan 12 15:32:56 2022 UTC.  Information about MeetBot at http://wiki.debian.org/MeetBot . (v 0.1.4)15:32
opendevmeetMinutes:        https://meetings.opendev.org/meetings/cinder_bs/2022/cinder_bs.2022-01-12-15.03.html15:32
opendevmeetMinutes (text): https://meetings.opendev.org/meetings/cinder_bs/2022/cinder_bs.2022-01-12-15.03.txt15:32
opendevmeetLog:            https://meetings.opendev.org/meetings/cinder_bs/2022/cinder_bs.2022-01-12-15.03.log.html15:32
rosmaitathanks enriquetaso15:33
enriquetaso\o/15:37
opendevreviewMerged openstack/cinder master: Volume transfers: Remove duplicate policy check  https://review.opendev.org/c/openstack/cinder/+/82413115:57
opendevreviewMerged openstack/cinder stable/xena: Reject bad img formats for uploaded encrypted vols  https://review.opendev.org/c/openstack/cinder/+/82415815:57
opendevreviewEric Harney proposed openstack/cinder master: Tests: Split test_create_snapshot_force into two tests  https://review.opendev.org/c/openstack/cinder/+/80429316:08
*** dviroel|lunch is now known as dviroel16:11
opendevreviewBrian Rosmaita proposed openstack/cinder master: Remove block-box files  https://review.opendev.org/c/openstack/cinder/+/82447217:44
opendevreviewRajat Dhasmana proposed openstack/cinder stable/xena: Volume transfers: Remove duplicate policy check  https://review.opendev.org/c/openstack/cinder/+/82447417:46
opendevreviewMerged openstack/cinderlib master: Migrate jobs from CentOS 8 to CentOS 8 Stream  https://review.opendev.org/c/openstack/cinderlib/+/82426119:32
*** dviroel is now known as dviroel|afk20:04
opendevreviewEric Harney proposed openstack/cinder master: WIP: RBD: Attempt remove() before trash_move()  https://review.opendev.org/c/openstack/cinder/+/82372620:14
opendevreviewAlan Bishop proposed openstack/os-brick master: DNM: Add delay to troubleshoot FIPS failures  https://review.opendev.org/c/openstack/os-brick/+/82449021:55
opendevreviewAlan Bishop proposed openstack/cinder master: WIP/DNM: Add fips check jobs  https://review.opendev.org/c/openstack/cinder/+/79053522:02
opendevreviewAlan Bishop proposed openstack/cinder master: WIP/DNM: Add fips check jobs  https://review.opendev.org/c/openstack/cinder/+/79053523:34

Generated by irclog2html.py 2.17.3 by Marius Gedminas - find it at https://mg.pov.lt/irclog2html/!