| *** mhen_ is now known as mhen | 03:02 | |
| *** erlon42 is now known as erlon | 13:01 | |
| opendevreview | Raghavendra Tilay proposed openstack/cinder master: [WIP] HPE 3par: Fix session share issue https://review.opendev.org/c/openstack/cinder/+/971158 | 13:07 |
|---|---|---|
| jbernard | #startmeeting cinder | 14:05 |
| opendevmeet | Meeting started Wed Dec 17 14:05:18 2025 UTC and is due to finish in 60 minutes. The chair is jbernard. Information about MeetBot at http://wiki.debian.org/MeetBot. | 14:05 |
| opendevmeet | Useful Commands: #action #agreed #help #info #idea #link #topic #startvote. | 14:05 |
| opendevmeet | The meeting name has been set to 'cinder' | 14:05 |
| 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 lutimura | 14:05 |
| jbernard | #topic roll call | 14:05 |
| jbernard | o/ hello all | 14:05 |
| Sai | o/ | 14:05 |
| kaisers | o/ | 14:05 |
| jbernard | this will likely be a very short one, i think many are already away for holidays | 14:05 |
| Anoop_Shukla | o/ | 14:06 |
| agalica | O/ | 14:06 |
| harsh | 0/ | 14:06 |
| rosmaita | o/ | 14:07 |
| sp-bmilanov | o/ | 14:07 |
| jayaanand | o/ | 14:08 |
| jbernard | #topic annoucements | 14:08 |
| jbernard | oy, I've had nearly 2 weeks of snow days, still trying to get caught up on things | 14:09 |
| jbernard | we have some specs still to review | 14:09 |
| jbernard | we did complete the openapi reviews, | 14:09 |
| jbernard | i think all of them have merged and that work should now be complete | 14:09 |
| jbernard | the next two wednesday meetings are canceled for holidays | 14:10 |
| harsh | okay | 14:10 |
| jbernard | i hope everyone has a good holidays and enjoys the time away | 14:11 |
| agalica | Sounds like you’re already knee deep in yours | 14:11 |
| agalica | ;) | 14:11 |
| jbernard | i has not been the december i was hoping for so far lol :P | 14:12 |
| jbernard | #topic open discussion | 14:12 |
| jbernard | i dont have anything specific | 14:12 |
| jbernard | im behind on spec reviews, but trying to get to those this wekk | 14:13 |
| jbernard | unless there is anything, we can wrap | 14:16 |
| kaisers | i have a small question | 14:16 |
| jbernard | sure | 14:16 |
| kaisers | I have been working on doing a range of fixes on the quobyte driver after the CI is no re-created. i am torn between having these fixes stand separately or having them in a dependency chain, is there a preferred way? | 14:17 |
| jbernard | i think you have 2 ways you can choose | 14:18 |
| jbernard | you can create a dependency chain, which is just the stack of patches you have committed locally | 14:18 |
| jbernard | or you can submit them seperately and use depends-on in the commit message to indicate dependencies | 14:19 |
| jbernard | i *think* the first option is easier | 14:19 |
| kaisers | ok, depends-on is a good point | 14:19 |
| jbernard | as far as preferred, as long as the commits are well formed and passing zuul, i think whatever you choose is okay | 14:20 |
| kaisers | ok, thnx. | 14:20 |
| sp-bmilanov | hi, one question from me -- what's the latest approach to requesting reviews? did https://review.opendev.org/c/openstack/cinder/+/847536 fall through the cracks at some point? it was on the etherpad lists for dalmatian, epoxy, and flamingo, and at some point was the first thing in the etherpad for gazpacho, now it's way way down? all points raised by Cinder Core have been discussed and agreed on for a year now | 14:20 |
| sp-bmilanov | I see the review festival etherpad, can I just add changes there directly? | 14:22 |
| jbernard | sp-bmilanov: still teh same, i dont think the position in the pad indicates importance | 14:22 |
| jbernard | sp-bmilanov: it's a larger change (by the numbers) so it probably doesn't get caught by the festival filter | 14:23 |
| harsh | Will we have the festival review meeting this friday ? | 14:24 |
| sp-bmilanov | jbernard: ah, right | 14:24 |
| kaisers | just typed that, too | 14:24 |
| jbernard | rosmaita, eharney, yall are on the reviewer list, can this be added to the queue | 14:25 |
| jbernard | sp-bmilanov: it does have 2 unresolved comments, resolving those would help | 14:25 |
| jbernard | reviewers may be waiting for those to go away | 14:25 |
| jbernard | i will be out friday, rosmaita - what's your position on the festival? | 14:25 |
| rosmaita | i will be around, we can still have it | 14:26 |
| jbernard | ok, sp-bmilanov that may be a good place to get feedback | 14:27 |
| sp-bmilanov | I don't think I can do anything useful with the unresolved comments, they are waiting for some feedback that has still not come back | 14:27 |
| agalica | The one where rosmaita ruins christmas | 14:28 |
| sp-bmilanov | jbernard: good idea | 14:28 |
| jbernard | sp-bmilanov: ok, let fernando know when you can, the friday meeting might be the best place | 14:29 |
| jbernard | agalica: if it has to be somebody, i think rosmaita is the best choice | 14:29 |
| rosmaita | :D | 14:30 |
| agalica | Haha | 14:30 |
| jbernard | :) | 14:30 |
| erlon | \o late | 14:34 |
| jbernard | erlon: that's okay, got anything? | 14:35 |
| jbernard | we're ready to wrap up if not | 14:35 |
| jbernard | alright, | 14:38 |
| jbernard | enjoy the break everyone, happy holidays and new year | 14:38 |
| erlon | nothing particularly | 14:38 |
| erlon | sp-bmilanov Ill add your patch to my review queue | 14:38 |
| jbernard | erlon: thanks | 14:38 |
| jbernard | erlon: rosmaita is holding the review festival friday as well | 14:39 |
| jbernard | ok, that's it | 14:39 |
| jbernard | #endmeeting | 14:39 |
| opendevmeet | Meeting ended Wed Dec 17 14:39:20 2025 UTC. Information about MeetBot at http://wiki.debian.org/MeetBot . (v 0.1.4) | 14:39 |
| opendevmeet | Minutes: https://meetings.opendev.org/meetings/cinder/2025/cinder.2025-12-17-14.05.html | 14:39 |
| opendevmeet | Minutes (text): https://meetings.opendev.org/meetings/cinder/2025/cinder.2025-12-17-14.05.txt | 14:39 |
| opendevmeet | Log: https://meetings.opendev.org/meetings/cinder/2025/cinder.2025-12-17-14.05.log.html | 14:39 |
| sp-bmilanov | thanks jbernard! | 14:40 |
| erlon | sp-bmilanov what protocol did your driver supported before the patch? nfs? | 14:43 |
| sp-bmilanov | a StorPool-proprietary block protocol | 14:43 |
| erlon | So you guys have a nova connector for that | 14:44 |
| sp-bmilanov | yep | 14:45 |
| sp-bmilanov | > erlon: sp-bmilanov Ill add your patch to my review queue | 14:53 |
| sp-bmilanov | oh, cool, thanks, I just saw that | 14:53 |
| sp-bmilanov | sfernand: ping re: https://review.opendev.org/c/openstack/cinder/+/847536 some outstanding comments (else we can discuss them on Friday) | 15:03 |
| cardoe | jbernard: how can I get https://review.opendev.org/c/openstack/cinder/+/961436 in the review pipeline? I put it on the etherpad you previously mentioned | 15:17 |
| noonedeadpunk | rosmaita: hey! do you have any suggestions around how to approach this spec in a better way about changing config? https://review.opendev.org/c/openstack/cinder-specs/+/962306 | 15:37 |
| rosmaita | noonedeadpunk: looking | 15:39 |
| noonedeadpunk | and in the implementation a folk pointed out that sucxh possibility overall would be helpful and they do it in their downstream driver | 15:39 |
| noonedeadpunk | I'm really open to suggestions/discussion around this one... | 15:40 |
| rosmaita | will need to think some more, left a few minor comments though for now | 16:19 |
| rosmaita | noonedeadpunk: ^^ | 16:19 |
| noonedeadpunk | ++ | 16:20 |
| noonedeadpunk | will address them | 16:21 |
| jbernard | cardoe: well, i hope the review priority will help, there is a review meeting on friday (https://etherpad.opendev.org/p/cinder-festival-of-reviews), that is probably the best place to ask this week | 16:27 |
| cardoe | okay thanks | 16:28 |
| jbernard | cardoe: i will have quite a bit of time over the holidays and planning to go through the review etherpad as well | 16:28 |
| jbernard | cardoe: is it there as well? | 16:28 |
| jbernard | cardoe: https://etherpad.opendev.org/p/cinder-gazpacho-reviews | 16:29 |
| cardoe | yep. It's under jayaanand's name | 16:29 |
| jbernard | cardoe: ok, great | 16:29 |
| cardoe | It's also under my name | 16:29 |
| cardoe | Under the backports section | 16:30 |
| cardoe | It's already been backported to 2025.2 just looking for it to go to 2025.1 | 16:30 |
| jbernard | kk | 16:30 |
| cardoe | Folks gave some review cause they disagreed with some of it. But then they pointed out its on master and backported to 2025.2 so they removed their -1 | 16:30 |
| cardoe | The part that people don't like isn't what I need. It's got the side effect of fixing multi-attach generically for Ironic. | 16:31 |
| jbernard | ok, should be relatively easy then | 16:33 |
Generated by irclog2html.py 4.0.0 by Marius Gedminas - find it at https://mg.pov.lt/irclog2html/!