| *** mhen_ is now known as mhen | 02:55 | |
| harsh | Received one +2 on : [IBM SVf Driver] Added support for volumegroup-snapshot for temporary volumegroup | https://review.opendev.org/c/openstack/cinder/+/926844 | 06:56 |
|---|---|---|
| harsh | Requesting the core reviewers for a second +2 please. | 06:57 |
| opendevreview | Dakshina Purohit proposed openstack/cinder master: [IBM SVf driver] Adding support for individual volume snapshots https://review.opendev.org/c/openstack/cinder/+/949006 | 07:54 |
| opendevreview | Dakshina Purohit proposed openstack/cinder master: Doc: Documentation updates for ibm-svf-volumegroup feature. https://review.opendev.org/c/openstack/cinder/+/958465 | 08:17 |
| Jyotsna | test message | 11:10 |
| harsh | hello.. I see that there is a cinder API : group-type-update which can be used to retype a group. But i don't find the driver.py code for this API which calls the backend driver code. Can someone please point me in the right direction? | 11:15 |
| Jyotsna | Hi Everyone, We have proposed new cinder driver (New cinder driver for HPE Alletra MP) which has few critical features which customers are awaiting from long time. | 11:16 |
| Jyotsna | May I request core reviewers to review the driver and help us get into GG release. | 11:16 |
| Jyotsna | https://review.opendev.org/c/openstack/cinder/+/969373 Refer link for review | 11:18 |
| Jyotsna | @erlon you have provided some comments on new driver https://review.opendev.org/c/openstack/cinder/+/969373, we have addressed it. Can you please re-review the changes? | 11:39 |
| opendevreview | Prasant Padhi proposed openstack/cinder master: Dell Powerstore: Add support driver assisted volume migration This patch implements volume migration for Powerstore storage array https://review.opendev.org/c/openstack/cinder/+/971952 | 12:52 |
| opendevreview | Silvan Kaiser proposed openstack/cinder master: Fix missing encryption params in Quobyte driver https://review.opendev.org/c/openstack/cinder/+/899706 | 13:24 |
| opendevreview | Silvan Kaiser proposed openstack/cinder master: Fix copy_image_to_volume for Quobyte driver https://review.opendev.org/c/openstack/cinder/+/922082 | 13:24 |
| opendevreview | Silvan Kaiser proposed openstack/cinder master: Report provisioning in driver stats for Quobyte driver https://review.opendev.org/c/openstack/cinder/+/920875 | 13:24 |
| opendevreview | Silvan Kaiser proposed openstack/cinder master: Quobyte: adapt to image_utils fixes https://review.opendev.org/c/openstack/cinder/+/953966 | 13:24 |
| opendevreview | Silvan Kaiser proposed openstack/cinder master: quobyte: Fix extending attached volumes #1870367 https://review.opendev.org/c/openstack/cinder/+/966186 | 13:45 |
| RanjithRethinam | hi | 14:01 |
| raghavendrat | hi | 14:04 |
| VolodymyrBoiko[m] | hi | 14:04 |
| jbernard | #startmeeting cinder | 14:04 |
| opendevmeet | Meeting started Wed Jan 14 14:04:53 2026 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 | #link https://etherpad.opendev.org/p/cinder-gazpacho-meetings | 14:05 |
| jbernard | 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: courtesy reminder | 14:05 |
| jbernard | #topic roll call | 14:05 |
| rosmaita | o/ | 14:05 |
| Luzi | o/ | 14:05 |
| yuval | o/ | 14:06 |
| gireesh | o/ | 14:06 |
| agalica | O/ | 14:06 |
| raghavendrat | hello | 14:07 |
| Anoop_Shukla | O/ | 14:07 |
| RanjithRethinam | o/ | 14:07 |
| jbernard | hello everyone | 14:07 |
| jbernard | thanks for joining | 14:07 |
| jbernard | #topic annoucements | 14:07 |
| jbernard | we are currently in R12 as per the cycle schedule at https://releases.openstack.org/gazpacho/schedule.html | 14:08 |
| jbernard | make that R11 | 14:08 |
| jbernard | looking at early Feb (first week) for a midcycle meetup | 14:09 |
| jbernard | how does that sound? we could do last week of Jan too | 14:10 |
| yuval | I have a bug fix so my deadline is 20feb? | 14:10 |
| harshailani | 0/ | 14:10 |
| jbernard | yuval: bug fix in os-brick? | 14:10 |
| agalica | I personally vote for mid-Feb | 14:10 |
| yuval | no, cinder | 14:10 |
| yuval | I also have a change is os-brick though | 14:11 |
| yuval | I wrote it in the agenda for today meeting | 14:11 |
| jbernard | yuval: ok, hang on for a moment | 14:12 |
| jbernard | agalica: that's doable, it's just a function of how much time we want to have between the meetup and feature freeze | 14:12 |
| jbernard | which is last week of Feb | 14:13 |
| RanjithRethinam | Hi,can someone pls review this PR https://review.opendev.org/c/openstack/cinder/+/971739, its reviewed and approved by Sean, and VMStore CI and Zuul looks happy | 14:13 |
| agalica | Gotcha! | 14:15 |
| jbernard | i will assume silence means any week is fine, mid Feb (13th-ish) unless there are objections, ill send a mail to the list | 14:15 |
| jayaanand | hi | 14:16 |
| jbernard | Luzi: really quick, simon left some formatting comments on your spec, are you able to incorporate those into an update? id like to land that as soon as we can | 14:16 |
| Luzi | yeah i will look through it | 14:16 |
| jbernard | #topic yuval's bugfix | 14:16 |
| jbernard | yuval: ^ you're up | 14:16 |
| jbernard | Luzi: thanks! | 14:16 |
| yuval | yea so basically the change is that I added configuration to os-brick | 14:17 |
| yuval | its only affecting lightbits driver | 14:18 |
| yuval | so you can set some variables like port and address service we use | 14:18 |
| yuval | we have configuration to many things cinder/nova... I dont see the issue here | 14:19 |
| yuval | I was requested to bring it up here to discussion | 14:19 |
| jbernard | i don't see a problem what that, are you anticipating pushback? | 14:20 |
| rosmaita | yeah, isn't it a code smell to have config opts in a library? | 14:20 |
| yuval | yes, there were people with bad feeling about it. so I am here to talk about it | 14:20 |
| yuval | yea rosmaita | 14:20 |
| rosmaita | at this point, i don't think any of the specialized connectors have config opts in os-brick | 14:21 |
| erlon | \o late joining ... | 14:21 |
| rosmaita | so i just want to make sure we think carefully about whether we want to set this precedent | 14:21 |
| yuval | we do have opts.py in os-brick | 14:22 |
| yuval | example: cfg.IntOpt('wait_mpath_device_attempts', | 14:22 |
| rosmaita | sure, but that will apply to many connectors | 14:23 |
| jbernard | rosmaita: for hardcoded configuration variables, what is the preferred alternative to effect a change in one of those? | 14:24 |
| rosmaita | maybe it's just me, but it seems different to have vendor specific opts in os-brick that could be handled by the cinder driver | 14:24 |
| yuval | only linuxscsi is using those opts | 14:24 |
| jbernard | rosmaita: wouldn't those need to be duplicated in all services consuming brick in that case? | 14:25 |
| rosmaita | not sure what you mean, these are lightos specific options we are talking about | 14:26 |
| yuval | if I will add my opts to opts.py its better? | 14:26 |
| kaisers | o/ | 14:26 |
| rosmaita | no, if we allow them, i think they should be defined in with the connector | 14:27 |
| jbernard | rosmaita: if nova calls brick/lightos, then cinder's brick-specific config would not be available? | 14:27 |
| eharney | yeah, these options would need to be set for both cinder and nova configs right? | 14:27 |
| erlon | I'm also for the opinion that the options should leave either on Cinder or Nova. Especially if it's a driver specific option (for that case in Cinder). You can always pass that option in the connector properties | 14:27 |
| yuval | I agree so it will stay isolated | 14:27 |
| jbernard | eharney: that's what im getting at | 14:27 |
| yuval | yea I will set them in both nova.conf and cinder.conf | 14:27 |
| rosmaita | right, so if we define them in cinder, then like erlon says, they would be passed in the connector properties | 14:28 |
| yuval | but I am doing this stuff before I init a connector (If I remember right) | 14:28 |
| rosmaita | so would only need to be set in one place? | 14:28 |
| eharney | putting them in brick seems likely to introduce complications if any changes are made in the future, because then you end up with a weird matrix where the nova/cinder conf depends on the version of brick deployed etc | 14:28 |
| erlon | yuval: I didn't find the link for the change can you share? | 14:30 |
| yuval | https://review.opendev.org/c/openstack/os-brick/+/957599/11/os_brick/initiator/connectors/lightos.py#155 | 14:31 |
| agalica | I’m on team erlon, eharney, and rosmaita here | 14:31 |
| jbernard | yuval: is there a technical reason the options cannot be passed in from the caller? that seems to be the perferred approach | 14:32 |
| yuval | I need to re-check - to answer that fully | 14:33 |
| eharney | it should be possible | 14:33 |
| yuval | I will check that | 14:34 |
| jbernard | ok, report back and let us know | 14:35 |
| yuval | ok - ok, I will give an example hold on | 14:35 |
| yuval | this change was requested because of using cinder/nova inside dockers. so if cinder is on docker a with network A and nova is on docker b on Network B I need to configure it per docker | 14:36 |
| yuval | I am from that docker want to check some service (we use) is alive through the network | 14:38 |
| yuval | I need to reach it with context of the Network I live in | 14:38 |
| opendevreview | Eric Harney proposed openstack/cinder master: Prevent database errors when quota fields overflow https://review.opendev.org/c/openstack/cinder/+/972476 | 14:40 |
| rosmaita | ok, so that sounds like you would actually want to be able to set the value in multiple places | 14:41 |
| opendevreview | Nilesh Thathagar proposed openstack/cinder master: Dell PowerFlex: Fixed volume detach issue https://review.opendev.org/c/openstack/cinder/+/973232 | 14:41 |
| jbernard | rosmaita: right, is that preferred over a shared brick config? | 14:42 |
| yuval | yes, the config files sounds the best way for me | 14:42 |
| yuval | each service have its own config | 14:42 |
| yuval | value need to be changed per server | 14:43 |
| jbernard | yes, you just need to make sure they're all in sync | 14:43 |
| yuval | the service (discovery-client) is not 1 for all other servicer (cinder/nova) and everybody checks it. the service lives in the machine parallel to the cinder/nova. 1 for each server | 14:45 |
| jayaanand | need core attraction for NetApp changes:https://review.opendev.org/c/openstack/cinder/+/968831 | 14:45 |
| yuval | ok, I understand why its confusing. I guess I didnt explain myself good enough in the commit | 14:46 |
| rosmaita | i think you have made a good case ... would be nice to say something in the commit message about why is makes sense for these to be defined in os_brick | 14:48 |
| erlon | yuval: I don't think I followed your explanation. Could you provide a clearer commit message that explains the problem you're trying to solve and why it needs to be addressed this way? | 14:48 |
| erlon | :) | 14:48 |
| yuval | I for sure update the commit | 14:48 |
| jbernard | #action yuval to update lightos brick commit message, further discussion if needed | 14:49 |
| yuval | but lets wait 2 more minutes - I will explain why its needed so it wont drag to the next meeting | 14:49 |
| jbernard | #action jbernard to send mail about midcycle | 14:49 |
| jbernard | yuval: sure, we have 11 minutes remaining, i thnk that's all on the agenda for today | 14:50 |
| erlon | one thing | 14:51 |
| yuval | I have an instance with docker network 10.10.10.1 the dc is at 10.10.10.2. I have another nova instance in docker with network 20.20.20.1 with dc in 20.20.20.2. I need to configure os brick to reach them | 14:51 |
| yuval | I can not put it in the connector | 14:51 |
| yuval | like in the cinder.conf | 14:52 |
| erlon | I wanted to call attention on this other OS brick patch: https://review.opendev.org/c/openstack/os-brick/+/955379 | 14:52 |
| erlon | It has been the queue for quite some time, and we are getting pressure from our customer to get it fixed | 14:52 |
| yuval | so we agree the change is needed? | 14:54 |
| RanjithRethinam | @jbernard can someone pls review this PR for VMStore Cinder driver: https://review.opendev.org/c/openstack/cinder/+/971739 | 14:54 |
| yuval | rosmaita: erlon: jbernard: | 14:54 |
| jbernard | RanjithRethinam: can you resolve the merge conflict? | 14:54 |
| harshailani | hello.. just wanted to ask a second +2 on the review: [IBM SVf Driver] Added support for volumegroup-snapshot for temporary volumegroup | https://review.opendev.org/c/openstack/cinder/+/926844 | 14:55 |
| jbernard | erlon: noted, ill try to mention it to rajat as he has reviewed previously | 14:56 |
| rosmaita | yuval: i think that makes sense ... os-brick has to use the discovery client, the location of the client may be specific to where os-brick is running, the cinder driver only knows the location of the discovery client cinder itself can use, so the driver cannot pass this info, it needs to be defined for each entity running os-brick | 14:57 |
| rosmaita | (that's my understanding, correct me if i'm inaccurate!) | 14:57 |
| yuval | exactly yea, so I will update my commit with more info, why that is needed + example | 14:58 |
| yuval | Thank you for your time and discussion | 14:58 |
| rosmaita | ok, cool ... also you mention this in your release note, but you might want to say in the commit message that you have defined sensible defaults so that this may not be needed for all installations using lightos | 14:59 |
| jbernard | one minute remaining, last call | 14:59 |
| raghavendrat | it would be great to receive review on below patch: | 14:59 |
| raghavendrat | New cinder driver for HPE Alletra MP | 14:59 |
| raghavendrat | https://review.opendev.org/c/openstack/cinder/+/969373 | 14:59 |
| erlon | rosmaita: when you add a config like that, where's the place the user need to set? cinder.conf? nova.conf? both? | 14:59 |
| raghavendrat | We have kept minimal code. Thanks | 15:00 |
| rosmaita | erlon: in the [os_brick] section of cinder.conf or nova.conf | 15:00 |
| rosmaita | both | 15:00 |
| erlon | raghavendrat: I have that to my review list and I'll review it later this week | 15:00 |
| raghavendrat | thanks erlon | 15:01 |
| erlon | hmm,right thanks for clarifying | 15:01 |
| jbernard | #endmeeting | 15:02 |
| opendevmeet | Meeting ended Wed Jan 14 15:02:39 2026 UTC. Information about MeetBot at http://wiki.debian.org/MeetBot . (v 0.1.4) | 15:02 |
| opendevmeet | Minutes: https://meetings.opendev.org/meetings/cinder/2026/cinder.2026-01-14-14.04.html | 15:02 |
| opendevmeet | Minutes (text): https://meetings.opendev.org/meetings/cinder/2026/cinder.2026-01-14-14.04.txt | 15:02 |
| opendevmeet | Log: https://meetings.opendev.org/meetings/cinder/2026/cinder.2026-01-14-14.04.log.html | 15:02 |
| jbernard | thanks everyone | 15:02 |
| raghavendrat | thank you | 15:02 |
| noonedeadpunk | simondodsley: if around I'd love to ask you about third point (ordering) wrt https://review.opendev.org/c/openstack/cinder-specs/+/958838 | 15:06 |
| noonedeadpunk | As I _think_ this is what happens now in fact | 15:06 |
| noonedeadpunk | though backup_create_containers is enforced on the backend level | 15:07 |
| noonedeadpunk | I think I am missing nuance in there you was trying to point out in review | 15:07 |
| opendevreview | Josephine Seifert proposed openstack/cinder-specs master: Re-propose Spec for the Cinder part of the Image Encryption https://review.opendev.org/c/openstack/cinder-specs/+/964777 | 15:11 |
| opendevreview | Josephine Seifert proposed openstack/cinder-specs master: Re-propose Spec for the Cinder part of the Image Encryption https://review.opendev.org/c/openstack/cinder-specs/+/964777 | 15:23 |
| Luzi | simondodsley, jbernard I think I addressed all the comments in the spec | 15:26 |
| opendevreview | Takashi Kajinami proposed openstack/cinder master: Migrate upper functional job to Python 3.13 https://review.opendev.org/c/openstack/cinder/+/970038 | 15:26 |
| zigo | In my Debian package, I backported this: https://review.opendev.org/c/openstack/python-openstackclient/+/971255 | 15:42 |
| zigo | Though now, it's failing in Debian autopkgtest: https://ci.debian.net/packages/p/python-openstackclient/testing/amd64/67788053/ | 15:42 |
| zigo | It doesn't make sense to me. Does someone understand? | 15:42 |
| zigo | I'm asking here because it concerns the volume API. | 15:42 |
| zigo | lutimura: stephenfin: ^ | 15:43 |
| opendevreview | Dmitriy Rabotyagov proposed openstack/cinder-specs master: Add spec for backup container name templates https://review.opendev.org/c/openstack/cinder-specs/+/962306 | 16:08 |
| opendevreview | Dmitriy Rabotyagov proposed openstack/cinder-specs master: Add spec for restricting cinder-backup container creation https://review.opendev.org/c/openstack/cinder-specs/+/958838 | 16:58 |
| opendevreview | Dakshina Purohit proposed openstack/cinder master: [IBM SVf Driver] Added support for volumegroup-snapshot for temporary volumegroup https://review.opendev.org/c/openstack/cinder/+/926844 | 18:06 |
| opendevreview | Dakshina Purohit proposed openstack/cinder master: [IBM SVf Driver] Added support for restorefromsnapshot CLI for volumegroups https://review.opendev.org/c/openstack/cinder/+/916722 | 18:06 |
| opendevreview | Dakshina Purohit proposed openstack/cinder master: [IBM SVf Driver] Added support of clone operation for volumegroup https://review.opendev.org/c/openstack/cinder/+/938771 | 18:06 |
| opendevreview | Dakshina Purohit proposed openstack/cinder master: Doc: Documentation updates for ibm-svf-volumegroup feature. https://review.opendev.org/c/openstack/cinder/+/929906 | 18:06 |
| opendevreview | Dakshina Purohit proposed openstack/cinder master: [IBM SVf driver] Adding support for individual volume snapshots https://review.opendev.org/c/openstack/cinder/+/949006 | 18:06 |
| opendevreview | Eric Harney proposed openstack/cinder master: WIP: cinder/monkey_patch.py for threading mode https://review.opendev.org/c/openstack/cinder/+/970332 | 19:31 |
| opendevreview | Alexey Khodos proposed openstack/cinder master: Add New Storage Driver for VMstore NFS https://review.opendev.org/c/openstack/cinder/+/971739 | 22:14 |
| *** agalica_ is now known as agalica | 23:28 | |
Generated by irclog2html.py 4.0.0 by Marius Gedminas - find it at https://mg.pov.lt/irclog2html/!