| opendevreview | Kanagaraj Manickam proposed openstack/cinder master: Update huawei driver https://review.opendev.org/c/openstack/cinder/+/837662 | 06:22 |
|---|---|---|
| opendevreview | Kanagaraj Manickam proposed openstack/cinder master: Update huawei driver https://review.opendev.org/c/openstack/cinder/+/837662 | 06:30 |
| gokhani | hi 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 https://docs.openstack.org/cinder/latest/reference/support-matrix.html#operation_multi_attach | 07:34 |
| gokhani | whoami-rajat__: thanks, unfortunately I am using generic nfs driver :( | 07:40 |
| lpetrut | whoami-rajat__ hi, thanks for reviewing the os-brick volume resize patch. I've addressed your comments: https://review.opendev.org/c/openstack/os-brick/+/762776 | 07:49 |
| opendevreview | Harsh Ailani proposed openstack/cinder master: [IBM DS8000] Fixed Detach for multi-attach volumes https://review.opendev.org/c/openstack/cinder/+/837699 | 08:04 |
| *** whoami-rajat__ is now known as whoami-rajat | 08:53 | |
| whoami-rajat | lpetrut, sure, will take a look | 08:57 |
| opendevreview | Victoria Martinez de la Cruz proposed openstack/devstack-plugin-ceph master: Deploy with cephadm https://review.opendev.org/c/openstack/devstack-plugin-ceph/+/826484 | 09:37 |
| opendevreview | Victoria Martinez de la Cruz proposed openstack/devstack-plugin-ceph master: Deploy with cephadm https://review.opendev.org/c/openstack/devstack-plugin-ceph/+/826484 | 10:04 |
| bingyanh__ | hi barbieri i esee you create the cherry pick: https://review.opendev.org/c/openstack/cinder/+/834961. it is a unified operation? | 10:05 |
| opendevreview | Francesco Pantano proposed openstack/devstack-plugin-ceph master: Deploy with cephadm https://review.opendev.org/c/openstack/devstack-plugin-ceph/+/826484 | 10:22 |
| *** dviroel|out is now known as dviroel | 11:11 | |
| opendevreview | Luigi Toscano proposed openstack/devstack-plugin-ceph master: [DNM][CI] Add CEPHADM_DEPLOY flag to py3 tests https://review.opendev.org/c/openstack/devstack-plugin-ceph/+/834223 | 12:01 |
| opendevreview | Francesco Pantano proposed openstack/devstack-plugin-ceph master: Deploy with cephadm https://review.opendev.org/c/openstack/devstack-plugin-ceph/+/826484 | 12:51 |
| opendevreview | Mounika Sreeram proposed openstack/cinder master: [SVf] Resize of GMCV volumes in group https://review.opendev.org/c/openstack/cinder/+/830787 | 13:31 |
| opendevreview | kiran pawar proposed openstack/cinder master: Use trusts for cinder backup per_user mode. https://review.opendev.org/c/openstack/cinder/+/785362 | 13:48 |
| opendevreview | kiran pawar proposed openstack/cinder master: Parallelize the cinder backup process https://review.opendev.org/c/openstack/cinder/+/779233 | 13:50 |
| whoami-rajat | Cinder meeting in #openstack-meeting-alt at 1400 UTC | 13:59 |
| whoami-rajat | jungleboyj rosmaita smcginnis tosky whoami-rajat m5z e0ne geguileo eharney walshh_ jbernard sfernand enriquetaso hemna fabiooliveira yuval | 13:59 |
| geguileo | whoami-rajat: thanks :-) | 13:59 |
| whoami-rajat | np | 14:00 |
| enriquetaso | #startmeeting cinder_bs | 15:02 |
| opendevmeet | Meeting 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 http://wiki.debian.org/MeetBot. | 15:02 |
| opendevmeet | Useful Commands: #action #agreed #help #info #idea #link #topic #startvote. | 15:02 |
| opendevmeet | The meeting name has been set to 'cinder_bs' | 15:02 |
| enriquetaso | Welcome to the Cinder Bug Meeting | 15:02 |
| whoami-rajat | Hi | 15:02 |
| enriquetaso | hi | 15:02 |
| enriquetaso | Four new bugs reported in this period of two weeks. Not bad. | 15:02 |
| rosmaita | o/ | 15:02 |
| enriquetaso | You can check the full report here: | 15:02 |
| enriquetaso | #link http://lists.openstack.org/pipermail/openstack-discuss/2022-April/028171.html | 15:02 |
| enriquetaso | #topic Encryptor connect_volume not changing the symlink | 15:02 |
| enriquetaso | #link https://bugs.launchpad.net/os-brick/+bug/1967790 | 15:03 |
| enriquetaso | This is a high importance bug that we discussed last week. | 15:03 |
| enriquetaso | Feel free to review geguileo patch:: | 15:03 |
| enriquetaso | #link https://review.opendev.org/c/openstack/os-brick/+/836391 | 15:03 |
| eharney | i'll be reviewing it | 15:04 |
| enriquetaso | thanks eharney | 15:04 |
| enriquetaso | moving on.. | 15:05 |
| enriquetaso | #topic cinder-manage db sync fails due to row size too large | 15:05 |
| opendevreview | Francesco Pantano proposed openstack/devstack-plugin-ceph master: Deploy with cephadm https://review.opendev.org/c/openstack/devstack-plugin-ceph/+/826484 | 15:05 |
| enriquetaso | #link https://bugs.launchpad.net/cinder/+bug/1968746 | 15:05 |
| rosmaita | i've been thinking about this one | 15:05 |
| enriquetaso | Running 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 |
| rosmaita | probably a "won't fix" | 15:05 |
| enriquetaso | Not sure if this is a particular case or if I should close the bug. | 15:05 |
| enriquetaso | thanks rosmaita | 15:05 |
| rosmaita | i think maybe the thing to do is put a statement somewhere in our docs about what versions we test with/support | 15:06 |
| rosmaita | basically the db versions packaged with the supported operating systems | 15:06 |
| eharney | because new versions of mysql don't hit this? | 15:06 |
| rosmaita | right | 15:06 |
| eharney | makes sense to me | 15:06 |
| enriquetaso | #action: add statement of the mysql version supported | 15:07 |
| rosmaita | but, it's good to have the bug because it does bring up some issues that maybe we need to pay attention to | 15:07 |
| rosmaita | enriquetaso: assign the bug to me and i will follow up | 15:07 |
| enriquetaso | done | 15:08 |
| enriquetaso | thanks brian! | 15:08 |
| rosmaita | thanks! | 15:08 |
| enriquetaso | #topic Concurrent migration of vms with the same multiattach volume fails | 15:08 |
| enriquetaso | #link https://bugs.launchpad.net/cinder/+bug/1968645 | 15:08 |
| enriquetaso | Attaching 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 |
| enriquetaso | No fix proposed to master yet. | 15:08 |
| enriquetaso | I'm not sure if we have a tempest test for a case like this one | 15:09 |
| rosmaita | i suspect not, that's quite a scenario | 15:09 |
| rosmaita | though kind of a common scenario, i guess, when you look at it | 15:10 |
| eharney | i suspect that a fix stopping this from the Nova side might be a better short term goal than a Cinder fix, but not sure | 15:10 |
| enriquetaso | stopping the migration ? | 15:11 |
| geguileo | eharney: +1 | 15:11 |
| eharney | this happens because our state management with multiattach volumes is stretched in odd ways when using multiattach | 15:11 |
| geguileo | enriquetaso: not allowing 2 concurrent migrations that share a volume | 15:11 |
| eharney | i'm not sure we can do much about it without large API changes | 15:11 |
| tosky | enriquetaso: I confirm we don't have a tempest test for that | 15:11 |
| eharney | right, a more tactical fix in Nova like what geguileo suggests seems much easier | 15:12 |
| geguileo | we may be able to make it work with current API, but it would still be a Nova fix | 15:12 |
| enriquetaso | cool, maybe I can add a comment with all this discussion on the bug report and see what nova team thinks | 15:12 |
| enriquetaso | +1 | 15:13 |
| eharney | this should probably also be documented a known issue somewhere since it would be a problem for evacuations etc | 15:13 |
| geguileo | enriquetaso: I would literally either kick it to the nova component or add the nova component to the bug | 15:13 |
| enriquetaso | geguileo, the nova component is already added | 15:13 |
| geguileo | because I think it may just be an issue that Nova needs to wait a second and then it would work | 15:13 |
| geguileo | enriquetaso: ok | 15:14 |
| enriquetaso | #action: document this as known issue | 15:14 |
| geguileo | it may just be that there's a race condition between the 2... | 15:14 |
| geguileo | or maybe not... | 15:14 |
| geguileo | I don't remember clearly the state transitions of the volume in that case :-( | 15:14 |
| enriquetaso | excellent | 15:15 |
| enriquetaso | I'll add a comment on the bug report tho | 15:16 |
| enriquetaso | Last one | 15:16 |
| enriquetaso | #topic reimage_volume failure message action does not exist | 15:16 |
| enriquetaso | #link https://bugs.launchpad.net/cinder/+bug/1968170 | 15:16 |
| eharney | so, one note on this one | 15:16 |
| enriquetaso | sure | 15:16 |
| eharney | it's just broken code, and i found this trivially with pylint | 15:17 |
| eharney | we are missing out on finding such things because our pylint gate job is not very useful | 15:17 |
| geguileo | pylint++ | 15:17 |
| eharney | i might change it soon to make it do what it actually should do | 15:17 |
| enriquetaso | #action: update pylint job | 15:17 |
| enriquetaso | Marked as low-hanging-fruit in case interns are looking for small bugs. | 15:18 |
| eharney | the 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 messages | 15:18 |
| enriquetaso | that's bad :( | 15:18 |
| eharney | and then someone touches a file two years later and sees a bunch of failures unrelated to what they were actually doing | 15:18 |
| eharney | it's not great | 15:18 |
| enriquetaso | We have a plan to fix it then, thanks Eric | 15:19 |
| enriquetaso | #topic Open Discussion | 15:19 |
| enriquetaso | Feel free to proposed any bug here | 15:20 |
| enriquetaso | OK, nothing. See you next week! | 15:25 |
| enriquetaso | #endmeeting | 15:25 |
| opendevmeet | Meeting ended Wed Apr 13 15:25:35 2022 UTC. Information about MeetBot at http://wiki.debian.org/MeetBot . (v 0.1.4) | 15:25 |
| opendevmeet | Minutes: https://meetings.opendev.org/meetings/cinder_bs/2022/cinder_bs.2022-04-13-15.02.html | 15:25 |
| opendevmeet | Minutes (text): https://meetings.opendev.org/meetings/cinder_bs/2022/cinder_bs.2022-04-13-15.02.txt | 15:25 |
| opendevmeet | Log: https://meetings.opendev.org/meetings/cinder_bs/2022/cinder_bs.2022-04-13-15.02.log.html | 15:25 |
| rosmaita | thanks, enriquetaso | 15:25 |
| enriquetaso | \o/ | 15:26 |
| *** dviroel is now known as dviroel|lunch | 15:28 | |
| simondodsley | who is responsible for the DellEMC Unity and PowerFlex drivers, and the IDM DS8K driver? I have a patch up (https://review.opendev.org/c/openstack/cinder/+/832130) that need to pass their CIs but they don't seem to have a CI... | 15:29 |
| ganso | jungleboyj: thanks for the review! | 15:38 |
| *** dviroel|lunch is now known as dviroel | 16:35 | |
| opendevreview | kiran pawar proposed openstack/cinder master: Use trusts for cinder backup per_user mode. https://review.opendev.org/c/openstack/cinder/+/785362 | 17:40 |
| shubjero | rosmaita: 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 but | 19:03 |
| shubjero | upgrading 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 dark | 19:03 |
| shubjero | This 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 |
| shubjero | Could this be required across other openstack databases? Just cinder? That stuff is going through my head now | 19:06 |
| shubjero | Could be a ticking time bomb | 19:06 |
| opendevreview | Walt proposed openstack/cinder master: Bugfix: Account for consumed space better https://review.opendev.org/c/openstack/cinder/+/826510 | 19:29 |
| rosmaita | shubjero: thanks for your comments ... i imagine it's not just the cinder DB. I was thinking we should "fix" this via documentation | 20:02 |
| opendevreview | Walt proposed openstack/cinder master: [Ceph-iscsi] Disable multiattach https://review.opendev.org/c/openstack/cinder/+/827554 | 20:09 |
| *** dviroel is now known as dviroel|afk | 20:22 | |
| opendevreview | Walt proposed openstack/cinder-specs master: Add reporting of capacity factors https://review.opendev.org/c/openstack/cinder-specs/+/837768 | 21:17 |
| shubjero | rosmaita: 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 rows | 22:43 |
Generated by irclog2html.py 2.17.3 by Marius Gedminas - find it at https://mg.pov.lt/irclog2html/!