Wednesday, 2022-04-13

opendevreviewKanagaraj Manickam proposed openstack/cinder master: Update huawei driver
opendevreviewKanagaraj Manickam proposed openstack/cinder master: Update huawei driver
gokhanihi folks, can I use multiattach with nfs driver which is installed on linux server? I tried it I get no weighed backend error. 07:23
whoami-rajat__gokhani, hi, AFAICS multiattach is not supported for generic nfs driver, it is however possible in netapp nfs driver
gokhaniwhoami-rajat__: thanks, unfortunately I am using generic nfs driver :( 07:40
lpetrutwhoami-rajat__ hi, thanks for reviewing the os-brick volume resize patch. I've addressed your comments:
opendevreviewHarsh Ailani proposed openstack/cinder master: [IBM DS8000] Fixed Detach for multi-attach volumes
*** whoami-rajat__ is now known as whoami-rajat08:53
whoami-rajatlpetrut, sure, will take a look08:57
opendevreviewVictoria Martinez de la Cruz proposed openstack/devstack-plugin-ceph master: Deploy with cephadm
opendevreviewVictoria Martinez de la Cruz proposed openstack/devstack-plugin-ceph master: Deploy with cephadm
bingyanh__hi barbieri i esee you create the cherry pick: it is a unified operation?10:05
opendevreviewFrancesco Pantano proposed openstack/devstack-plugin-ceph master: Deploy with cephadm
*** dviroel|out is now known as dviroel11:11
opendevreviewLuigi Toscano proposed openstack/devstack-plugin-ceph master: [DNM][CI] Add CEPHADM_DEPLOY flag to py3 tests
opendevreviewFrancesco Pantano proposed openstack/devstack-plugin-ceph master: Deploy with cephadm
opendevreviewMounika Sreeram proposed openstack/cinder master: [SVf] Resize of GMCV volumes in group
opendevreviewkiran pawar proposed openstack/cinder master: Use trusts for cinder backup per_user mode.
opendevreviewkiran pawar proposed openstack/cinder master: Parallelize the cinder backup process
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 yuval13:59
geguileowhoami-rajat: thanks :-)13:59
enriquetaso#startmeeting cinder_bs15:02
opendevmeetMeeting started Wed Apr 13 15:02:21 2022 UTC and is due to finish in 60 minutes.  The chair is enriquetaso. Information about MeetBot at
opendevmeetUseful Commands: #action #agreed #help #info #idea #link #topic #startvote.15:02
opendevmeetThe meeting name has been set to 'cinder_bs'15:02
enriquetasoWelcome to the Cinder Bug Meeting15:02
enriquetasoFour new bugs reported in this period of two weeks. Not bad.  15:02
enriquetasoYou can check the full report here:15:02
enriquetaso#link 15:02
enriquetaso#topic Encryptor connect_volume not changing the symlink15:02
enriquetaso#link 15:03
enriquetasoThis is a high importance bug that we discussed last week.15:03
enriquetasoFeel free to review geguileo patch::15:03
enriquetaso#link 15:03
eharneyi'll be reviewing it15:04
enriquetasothanks eharney 15:04
enriquetasomoving on..15:05
enriquetaso#topic cinder-manage db sync fails due to row size too large15:05
opendevreviewFrancesco Pantano proposed openstack/devstack-plugin-ceph master: Deploy with cephadm
enriquetaso#link 15:05
rosmaitai've been thinking about this one15:05
enriquetasoRunning the cinder-manage db sync to upgrade to Xena from Wallaby resulted in error. The reporter has cluster since Kilo and  fix the problem manually changing row_format from 'Compact' to 'Dynamic' by running: ALTER TABLE volumes ROW_FORMAT=DYNAMIC; 15:05
rosmaitaprobably a "won't fix"15:05
enriquetasoNot sure if this is a particular case or if I should close the bug.15:05
enriquetasothanks rosmaita 15:05
rosmaitai think maybe the thing to do is put a statement somewhere in our docs about what versions we test with/support15:06
rosmaitabasically the db versions packaged with the supported operating systems15:06
eharneybecause new versions of mysql don't hit this?15:06
eharneymakes sense to me15:06
enriquetaso#action: add statement of the mysql version supported 15:07
rosmaitabut, it's good to have the bug because it does bring up some issues that maybe we need to pay attention to15:07
rosmaitaenriquetaso: assign the bug to me and i will follow up15:07
enriquetasothanks brian!15:08
enriquetaso#topic Concurrent migration of vms with the same multiattach volume fails15:08
enriquetaso#link 15:08
enriquetasoAttaching one volume to multiple vms and trying to migrate all the vms at the same time fails. The report mentioned that the reason for the failure of vm migration is that the multiattach volume status changes to attaching after the vm migration process. At this time, when another vm migrates, it is judged that the volume status is attaching, which leads to the execution of attachment_create fail.15:08
enriquetasoNo fix proposed to master yet.15:08
enriquetasoI'm not sure if we have a tempest test for a case like this one15:09
rosmaitai suspect not, that's quite a scenario15:09
rosmaitathough kind of a common scenario, i guess, when you look at it15:10
eharneyi suspect that a fix stopping this from the Nova side might be a better short term goal than a Cinder fix, but not sure15:10
enriquetaso stopping the migration ?15:11
geguileoeharney: +115:11
eharneythis happens because our state management with multiattach volumes is stretched in odd ways when using multiattach15:11
geguileoenriquetaso: not allowing 2 concurrent migrations that share a volume15:11
eharneyi'm not sure we can do much about it without large API changes15:11
toskyenriquetaso: I confirm we don't have a tempest test for that15:11
eharneyright, a more tactical fix in Nova like what geguileo suggests seems much easier15:12
geguileowe may be able to make it work with current API, but it would still be a Nova fix15:12
enriquetasocool, maybe I can add a comment with all this discussion on the bug report and see what nova team thinks 15:12
eharneythis should probably also be documented a known issue somewhere since it would be a problem for evacuations etc15:13
geguileoenriquetaso: I would literally either kick it to the nova component or add the nova component to the bug15:13
enriquetasogeguileo, the nova component is already added15:13
geguileobecause I think it may just be an issue that Nova needs to wait a second and then it would work15:13
geguileoenriquetaso: ok15:14
enriquetaso#action: document this as known issue 15:14
geguileoit may just be that there's a race condition between the 2...15:14
geguileoor maybe not...15:14
geguileoI don't remember clearly the state transitions of the volume in that case  :-(15:14
enriquetasoexcellent 15:15
enriquetasoI'll add a comment on the bug report  tho15:16
enriquetasoLast one15:16
enriquetaso#topic reimage_volume failure message action does not exist15:16
eharneyso, one note on this one15:16
eharneyit's just broken code, and i found this trivially with pylint15:17
eharneywe are missing out on finding such things because our pylint gate job is not very useful15:17
eharneyi might change it soon to make it do what it actually should do15:17
enriquetaso#action: update pylint job15:17
enriquetasoMarked as low-hanging-fruit in case interns are looking for small bugs.  15:18
eharneythe behavior of it only looking at files that your patch touches is just confusing to people, and it means that we merge broken things and then never see the messages15:18
enriquetasothat's bad :(15:18
eharneyand then someone touches a file two years later and sees a bunch of failures unrelated to what they were actually doing15:18
eharneyit's not great15:18
enriquetasoWe have a plan to fix it then, thanks Eric15:19
enriquetaso#topic Open Discussion15:19
enriquetasoFeel free to proposed any bug here15:20
enriquetasoOK, nothing. See you next week!15:25
opendevmeetMeeting ended Wed Apr 13 15:25:35 2022 UTC.  Information about MeetBot at . (v 0.1.4)15:25
opendevmeetMinutes (text):
rosmaitathanks, enriquetaso15:25
*** dviroel is now known as dviroel|lunch15:28
simondodsleywho is responsible for the DellEMC Unity and PowerFlex drivers, and the IDM DS8K driver? I have a patch up ( that need to pass their CIs but they don't seem to have a CI...15:29
gansojungleboyj: thanks for the review!15:38
*** dviroel|lunch is now known as dviroel16:35
opendevreviewkiran pawar proposed openstack/cinder master: Use trusts for cinder backup per_user mode.
shubjerorosmaita: Hey, thanks for looking in to that cinder-manage db sync bug. I saw the bug notes from the meeting and I think some things need to be a little bit clarified. Ubuntu 20.04's default version of mariadb-server is 10.3 which is the version I am running. A new greenfield deployment would not have issues because table row defaults to Dynamic instead of compact but19:03
shubjeroupgrading versions of openstack and mariadb does not retroactively upgrade table rows to dynamic so the operators who have upgraded over time without explicit instructions to update this stuff may be left in the dark19:03
shubjeroThis detail about table rows makes me wonder if there are other's that need to be changed from Compact to Dynamic. I'm not super strong with db admin stuff so I don't know the ramifications of leaving well enough alone or proactively converting everything to Dynamic.19:05
shubjeroCould this be required across other openstack databases? Just cinder? That stuff is going through my head now19:06
shubjeroCould be a ticking time bomb19:06
opendevreviewWalt proposed openstack/cinder master: Bugfix: Account for consumed space better
rosmaitashubjero: thanks for your comments ... i imagine it's not just the cinder DB.  I was thinking we should "fix" this via documentation20:02
opendevreviewWalt proposed openstack/cinder master: [Ceph-iscsi]  Disable multiattach
*** dviroel is now known as dviroel|afk20:22
opendevreviewWalt proposed openstack/cinder-specs master: Add reporting of capacity factors
shubjerorosmaita: Yeah agreed, it's likely all of them but it might not be an issue unless various openstack projects are storing longer pieces of data in certain table rows22:43

Generated by 2.17.3 by Marius Gedminas - find it at!