| *** mhen_ is now known as mhen | 02:44 | |
| *** whoami-rajat_ is now known as whoami-rajat | 13:55 | |
| rosmaita | #startmeeting cinder | 14:00 |
|---|---|---|
| opendevmeet | Meeting started Wed Mar 12 14:00:59 2025 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 |
| rosmaita | #topic roll call | 14:01 |
| simondodsley | o/ | 14:01 |
| akawai | o/ | 14:01 |
| Sai | o/ | 14:02 |
| whoami-rajat | hey | 14:03 |
| rosmaita | looks like a light turnout ... i'll wait a few minutes, we went to daylight saving time in the USA this week, so people may be a bit confused | 14:03 |
| rosmaita | Jon has a conflict that is keeping him offline, so I'm covering for him today | 14:04 |
| simondodsley | Good old DST... | 14:04 |
| msaravan | Hi | 14:04 |
| rosmaita | good in some ways, bad in some ways | 14:04 |
| simondodsley | shall we keep it or not? that is the question... | 14:04 |
| tosky | o/ | 14:05 |
| rosmaita | simondodsley: are you pro- or anti- DST? | 14:05 |
| simondodsley | TBH I don't care | 14:05 |
| simondodsley | i live in FL so any change doesn't reallt affect me | 14:06 |
| jungleboyj | I would be happy keeping the time where it is now. Makes it better for me working with China. | 14:06 |
| rosmaita | i think we should have micro-time-zones | 14:06 |
| simondodsley | lol - can I have my house always be at 5pm | 14:07 |
| rosmaita | :D | 14:07 |
| rosmaita | ok, i guess we should get started | 14:07 |
| jungleboyj | LOL. | 14:07 |
| jungleboyj | Well, it is always 5 O'Clock somewhere. | 14:07 |
| rosmaita | #topic announcements | 14:07 |
| rosmaita | this is RC-1 target week | 14:08 |
| rosmaita | what that means is that the first Release Candidate will be released and a stable/2025.1 branch will be cut | 14:08 |
| rosmaita | the deadline for RC-1 is friday | 14:08 |
| rosmaita | after friday, bugs will have to be fixed in master and then backported to stable/2025.1 | 14:09 |
| rosmaita | until the coordinated OpenStack release on Wednesday April 2, only release-critical bugs will be allowed to be backported | 14:09 |
| simondodsley | that is concerning to me as I have a number of feature patches and bug fixes that have been waiting for some time | 14:09 |
| rosmaita | yeah, and i'm not sure what to say about that | 14:11 |
| simondodsley | i just need some core reviews - mainly for https://review.opendev.org/c/openstack/cinder/+/933675 as that is a big one we have been promising to customers. Needs one more core | 14:11 |
| rosmaita | in the meantime, if you come across an issue that you think is release-critical, you can put it on this etherpad | 14:11 |
| rosmaita | #link https://etherpad.opendev.org/p/cinder-epoxy-release-critical-bugs | 14:12 |
| whoami-rajat | I remember reviewing some of them last week, do we have them listed on basis of priority? | 14:12 |
| rosmaita | yikes, -56 +785 | 14:12 |
| rosmaita | whoami-rajat: not yet, that i am aware of | 14:12 |
| simondodsley | romaita: exactly - it's been there a long time | 14:13 |
| yuval | I have a very small patch only doc's need another core review: https://review.opendev.org/c/openstack/cinder/+/942672 | 14:13 |
| rosmaita | simondodsley: i will commit to looking that over today | 14:13 |
| simondodsley | thank you | 14:13 |
| rosmaita | yuval: done | 14:14 |
| yuval | Thansk | 14:14 |
| jungleboyj | simondodsley: Me as well. Just opened a tab. | 14:14 |
| rosmaita | just a reminder to cores, the cinder project policy on doc-only changes is that they only need one +2 | 14:14 |
| jungleboyj | ++ | 14:14 |
| rosmaita | jungleboyj: do you have time for https://review.opendev.org/c/openstack/cinder/+/933675 ? | 14:15 |
| rosmaita | (or were you talking about something else?) | 14:16 |
| jungleboyj | rosmaita: Yes, that one. | 14:16 |
| rosmaita | ok, if you can take that, i'll grab something else | 14:16 |
| jungleboyj | rosmaita: Sounds good. | 14:17 |
| rosmaita | actually, i will grab https://review.opendev.org/c/openstack/cinder/+/916408 since it fixes a feature | 14:17 |
| simondodsley | my other outstanding patches are small | 14:18 |
| whoami-rajat | this one is not release critical but it's a small one correcting DEBUG logging in RBD QoS code, the commit message mentions the before/after https://review.opendev.org/c/openstack/cinder/+/943666 | 14:18 |
| whoami-rajat | it turns out we were logging the module instead of the actual QoS values being applied | 14:18 |
| rosmaita | that would be misleading | 14:19 |
| whoami-rajat | yeah i was puzzled to not see the QoS properties in logs and later i realized what was happening ... | 14:20 |
| rosmaita | whoami-rajat: i will review that one if you hit this: https://review.opendev.org/c/openstack/cinder/+/915908 | 14:20 |
| whoami-rajat | we now have NVMe over RDMA so who needs iSER :D | 14:21 |
| whoami-rajat | sure, will take a look | 14:21 |
| rosmaita | thanks! | 14:22 |
| rosmaita | it has a child patch that would be an easy review, too | 14:22 |
| rosmaita | ok, but back to the main point, i.e., what's the top priority | 14:23 |
| rosmaita | there was a encryption standardization patch that is important | 14:23 |
| rosmaita | looks like the glance part is on hold for Flamingo, though | 14:25 |
| rosmaita | this is the cinder patch: https://review.opendev.org/c/openstack/cinder/+/926298 | 14:25 |
| rosmaita | this is the glance one: https://review.opendev.org/c/openstack/glance/+/926295 | 14:26 |
| whoami-rajat | yeah, it would be good to get Glance team opinion on it, I would rather have all the code in same cycle to avoid backward compat issues | 14:27 |
| rosmaita | ok, makes sense ... we'll hold off on that one | 14:27 |
| rosmaita | the other patch i was thinking of was abishop's display migration progress | 14:27 |
| rosmaita | i believe the nova side of that will not be in epoxy | 14:28 |
| whoami-rajat | :/ | 14:30 |
| rosmaita | this is the cinder patch: https://review.opendev.org/c/openstack/cinder/+/942716 | 14:32 |
| rosmaita | it has a depends-on against the nova patch, so guess it will have to wait for Flamingo | 14:33 |
| rosmaita | guess that's all for topics | 14:35 |
| rosmaita | anyone have any concerns they want to raise? | 14:35 |
| whoami-rajat | I've two patches added to the critical etherpad, the RBD one is pretty critical to me, but otherwise we will need to add them to KNOWN ISSUES in reno | 14:36 |
| whoami-rajat | s/critical/release critical | 14:36 |
| rosmaita | those both look worthy to me | 14:37 |
| rosmaita | vdhakad: are you or someone from ibm here? | 14:38 |
| rosmaita | ok, i left a note about that one on the agenda etherpad | 14:40 |
| rosmaita | looks like the pure patches mostly have one +2 already, so those shouldn't be too bad | 14:40 |
| rosmaita | and actually, jungleboyj already committed to the one on line 88 | 14:41 |
| rosmaita | the hitachi & oem patch ... already had 2 +2s, so I just +w'd it | 14:41 |
| akawai | rosmaita: thx | 14:42 |
| rosmaita | ok, if there is nothing else to discuss, we have 18 minutes of free reviewing time | 14:42 |
| whoami-rajat | just to confirm, we are not pursuing the two features i.e. standardize encryption and report migration status? | 14:43 |
| rosmaita | migration status : not pursuing for sure, since the nova patch is a dependency, and it won't be in nova epoxy | 14:44 |
| rosmaita | standardize encryption: hate to leave it sitting, but it would be good to understand the glance concerns and whether it makes sense to merge the cinder patch without the glance one | 14:45 |
| whoami-rajat | ack, thanks | 14:45 |
| rosmaita | so i will ping Jon ... he can contact the core team if he thinks it should make it into Flamingo | 14:46 |
| rosmaita | will have to merge tomorrow, though | 14:46 |
| rosmaita | anything else? | 14:46 |
| rosmaita | go forth and review!!! | 14:47 |
| rosmaita | see you next week | 14:47 |
| rosmaita | #endmeeting | 14:47 |
| opendevmeet | Meeting ended Wed Mar 12 14:47:29 2025 UTC. Information about MeetBot at http://wiki.debian.org/MeetBot . (v 0.1.4) | 14:47 |
| opendevmeet | Minutes: https://meetings.opendev.org/meetings/cinder/2025/cinder.2025-03-12-14.00.html | 14:47 |
| opendevmeet | Minutes (text): https://meetings.opendev.org/meetings/cinder/2025/cinder.2025-03-12-14.00.txt | 14:47 |
| opendevmeet | Log: https://meetings.opendev.org/meetings/cinder/2025/cinder.2025-03-12-14.00.log.html | 14:47 |
| jungleboyj | Thanks! | 14:48 |
Generated by irclog2html.py 2.17.3 by Marius Gedminas - find it at https://mg.pov.lt/irclog2html/!