| *** mhen_ is now known as mhen | 01:46 | |
| opendevreview | Volodymyr Boiko proposed openstack/cinder master: VAST Data Volume Driver https://review.opendev.org/c/openstack/cinder/+/939005 | 07:18 |
|---|---|---|
| *** iurygregory_ is now known as iurygregory | 10:57 | |
| opendevreview | Konrad Gube proposed openstack/cinder master: Support waiting for volume extend completion https://review.opendev.org/c/openstack/cinder/+/891602 | 12:58 |
| opendevreview | Konrad Gube proposed openstack/cinder master: NFS: Add online extend support https://review.opendev.org/c/openstack/cinder/+/873686 | 12:58 |
| opendevreview | Konrad Gube proposed openstack/cinder master: Netapp NFS: Add online extend support https://review.opendev.org/c/openstack/cinder/+/873889 | 12:58 |
| opendevreview | Konrad Gube proposed openstack/devstack-plugin-nfs master: Enable attached volume extend tests https://review.opendev.org/c/openstack/devstack-plugin-nfs/+/896196 | 13:35 |
| opendevreview | Merged openstack/cinder stable/2024.2: Always upload image volumes in a native thread https://review.opendev.org/c/openstack/cinder/+/934380 | 14:04 |
| jbernard | #startmeeting cinder | 14:04 |
| opendevmeet | Meeting started Wed Oct 22 14:04:44 2025 UTC and is due to finish in 60 minutes. The chair is jbernard. Information about MeetBot at http://wiki.debian.org/MeetBot. | 14:04 |
| opendevmeet | Useful Commands: #action #agreed #help #info #idea #link #topic #startvote. | 14:04 |
| opendevmeet | The meeting name has been set to 'cinder' | 14:04 |
| jbernard | courtesy ping: jungleboyj rosmaita smcginnis tosky whoami-rajat m5z e0ne geguileo eharney jbernard hemna fabiooliveira yuval tobias-urdin adiare happystacker dosaboy hillpd msaravan sp-bmilanov Luzi sfernand simondodsley zaubea nileshthathagar flelain wizardbit agalica | 14:04 |
| jbernard | #topic roll call | 14:04 |
| jbernard | #link https://etherpad.opendev.org/p/cinder-gazpacho-meetings | 14:05 |
| agalica | o/ | 14:05 |
| rosmaita | o/ | 14:05 |
| hemna | mep | 14:05 |
| hvlcchao1 | o/ | 14:05 |
| yuval | 0/ | 14:05 |
| jbernard | o/ | 14:05 |
| erlon | \o | 14:05 |
| jungleboyj | o/ | 14:06 |
| raineszm | o/ | 14:06 |
| whoami-rajat | hi | 14:06 |
| opendevreview | Eric Harney proposed openstack/cinder master: WIP: image_utils: Detect missing device before calling qemu-img convert https://review.opendev.org/c/openstack/cinder/+/964541 | 14:06 |
| raineszm | I hope I wasn't too presumptuous in adding an entry for today's meeting. | 14:07 |
| jbernard | raineszm: not at all, that is what you're supposed to do | 14:07 |
| raineszm | great. ty | 14:08 |
| jbernard | welcome everyone | 14:08 |
| jbernard | #topic annoucements | 14:08 |
| jbernard | really quickly, next week is PTG | 14:08 |
| jbernard | im working on finalizing the scheudle, but roughly we will have tuesday and wednesday for topics | 14:09 |
| jbernard | thurs and fri can be used for collaborative reveiws, which will probably be more helpful than anything else | 14:09 |
| jbernard | we have a few cross project sessions | 14:09 |
| jbernard | there is an eventlet removal session on tuesday (across all projects) | 14:10 |
| jbernard | wednesday we have a nova/cinder and a nova/cinder/ironic session | 14:10 |
| jbernard | it looks like the nova/cinder/glance (encryption api) cross project will be on thursday | 14:10 |
| jbernard | (waiting for final word from cyril) | 14:10 |
| jbernard | most of the sessions will be concentrated between 1400 to 1700 UTC | 14:11 |
| jbernard | im not sure if we want to try meetpad again (over google) but we can sort that out later | 14:12 |
| jbernard | historicaly ive had trouble with the recordings, but that's most probably my fault | 14:12 |
| jbernard | that's about all, i will send out a mail with more details and update the etherpad | 14:13 |
| rosmaita | well, if the recordings are important, we should use whatever makes recording most reliable | 14:13 |
| jbernard | that, i think, is why we've reverted to goolge in the past | 14:13 |
| rosmaita | yep | 14:13 |
| jbernard | it works, but it's different from what most others are doing | 14:13 |
| jbernard | and can create some confusion, but i dont think it's signficant | 14:14 |
| jayaanand | hi | 14:14 |
| rosmaita | when was the last time we used meetpad? i don't remember | 14:14 |
| erlon | If it would be nice if it was possible to get automated notes on those. Not sure if you guys have tried that before, but it's pretty handful the times I used | 14:14 |
| jbernard | last ptg we tried initially | 14:14 |
| jbernard | and failed and switch back to google | 14:14 |
| jbernard | erlon: i think i can enable gemini to take notes from the live sessions | 14:15 |
| erlon | Does meat pad supports automated notes? | 14:15 |
| jbernard | erlon: is that what you mean? | 14:15 |
| erlon | yeah, that's what I used on google meets | 14:15 |
| jbernard | if we use google then i think it's straightforward, i need to look into meetpad if we try to go that route | 14:15 |
| jbernard | that's all i have for annoucements, questions about next week? | 14:17 |
| rosmaita | i must admit that i haven't been paying attention, is meetpad currently the preferred PTG meeting software? | 14:17 |
| rosmaita | i notice that starlingx is using linux foundation zoom | 14:17 |
| erlon | Are you organizing the schedule and planning for the ptg anywere (etherpad, docs)? | 14:17 |
| erlon | Zoom also has meeting notes feature | 14:18 |
| jbernard | we have a planning etherpad (https://etherpad.opendev.org/p/2026.1-ptg-cinder-planning) quite sparse at the moment, i will be transferring notes and shedule to the proper etherpad today | 14:18 |
| jbernard | let me just say, if anything comes up just ask in the channel | 14:20 |
| jbernard | most things are open to discussion and i welcome all input | 14:20 |
| jbernard | #topic stable releases | 14:21 |
| jbernard | rosmaita: just wanted to check in with you, how is that going? | 14:21 |
| jbernard | do you need anythign from me? | 14:21 |
| rosmaita | just need +1s on the patches :D | 14:22 |
| jbernard | ok, will do | 14:22 |
| rosmaita | so, at this point, no more merges into Caracal, that is, stable/2024.1 | 14:23 |
| jbernard | i think we landed everything we were targeting, no? | 14:23 |
| rosmaita | yep | 14:23 |
| rosmaita | there are no open non-WIP patches | 14:23 |
| rosmaita | and i guess i should abandon all of them | 14:24 |
| jbernard | yeah, else they just clutter the dashboard | 14:25 |
| rosmaita | that's all from me | 14:25 |
| jbernard | rosmaita: thanks for all of it! | 14:25 |
| opendevreview | Merged openstack/cinder stable/2024.2: [Pure Storage] Manage Volume from GUI fails https://review.opendev.org/c/openstack/cinder/+/944301 | 14:25 |
| rosmaita | np | 14:25 |
| opendevreview | Merged openstack/cinder stable/2024.2: [Pure Storage] Fix issue with LACP ports not being identified https://review.opendev.org/c/openstack/cinder/+/945785 | 14:25 |
| jayaanand | we have multiple back-porting patches from NetApp. Can someone take a look | 14:26 |
| jayaanand | https://review.opendev.org/c/openstack/cinder/+/956755 | 14:26 |
| jayaanand | https://review.opendev.org/c/openstack/cinder/+/959293 | 14:26 |
| jayaanand | https://review.opendev.org/c/openstack/cinder/+/961431 | 14:27 |
| jayaanand | https://review.opendev.org/c/openstack/cinder/+/961432 | 14:27 |
| jbernard | #topic rbd driver blocksize | 14:27 |
| jbernard | #link https://review.opendev.org/c/openstack/cinder/+/658283 | 14:27 |
| jbernard | raineszm: that's you i believe | 14:27 |
| raineszm | So this is an /old/ patch that stalled out but is probably worth reviving. | 14:27 |
| jbernard | jayaanand: if they're just review requests, put them in the etherpad | 14:28 |
| raineszm | The basic question that seem to have come up that stalled the review was that there was a question about where to add the new options. | 14:28 |
| jayaanand | sure than you! | 14:28 |
| jbernard | raineszm: i vaguely remember - why can't we make it an option at the driver level instead of just rbd? | 14:28 |
| raineszm | That seems like a good idea long term. | 14:28 |
| raineszm | The only concern is how to handle the other drivers that already have an analogous option | 14:29 |
| jbernard | hmm, do you have an example? | 14:29 |
| raineszm | So there's combination of refactor and new feature there | 14:29 |
| raineszm | yes 1 sec | 14:29 |
| hemna | +A'd | 14:29 |
| erlon | How common is this option across the drivers? Is this a libvirt feature that can be used by any backend? | 14:31 |
| raineszm | Yes. So any driver can set the metadata | 14:31 |
| raineszm | and libvirt picks it up | 14:31 |
| raineszm | the solidfire driver has implemented it in the past I think | 14:32 |
| raineszm | just trying to find a permalink | 14:32 |
| erlon | Are the connectors (nfs, iscsi, fc) already passing that to libvirt on the nova side? | 14:32 |
| raineszm | It seems so. So basically the logical_block_size and physical_block_size have default values that are advertised | 14:34 |
| raineszm | in the connection data | 14:34 |
| raineszm | the addition here is to allow setting those as an option | 14:34 |
| raineszm | At least one other driver sets those by querying the disk geometry | 14:35 |
| raineszm | Currently in the icsci driver it seems | 14:36 |
| raineszm | https://opendev.org/openstack/cinder/src/commit/d02171164bdd702b12b59888b744d172f30d712d/cinder/volume/targets/iscsi.py#L127 | 14:36 |
| erlon | right, I see that in nova. I think my concern would be mostly related to the ability to backport this fix if we increase its scope | 14:38 |
| raineszm | and here https://opendev.org/openstack/cinder/src/commit/d02171164bdd702b12b59888b744d172f30d712d/cinder/volume/driver.py#L2782 in the driver base | 14:38 |
| erlon | What I like that idea to have this re-used across all drivers | 14:39 |
| raineszm | I agree. I think that long term it should be moved to the driver level | 14:40 |
| raineszm | The question is whether to do that combined with the fix for rbd or to do it in two steps | 14:40 |
| raineszm | The motivation for the original patch being that it allows to fix performance issues with rbd and windows vms | 14:41 |
| raineszm | jbernard: what do you think? | 14:42 |
| erlon | yes, and I like the way the options are put in pairs like 512e -> "4096 512", it seems more meaningful too the user. | 14:42 |
| jbernard | i lean towards doing it once, even if it takes a bit more (at the driver layer) | 14:43 |
| erlon | How about backporting? It will be bigger | 14:44 |
| jbernard | that is true, but it's not impossible, especially if care is taken in writing and review | 14:44 |
| erlon | But apparently not impossible. | 14:45 |
| erlon | :) | 14:45 |
| erlon | yeah, agreed | 14:45 |
| raineszm | Haha. Okay. Fair. | 14:45 |
| raineszm | Yeah I'm game to hoist it up to the driver level. | 14:45 |
| raineszm | If that's what we want to do | 14:45 |
| jbernard | my sense is that faster fixes that require followup, more often than not, never get said followup | 14:45 |
| erlon | If we can make it back to Caracal, I think I can make it happen | 14:45 |
| raineszm | I'll start taking a look at that then. | 14:46 |
| raineszm | thanks for chatting in through. | 14:46 |
| raineszm | *it | 14:46 |
| jbernard | np, thanks for working on it | 14:47 |
| erlon | lets discuss more details on the option naming in the ptg | 14:47 |
| jbernard | sure | 14:47 |
| jbernard | #topic open discussion | 14:48 |
| erlon | \o | 14:48 |
| tobias-urdin | can i shamelessly plug my (now old) open patches :) https://review.opendev.org/q/project:openstack/cinder+status:open+owner:%22Tobias+Urdin%22 | 14:48 |
| agalica | jbernard: would it be possible to get cores assigned to our issues? We have 3 issues we need to backport - targeting for december. These are all important. One of them already has a core and another reviewer (and we're waiting on Rajat again on that one) | 14:48 |
| erlon | just a nudge on whoami-rajat to get a look on the brick iscsi patch | 14:48 |
| jbernard | tobias-urdin: you may :) | 14:49 |
| agalica | tobias-urdin: do you need reviewers: My team can review if you do | 14:49 |
| agalica | non-core | 14:49 |
| agalica | My patches are these: Replication: https://review.opendev.org/c/openstack/cinder/+/869991 | 14:49 |
| agalica | Snapshot Expansion: https://review.opendev.org/c/openstack/cinder/+/850830 | 14:49 |
| agalica | B20: https://review.opendev.org/c/openstack/cinder/+/959901 | 14:49 |
| agalica | whoami-rajat: Polite nudge here as well that we have updated our B20 patch with the changes that hopefully satisfy your comments | 14:50 |
| jbernard | agalica: cores assigned, let me think on that. it's not been our practice in the past but maybe we can do something to get closer and improve turnaround | 14:50 |
| jbernard | it's always voluntary of course, as this is open source, but maybe i can help coordinate better | 14:51 |
| agalica | jbernard: thanks. we're really struggling here, and no one seems to want to review. we're already behind in that we were meant to get merged into 2025.1 and missed that. Things are getting hot. | 14:51 |
| tobias-urdin | thanks! they are quite old now but i think i rebased them recently, any feedback welcome :) | 14:51 |
| agalica | we are happy to review many patches if tehre are | 14:51 |
| agalica | ok tobias - any one in particular, or all of them? | 14:52 |
| jbernard | agalica: ack | 14:52 |
| erlon | @agalica the volume extend patch has a merge conflict. Can you rebase it? | 14:52 |
| agalica | erlon: yes, we will do that today | 14:53 |
| erlon | Volume Replication too | 14:53 |
| agalica | ok, will do that today | 14:53 |
| agalica | tobias-urden: do you have any specific patches you want reviewed in that list, or all of them? | 14:53 |
| jbernard | agalica: i would go through them and review the ones that are passing ci without merge conflict | 14:54 |
| agalica | ok, thanks - will do that | 14:55 |
| wizardbit | Hey jbernard, were you able to get an update from Rajat on https://review.opendev.org/c/openstack/os-brick/+/955379 ? | 14:56 |
| jbernard | whoami-rajat: ^ | 14:56 |
| wizardbit | That will ping him? | 14:59 |
| jbernard | maybe ;) | 14:59 |
| jbernard | it's quite late now, he may see it in his scrollback buffer | 15:00 |
| jbernard | we should be able to address all of these in next week's meetings thoug | 15:00 |
| jbernard | we're at time, anything else? | 15:01 |
| Anoop_Shukla | https://review.opendev.org/c/openstack/cinder/+/956221 | 15:01 |
| jbernard | Anoop_Shukla: ack | 15:01 |
| jbernard | ok, thank you everyone, see you next week | 15:01 |
| jbernard | #endmeeting | 15:01 |
| opendevmeet | Meeting ended Wed Oct 22 15:01:55 2025 UTC. Information about MeetBot at http://wiki.debian.org/MeetBot . (v 0.1.4) | 15:01 |
| opendevmeet | Minutes: https://meetings.opendev.org/meetings/cinder/2025/cinder.2025-10-22-14.04.html | 15:01 |
| opendevmeet | Minutes (text): https://meetings.opendev.org/meetings/cinder/2025/cinder.2025-10-22-14.04.txt | 15:01 |
| opendevmeet | Log: https://meetings.opendev.org/meetings/cinder/2025/cinder.2025-10-22-14.04.log.html | 15:01 |
| tobias-urdin | agalica: any of them, they've been around for a while, thanks! | 15:02 |
| agalica | ok, we will take a look at them this week! If you are able to look at my patches too, we would be really appreciative. I will ping you the links | 15:03 |
| agalica | tobias-urden: ^ | 15:03 |
| agalica | Snapshot Expansion: https://review.opendev.org/c/openstack/cinder/+/850830 | 15:04 |
| agalica | B20: https://review.opendev.org/c/openstack/cinder/+/959901 | 15:04 |
| agalica | sorry about that, wrong window | 15:04 |
| tobias-urdin | agalica: ack, thx | 15:17 |
| opendevreview | Hao Chen proposed openstack/cinder master: Hitachi: Volume Replication for VSP Driver https://review.opendev.org/c/openstack/cinder/+/869991 | 15:25 |
| agalica | erlon: we rebased https://review.opendev.org/c/openstack/cinder/+/869991 to our B20 patch (https://review.opendev.org/c/openstack/cinder/+/959901). The latter shows "active", while the former still shows "Merge Conflict" despite having the correct parent. I am wondering if it's because it depends on the first patch being merged? Do you know the answer to this? | 15:38 |
| opendevreview | anthony gamboa proposed openstack/cinder master: Hitachi: add support for Hitachi VSP One B20 https://review.opendev.org/c/openstack/cinder/+/959901 | 15:41 |
| erlon | There are many abandoned patches from the 20 version. There might be some mix there | 15:46 |
| agalica | erlon: yeah, we have it based on the correct one. We think we know what we need to do now and we're working through it today. Looks like we're going to have to do some manual rebase merges | 15:48 |
| opendevreview | Raghavendra Tilay proposed openstack/cinder stable/2024.2: HPE 3par - skip license check for new wsapi https://review.opendev.org/c/openstack/cinder/+/964558 | 16:12 |
| opendevreview | Merged openstack/cinder stable/2024.2: Huawei hypermetro: RECOVERYPOLICY typo https://review.opendev.org/c/openstack/cinder/+/944797 | 17:51 |
| opendevreview | Merged openstack/cinder stable/2025.1: tgt target: Provide unique scsi_sn and scsi_id https://review.opendev.org/c/openstack/cinder/+/952279 | 18:15 |
| opendevreview | Merged openstack/cinder stable/2025.1: Show volume attachment host information for services https://review.opendev.org/c/openstack/cinder/+/964155 | 18:15 |
| opendevreview | Merged openstack/cinder stable/2024.2: [Pure Storage] Remove SafeMode PGs from Replication Pods https://review.opendev.org/c/openstack/cinder/+/944565 | 19:23 |
| opendevreview | Hao Chen proposed openstack/cinder master: Hitachi: Volume Replication for VSP Driver https://review.opendev.org/c/openstack/cinder/+/869991 | 20:39 |
| opendevreview | Volodymyr Boiko proposed openstack/cinder master: VAST Data Volume Driver https://review.opendev.org/c/openstack/cinder/+/939005 | 20:40 |
| agalica_ | erlon: https://review.opendev.org/c/openstack/cinder/+/869991 is now free of conflicts. We're still working on the other one. Alerting you in case you were planning on reviewing it. | 20:45 |
| opendevreview | Anthony Galica proposed openstack/cinder master: Hitachi: support extending volume has snapshot(s) https://review.opendev.org/c/openstack/cinder/+/850830 | 20:51 |
| opendevreview | Hao Chen proposed openstack/cinder master: Hitachi: support extending volume has snapshot(s) https://review.opendev.org/c/openstack/cinder/+/850830 | 21:14 |
| agalica_ | erlon: second one up now too: https://review.opendev.org/c/openstack/cinder/+/850830 | 21:15 |
| opendevreview | Merged openstack/cinder stable/2025.1: [Pure Storage] Resolve EG1 arrays not reporting DRR https://review.opendev.org/c/openstack/cinder/+/960278 | 21:21 |
| opendevreview | Merged openstack/cinder stable/2025.1: [Pure Storage] Cinder manage quota breach deletion fix https://review.opendev.org/c/openstack/cinder/+/960280 | 21:25 |
| opendevreview | Merged openstack/cinder stable/2025.1: [Pure Storage] Fix volume reconnect error https://review.opendev.org/c/openstack/cinder/+/960277 | 21:30 |
| opendevreview | Merged openstack/cinder stable/2025.1: Add testing for optimized volume upload https://review.opendev.org/c/openstack/cinder/+/956169 | 22:58 |
| opendevreview | Merged openstack/cinder stable/2025.1: NVMe-oF Target: Fix incorrect check for initiator in connector data https://review.opendev.org/c/openstack/cinder/+/950657 | 23:20 |
| opendevreview | Merged openstack/cinder stable/2024.2: Driver assisted migration on retype when it's safe https://review.opendev.org/c/openstack/cinder/+/964454 | 23:28 |
Generated by irclog2html.py 4.0.0 by Marius Gedminas - find it at https://mg.pov.lt/irclog2html/!