| *** mhen_ is now known as mhen | 02:39 | |
| opendevreview | Nilesh Thathagar proposed openstack/cinder master: Dell PowerFlex: Fixed volume detach issue https://review.opendev.org/c/openstack/cinder/+/973232 | 05:13 |
|---|---|---|
| opendevreview | Sandip Rajbanshi proposed openstack/cinder master: [IBM SVf Driver] Added support for volumegroup-snapshot for temporary volumegroup https://review.opendev.org/c/openstack/cinder/+/926844 | 07:00 |
| opendevreview | Sandip Rajbanshi proposed openstack/cinder master: [IBM SVf Driver] Added support for restorefromsnapshot CLI for volumegroups https://review.opendev.org/c/openstack/cinder/+/916722 | 07:00 |
| opendevreview | Sandip Rajbanshi proposed openstack/cinder master: [IBM SVf Driver] Added support of clone operation for volumegroup https://review.opendev.org/c/openstack/cinder/+/938771 | 07:00 |
| opendevreview | Sandip Rajbanshi proposed openstack/cinder master: Doc: Documentation updates for ibm-svf-volumegroup feature. https://review.opendev.org/c/openstack/cinder/+/929906 | 07:00 |
| opendevreview | Sandip Rajbanshi proposed openstack/cinder master: [IBM SVf driver] Adding support for individual volume snapshots https://review.opendev.org/c/openstack/cinder/+/949006 | 07:00 |
| opendevreview | Sandip Rajbanshi proposed openstack/cinder master: [IBM SVf driver] Adding support for Individual volume clone https://review.opendev.org/c/openstack/cinder/+/951157 | 07:00 |
| opendevreview | Sandip Rajbanshi proposed openstack/cinder master: Doc: Documentation updates for ibm-svf-volumegroup feature. https://review.opendev.org/c/openstack/cinder/+/958465 | 07:00 |
| opendevreview | Sandip Rajbanshi proposed openstack/cinder master: [IBM SVf driver] Adding support for PBHA partition backend https://review.opendev.org/c/openstack/cinder/+/943364 | 07:00 |
| opendevreview | Sandip Rajbanshi proposed openstack/cinder master: [IBM SVf driver] PBHA Volume Extend Support https://review.opendev.org/c/openstack/cinder/+/949483 | 07:00 |
| opendevreview | Sandip Rajbanshi proposed openstack/cinder master: [IBM SVf driver] Adding support for PBHA volumegroup update https://review.opendev.org/c/openstack/cinder/+/950860 | 07:00 |
| opendevreview | Sandip Rajbanshi proposed openstack/cinder master: [IBM SVf driver] Support PBHA properties reporting https://review.opendev.org/c/openstack/cinder/+/952422 | 07:00 |
| opendevreview | Sandip Rajbanshi proposed openstack/cinder master: [IBM SVf driver] Support PBHA group-snapshot and restore https://review.opendev.org/c/openstack/cinder/+/952432 | 07:00 |
| opendevreview | Sandip Rajbanshi proposed openstack/cinder master: [IBM SVf driver] Adding policing for PBHA partition https://review.opendev.org/c/openstack/cinder/+/958138 | 07:00 |
| opendevreview | Sandip Rajbanshi proposed openstack/cinder master: [IBM SVf driver] Support location affinity to hosts in PBHA partition. https://review.opendev.org/c/openstack/cinder/+/963620 | 07:00 |
| opendevreview | Sandip Rajbanshi proposed openstack/cinder master: [IBM SVf driver] Adding support for PBHA volume manage operation https://review.opendev.org/c/openstack/cinder/+/962244 | 07:00 |
| opendevreview | Sandip Rajbanshi proposed openstack/cinder master: Doc: Documentation updates for ibm-svf-pbha feature. https://review.opendev.org/c/openstack/cinder/+/967563 | 07:00 |
| opendevreview | Sandip Rajbanshi proposed openstack/cinder master: [IBM SVf Driver] Add basic support for Policy-Based Replication (PBR) https://review.opendev.org/c/openstack/cinder/+/965442 | 07:00 |
| opendevreview | Sandip Rajbanshi proposed openstack/cinder master: [IBM SVf Driver] List replication-policies (PBR) https://review.opendev.org/c/openstack/cinder/+/965813 | 07:00 |
| opendevreview | Sandip Rajbanshi proposed openstack/cinder master: [IBM SVf Driver] Add support for failover/failback in PBR https://review.opendev.org/c/openstack/cinder/+/974983 | 07:00 |
| opendevreview | Sandip Rajbanshi proposed openstack/cinder master: [IBM SVf driver] Doc: Documentation updates for ib m-svf-callhome Feature https://review.opendev.org/c/openstack/cinder/+/974989 | 08:43 |
| opendevreview | Sandip Rajbanshi proposed openstack/cinder master: [IBM SVf driver] Doc: Documentation updates for ib m-svf-callhome Feature https://review.opendev.org/c/openstack/cinder/+/974989 | 08:44 |
| opendevreview | Sandip Rajbanshi proposed openstack/cinder master: Doc: Documentation updates for ibm-svf-callhome Feature https://review.opendev.org/c/openstack/cinder/+/974989 | 08:46 |
| opendevreview | Max proposed openstack/os-brick master: feat: add nfs 4.2 option https://review.opendev.org/c/openstack/os-brick/+/960573 | 09:13 |
| opendevreview | Harsh Ailani proposed openstack/cinder master: [IBM SVf Driver] Added support for volumegroup-snapshot for temporary volumegroup https://review.opendev.org/c/openstack/cinder/+/926844 | 09:33 |
| harsh | jbernard and rosmaita : I have addressed the comments on : [IBM SVf Driver] Added support for volumegroup-snapshot for temporary volumegroup | https://review.opendev.org/c/openstack/cinder/+/926844 | 09:33 |
| harsh | jbernard: could you please add your +2 again and rosmaita: i have fixed your comments on the patch as well. | 09:34 |
| opendevreview | Takashi Kajinami proposed openstack/cinder-specs master: Replace obsolete PCRE packages https://review.opendev.org/c/openstack/cinder-specs/+/974071 | 10:13 |
| opendevreview | Nimesh Desai proposed openstack/cinder master: [IBM SVf Driver] Added support for volumegroup-snapshot for temporary volumegroup https://review.opendev.org/c/openstack/cinder/+/926844 | 10:38 |
| opendevreview | Nimesh Desai proposed openstack/cinder master: [IBM SVf Driver] Added support for restorefromsnapshot CLI for volumegroups https://review.opendev.org/c/openstack/cinder/+/916722 | 10:38 |
| opendevreview | Nimesh Desai proposed openstack/cinder master: [IBM SVf Driver] Added support of clone operation for volumegroup https://review.opendev.org/c/openstack/cinder/+/938771 | 10:38 |
| opendevreview | Nimesh Desai proposed openstack/cinder master: Doc: Documentation updates for ibm-svf-volumegroup feature. https://review.opendev.org/c/openstack/cinder/+/929906 | 10:38 |
| opendevreview | Nimesh Desai proposed openstack/cinder master: [IBM SVf driver] Adding support for individual volume snapshots https://review.opendev.org/c/openstack/cinder/+/949006 | 10:38 |
| opendevreview | Nimesh Desai proposed openstack/cinder master: [IBM SVf driver] Adding support for Individual volume clone https://review.opendev.org/c/openstack/cinder/+/951157 | 10:38 |
| MaxLamprecht[m] | rosmaita: hi, I addressed your feedback on https://review.opendev.org/c/openstack/os-brick/+/960573. Would be great if you could have another look :) | 12:34 |
| MaxLamprecht[m] | Also I have https://review.opendev.org/c/openstack/cinder/+/944016 which removes not needed behavior from the netapp drivers rest version(has already multiple +1 from the netapp folks) | 12:36 |
| opendevreview | Sandip Rajbanshi proposed openstack/cinder master: [IBM SVf driver] Adding support for CINDER plugin registration https://review.opendev.org/c/openstack/cinder/+/951829 | 12:43 |
| opendevreview | Sandip Rajbanshi proposed openstack/cinder master: [IBM SVf driver] Doc: Documentation updates for ib m-svf-callhome Feature https://review.opendev.org/c/openstack/cinder/+/974989 | 12:43 |
| MaxLamprecht[m] | Review would be also great here: https://review.opendev.org/c/openstack/cinder/+/972653. continue_backup should have the same error handling as create_backup | 12:43 |
| raghavendrat | hi | 14:01 |
| RanjithRethinam | hi | 14:02 |
| agalica | hello | 14:03 |
| harsh | o/ | 14:03 |
| Luzi | hi | 14:03 |
| erlon | \o | 14:03 |
| jbernard | heya all, brian is going to run the meeting this morning, im still snowed in and school is closed | 14:04 |
| simondodsley | o/ | 14:04 |
| mhen | o/ | 14:05 |
| lutimura | o/ | 14:05 |
| opendevreview | Ivan Anfimov proposed openstack/cinder-specs master: Remove outdated comment in setup.py https://review.opendev.org/c/openstack/cinder-specs/+/974680 | 14:08 |
| rosmaita | and i was stuck in a meeting and forgot | 14:12 |
| rosmaita | #startmeeting cinder | 14:12 |
| opendevmeet | Meeting started Wed Jan 28 14:12:18 2026 UTC and is due to finish in 60 minutes. The chair is rosmaita. Information about MeetBot at http://wiki.debian.org/MeetBot. | 14:12 |
| opendevmeet | Useful Commands: #action #agreed #help #info #idea #link #topic #startvote. | 14:12 |
| opendevmeet | The meeting name has been set to 'cinder' | 14:12 |
| rosmaita | forgot the reminder ping, so here it comes | 14:12 |
| rosmaita | 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 kaisers | 14:13 |
| rosmaita | cinder meeting happening now! | 14:13 |
| rosmaita | looks like we already did roll call, but let's do it again for the record | 14:13 |
| rosmaita | #topic roll call | 14:13 |
| erlon | hi all again | 14:13 |
| agalica | o7 | 14:13 |
| harsh | o/ | 14:15 |
| Sai | o/ | 14:15 |
| RanjithRethinam | hi | 14:15 |
| rosmaita | simondodsley mhen Luzi and lutimura are here as well | 14:16 |
| rosmaita | ok, let's get started | 14:16 |
| rosmaita | #topic announcements | 14:16 |
| rosmaita | looks like there aren't any! | 14:16 |
| RanjithRethinam | can we pls approve this PR: https://review.opendev.org/c/openstack/cinder/+/971739. VMStore CI passed, zuul looks happy, all review comments are addressed | 14:19 |
| nimeshdesai | This patch was reviewed in the past and also received +2. Need reviews again - 926844: [IBM SVf Driver] Added support for volumegroup-snapshot for temporary volumegroup | https://review.opendev.org/c/openstack/cinder/+/926844 | 14:19 |
| nimeshdesai | Also need one more +2 for - 959683 - [SVf] Define meaningful names for error constants | 14:19 |
| rosmaita | those aren't quite "announcements", but OK | 14:20 |
| rosmaita | #topic ai-generated code policy | 14:20 |
| agalica | vibe coding now required? | 14:20 |
| agalica | /s | 14:20 |
| rosmaita | i don't even know what that meand? | 14:21 |
| rosmaita | and i can't type | 14:21 |
| agalica | telling the AI to write the program and then accepting the output without checking it | 14:21 |
| agalica | more or less :) | 14:21 |
| rosmaita | gotcha | 14:21 |
| lutimura | the fact that you don't know what it is is a good sign, i guess | 14:21 |
| rosmaita | yeah, first of all, we don't want to do that! | 14:21 |
| rosmaita | we have seen a few patches lately that look like they have been ai-assisted at least | 14:22 |
| rosmaita | that is ok, but | 14:22 |
| erlon | "Apply a higher level of scrutiny to contributions created using AI tools" | 14:22 |
| rosmaita | openstack requirement is that you must identify this clearly in the commit message | 14:22 |
| rosmaita | and say where the ai-assistance happened so that we can do what erlon said | 14:23 |
| jbernard | https://openinfra.org/legal/ai-policy | 14:23 |
| rosmaita | so please review ^^ and pay attention to the "contributor checklist" and the "reviewer checklist" | 14:23 |
| erlon | I haven't noticed any of that in my reviews. I wonder if people will really disclose the tools they used | 14:23 |
| rosmaita | eric did note this in one of his patches: https://review.opendev.org/c/openstack/cinder/+/972476 | 14:25 |
| rosmaita | but he didn't say what exactly Claude Code did | 14:26 |
| erlon | ah nice | 14:26 |
| rosmaita | i think i've seen abhishekk over at glance say specifically that Claude code was used for unit test generation | 14:27 |
| rosmaita | in any case, please read and follow the policy | 14:27 |
| rosmaita | #link https://openinfra.org/legal/ai-policy | 14:27 |
| rosmaita | that's all from me | 14:28 |
| rosmaita | next item is 2 review requests for patches i thought had merged already | 14:29 |
| rosmaita | both have 1 +2 | 14:30 |
| erlon | yeah, they are around for quite a while | 14:31 |
| erlon | rajat had an objection in the os brick one, but from our discussions on the ptg, I understood that he understood my rationale on his comment | 14:32 |
| erlon | But he never came back to re-review it | 14:32 |
| erlon | And we are dragging this for very long | 14:32 |
| rosmaita | which patch is that? the 2 on the agenda are both cinder patches | 14:33 |
| eharney | re: AI policy, https://review.opendev.org/c/openstack/cinder/+/959105 was a concern | 14:33 |
| eharney | wait, wrong patch, never mind | 14:34 |
| rosmaita | btw, i will commit to reviewing both https://review.opendev.org/c/openstack/cinder/+/836083 and https://review.opendev.org/c/openstack/cinder/+/658283 today, | 14:34 |
| rosmaita | so please nobody bug me for other reviews | 14:34 |
| erlon | sorry, pasted the wrong one | 14:34 |
| erlon | this is the one I meant; https://review.opendev.org/c/openstack/os-brick/+/955379 | 14:35 |
| erlon | rosmaita: replaced/fixed it in the agenda too | 14:36 |
| rosmaita | ok, well, i am only committing to the ones i listed above, i don't want to get involved in the multiple sessions controversy | 14:37 |
| rosmaita | but, that os-brick patch has one +2 | 14:38 |
| eharney | IIRC the concept for 955379 is fine, but i haven't dug through the code yet | 14:40 |
| rosmaita | #topic open discussion | 14:42 |
| erlon | eharney: I would appreciate a second core looking at that, since I havent being able to get rajats opinion on that. | 14:42 |
| erlon | I mean, his opinion after my explanation | 14:42 |
| rosmaita | ok, anybody got anything? | 14:45 |
| raghavendrat | hi, may i ask for reviews | 14:46 |
| rosmaita | you may ask ... | 14:46 |
| simondodsley | there is a whole etherpad full of review requests... | 14:47 |
| raghavendrat | we have 3 patches. | 14:47 |
| simondodsley | those need to be addressed as people have been using that | 14:47 |
| rosmaita | yes, that is a better idea, please update the etherpad | 14:47 |
| raghavendrat | 1] New cinder driver for HPE Alletra MP | 14:47 |
| simondodsley | i don't think you should try and circumvent the process | 14:47 |
| raghavendrat | https://review.opendev.org/c/openstack/cinder/+/969373 | 14:48 |
| raghavendrat | i updated etherpad | 14:48 |
| simondodsley | https://etherpad.opendev.org/p/cinder-gazpacho-reviews | 14:48 |
| rosmaita | #link https://etherpad.opendev.org/p/cinder-gazpacho-reviews | 14:48 |
| rosmaita | jinx! | 14:48 |
| opendevreview | Eric Harney proposed openstack/cinder master: Prevent database errors when quota fields overflow https://review.opendev.org/c/openstack/cinder/+/972476 | 14:48 |
| simondodsley | please add at the bottom of the lists - for fairness sake | 14:49 |
| rosmaita | ok, people with time for reviews, please look at the etherpad ^^ to help out your fellow community members | 14:49 |
| rosmaita | and if you use AI, please follow the guidelines and say so and where it was used | 14:49 |
| RanjithRethinam | Add New Storage Driver for VMstore NFS - https://review.opendev.org/c/openstack/cinder/+/971739 (added to the etherpad) | 14:50 |
| simondodsley | one more question - will the PTGs ever be in-person again? I hate these virtual ones | 14:50 |
| rosmaita | simondodsley: don't know, guess it's a foundation question | 14:50 |
| simondodsley | given we don't really have Summit sanymore, we don't get to meet up properly now | 14:50 |
| simondodsley | i'll ask Kendall | 14:51 |
| rosmaita | yeah, now that we are part of the Linux Foundation, there should be some possibilities for co-location with some other meetings | 14:51 |
| rosmaita | ok, that's all for today ... have a productive week! | 14:52 |
| rosmaita | #endmeeting | 14:52 |
| opendevmeet | Meeting ended Wed Jan 28 14:52:48 2026 UTC. Information about MeetBot at http://wiki.debian.org/MeetBot . (v 0.1.4) | 14:52 |
| opendevmeet | Minutes: https://meetings.opendev.org/meetings/cinder/2026/cinder.2026-01-28-14.12.html | 14:52 |
| opendevmeet | Minutes (text): https://meetings.opendev.org/meetings/cinder/2026/cinder.2026-01-28-14.12.txt | 14:52 |
| opendevmeet | Log: https://meetings.opendev.org/meetings/cinder/2026/cinder.2026-01-28-14.12.log.html | 14:52 |
| opendevreview | Jason Paroly proposed openstack/cinder-tempest-plugin master: Fix volume type test extra_specs https://review.opendev.org/c/openstack/cinder-tempest-plugin/+/974914 | 15:57 |
| *** fprzewozny is now known as fprzewozn | 18:59 | |
| opendevreview | Zachary Mark Raines proposed openstack/cinder master: Add 512e/4k disk geometry configuration https://review.opendev.org/c/openstack/cinder/+/658283 | 19:23 |
| opendevreview | Ade Lee proposed openstack/cinder master: Add region_name support for Glance and Swift backup driver https://review.opendev.org/c/openstack/cinder/+/973447 | 19:44 |
| opendevreview | Ade Lee proposed openstack/cinder master: Add region_name support for Barbican key migration https://review.opendev.org/c/openstack/cinder/+/973448 | 19:44 |
| opendevreview | Abhishek Gupta proposed openstack/cinder master: Dell PowerMax: Improve UT coverage https://review.opendev.org/c/openstack/cinder/+/959105 | 20:16 |
Generated by irclog2html.py 4.0.0 by Marius Gedminas - find it at https://mg.pov.lt/irclog2html/!