Wednesday, 2022-11-16

opendevreviewMasayuki Igawa proposed openstack/os-brick stable/victoria: RBD: Fix disconnect_volume for encrypted volumes  https://review.opendev.org/c/openstack/os-brick/+/86461702:03
opendevreviewManojkatari proposed openstack/cinder stable/wallaby: [PURE] support IPv6 / add parameter pure_iscsi_cidr_list  https://review.opendev.org/c/openstack/cinder/+/86418505:59
*** amoralej|off is now known as amoralej07:00
enriquetasoHello geguileo, sorry for bothering you but last week in the bug meeting we talked about  https://bugs.launchpad.net/cinder/+bug/199520410:53
enriquetasohttps://meetings.opendev.org/meetings/cinder_bs/2022/cinder_bs.2022-11-09-15.00.log.html#l-1010:53
enriquetasoand I'm still not sure the importance of the bug, would you mind to take a look a see if it's valid and makes sense?10:54
opendevreviewDmitriy Rabotyagov proposed openstack/devstack-plugin-ceph master: Use default Ubuntu Jammy ceph repos  https://review.opendev.org/c/openstack/devstack-plugin-ceph/+/86352411:06
opendevreviewDmitriy Rabotyagov proposed openstack/devstack-plugin-ceph master: Use default Ubuntu Jammy ceph repos  https://review.opendev.org/c/openstack/devstack-plugin-ceph/+/86352411:08
*** dviroel|afk is now known as dviroel11:20
*** amoralej is now known as amoralej|off11:47
geguileoenriquetaso: sure, give me 10 minutes... I'm trying to configure my IRC client (weechat) to connect to slack...13:09
opendevreviewDr. Jens Harbott proposed openstack/cinder master: WIP: Test Ubuntu 22.04 (Jammy) migration  https://review.opendev.org/c/openstack/cinder/+/86160913:11
TusharTgitehi core reviewers need a review on this https://review.opendev.org/c/openstack/cinder/+/860212  https://review.opendev.org/c/openstack/cinder/+/86022713:12
*** abhishekk is now known as akekane|home13:26
senriquegeguileo, thanks!13:27
senriqueTusharTgite, done13:32
TusharTgitesenrique: thanks13:36
opendevreviewMerged openstack/cinder master: Set backup status to error on VolumeNotFound  https://review.opendev.org/c/openstack/cinder/+/86410013:38
opendevreviewMerged openstack/cinder master: DataCore: Add note about unique CHAP storage paths  https://review.opendev.org/c/openstack/cinder/+/84986913:38
opendevreviewMerged openstack/cinder master: Update mypy to 0.981  https://review.opendev.org/c/openstack/cinder/+/85987713:38
*** amoralej|off is now known as amoralej13:46
geguileosenrique: ok, check, that's the expected behavior, which doesn't mean that it's not a problem...13:54
geguileosenrique: it's the whole issue of each scheduler keeping an updated value only locally13:54
whoami-rajatCinder meeting in #openstack-meeting-alt at 1400 UTC13:59
whoami-rajatjungleboyj rosmaita smcginnis tosky whoami-rajat m5z e0ne geguileo eharney walshh_ jbernard sfernand enriquetaso hemna fabiooliveira yuval tobias-urdin adiare13:59
enriquetasothanks13:59
whoami-rajatnp14:00
enriquetasohi15:03
enriquetasocinder bug meeting15:03
enriquetasovery quick15:03
enriquetaso#startmeeting cinder_bs15:03
opendevmeetMeeting started Wed Nov 16 15:03:42 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
enriquetasoganso, whoami-rajat 15:03
enriquetasodo you want to discuss the topic now?15:04
enriquetasoIt will be recorded on the bug meeting15:04
gansoyea very quick15:04
gansoit is just an FYI on https://review.opendev.org/c/openstack/cinder/+/81268515:04
whoami-rajatI've no issues now or later15:04
enriquetaso#topic Filter reserved image properties15:04
gansowe had a lot of interesting discussion on that patch, at some point we intended to have a discussion here15:04
enriquetaso#link https://review.opendev.org/c/openstack/cinder/+/81268515:04
gansobut it seems that yesterday we ended up agreeing on the approach15:05
rosmaitaganso: what's your real name? (are you Rodrigo or Rafael?)15:05
gansoso it might need not a discussion15:05
gansorosmaita: I'm Rodrigo15:05
rosmaitaok, thanks15:05
gansoso I suppose Rafael is changing the patch to have the metadata removed only on upload-back-to-glance15:06
gansoif anyone disagrees or would like to discuss it further, we can do so, otherwise, might not need any further discussing15:06
whoami-rajatI also kind of agree on that approach15:06
whoami-rajatif glance doesn't want to supply some metadata to the consumer (nova, cinder) then glance should not supply it in the first place15:07
whoami-rajatif glance is supplying it then it makes sense to keep it and only filter out when it creates issues15:07
rosmaitanot surprisingly, i agree with that15:08
whoami-rajatthat provides us flexibility, since once metadata is removed, we can't recover it but if it stays, we can always filter it later15:09
whoami-rajatso +1 to rosmaita's suggestion15:10
whoami-rajatganso, anything else on this? don't want to block enriquetaso's meeting for long :)15:11
gansowhoami-rajat: nothing else =)15:11
gansothanks enriquetaso for this meeting15:11
whoami-rajatgreat, thanks15:11
enriquetasocool 15:11
whoami-rajatthanks enriquetaso !15:11
enriquetasoOK, Only one bug for today's meeting. List of bugs:15:11
enriquetasono worries 15:12
enriquetaso#link https://lists.openstack.org/pipermail/openstack-discuss/2022-November/031212.html15:12
enriquetaso#topic Cinderlib Gate broken in Zed15:12
enriquetaso#link https://bugs.launchpad.net/cinderlib/+bug/199673815:12
enriquetasoAs discussed last week at the cinder meeting. I've opened a bug for the cinderlib gate failures. 15:12
enriquetasowhoami-rajat, rosmaita 15:12
enriquetaso^15:12
enriquetasoLet me know if the description is wrong so I can update it. 15:12
rosmaitalooks good, thank you for filing the bug15:12
whoami-rajati don't see "horribly broken" in title but it should be fine15:13
enriquetasolol15:13
enriquetasoi can update the title :P15:13
rosmaitaenriquetaso: you should!15:13
whoami-rajatlooks good, thanks for reporting it15:13
enriquetaso#action: (enriquetaso) update the titel15:13
enriquetasotitle*15:13
enriquetasoOK, moving on15:13
rosmaitaok, i will assign it to myself with the goal of reporting status at the festival of bugs on friday15:13
enriquetasothanks15:14
whoami-rajatrosmaita++15:14
enriquetaso#topic "[OpenStack Yoga] Creating a VM fails when stopping only one rabbitmq.15:14
enriquetaso#link https://bugs.launchpad.net/cinder/+bug/199025715:14
enriquetasoI've removed cinder from that bug15:14
enriquetasoit doesn't look like a cinder bug, but if you think it should, please let me know15:15
enriquetaso#topic open discussion 15:15
enriquetasoany bugs to discuss ?15:15
rosmaitacircling back to that cinderclient snapshot-create fix, i think we should backport it as far as xena15:15
rosmaitathat's bug https://bugs.launchpad.net/python-cinderclient/+bug/199588315:15
whoami-rajatwe need more info on 1990257, if they're creating the instance BFV then it might be a cinder problem15:16
rosmaitai think we need to un-target from Wallaby, and add Zed, Yoga, and Xena instead15:16
enriquetasoture15:16
enriquetasotrue 15:16
whoami-rajati will add a comment there15:17
enriquetaso#action(enriquetaso): un-target wallaby and add zed, yoga and xena for bug199025715:17
enriquetasothanks whoami-rajat 15:17
enriquetasothanks for updating rosmaita 15:18
rosmaitanp, i actually brought it up becasue i didn't know if you've used the "target to series" functionality in launchpad?15:19
enriquetasoi'm trying to use them yes15:19
rosmaitacool, we have an action item for jon bernard on this15:20
enriquetasojbernard :)15:20
rosmaitait's showing that you can nominate kilo through wallaby ... x, y, z aren't available15:20
enriquetaso#action: add a comment in 1990257 asking if they are using intance BFV15:20
enriquetaso:/15:21
whoami-rajatenriquetaso, added a comment15:21
rosmaitaoh, that may be because you already did the targeting15:21
enriquetasoi've updated it rosmaita 15:21
enriquetasoyes, sorry15:21
rosmaitanot sure why it picked Zed as the tracking branch, though15:21
enriquetasoyes.. that's wrong15:22
enriquetasoi'll let you do it, maybe multiple updates from diff people confuses launchpad 15:22
rosmaitano, it's a launchpad thing, nothing you did15:22
enriquetasothanks!15:23
rosmaitathe 2023.1 series isn't defined for cinderclient yet15:23
rosmaitaand Zed is showing as the focus of development ... so that's why the bug is tracking zed15:23
enriquetasothe fix is in master tho15:24
rosmaitayeah, but launchpad only knows about the releases we tell it about15:24
enriquetasou.u OK15:25
enriquetasoDo we have any other bug to discuss?15:25
enriquetasoOK, closing the meeting then :)15:27
enriquetasothank you!!15:27
whoami-rajatthanks enriquetaso 15:27
enriquetaso#endmeeting15:27
opendevmeetMeeting ended Wed Nov 16 15:27:31 2022 UTC.  Information about MeetBot at http://wiki.debian.org/MeetBot . (v 0.1.4)15:27
opendevmeetMinutes:        https://meetings.opendev.org/meetings/cinder_bs/2022/cinder_bs.2022-11-16-15.03.html15:27
opendevmeetMinutes (text): https://meetings.opendev.org/meetings/cinder_bs/2022/cinder_bs.2022-11-16-15.03.txt15:27
opendevmeetLog:            https://meetings.opendev.org/meetings/cinder_bs/2022/cinder_bs.2022-11-16-15.03.log.html15:27
enriquetasothank whoami-rajat 15:27
sfv880Hello reviewers, Sorry to bother you, I replied and fixed on https://review.opendev.org/c/openstack/cinder/+/850320 and https://review.opendev.org/c/openstack/cinder/+/850321 - could you please review? Thank you so much!15:29
rosmaitasfv880: they both add more lines of tests than lines of code, which is a really good thing!15:31
sfv880rosmaita: Thank you!15:32
*** dviroel is now known as dviroel|lunch16:32
*** dviroel|lunch is now known as dviroel17:27
*** amoralej is now known as amoralej|off17:46
*** dviroel is now known as dviroel|afk19:34
opendevreviewNahim Alves de Souza proposed openstack/cinder master: Netapp ONTAP: Fix volume attachment with multiple HA pairs  https://review.opendev.org/c/openstack/cinder/+/86371121:03
opendevreviewMerged openstack/devstack-plugin-ceph master: Use default Ubuntu Jammy ceph repos  https://review.opendev.org/c/openstack/devstack-plugin-ceph/+/86352421:45

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