| *** mhen_ is now known as mhen | 02:27 | |
| jbernard | #startmeeting cinder | 14:01 |
|---|---|---|
| opendevmeet | Meeting started Wed Nov 20 14:01:15 2024 UTC and is due to finish in 60 minutes. The chair is jbernard. Information about MeetBot at http://wiki.debian.org/MeetBot. | 14:01 |
| opendevmeet | Useful Commands: #action #agreed #help #info #idea #link #topic #startvote. | 14:01 |
| opendevmeet | The meeting name has been set to 'cinder' | 14:01 |
| sp-bmilanov | o/ | 14:01 |
| tosky | o/ | 14:01 |
| akawai | o/ | 14:01 |
| whoami-rajat | Hi | 14:01 |
| jhorstmann | o/ | 14:01 |
| rosmaita | o/ | 14:01 |
| jbernard | o/ | 14:02 |
| msaravan | Hi | 14:04 |
| manudinesh_ | Hi | 14:05 |
| jbernard | hello everyone | 14:06 |
| jbernard | #topic annoucements | 14:06 |
| jbernard | i don't really have any :) | 14:06 |
| jbernard | it's the week after M1 | 14:07 |
| jbernard | i did update the wiki to include our PTG summary for epoxy | 14:07 |
| jbernard | otherwise, reviews, communication, emails, etc | 14:07 |
| jbernard | let's pick up where we left off, with sp-bmilanov | 14:08 |
| jbernard | #topic multi-transport protocol volume driver support | 14:08 |
| jbernard | here's the link for our working etherpad | 14:09 |
| jbernard | #link https://etherpad.opendev.org/p/cinder-epoxy-meetings | 14:09 |
| sp-bmilanov | hi, the main use case for this would be live instance migration between transport protocols.. currently, it cannot be done with the driver-per-protocol approach, I think | 14:09 |
| jbernard | simondodsley: your last message from last week's meeting was a suggestion to split the driver up | 14:10 |
| jbernard | whoami-rajat: curious your stance | 14:10 |
| jbernard | log from last week: https://meetings.opendev.org/meetings/cinder/2024/cinder.2024-11-13-14.01.log.html | 14:10 |
| jbernard | sp-bmilanov: have you had a chance to evaluate the nova flavors suggestion? | 14:11 |
| sp-bmilanov | using nova flavors does not help with the use case, no | 14:12 |
| jbernard | whoami-rajat: do you still object to this approach? | 14:14 |
| whoami-rajat | i didn't get much time to look into this but what i was curious about is with multiple backends set up (each having it's own cinder-volume service), how do we determine which service/driver to call for the attachment update call | 14:15 |
| jbernard | i think you could call either one, as they both support all available protocols | 14:16 |
| whoami-rajat | what i mean is, if we setup one backend for netapp iscsi and one for netapp FC, and we call self.driver.initialize_connection from manager, how do we determine which service gets called | 14:17 |
| whoami-rajat | that i wanted to figure out | 14:17 |
| whoami-rajat | probably i can take it as an action item and update my comment on the patch | 14:18 |
| jbernard | ok | 14:19 |
| jbernard | #topic antelope has become unmaintained (2023.1) and the stable branch is no longer present | 14:20 |
| jbernard | im not sure there's much to add, just a heads up | 14:21 |
| sp-bmilanov | jbernard: can I have 2 more minutes please | 14:21 |
| tosky | about that: after the dropping of stable/2023.1 (now unmaintained), we need to merge https://review.opendev.org/c/openstack/cinder-tempest-plugin/+/934275 to unblock cinder-tempest-plugin | 14:21 |
| tosky | jbernard: that's urgent ^^ | 14:22 |
| sp-bmilanov | right, I can ping whoami-rajat in #openstack-cinder later | 14:22 |
| jbernard | there is also https://review.opendev.org/c/openstack/cinder/+/935378 | 14:23 |
| whoami-rajat | sp-bmilanov, yeah sorry for holding this, will try to resolve the query by this week | 14:23 |
| jbernard | tosky: will take a look after the meeting | 14:23 |
| jbernard | #topic rally ci job | 14:24 |
| jbernard | https://zuul.opendev.org/t/openstack/builds?job_name=cinder-rally-task&project=openstack%2Fcinder&branch=master&result=SUCCESS&skip=0 | 14:24 |
| jbernard | jens brought up that the rally job has been failing for some time | 14:24 |
| jbernard | do we want to fix this, or let it go to save resources? | 14:25 |
| whoami-rajat | with the link above, i only see the job passing | 14:27 |
| whoami-rajat | okay i removed the SUCCESS filter | 14:27 |
| tosky | yep | 14:27 |
| jbernard | 2024-11-18 15:57:21 frickler last success for cinder-rally-task was half a year ago, if nobody wants to or is able to fix it, maybe stop running it and wasting CI | 14:27 |
| jbernard | resources? | 14:28 |
| whoami-rajat | checked one of the test, i think it's related to cinderv3 vs block-storage endpoint change | 14:28 |
| whoami-rajat | Unexpected error: Validation error: There is no 'cinderv3' service in your environment | 14:28 |
| whoami-rajat | #link https://68dc1c67d2d925924853-d78b0c94e26e635eac828273888f290f.ssl.cf2.rackcdn.com/935615/2/check/cinder-rally-task/e5228ed/results/report.html#/CinderVolumes.create_and_attach_volume | 14:28 |
| whoami-rajat | maybe a change here is needed | 14:29 |
| whoami-rajat | #link https://github.com/openstack/rally-openstack/blob/master/rally-jobs/cinder.yaml#L40 | 14:29 |
| jbernard | that might do it | 14:30 |
| whoami-rajat | do we know what kind of coverage does the rally job provide us? | 14:30 |
| jbernard | so we /do/ want to keep rally it sounds like | 14:30 |
| whoami-rajat | just to see if it's not overlapping with our existing tests | 14:30 |
| jbernard | or maybe that's undecided | 14:30 |
| jbernard | im not sure, im wondering the same thing - how much of it overlaps with tempest | 14:30 |
| tosky | it should be more performance testing, which tempest doesn't cover | 14:30 |
| tosky | it can also run tempest tests (or it used to, I haven't touched it in ages), but I'm not sure it's used for that purpose inside that job | 14:31 |
| jbernard | ahh, i see | 14:31 |
| whoami-rajat | tosky, do you mean concurrency testing? like creating 5 volumes simultaneously | 14:31 |
| tosky | whoami-rajat: in some specific deadline | 14:32 |
| whoami-rajat | okay, i remember working on something similar here | 14:32 |
| whoami-rajat | https://github.com/openstack/cinder-tempest-plugin/blob/master/cinder_tempest_plugin/api/volume/test_multiple_volume_from_resource.py | 14:32 |
| whoami-rajat | but it would be worth checking the current coverage of rally to determine if we need the job | 14:33 |
| tosky | whoami-rajat: but your test didn't consider the execution time, which is iirc what rally is meant for | 14:34 |
| tosky | agreed that we should check what we are really testing | 14:34 |
| whoami-rajat | i see your point | 14:34 |
| jbernard | ok, i will post a patch to rally-openstack to s/cinderv3/block-storage | 14:37 |
| jbernard | hopefully that will unblock things | 14:37 |
| tosky | not sure rally-openstack is branched (probably it is), that's probably the only thing to check | 14:38 |
| jbernard | tosky: good to note | 14:38 |
| jbernard | tosky: thanks | 14:38 |
| jbernard | #topic upgrade db schema issues | 14:39 |
| jbernard | zigo noticed a db migration failure when upgrading to bobcat or caracal | 14:39 |
| jbernard | not sure if anyone has anything to add, i just wanted to raise it - we'll need to push some kind of fix for that | 14:41 |
| jbernard | there is a bug (with patch): https://bugs.launchpad.net/cinder/+bug/2070475 | 14:41 |
| jbernard | but the patch doesn't cover all cases, according to zigo's testing | 14:42 |
| jbernard | where non-deleted volumes can still have use_quota=NULL | 14:42 |
| jbernard | something to be aware of | 14:42 |
| jbernard | we can open things up for discussion now, there are review requests but I won't list them all here. If you have review cycles, please consider those | 14:44 |
| jbernard | #topic open discussion | 14:44 |
| manudinesh | Hi All, can we take up this one: https://bugs.launchpad.net/cinder/+bug/2084117 | 14:45 |
| manudinesh | We see error while trying to deploy vSCSI based VM [ It tries to create InitiatorGroup. We were told vSCSI based VMs should use underlying host's IG mapping]...Can someone clarrify...Thanks | 14:47 |
| jbernard | this looks like a netapp connection issue, is anyone fom NetApp around today? | 14:47 |
| manudinesh | This is in NetApp driver | 14:47 |
| whoami-rajat | sp-bmilanov, hey, i was able to get my answer, so i had a proposal in mind to create multiple backends (one for each protocol) and use them depending on what the compute node supports but looks like based on the host/cluster name, we decide which cinder-volume service we will redirect the rpc call to | 14:48 |
| manudinesh | Can you please help me with NetApp point-of-contact mail-id...I will try to reachout directly...Thanks | 14:48 |
| whoami-rajat | basically if the volume was created in an iSCSI backend, all volume operations will redirect to the same backend | 14:48 |
| whoami-rajat | so even though i don't like the idea of one backend class supporting all protocols, maybe we can make this exception for the storpool driver | 14:49 |
| whoami-rajat | since we don't have any mechanism to dynamically redirect calls based on the storage protocol in the connector properties | 14:49 |
| jbernard | whoami-rajat: do you know if we have a up-to-date contact list for drivers? | 14:49 |
| rosmaita | can probably just see who's pushed patches recently | 14:50 |
| whoami-rajat | jbernard, i don't think it has been updated in a while but msaravan is the go to person for netapp issues | 14:50 |
| rosmaita | https://review.opendev.org/q/project:openstack/cinder+dir:cinder/volume/drivers/netapp | 14:50 |
| msaravan | Sorry @manudinesh : I am looking at the bug.. | 14:51 |
| manudinesh | sure msaravan..Thanks | 14:51 |
| sp-bmilanov | whoami-rajat: ack | 14:51 |
| msaravan | Can we sync up sometime this week or early next week.. we understood the igroup issue, need some more information | 14:52 |
| manudinesh | sure, we can ..Thanks | 14:52 |
| whoami-rajat | sp-bmilanov, can you help me with your patch link, will update my comments | 14:52 |
| jhorstmann | I have pushed a first version of the dm-clone driver spec: https://review.opendev.org/c/openstack/cinder-specs/+/935347 It is still missing the specifics about snapshots, but some feedback would be welcome before I make things more complicated by adding snapshots | 14:53 |
| sp-bmilanov | whoami-rajat: yep, https://review.opendev.org/c/openstack/cinder/+/847536 | 14:53 |
| jbernard | jhorstmann: i added it to the etherpad, will take a look, thanks | 14:54 |
| sp-bmilanov | whoami-rajat: I will also namespace the newly-added options, there's still a comment for that | 14:55 |
| whoami-rajat | jbernard, the spec freeze is M2 right? | 14:55 |
| jbernard | whoami-rajat: that's what i was thinking | 14:55 |
| jbernard | whoami-rajat: but i haven't sumitted it to be official yet | 14:56 |
| whoami-rajat | so epoxy-2 is Jan 9 | 14:56 |
| jbernard | M2 is first week of Jan | 14:56 |
| whoami-rajat | #link https://releases.openstack.org/epoxy/schedule.html#e-2 | 14:56 |
| jbernard | https://releases.openstack.org/epoxy/schedule.html | 14:56 |
| whoami-rajat | we have time but a lot of folks will be out next month | 14:56 |
| whoami-rajat | would be useful to have some feedback on jhorstmann spec before end of the year break | 14:57 |
| jbernard | jhorstmann: you may want to start on the snapshot details | 14:57 |
| whoami-rajat | though, i will be around | 14:57 |
| jhorstmann | M2 is also the new driver deadline, right? | 14:57 |
| whoami-rajat | jhorstmann, yeah right | 14:57 |
| jbernard | hmm, it's technically up to us | 14:57 |
| whoami-rajat | :D | 14:58 |
| whoami-rajat | Cinder team makes exceptions for dedicated contributors for sure | 14:58 |
| jbernard | let's see how the spec shapes up | 14:58 |
| jhorstmann | jbernard, I will start with the snapshot part | 14:58 |
| jbernard | if things look okay, we can make some adjustments | 14:59 |
| jbernard | k, we're nearly out of time | 14:59 |
| jbernard | last call | 14:59 |
| jbernard | thanks everyone! | 15:00 |
| jbernard | #endmeeting | 15:00 |
| opendevmeet | Meeting ended Wed Nov 20 15:00:05 2024 UTC. Information about MeetBot at http://wiki.debian.org/MeetBot . (v 0.1.4) | 15:00 |
| opendevmeet | Minutes: https://meetings.opendev.org/meetings/cinder/2024/cinder.2024-11-20-14.01.html | 15:00 |
| opendevmeet | Minutes (text): https://meetings.opendev.org/meetings/cinder/2024/cinder.2024-11-20-14.01.txt | 15:00 |
| opendevmeet | Log: https://meetings.opendev.org/meetings/cinder/2024/cinder.2024-11-20-14.01.log.html | 15:00 |
| sp-bmilanov | thank you! | 15:00 |
| whoami-rajat | thanks everyone! | 15:00 |
| zigo | jbernard: My patch is ugly, but works: https://salsa.debian.org/openstack-team/services/cinder/-/blob/debian/dalmatian/debian/patches/fix-db-schema-migration.patch?ref_type=heads | 15:15 |
Generated by irclog2html.py 2.17.3 by Marius Gedminas - find it at https://mg.pov.lt/irclog2html/!