| opendevreview | Alexander Deiter proposed openstack/cinder master: Fix Infinidat driver multi-attach feature https://review.opendev.org/c/openstack/cinder/+/850321 | 08:11 |
|---|---|---|
| opendevreview | Gorka Eguileor proposed openstack/cinder master: Tests: Fix NFS tests https://review.opendev.org/c/openstack/cinder/+/854269 | 08:41 |
| opendevreview | Gorka Eguileor proposed openstack/cinder master: Tests: Fix cmd tests https://review.opendev.org/c/openstack/cinder/+/854270 | 08:41 |
| opendevreview | Gorka Eguileor proposed openstack/cinder master: Tests: Fix PowerMax tests https://review.opendev.org/c/openstack/cinder/+/854271 | 08:41 |
| opendevreview | Gorka Eguileor proposed openstack/cinder master: Tests: Fix versions view https://review.opendev.org/c/openstack/cinder/+/854272 | 08:41 |
| opendevreview | Gorka Eguileor proposed openstack/cinder master: Tests: Randomize tests https://review.opendev.org/c/openstack/cinder/+/854273 | 08:41 |
| opendevreview | Gorka Eguileor proposed openstack/cinder master: Ceph backup: Close source file https://review.opendev.org/c/openstack/cinder/+/854274 | 08:41 |
| opendevreview | Gorka Eguileor proposed openstack/cinder master: RBD: Ensure that everything runs in native threads https://review.opendev.org/c/openstack/cinder/+/854275 | 08:41 |
| geguileo | whoami-rajat: we may want to get this fix in as well https://review.opendev.org/c/openstack/os-brick/+/854281 | 08:41 |
| geguileo | whoami-rajat: it helps with cinder unit tests not returning to the shell | 08:42 |
| geguileo | whoami-rajat: I explain it in the other half of the solution (Cinder patch): https://review.opendev.org/c/openstack/cinder/+/854274/ | 08:42 |
| whoami-rajat | geguileo, ack, will take a look | 08:43 |
| whoami-rajat | geguileo, can you update your patch with Brian's suggestion and we can get it in quickly https://review.opendev.org/c/openstack/os-brick/+/836057 | 08:43 |
| opendevreview | Gorka Eguileor proposed openstack/os-brick master: NVMe-oF: Get system uuid in privsep https://review.opendev.org/c/openstack/os-brick/+/836057 | 08:50 |
| geguileo | whoami-rajat: done | 08:50 |
| opendevreview | Gorka Eguileor proposed openstack/cinder master: RBD: Ensure that everything runs in native threads https://review.opendev.org/c/openstack/cinder/+/854275 | 09:02 |
| opendevreview | Gorka Eguileor proposed openstack/cinder master: Support os-brick specific lock_path https://review.opendev.org/c/openstack/cinder/+/849325 | 09:09 |
| geguileo | rosmaita: I have updated the os-brick lock paths patches: ^ cinder and nova https://review.opendev.org/c/openstack/nova/+/849328 | 09:10 |
| opendevreview | Gorka Eguileor proposed openstack/cinder master: Deadlock prevention support in synchronize https://review.opendev.org/c/openstack/cinder/+/848899 | 09:22 |
| opendevreview | Gorka Eguileor proposed openstack/cinder master: PowerMax: Fix deadlock moving SGs https://review.opendev.org/c/openstack/cinder/+/848900 | 09:22 |
| opendevreview | Gorka Eguileor proposed openstack/cinder master: Fix leftover devices when copying data https://review.opendev.org/c/openstack/cinder/+/731261 | 10:00 |
| *** janders2 is now known as janders | 11:06 | |
| *** manpreetk_ is now known as manpreetk | 11:06 | |
| *** open10k8s_ is now known as open10k8s | 11:06 | |
| *** maaritamm_ is now known as maaritamm | 11:06 | |
| *** jungleboyj_ is now known as jungleboyj | 11:06 | |
| *** simondodsley_ is now known as simondodsley | 11:06 | |
| *** PrinzElvis_ is now known as PrinzElvis | 11:06 | |
| *** vkmc_ is now known as vkmc | 11:06 | |
| *** TheJulia_ is now known as TheJulia | 11:06 | |
| *** sfinucan is now known as stephenfin | 11:06 | |
| *** arne_wiebalck_ is now known as arne_wiebalck | 11:24 | |
| *** dviroel is now known as dviroel|rover | 11:26 | |
| opendevreview | Rajat Dhasmana proposed openstack/cinder stable/train: doc: clarify usage of use_multipath_for_image_xfer parameter https://review.opendev.org/c/openstack/cinder/+/854142 | 12:05 |
| opendevreview | Gorka Eguileor proposed openstack/cinder master: Expose group in Volume OVO https://review.opendev.org/c/openstack/cinder/+/779578 | 12:27 |
| opendevreview | Gorka Eguileor proposed openstack/cinder master: 3PAR: Fix extend error for online copy https://review.opendev.org/c/openstack/cinder/+/756709 | 13:30 |
| opendevreview | Gorka Eguileor proposed openstack/cinder master: 3PAR: Allow disabling online copy for cloning https://review.opendev.org/c/openstack/cinder/+/756710 | 13:30 |
| opendevreview | Gorka Eguileor proposed openstack/cinder master: 3PAR: Fix race in concurrent attachments https://review.opendev.org/c/openstack/cinder/+/756711 | 13:30 |
| opendevreview | Gorka Eguileor proposed openstack/cinder master: Deadlock prevention support in synchronize https://review.opendev.org/c/openstack/cinder/+/848899 | 13:49 |
| opendevreview | Gorka Eguileor proposed openstack/cinder master: PowerMax: Fix deadlock moving SGs https://review.opendev.org/c/openstack/cinder/+/848900 | 13:49 |
| whoami-rajat | Cinder meeting in #openstack-meeting-alt at 1400 UTC | 14:00 |
| whoami-rajat | jungleboyj rosmaita smcginnis tosky whoami-rajat m5z e0ne geguileo eharney walshh_ jbernard sfernand enriquetaso hemna fabiooliveira yuval tobias-urdin adiare | 14:00 |
| geguileo | whoami-rajat: thanks | 14:00 |
| whoami-rajat | np | 14:01 |
| geguileo | eharney: my very first patch blocked by mypy!!! :-) https://review.opendev.org/c/openstack/cinder/+/846054 | 14:04 |
| eharney | geguileo: :) | 14:04 |
| opendevreview | Gorka Eguileor proposed openstack/cinder master: Fix cinderlib RBD keyring issue https://review.opendev.org/c/openstack/cinder/+/846054 | 14:15 |
| opendevreview | Merged openstack/os-brick master: NVMe-oF: Get system uuid in privsep https://review.opendev.org/c/openstack/os-brick/+/836057 | 14:18 |
| *** dansmith_ is now known as dansmith | 14:29 | |
| enriquetaso | Hello | 15:03 |
| enriquetaso | #startmeeting cinder_bs | 15:03 |
| opendevmeet | Meeting started Wed Aug 24 15:03:46 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 |
| opendevmeet | Useful Commands: #action #agreed #help #info #idea #link #topic #startvote. | 15:03 |
| opendevmeet | The meeting name has been set to 'cinder_bs' | 15:03 |
| enriquetaso | Full list of bugs | 15:03 |
| enriquetaso | #link https://lists.openstack.org/pipermail/openstack-discuss/2022-August/030084.html | 15:03 |
| enriquetaso | We have three bugs to discuss today | 15:04 |
| rosmaita | o/ | 15:04 |
| enriquetaso | #topic I can't attach a nfs+luksv1 volume because _attach_encryptor only supports iscsi protocol | 15:04 |
| enriquetaso | hi rosmaita | 15:04 |
| enriquetaso | #link https://bugs.launchpad.net/nova/+bug/1987311 | 15:04 |
| enriquetaso | I've opened a bug against Nova regarding NFS encryption. I'll try to reach the nova team this week. | 15:04 |
| rosmaita | nice bug report there | 15:05 |
| enriquetaso | ha, yep, thanks eharney for debugging this with me | 15:05 |
| enriquetaso | moving on | 15:07 |
| enriquetaso | #topic [docs] Cannot start VM from disk with burst QoS. | 15:07 |
| enriquetaso | #link https://bugs.launchpad.net/cinder/+bug/1982945 | 15:07 |
| enriquetaso | This bug was originally a nova bug but was moved this week i think. I marked this as a documentation bug but I could be wrong. | 15:07 |
| enriquetaso | Last bug: | 15:09 |
| enriquetaso | #topic RBD support extends for in-use volume | 15:09 |
| enriquetaso | #link https://bugs.launchpad.net/cinder/+bug/1900406 | 15:09 |
| enriquetaso | This is a last-month bug that I forgot to mention. | 15:10 |
| enriquetaso | The configuration rbd_volume_local_attach = False but looks like Cinder ignores it and allows rbd to extend the volume anyway. | 15:10 |
| enriquetaso | I marked it as low importance because i think it may be an easy fix | 15:10 |
| eharney | rbd_volume_local_attach is a nova option... i guess what this is saying is that when using that, extend should not be allowed, but it is? | 15:10 |
| opendevreview | Gorka Eguileor proposed openstack/cinder master: Expose group in Volume OVO https://review.opendev.org/c/openstack/cinder/+/779578 | 15:11 |
| rosmaita | enriquetaso: btw, i agree with your idea to start https://bugs.launchpad.net/cinder/+bug/1982945 as a doc bug | 15:11 |
| eharney | what would the fix be? | 15:12 |
| enriquetaso | eharney, yes, that's what I understood. I haven't check in the code but maybe it's a simple CONF check while extending | 15:13 |
| eharney | well, i think this bug at least needs a better description of what's happening and a proposal, for it to be tagged as low-hanging-fruit | 15:13 |
| geguileo | that would be a nova issue, right? | 15:14 |
| eharney | it's barely clear to me what is happening there, much less a new-comer | 15:14 |
| geguileo | os-brick doesn't know about that | 15:14 |
| eharney | geguileo: probably | 15:14 |
| geguileo | if you tell os-brick to extend, then it will try to do so | 15:14 |
| enriquetaso | i'll remove the low-hanging-fruit :P my bad | 15:14 |
| geguileo | enriquetaso: oh, it should be easy | 15:15 |
| eharney | the bug also doesn't state what happens | 15:15 |
| eharney | or what is supposed to happen | 15:15 |
| eharney | well, ok | 15:15 |
| enriquetaso | maybe i should link nova team to the bug as well since rbd_volume_local_attach is a nova option | 15:15 |
| enriquetaso | so, we need more description on the bug, sadly it's almost one year-old but, hope the reporter reply | 15:16 |
| geguileo | I would just ask in the bug what they mean | 15:17 |
| geguileo | because I have read it now, and I don't even understand what they are talking about | 15:17 |
| geguileo | Nova seems to do the right thing on the driver side and doesn't call os-brick | 15:17 |
| geguileo | so it just calls cinder to extend, and the librbd on the hypervisor will detect the new size and it should just work | 15:18 |
| geguileo | this sentence is weird: rbd and libvirt will extend rbd image. rbd should not extend rbd image. | 15:18 |
| opendevreview | Alexander Deiter proposed openstack/cinder master: Fix Infinidat driver to backup attached volume https://review.opendev.org/c/openstack/cinder/+/851640 | 15:21 |
| enriquetaso | OK | 15:21 |
| enriquetaso | so the bug is less clear than I thought | 15:22 |
| enriquetaso | I'll ask for more information and what they mean | 15:22 |
| eharney | it might be more productive to not worry about the report too much, and just go assess what kind of test coverage we have for rbd extend, both local attach and not | 15:23 |
| eharney | if we are interested in chasing this | 15:23 |
| enriquetaso | i can do that as well | 15:23 |
| eharney | we should be able to find out whether it works or not without the reporter's involvement | 15:23 |
| *** dviroel|rover is now known as dviroel|rover|lunch | 15:24 | |
| sfv880 | Hello! if possible, I would like to ask you to review the fixes for the Infinidat driver: | 15:25 |
| sfv880 | Fix Infinidat driver to return all iSCSI portals | https://review.opendev.org/c/openstack/cinder/+/849022 | 15:25 |
| sfv880 | Fix Infinidat driver to use TLS/SSL communication | https://review.opendev.org/c/openstack/cinder/+/849846 | 15:25 |
| sfv880 | Fix Infinidat driver generic volume migration | https://review.opendev.org/c/openstack/cinder/+/850320 | 15:25 |
| sfv880 | Fix Infinidat driver multi-attach feature | https://review.opendev.org/c/openstack/cinder/+/850321 | 15:25 |
| sfv880 | Fix Infinidat driver to backup attached volume | https://review.opendev.org/c/openstack/cinder/+/851640 | 15:25 |
| sfv880 | Thank you very much! | 15:25 |
| enriquetaso | lol | 15:25 |
| enriquetaso | eharney, makes sense, i'll check that and add some comments on the bug report with this information | 15:26 |
| enriquetaso | OK, last 4 minutes | 15:26 |
| enriquetaso | #topic open discussion | 15:26 |
| enriquetaso | do we have any bug to discuss ? | 15:26 |
| rosmaita | nothing from me | 15:29 |
| enriquetaso | eharney, {0} tempest.api.volume.test_volumes_extend.VolumesExtendAttachedTest.test_extend_attached_volume [28.431093s] ... ok looks like it's fine | 15:29 |
| enriquetaso | anyway i need to see more carefully | 15:29 |
| enriquetaso | rosmaita, OK :) | 15:29 |
| eharney | that doesn't cover local attach vs not | 15:30 |
| eharney | also need to see how much behavior that actually checks | 15:30 |
| eharney | is it supposed to extend inside the instance, etc | 15:31 |
| opendevreview | Takashi Natsume proposed openstack/cinder master: Fix a deprecation warning about regex https://review.opendev.org/c/openstack/cinder/+/854428 | 15:31 |
| enriquetaso | eharney, thanks | 15:31 |
| enriquetaso | OK, thanks everyone! | 15:31 |
| enriquetaso | #endmeeting | 15:31 |
| opendevmeet | Meeting ended Wed Aug 24 15:31:53 2022 UTC. Information about MeetBot at http://wiki.debian.org/MeetBot . (v 0.1.4) | 15:31 |
| opendevmeet | Minutes: https://meetings.opendev.org/meetings/cinder_bs/2022/cinder_bs.2022-08-24-15.03.html | 15:31 |
| opendevmeet | Minutes (text): https://meetings.opendev.org/meetings/cinder_bs/2022/cinder_bs.2022-08-24-15.03.txt | 15:31 |
| opendevmeet | Log: https://meetings.opendev.org/meetings/cinder_bs/2022/cinder_bs.2022-08-24-15.03.log.html | 15:31 |
| opendevreview | Eric Harney proposed openstack/cinder-tempest-plugin master: Add volume dependency test coverage https://review.opendev.org/c/openstack/cinder-tempest-plugin/+/854287 | 15:43 |
| opendevreview | Eric Harney proposed openstack/cinder master: RBD: Flattening of child volumes during deletion https://review.opendev.org/c/openstack/cinder/+/835384 | 15:44 |
| opendevreview | Gorka Eguileor proposed openstack/cinder master: TOX: Document install_command usage https://review.opendev.org/c/openstack/cinder/+/840525 | 15:46 |
| opendevreview | Merged openstack/cinder master: Tests: Fix service stopping on cleanup https://review.opendev.org/c/openstack/cinder/+/854259 | 15:46 |
| opendevreview | Prajakta proposed openstack/cinder stable/yoga: Doc: To update the portset feature details in IBM Spectrum Virtualize user guide https://review.opendev.org/c/openstack/cinder/+/853788 | 16:05 |
| *** dviroel|rover|lunch is now known as dviroel|rover | 16:21 | |
| opendevreview | Victoria Martinez de la Cruz proposed openstack/devstack-plugin-ceph master: Use a standard port for NFS https://review.opendev.org/c/openstack/devstack-plugin-ceph/+/854436 | 16:42 |
| opendevreview | Eric Harney proposed openstack/cinder master: RBD: Flattening of child volumes during deletion https://review.opendev.org/c/openstack/cinder/+/835384 | 18:23 |
| opendevreview | Gorka Eguileor proposed openstack/cinder master: Expose group in Volume OVO https://review.opendev.org/c/openstack/cinder/+/779578 | 18:36 |
| opendevreview | Gorka Eguileor proposed openstack/os-brick master: RBD: Improve IOWrapper's close & flush methods https://review.opendev.org/c/openstack/os-brick/+/854281 | 19:46 |
| opendevreview | Alexander Deiter proposed openstack/cinder master: Fix Infinidat driver to backup attached volume https://review.opendev.org/c/openstack/cinder/+/851640 | 20:39 |
| *** dviroel|rover is now known as dviroel|rover|brb | 22:30 | |
| *** dviroel|rover|brb is now known as dviroel|rover | 23:26 | |
Generated by irclog2html.py 2.17.3 by Marius Gedminas - find it at https://mg.pov.lt/irclog2html/!