| opendevreview | jayaanand borra proposed openstack/cinder master: NetApp ZAPI: Replace urllib with requests library in NaServer https://review.opendev.org/c/openstack/cinder/+/992289 | 05:07 |
|---|---|---|
| opendevreview | Nilesh Thathagar proposed openstack/cinder master: Dell PowerFlex Driver: Fix NVMe/TCP host creation with long host names https://review.opendev.org/c/openstack/cinder/+/996392 | 05:52 |
| opendevreview | Susmita Poddar proposed openstack/cinder master: HPE 3par - support for autoSynchronous as part of RCG creation https://review.opendev.org/c/openstack/cinder/+/967158 | 05:57 |
| opendevreview | jayaanand borra proposed openstack/cinder master: NetApp ZAPI: Replace urllib with requests library in NaServer https://review.opendev.org/c/openstack/cinder/+/992289 | 07:57 |
| opendevreview | jayaanand borra proposed openstack/cinder master: NetApp ZAPI: Replace urllib with requests library in NaServer https://review.opendev.org/c/openstack/cinder/+/992289 | 08:05 |
| opendevreview | Nilesh Thathagar proposed openstack/cinder master: Dell PowerFlex Driver: Fix NVMe/TCP host creation with long host names https://review.opendev.org/c/openstack/cinder/+/996392 | 08:12 |
| opendevreview | Anoop Kumar Shukla proposed openstack/cinder master: NetApp: enrich periodic Cinder EMS payloads https://review.opendev.org/c/openstack/cinder/+/996205 | 10:08 |
| Anoop_Shukla | @rosmaita, Jon, Eric. I recently looked at a patch which changes driver sleep from green thread.sleep() to time.sleep(). https://review.opendev.org/c/openstack/cinder/+/950695/11/cinder/volume/drivers/netapp/dataontap/client/api.py | 11:31 |
| Anoop_Shukla | Although OpenStack documentation recommends using greenthread.sleep(): https://docs.openstack.org/nova/14.0.4/threading.html. So I am confused on how this change affects the driver code. Since the changes in above patch are across drivers, and after reading some documents, it ends up pausing the OS thread - what is the recommendation here? Since the changes are on api.py, it would affect all our concurrent operations which | 11:33 |
| Anoop_Shukla | uses REST APIs on platform. | 11:33 |
| opendevreview | Rajat Dhasmana proposed openstack/cinder-specs master: Propose replication v3 spec https://review.opendev.org/c/openstack/cinder-specs/+/985952 | 12:11 |
| opendevreview | Anoop Kumar Shukla proposed openstack/cinder master: NetApp: enrich periodic Cinder EMS payloads https://review.opendev.org/c/openstack/cinder/+/996205 | 13:12 |
| opendevreview | Nilesh Thathagar proposed openstack/cinder master: Dell PowerFlex Driver: Fix NVMe/TCP host creation with long host names https://review.opendev.org/c/openstack/cinder/+/996392 | 13:39 |
| rosmaita | Cinder meeting in #openstack-cinder at 1400 UTC | 13:59 |
| 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 aloke_dev Anoop_Shukla erlon Jyotsna | 13:59 |
| whoami-rajat | hi | 14:00 |
| simondodsley | o/ | 14:00 |
| jungleboyj | o/ Lurking. | 14:00 |
| rosmaita | wait, let me start | 14:00 |
| Jyotsna | o/ | 14:00 |
| auniyal | o/ | 14:00 |
| hvlcchao1 | o/ | 14:00 |
| rosmaita | #startmeeting cinder | 14:00 |
| opendevmeet | Meeting started Wed Jul 8 14:00:24 2026 UTC and is due to finish in 60 minutes. The chair is rosmaita. 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 |
| whoami-rajat | hi (again) | 14:00 |
| simondodsley | o/ | 14:00 |
| rosmaita | #link https://etherpad.opendev.org/p/cinder-hibiscus-meetings | 14:00 |
| rosmaita | #topic roll call | 14:00 |
| rosmaita | ok, now you can say hi! | 14:00 |
| rosmaita | hi | 14:01 |
| jungleboyj | o/ Hello | 14:01 |
| jayaanand | o/ | 14:01 |
| fqi | o/ | 14:01 |
| eharney | hi | 14:01 |
| rosmaita | #chair jbernard | 14:02 |
| opendevmeet | Current chairs: jbernard rosmaita | 14:02 |
| rosmaita | something came up and Jon will be delayed today | 14:02 |
| agalica_ | o/ | 14:03 |
| rosmaita | ok, guess we should get started | 14:04 |
| rosmaita | #topic announcements | 14:04 |
| rosmaita | Just a reminder that the Festival of Reviews is this friday, at the end of this week | 14:04 |
| rosmaita | it got postponed due to a holiday in the usa | 14:05 |
| rosmaita | and, the periodic weekly jobs are all green for cinderclient and cinderclient-brick-ext | 14:05 |
| rosmaita | so, hooray | 14:05 |
| whoami-rajat | \o/ | 14:06 |
| jungleboyj | \o/ | 14:06 |
| rosmaita | and that's all i've got, though i imagine there are some deadlines lurking | 14:06 |
| rosmaita | well, scrolling down to june 10 in the agenda, looks like the third-party-CI-must-be-working-for-all-protocols deadline was July 3 | 14:07 |
| rosmaita | so hopefully everyone has their mojo working | 14:07 |
| rosmaita | the new driver deadline is coming up pretty soon, July 24 | 14:08 |
| rosmaita | anyone else have an announcement they would like to share? | 14:09 |
| rosmaita | apparently not ... let's move along | 14:11 |
| rosmaita | #topic Follow up on Strategy for security issues | 14:11 |
| rosmaita | ok, so last week we took an initial look at | 14:12 |
| rosmaita | #link https://lists.openstack.org/archives/list/openstack-discuss@lists.openstack.org/thread/44MLENT3AXUML7CYYLNWO6Z7LO6CQT6W/ | 14:12 |
| rosmaita | we've got several things going on, i will try to summarize the issues | 14:13 |
| rosmaita | 1. cinder needs a document telling operators about "safe" vs. "unsafe" deployment configurations | 14:13 |
| Anoop_Shukla | o/ | 14:14 |
| rosmaita | 2. cinder needs a document for driver developers outlining expectations for security-conscious driver development | 14:14 |
| rosmaita | 3. maybe we need a driver security matrix, or a column in the current matrix if we can jam all the info in there | 14:14 |
| rosmaita | 4. we need a policy to mark drivers that are unsafe security-wise as 'unsupported' (though maybe that's part of #2) | 14:16 |
| rosmaita | 5. we need to encourage vendor drivers to fix the "bad" bugs, e.g., hard-coded 'http' right away | 14:16 |
| rosmaita | 6. we need to encourage vendor drivers to change default settings for options to secure values | 14:17 |
| rosmaita | i think that's all? | 14:17 |
| rosmaita | before opening this up to discussion, my recommendation is that we do #4 first | 14:18 |
| rosmaita | that way we will have some encouragement for vendors to prioritize 5 and 6 | 14:18 |
| jungleboyj | That would make sense as the starting point and to identify where the problems are. | 14:19 |
| rosmaita | and it will give us a preliminary document for #2 | 14:19 |
| rosmaita | ok, let's discuss | 14:19 |
| whoami-rajat | maybe can use the driver review skill to document safe vs unsafe practices that drivers are doing and document them https://review.opendev.org/c/openstack/agentic-workflows/+/995502 | 14:20 |
| rosmaita | i think that is a good idea | 14:20 |
| whoami-rajat | it uses the cinder coresec persona for finding out potential vulnerabilities https://review.opendev.org/c/openstack/agentic-workflows/+/994081/4/personas/cinder-coresec.md | 14:21 |
| simondodsley | agreed - i'll have a look at the skill and add any comments | 14:21 |
| rosmaita | i don't want to get too far off topic, but whoami-rajat have you mentioned to the cinder community that you have those patches up for the agentic-AI stuff? | 14:21 |
| whoami-rajat | thanks simondodsley | 14:21 |
| whoami-rajat | rosmaita, no i didn't, can do it quickly to not take a lot of time | 14:22 |
| whoami-rajat | I've proposed series of patches to add cinder specific knowledge, personas and skills https://review.opendev.org/q/topic:%22cinder-agentic-work%22 | 14:22 |
| whoami-rajat | sorry | 14:22 |
| whoami-rajat | #link https://review.opendev.org/q/topic:%22cinder-agentic-work%22 | 14:22 |
| whoami-rajat | surely they are long md files as of now but will try to use/modify/iterate to optimize them further | 14:23 |
| whoami-rajat | feel free to take a look | 14:23 |
| whoami-rajat | that's all! thanks | 14:23 |
| rosmaita | thanks! | 14:23 |
| rosmaita | note to the community -- you may not have seen them because they're not proposed to the cinder repo | 14:24 |
| rosmaita | but please take a look | 14:24 |
| simondodsley | on it | 14:25 |
| rosmaita | even if you don't use AI agents, the md files are pretty human-readable, so you can see what they are doing and comment | 14:25 |
| whoami-rajat | yeah, overtime I'm planning to migrate the knowledge to cinder docs (that is also beneficial for human readers) and try to use links as much as possible in the agentic workflow project | 14:25 |
| rosmaita | ok, back on topic ... i think whoami-rajat's agentic AI stuff will help us identify problematic drivers | 14:25 |
| rosmaita | but we still need to have a specific policy about what to do about problematic drivers | 14:26 |
| whoami-rajat | I can provide some input by next week using that (but don't rely too much on it!) | 14:26 |
| rosmaita | i will take an action item to get something posted before the Festival of Reviews | 14:27 |
| simondodsley | real security issues like http only or lack of TLS support should be, IMHO, be flagged as `unsupported` until fixed | 14:27 |
| rosmaita | i think discussion will be easier once there is something posted to criticize | 14:27 |
| rosmaita | simondodsley: agree | 14:28 |
| simondodsley | i don't think lack of CHAP for iSCSI is an immeadiate `unsupported` as i don't think people really use it | 14:28 |
| aloke_dev | Hi Reviewers, my apologies if it is off topic, but I wanted to check regarding PR: https://review.opendev.org/c/openstack/cinder/+/986072 (Introducing Cinder Driver for InfoScale). We have enabled the 3rd party CI and it is green. Also this PR has got +1. Waiting for feedback from reviewers to move this toward +2. | 14:28 |
| simondodsley | it should be the next iteration though | 14:28 |
| rosmaita | yeah, i think we will start with the real bad stuff and then iterate | 14:29 |
| simondodsley | flipping CHAP to default on is a risk without a lot of notice | 14:29 |
| rosmaita | aloke_dev: let's hold that for open discussion (in about 15 minutes) | 14:29 |
| rosmaita | #topic RBAC enforcement in devstack | 14:30 |
| simondodsley | oh yeah - that was fun... | 14:30 |
| rosmaita | looks like there are some changes that need to be reviewed; luckily we have a Festival at the end of the week | 14:30 |
| rosmaita | simondodsley: yes, thank you for identifying that barbican issue | 14:31 |
| rosmaita | i would prefer not to talk about RBAC right now unless absolutely necessary | 14:32 |
| simondodsley | lol | 14:32 |
| rosmaita | ok, we will get that stuff taken care of on Friday | 14:32 |
| rosmaita | #topic Pure storage reports group replication as reported but the respective methods are not implemented | 14:33 |
| rosmaita | whoami-rajat: that's you | 14:33 |
| whoami-rajat | hey | 14:33 |
| simondodsley | yes - i'll take a look at that and report back | 14:33 |
| whoami-rajat | thanks, that's all i wanted | 14:33 |
| rosmaita | excellent, that was easy | 14:34 |
| whoami-rajat | yep, simondodsley \o/ | 14:34 |
| simondodsley | i assume this is due to the new replication temeest tests | 14:34 |
| rosmaita | #topic HPE Alletra MP driver needs reviews | 14:34 |
| whoami-rajat | no, i found out while i was enabling group replication in RBD | 14:34 |
| simondodsley | ok | 14:34 |
| Anoop_Shukla | I had one topic about the greenthread.sleep vs time.sleep - https://docs.openstack.org/nova/14.0.4/threading.html document suggests and recommends to use greenthreads..while we recently have changed to time.sleep. | 14:34 |
| whoami-rajat | #link https://review.opendev.org/q/topic:%22rbd-group-replication%22 | 14:34 |
| rosmaita | #link https://review.opendev.org/c/openstack/cinder/+/969373 | 14:35 |
| rosmaita | ok, we have 3 discussions happening at once | 14:35 |
| Anoop_Shukla | Sorry..my bad..we can take it at last | 14:36 |
| rosmaita | anything more on the Pure group replication stuff? | 14:36 |
| rosmaita | ok, that has been discussed, thank you simon and Rajat | 14:37 |
| whoami-rajat | sorry and thanks, nothing from my side | 14:37 |
| rosmaita | ok, next item is the HPE alletramp driver | 14:37 |
| rosmaita | HPE has been really good about revising and responding to requests | 14:38 |
| Jyotsna | Thanks <rosmaita> | 14:38 |
| rosmaita | whoami-rajat: i wonder if your agentic driver stuff is ready to let loose on that driver? | 14:38 |
| rosmaita | my perception is that the driver is really close, and we should get it completed as soon as possible to make room for the new drivers this cycle | 14:39 |
| Jyotsna | We need attention again from reviewers so that this time we dont loose chance to get the driver. | 14:39 |
| rosmaita | exactly | 14:39 |
| Jyotsna | We have been chasing this from almost a year and i would appreciate all the reviwers patience and efforts they spent in reviweing!! | 14:40 |
| rosmaita | looks like agalica and erlon have looked at it most recently | 14:40 |
| rosmaita | and eric | 14:41 |
| Jyotsna | yes they have reviewed it and we have addressed all the comments from them . All CIs are green and Zuul verified | 14:41 |
| rosmaita | ok, well, reviewers please try to take a look today and tomorrow | 14:42 |
| Jyotsna | Will it be possible for reviewers to provide +2 and move this towards merge. | 14:42 |
| rosmaita | otherwise, i think we will have to do it at the Festival on friday | 14:42 |
| rosmaita | which will pretty much mean that no other patches will get reviewed | 14:42 |
| rosmaita | but, it that's what it takes ... | 14:42 |
| Jyotsna | Thanks <rosmaita> for bringing everyone's attention on patch. | 14:43 |
| rosmaita | np, thank you for your patience and willingness to revise | 14:43 |
| whoami-rajat | I can't guarantee it but will report if the skill finds anything useful worth flagging | 14:44 |
| rosmaita | ok, let's move to open discussion .. we have 2 things already, the greenthread.sleep question, and i forget what the other one was | 14:44 |
| rosmaita | #topic open discussion | 14:44 |
| eharney | Anoop_Shukla: there is almost no situation where we should be calling greenthread.sleep instead of time.sleep | 14:44 |
| Anoop_Shukla | https://docs.openstack.org/cinder/newton/devref/threading.html The documentations suggest otherwise though | 14:45 |
| eharney | as we are moving toward removing eventlet and running in threaded mode, we will be removing eventlet-specific code | 14:45 |
| Anoop_Shukla | Okay..so the follow up question was, is there any performance implication that the drivers need to test for? | 14:46 |
| eharney | looks like we need to put it on the todo list to rewrite that doc | 14:46 |
| Anoop_Shukla | Since time.sleep would pause the OS thread? | 14:46 |
| whoami-rajat | isn't that doc Pro-eventlet rather than in favor of removing it, also I'm not sure if it's for the latest release | 14:46 |
| rosmaita | i thought time got monkey-patched so that you didn't have to worry about using the greenthread.sleep directly | 14:46 |
| whoami-rajat | #link https://docs.openstack.org/nova/latest/reference/threading.html#native-threading | 14:47 |
| eharney | correct | 14:47 |
| whoami-rajat | this is the latest one | 14:47 |
| eharney | so there isn't really a reason to want to do it | 14:47 |
| Anoop_Shukla | Yes..that seems to be the problem. There are documentations that suggest otherwise. | 14:47 |
| Anoop_Shukla | Okay good | 14:47 |
| Anoop_Shukla | So the latest documentation is updated then.. | 14:47 |
| rosmaita | we are at a transition point, because eventlet is not supported in modern pythons | 14:48 |
| rosmaita | and python has evolved so that hopefully eventlet is no longer necessary | 14:48 |
| Anoop_Shukla | In current code, time.sleep(0) does the same thing as greenthread.sleep(0) if time module is patched through eventlet.monkey_patch(). To be explicit, we recommend contributors use greenthread.sleep() instead of time.sleep() | 14:48 |
| Anoop_Shukla | The latest documentation still talks about recommending greenthread.sleep | 14:49 |
| whoami-rajat | it's in the eventlet section and not in native threading | 14:49 |
| whoami-rajat | Before the Flamingo release all OpenStack services used the green thread model of threading, implemented through using the Python eventlet and greenlet libraries. | 14:49 |
| Anoop_Shukla | okay | 14:50 |
| Anoop_Shukla | So we expect no impact of the change on performance? | 14:50 |
| rosmaita | Anoop_Shukla: thank you for bringing this up | 14:50 |
| rosmaita | well, we don't actually know about performance | 14:51 |
| eharney | driver maintainers will want to check for performance issues (and ensure that any dependencies are actually using thread-safe python libs etc) | 14:51 |
| rosmaita | i think short term, we should put a warning box on the pages that mention working with eventlet to say it's no longer relevant | 14:52 |
| rosmaita | unless someone else want to do it, i will take an action to do that right after this meeting | 14:53 |
| Anoop_Shukla | Thanks @rosmaita as it may confuse the new driver enthusiasts.. | 14:53 |
| Anoop_Shukla | We will try to performance test the change and evaluate if there are any bottleneck observed with the patched code | 14:54 |
| rosmaita | #action rosmaita eventlet-removal warnings in dev docs | 14:54 |
| rosmaita | Anoop_Shukla: that will be nice | 14:54 |
| rosmaita | ok there was another issue that i told someone to wait for open discussion | 14:54 |
| rosmaita | that time is now | 14:55 |
| aloke_dev | Hi, I'd like to check on PR https://review.opendev.org/c/openstack/cinder/+/986072 (InfoScale Cinder Driver). The 3rd party CI is now active and green, and the patch has a +1. I'm looking for reviewer feedback to help move this toward a +2. Since this is my first contribution, please let me know if I need to adjust anything to align with the workflow. | 14:55 |
| rosmaita | thank you! | 14:55 |
| Anoop_Shukla | With greenthread.sleep(): While one green thread sleeps, dozens of other green threads continue executing I/O tasks or logic. High network throughput is maintained. With time.sleep(): The event loop freezes entirely. No other green thread can run during the countdown period, destroying application concurrency. | 14:55 |
| fqi | I have updated the cinder spec for retype volume with migration including snapshots. I would appreciate it if you could please review it. https://review.opendev.org/c/openstack/cinder-specs/+/989322 | 14:55 |
| rosmaita | aloke_dev: do you know about the new driver review checklist? | 14:57 |
| eharney | fqi: i am still unclear about how many backends can synthesize snapshots out of temporary volumes, or how that works exactly | 14:57 |
| aloke_dev | Yes, I followed the guidelines @rosmaita. | 14:57 |
| rosmaita | aloke_dev: excellent! | 14:57 |
| rosmaita | well, i think we need to get the HPE alletramp out of the way, and that will free up bandwidth for infoscale | 14:58 |
| rosmaita | aloke_dev: in the meantime, take a look at the bugs listed in https://lists.openstack.org/archives/list/openstack-discuss@lists.openstack.org/thread/44MLENT3AXUML7CYYLNWO6Z7LO6CQT6W/ | 14:59 |
| rosmaita | just to make sure that they don't apply to you | 14:59 |
| rosmaita | fqi: we are running out of time, but if you want to discuss some more, people will probably stick around for a little while | 15:00 |
| rosmaita | ok, thanks everyone and don't forget the Festival of Reviews on friday | 15:01 |
| rosmaita | #endmeeting | 15:01 |
| opendevmeet | Meeting ended Wed Jul 8 15:01:06 2026 UTC. Information about MeetBot at http://wiki.debian.org/MeetBot . (v 0.1.4) | 15:01 |
| opendevmeet | Minutes: https://meetings.opendev.org/meetings/cinder/2026/cinder.2026-07-08-14.00.html | 15:01 |
| opendevmeet | Minutes (text): https://meetings.opendev.org/meetings/cinder/2026/cinder.2026-07-08-14.00.txt | 15:01 |
| opendevmeet | Log: https://meetings.opendev.org/meetings/cinder/2026/cinder.2026-07-08-14.00.log.html | 15:01 |
| aloke_dev | Sure, I will have a look! Thank you <rosmaita>! | 15:01 |
| fqi | Eric, Sorry I have to be off for a while. I will talk to you later when you are free. | 15:06 |
| opendevreview | Brian Rosmaita proposed openstack/cinder master: [docs] Eventlet is not a preferred threading model https://review.opendev.org/c/openstack/cinder/+/996495 | 15:38 |
| rosmaita | eharney: please do a sanity check ^^ | 15:39 |
| opendevreview | Simon Dodsley proposed openstack/cinder master: [Pure Storage] Implement Tiramisu group replication actions https://review.opendev.org/c/openstack/cinder/+/996497 | 15:58 |
| opendevreview | Merged openstack/devstack-plugin-ceph master: Allow customizing NFS-Ganesha port https://review.opendev.org/c/openstack/devstack-plugin-ceph/+/983103 | 16:15 |
| opendevreview | jayaanand borra proposed openstack/cinder master: Implement per-pool image-volume cache promotion https://review.opendev.org/c/openstack/cinder/+/988315 | 16:58 |
| opendevreview | Simon Dodsley proposed openstack/cinder master: doc: Add driver security disclosure matrix https://review.opendev.org/c/openstack/cinder/+/996554 | 21:52 |
| opendevreview | Simon Dodsley proposed openstack/cinder master: doc: Add driver security disclosure matrix https://review.opendev.org/c/openstack/cinder/+/996554 | 22:03 |
| opendevreview | Simon Dodsley proposed openstack/cinder master: doc: Add driver security disclosure matrix https://review.opendev.org/c/openstack/cinder/+/996554 | 22:34 |
Generated by irclog2html.py 4.1.0 by Marius Gedminas - find it at https://mg.pov.lt/irclog2html/!