| *** erlon9 is now known as erlon | 03:07 | |
| opendevreview | jayaanand borra proposed openstack/cinder-tempest-plugin master: Add image-cache cross-pool tempest tests https://review.opendev.org/c/openstack/cinder-tempest-plugin/+/989987 | 03:10 |
|---|---|---|
| opendevreview | Adam Harwell proposed openstack/cinder master: image_utils: use concurrency-aware tpool wrapping https://review.opendev.org/c/openstack/cinder/+/995001 | 06:01 |
| rm_work | ok, that cleaned up a lot with the changes that were incorporated into the parent, should be more minimal/focused now | 06:07 |
| *** erlon5 is now known as erlon | 09:34 | |
| opendevreview | Prudhvi proposed openstack/cinder master: Implement per-pool image-volume cache promotion https://review.opendev.org/c/openstack/cinder/+/988315 | 09:38 |
| *** auniyal2 is now known as auniyal_1 | 10:11 | |
| *** auniyal_1 is now known as auniyal__ | 10:11 | |
| opendevreview | Rajat Dhasmana proposed openstack/cinder-specs master: Propose replication v3 spec https://review.opendev.org/c/openstack/cinder-specs/+/985952 | 10:43 |
| opendevreview | jayaanand borra proposed openstack/cinder master: Implement per-pool image-volume cache promotion https://review.opendev.org/c/openstack/cinder/+/988315 | 10:45 |
| opendevreview | Thomas Goirand proposed openstack/cinder master: Use sample_max to make cinder reproducible https://review.opendev.org/c/openstack/cinder/+/996308 | 11:33 |
| opendevreview | Thomas Goirand proposed openstack/cinder master: Use sample_max to make cinder reproducible https://review.opendev.org/c/openstack/cinder/+/996308 | 12:36 |
| opendevreview | Leo Jenkins proposed openstack/cinder-specs master: Support for Hostname-based Ceph Mon References https://review.opendev.org/c/openstack/cinder-specs/+/1000721 | 13:15 |
| jbernard | #startmeeting cinder | 14:00 |
| opendevmeet | Meeting started Wed Aug 12 14:00:36 2026 UTC and is due to finish in 60 minutes. The chair is jbernard. Information about MeetBot at http://wiki.debian.org/MeetBot. | 14:00 |
| opendevmeet | Useful Commands: #action #agreed #help #info #idea #link #topic #startvote. | 14:00 |
| opendevmeet | The meeting name has been set to 'cinder' | 14:00 |
| yuval | is there a meeting? | 14:00 |
| jbernard | #link https://etherpad.opendev.org/p/cinder-hibiscus-meetings | 14:00 |
| yuval | 0/ | 14:00 |
| 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 aloke_dev Anoop_Shukla erlon Jyotsna: courtesy ping (cinder meeting) | 14:00 |
| jbernard | #topic roll call | 14:01 |
| jbernard | o/ | 14:01 |
| simondodsley | o/ | 14:01 |
| agalica | o/ | 14:01 |
| Jyotsna | o/ | 14:01 |
| Luzi | o/ | 14:01 |
| Anoop_Shukla | O/ | 14:01 |
| jbernard | hey everyone, im alive :) i managed to get quick sick a couple of weeks ago, feeling mostly better | 14:01 |
| kaisers | o/ | 14:02 |
| rosmaita | o/ | 14:02 |
| fqi | o/ | 14:03 |
| mhen | o/ | 14:03 |
| jbernard | #topic annoucements | 14:04 |
| jbernard | quickly, periodic tasks are all green | 14:04 |
| rosmaita | \o/ | 14:04 |
| jbernard | we need (probably) to have a second midcycle | 14:05 |
| Leo[m] | o/ | 14:05 |
| jbernard | and im thinking we should shoot for soon if possible | 14:05 |
| jbernard | the earliest would be next week | 14:05 |
| jbernard | for reference, this is a single day session lasing 2-3 hours | 14:06 |
| jbernard | is anyone opposed to holding it next friday? | 14:07 |
| jbernard | is a different day or week preferred? | 14:07 |
| rosmaita | that conflicts with the festival of reviews | 14:07 |
| erlon | \o | 14:07 |
| simondodsley | i'm open to that - replace the festival with it | 14:08 |
| jbernard | ahh, right. we could overwrite the festival, or push to the following week | 14:08 |
| rosmaita | another possibility would be to hold it on wednesday, 1400-1600 | 14:08 |
| rosmaita | everyone already has 1400-1500 blocked off | 14:08 |
| jbernard | iow overwrite out weekly meeting next week | 14:08 |
| jbernard | that works too, i would vote for that over friday personally | 14:09 |
| simondodsley | either | 14:09 |
| erlon | both work for me , but Friday is better | 14:09 |
| yuval | wedenesday is better | 14:09 |
| yuval | :) | 14:09 |
| rosmaita | i'd prefer wednesday, we are rapidly approaching M-3 | 14:10 |
| yuval | whats M-3 stand for? | 14:10 |
| yuval | milestone? | 14:10 |
| jbernard | yep | 14:11 |
| yuval | got it cause 28 is freeze right? | 14:11 |
| jbernard | ok, lets plan for wednesday next week, in place of our regular meeting | 14:12 |
| jbernard | i will send mail | 14:12 |
| agalica | the 28th of this month is freeze? | 14:12 |
| jbernard | that will give me some time to try to catch up on some of my backlog | 14:12 |
| jbernard | agalica: https://releases.openstack.org/hibiscus/schedule.html feature freeze | 14:14 |
| agalica | gotcha, thx | 14:14 |
| agalica | oof, none of my patches are really getting traction. thought there was more time | 14:14 |
| rosmaita | there is never enough time! | 14:15 |
| jbernard | #topic CBT support series (simon) | 14:17 |
| simondodsley | thanks - this is just a heads up to the vendors that we are creating CBT support | 14:18 |
| simondodsley | there is a series of patches listed in the reviews etherpad if people want to build their own support off those | 14:18 |
| Jyotsna | Yes we do have CNT in our roadmap for new driver, perhaps the driver get merge on time. | 14:19 |
| Jyotsna | *CBT | 14:19 |
| simondodsley | theree is also a spec which has been used to create the patch series: https://review.opendev.org/c/openstack/cinder-specs/+/985089 | 14:19 |
| yuval | are there driver that already implemented CBT? | 14:19 |
| simondodsley | the spec hasn't been approved yet, but it shouldbe good | 14:20 |
| simondodsley | only Ceph supports CBT today | 14:20 |
| Anoop_Shukla | NetApp plans to implement CBT in the next release. | 14:20 |
| yuval | k | 14:20 |
| simondodsley | this is for any other external system that wants to implement | 14:20 |
| Jyotsna | Thanks simon for sharing | 14:20 |
| simondodsley | there is full usage documentation in the patch series as well | 14:21 |
| simondodsley | including LVM support | 14:21 |
| Anoop_Shukla | Quick question: CBT will be applicable for all protocols yes? Including NFS? | 14:21 |
| simondodsley | it was designed so 3rd party backup vendors can use it as well | 14:21 |
| jbernard | nice, we can use some midcycle time to nail that down | 14:21 |
| jbernard | Anoop_Shukla: it should be, if you find an issue please raise it | 14:22 |
| simondodsley | Anoop_Shulka: that will be down to the vendors implementation | 14:22 |
| Anoop_Shukla | Right..we are currently analysing the support from our platform. That question came up during that analysis..so just making sure to clarify | 14:23 |
| jbernard | #topic NVMe-FC support series (simon) | 14:24 |
| simondodsley | Same again - vendor heads up - this is adding full NVMe-FC support to cinder and os-brick, including support for FCZM. There is LVM support as well, but Ubuntu doesn't ship nvme_fcloop so we can create tempest tests just yet | 14:24 |
| Anoop_Shukla | Another question was, are drivers allowed to have a binary dependency or a client dependency on the driver code? Or the binary client cannot be shipped via the openstack code? | 14:24 |
| simondodsley | the centos-9-stream nodeset is flakey so we can't get that to work currently, which would be the only place we could do the tempest tests | 14:25 |
| simondodsley | Anoop_Shukla: I don't beleive binaries can be shipped with openstack.... | 14:26 |
| jbernard | Anoop_Shukla: dependencies can be defined from what setuptools allows (requirements.txt), but things external to pypi are not available | 14:26 |
| jbernard | Anoop_Shukla: basically, no ;) | 14:26 |
| jbernard | Anoop_Shukla: those things (could) get pulled in a packaging time | 14:27 |
| Anoop_Shukla | Okay | 14:27 |
| Anoop_Shukla | Got it | 14:27 |
| yuval | appriciate for the heads up - what actually I need to validate/check? | 14:27 |
| simondodsley | you will need your vendor CI to run on that protocol and report back | 14:27 |
| yuval | if my backend does not use the NVME-FC still? | 14:28 |
| simondodsley | this add won't affect anyother nvme protocols, roce v2 or tcp | 14:29 |
| yuval | ok thanks | 14:29 |
| jbernard | thanks for working on those simon | 14:30 |
| jbernard | #topic drivers security stance (rosmaita) | 14:30 |
| rosmaita | #link https://review.opendev.org/c/openstack/cinder/+/996722 | 14:30 |
| rosmaita | i'd like to get some action on this patch | 14:31 |
| Anoop_Shukla | https://review.opendev.org/c/openstack/cinder/+/999310 | 14:31 |
| opendevreview | Merged openstack/cinder master: Imported Translations from Zanata https://review.opendev.org/c/openstack/cinder/+/1000238 | 14:31 |
| rosmaita | it's a little vague about what exactly we will be requiring | 14:31 |
| rosmaita | but i think that's ok, because the key point is to get the timeline set | 14:31 |
| Anoop_Shukla | NetApp patch to improve Cinder driver security stance - https://review.opendev.org/c/openstack/cinder/+/999310 | 14:31 |
| rosmaita | for when we will start kicking out/marking unsupported drivers | 14:32 |
| rosmaita | Anoop_Shukla: very nice! | 14:32 |
| Anoop_Shukla | Default to https and cert verify default to true for improving the security stance | 14:32 |
| rosmaita | simon has a big patch up that we can use to discuss details | 14:32 |
| Anoop_Shukla | Need some eyes to the patch as we target to merge it in this release | 14:33 |
| rosmaita | https://review.opendev.org/c/openstack/cinder/+/996554 | 14:33 |
| rosmaita | but my patch just says that we will start enforcing this with the Indri release, that is, 2027.1 | 14:34 |
| simondodsley | Anoop_Shukla: I'll look at it today | 14:35 |
| jbernard | rosmaita: does this count as a sufficient initial step related to what the foundation is hoping/asking for currently? | 14:35 |
| rosmaita | i think so ... the foundation is worried that bugs will be filed but not implemented | 14:36 |
| simondodsley | Anoop_Shukla - you should recheck to pick up the fix that reolves the openstacksdk-functional-devstack gate failure | 14:36 |
| rosmaita | so this gives us an enforcement mechanism | 14:37 |
| Anoop_Shukla | Thanks Simon..will pick up the change. Do you have the patch handy? | 14:37 |
| simondodsley | it's merged - so a recheck will work now | 14:37 |
| rosmaita | Anoop_Shukla: you may need to rebase on master | 14:38 |
| jbernard | rosmaita: excellent, thanks | 14:38 |
| simondodsley | true | 14:38 |
| Anoop_Shukla | @rosmaita: I think Manila is taking a stance of security in H release.. | 14:38 |
| rosmaita | well, since it would have to be done in the next week or so, i though H was too early | 14:38 |
| rosmaita | but, we could say H | 14:39 |
| Anoop_Shukla | Hmm..sooner the better :) | 14:39 |
| rosmaita | and then mark all the drivers (except maybe netapp, if your patch is merged) as unsupported | 14:39 |
| simondodsley | the doc pathc i created will have to be manually changed whenever a driver changes its status - there isn't an automatic update | 14:39 |
| Anoop_Shukla | haha | 14:39 |
| rosmaita | well, we aren't saying that nothing needs to happen until Indri (from the perspective of driver development) | 14:40 |
| simondodsley | and my driver-that is fine too rosmaita: | 14:40 |
| simondodsley | lol | 14:40 |
| rosmaita | ok, netapp and everpure | 14:40 |
| rosmaita | anyway, please leave comments on the patch, and we should discuss simon's patch at the midcycle next week | 14:42 |
| yuval | open discussion? | 14:43 |
| rosmaita | i think mhen has something first? | 14:43 |
| mhen | yes, thanks | 14:43 |
| mhen | (oh and sorry if putting this under "topics" directly was bad etiquette - should I have put it under open discussion?) | 14:43 |
| mhen | so, this is in the context of the ongoing image encryption contribution | 14:44 |
| rosmaita | no, it's a topic, i think | 14:44 |
| mhen | we were asked to address a resize function which is primarily used in remotefs/nfs based back | 14:44 |
| mhen | then we found volume encryption (which we need) not working with nfs right now and filed a launchpad bug | 14:44 |
| mhen | the potential fix from eharney linked in launchpad introduces behavior in Cinder that we see as problematic | 14:44 |
| mhen | #link: https://review.opendev.org/c/openstack/cinder/+/973870/comment/d923c431_09cca9b7/ | 14:44 |
| mhen | it breaks usage in Nova and `image create --volume` as it introduces qcow2-based encryption to Cinder, which is a behavior other code parts don't expect from Cinder | 14:44 |
| mhen | (also not known to os-brick) | 14:44 |
| mhen | so, one question would be if Cinder really would want to introduce new encryption mechanisms just to fix nfs? (with all the impact on os-brick, Nova etc.) | 14:45 |
| eharney | i haven't gone over the latest comment on 973870. but nfs volume qcow2 encryption is already present in Cinder | 14:45 |
| mhen | and more broadly: could we move forward with image encryption without supporting nfs/remotefs-based backends for the time being while this is being addressed? | 14:45 |
| mhen | eharney: it is? then I haven't looked close enough I guess | 14:46 |
| mhen | anyway, as written in the comment it breaks a few things | 14:46 |
| mhen | shouldn't that be known to os-brick then as well? | 14:46 |
| mhen | or am I misunderstanding things? | 14:46 |
| eharney | there are some holes in it (especially around handling volume<->image etc) but you can create encrypted volumes with that method and attach them in nova currently | 14:48 |
| Anoop_Shukla | So the encrypted image now gets decrypted before raw conversion after this patch? Just trying to understand the flow. | 14:49 |
| mhen | normally, our image encryption implementation assumes Cinder is using raw LUKS in every place, so we make sure that the image is converted to that encryptiono | 14:50 |
| mhen | if the image is already raw LUKS encrypted, it is used as-is | 14:50 |
| mhen | otherwise it is re-encrypted basically | 14:50 |
| Anoop_Shukla | okay | 14:50 |
| mhen | we do require an encrypted target volume though | 14:50 |
| Anoop_Shukla | right | 14:51 |
| mhen | however, we never expect Cinder to produce encrypted qcow2 currently | 14:51 |
| mhen | anyway, eharney if you have any pointers to existing encrypted qcow2 usage in Cinder feel free to comment on my comment and I'll look into it | 14:52 |
| mhen | we don't need to go into detail here (looking at the time) | 14:52 |
| mhen | I just wanted to know if it would be feasible to skip those backends for now in the image encryption contribution and add support later | 14:53 |
| rosmaita | mhen: let's definitely follow up at the midcycle next week | 14:53 |
| mhen | ok | 14:53 |
| jbernard | #topic open discussion | 14:55 |
| yuval | open discussion? | 14:56 |
| Jyotsna | HPE Alletra MP new driver patch (Recieved +2 and workflow +1 and ready for merge) https://review.opendev.org/c/openstack/cinder/+/969373 Requirement Patch: https://review.opendev.org/c/openstack/requirements/+/974322 requirement patch - Addressed the comments from zigo and released flowkit version + update patch with latest version. Require appoval from reviewers. As we are approaching towards feature freeze very less time left for new driver | 14:56 |
| yuval | I want to ask the core reviewers - what will help you for review? since I think there are a lot of patches going on | 14:57 |
| jbernard | Leo[m]: thanks for the hostname-based ceph patch, that's been on the list for a looong time | 14:57 |
| jbernard | yuval: ill be working a bit extra this week/weekend to try to catch up on what i can, ping me later this week, i should be able to make at least one pass | 14:58 |
| agalica | yuval: having solid reviews ahead of time on patches definitely helps. if there's less issues to find it's easier to review | 14:59 |
| agalica | speaking from a "community should be reviewing as well" perspective | 14:59 |
| Anoop_Shukla | May be we can sign up for patches voluntarily to offload.. | 15:00 |
| simondodsley | Make sure all patches are in the review etherpad, or we don't know to look at them | 15:00 |
| rosmaita | Jyotsna: i will check on the requirements patch later, and then prod some people once zuul is +1 (i only have +1 in that repo) | 15:00 |
| Jyotsna | sure thanks <rosmaita> i have ran recheck for zuul to pass so, if it passes we will need attention | 15:02 |
| simondodsley | and also make sure the patches go in the correct section of the etherpad | 15:02 |
| jbernard | ok, last call | 15:04 |
| Jyotsna | can we get attention from Sean McGinnis and Jens for requirement patch as they have +2 rights | 15:05 |
| jbernard | just to recap, we review session this friday and we will have our midcycle next wednesday in this time slot (plus an hour or so extended) | 15:05 |
| Jyotsna | Can core reviewers help here in getting there attention after +1 | 15:05 |
| agalica | review session is next friday, no? | 15:06 |
| jbernard | yes, sorry, next friday! | 15:06 |
| jbernard | lots of facetime next week | 15:06 |
| agalica | googletime, not facetime | 15:07 |
| jbernard | yup, in the general sense | 15:08 |
| agalica | sorry, that was a lame apple vs google joke | 15:08 |
| agalica | I should probably be destroyed | 15:08 |
| agalica | get that on the record | 15:09 |
| jbernard | done! | 15:09 |
| jbernard | ok, thanks everybody :) | 15:09 |
| jbernard | #endmeeting | 15:09 |
| opendevmeet | Meeting ended Wed Aug 12 15:09:20 2026 UTC. Information about MeetBot at http://wiki.debian.org/MeetBot . (v 0.1.4) | 15:09 |
| opendevmeet | Minutes: https://meetings.opendev.org/meetings/cinder/2026/cinder.2026-08-12-14.00.html | 15:09 |
| opendevmeet | Minutes (text): https://meetings.opendev.org/meetings/cinder/2026/cinder.2026-08-12-14.00.txt | 15:09 |
| opendevmeet | Log: https://meetings.opendev.org/meetings/cinder/2026/cinder.2026-08-12-14.00.log.html | 15:09 |
| agalica | bed time for me I guess, night all | 15:09 |
| agalica | lol | 15:09 |
| yuval | thanks! | 15:09 |
| Jyotsna | thanks | 15:10 |
| zigo | Jyotsna: Added my +1, thanks for considering my remarks, much appreciated. | 15:13 |
| Jyotsna | thanks a lot zigo | 15:13 |
| Jyotsna | <rosmaita> are you around? we got zuul verified on patch and +1 from zigo can you please also provide +1 | 15:15 |
| Jyotsna | also can you request +2 reviewers for some attention | 15:15 |
| Jyotsna | <zigo> can you please provide +1 on patch? | 15:16 |
| Jyotsna | sorry i am not able to see +1 from you | 15:17 |
| rosmaita | Jyotsna: looking | 15:17 |
| zigo | Jyotsna: When running tests, I still have hpe_storage_flowkit_py/tests/v3/test_session.py and hpe_storage_flowkit_py/tests/v3/test_task.py that are failing because they are trying to use the "src" folder. | 15:17 |
| Jyotsna | it is addressed in init_py as an alias is added | 15:18 |
| Jyotsna | so even if they are using src in import it will not fail | 15:18 |
| zigo | Jyotsna: That's what I'm getting when building the Debian package: | 15:21 |
| zigo | https://paste.opendev.org/show/bKLMOTq0rW4MewtFq0tK/ | 15:21 |
| zigo | So, the issue seems to be mocking, not import. | 15:21 |
| zigo | I can write a debian specific patch for it though... | 15:21 |
| Jyotsna | ok i will check this | 15:22 |
| zigo | Jyotsna: I'm actually trying to fix it ! :) | 15:23 |
| zigo | Hum... not sure how, actually. | 15:28 |
| *** dviroel is now known as dviroel_lunch | 15:53 | |
| opendevreview | Simon Dodsley proposed openstack/cinder master: Add changed-block tracking contract and os-list_changed_blocks API https://review.opendev.org/c/openstack/cinder/+/990923 | 16:04 |
| opendevreview | Simon Dodsley proposed openstack/cinder master: Consume Changed Block Tracking in cinder-backup https://review.opendev.org/c/openstack/cinder/+/1000747 | 16:04 |
| opendevreview | Simon Dodsley proposed openstack/cinder-tempest-plugin master: Add Changed Block Tracking (CBT) scenario tests https://review.opendev.org/c/openstack/cinder-tempest-plugin/+/995741 | 16:06 |
| *** dviroel_lunch is now known as dviroel | 17:17 | |
| opendevreview | Anoop Kumar Shukla proposed openstack/cinder master: NetApp: enrich periodic Cinder EMS payloads https://review.opendev.org/c/openstack/cinder/+/996205 | 17:19 |
Generated by irclog2html.py 4.1.0 by Marius Gedminas - find it at https://mg.pov.lt/irclog2html/!