| *** mhen_ is now known as mhen | 02:30 | |
| jbernard | #startmeeting cinder | 14:10 |
|---|---|---|
| opendevmeet | Meeting started Wed Mar 26 14:10:34 2025 UTC and is due to finish in 60 minutes. The chair is jbernard. Information about MeetBot at http://wiki.debian.org/MeetBot. | 14:10 |
| opendevmeet | Useful Commands: #action #agreed #help #info #idea #link #topic #startvote. | 14:10 |
| opendevmeet | The meeting name has been set to 'cinder' | 14:10 |
| whoami-rajat_ | Hi | 14:10 |
| sfernand | hi | 14:10 |
| msaravan | Hi | 14:10 |
| akawai | o/ | 14:10 |
| vdhakad | hi | 14:10 |
| simondodsley | o/ | 14:11 |
| jbernard | o/ | 14:11 |
| jbernard | welcome everyone | 14:11 |
| jbernard | #link https://etherpad.opendev.org/p/cinder-epoxy-meetings | 14:11 |
| rosmaita | o/ | 14:12 |
| jbernard | also | 14:12 |
| jbernard | #link https://etherpad.opendev.org/p/cinder-epoxy-release-critical-bugs | 14:12 |
| simondodsley | is there time to add to the critical bugs list? | 14:12 |
| jbernard | simondodsley: whacha got? | 14:13 |
| jbernard | if you have a critial bug, we should take a look | 14:13 |
| jbernard | today we have an otherwise empty agenda | 14:14 |
| jbernard | so I would like to process the rc-bug list as best as possible | 14:14 |
| jbernard | this: https://etherpad.opendev.org/p/cinder-epoxy-release-critical-bugs | 14:14 |
| simondodsley | I have 2 - one of which had to be reverted (yes, that one rosmaita...) and is a redo | 14:14 |
| akawai | wait | 14:14 |
| akawai | I'll leave Hitachi driver team this Apr. | 14:14 |
| akawai | I would like to introduce you new members, hvlcchao1 and agalica. Please help them. | 14:14 |
| akawai | Additionally, Hitachi CI is rebuilding in other site. so, it would not work for a while. | 14:15 |
| akawai | Thank you guys! | 14:15 |
| jbernard | akawai: thanks, sorry to see you go, and we wish you luck in next adventure | 14:15 |
| rosmaita | thanks akawai and welcome hvlcchao1 and agalica | 14:15 |
| akawai | thx! | 14:15 |
| jbernard | simon is updating the doc | 14:16 |
| jbernard | rosmaita: do you want to start at the top? | 14:16 |
| rosmaita | ok | 14:16 |
| rosmaita | i haven't reviewed 938265 yet, so will do that now | 14:18 |
| jbernard | whoami-rajat_: on https://review.opendev.org/c/openstack/cinder/+/938265 | 14:19 |
| jbernard | whoami-rajat_: tobias raised an issue | 14:19 |
| jbernard | i believe the issue raised not new, it is also the case with the current copy logic | 14:20 |
| jbernard | tosky: ^ curious if you have input as well | 14:21 |
| eharney | i'll leave a comment on 938265 | 14:21 |
| whoami-rajat_ | Looking | 14:22 |
| jbernard | thanks | 14:22 |
| whoami-rajat_ | We are reverting to the original logic, if the listed scenario is an issue, the optimisation that eharney added is the way to go i.e. use same format for image as volume | 14:23 |
| whoami-rajat_ | I am doing nothing new there, just adding the old logic back | 14:23 |
| whoami-rajat_ | Thanks eharney for your comment | 14:24 |
| jbernard | that's what thought as well, this looks okay to me | 14:27 |
| jbernard | done | 14:29 |
| vdhakad | Hi rosmaita, I saw your note on https://review.opendev.org/c/openstack/cinder/+/925450 and I'm okay with holding off merge, I'd appreciate if you/other core reviewers could take a look at it and provide review comments if any. Thanks! | 14:30 |
| rosmaita | vdhakad: thanks for understanding, we can work on that as soon as the final release candidate is out | 14:31 |
| jbernard | im looking at the cgroup change from vigo, v2 support was added in cgroup-tools 2.x | 14:33 |
| whoami-rajat_ | I can take a look at patches that needs review | 14:37 |
| jbernard | is anyone looking at alan's migration patch? | 14:41 |
| jbernard | https://review.opendev.org/c/openstack/cinder/+/938827 | 14:42 |
| jbernard | simondodsley: what do we need to know about the two you've added? | 14:45 |
| simondodsley | the first one is a redo of a patch that made it into 2025.1 but had to be reverted as another patch exposed a unit test bug. This is the redo. rosmaita had nightmares about the revert!!! | 14:46 |
| rosmaita | we need a volunteer to cherry-pick https://review.opendev.org/c/openstack/cinder/+/938490 | 14:46 |
| rosmaita | to stable/2025.1 | 14:46 |
| rosmaita | you can do it in the gerrit web interface | 14:46 |
| rosmaita | after you cherry pick, | 14:46 |
| rosmaita | you will need to edit the commit message to add this line after the Change-Id: | 14:46 |
| rosmaita | (cherry picked from commit d41586a115aabf01a756f6ee6ef58a813f87b546) | 14:46 |
| simondodsley | both patches are critical for customers who are experiencing issues and one of them likes to run the latest release | 14:46 |
| simondodsley | rosmaita - i can do that | 14:48 |
| rosmaita | simondodsley: thanks! | 14:48 |
| simondodsley | Done | 14:49 |
| simondodsley | https://review.opendev.org/c/openstack/cinder/+/945614 | 14:49 |
| jbernard | simondodsley: you still have the CI issue, can you confirm those patches are being tested and passing? | 14:55 |
| simondodsley | yes | 14:55 |
| simondodsley | that CI issue is causing me real grief!! | 14:55 |
| rosmaita | I did a cherry pick of https://review.opendev.org/c/openstack/cinder/+/945616 so eharney and jbernard can +2 the backport after the master patch merges | 14:56 |
| rosmaita | eharney or simondodsley: can you do the cherry-pick of https://review.opendev.org/c/openstack/cinder/+/916408 so jbernard and i can +2 it | 14:57 |
| eharney | will do that now | 14:58 |
| rosmaita | thanks | 14:58 |
| hemna | mep | 15:01 |
| jbernard | hemna: https://etherpad.opendev.org/p/cinder-epoxy-release-critical-bugs | 15:01 |
| hemna | need reviews? | 15:02 |
| jbernard | most are in good shape, the one exception is alan's migration patch | 15:02 |
| jbernard | https://review.opendev.org/c/openstack/cinder/+/938827 | 15:02 |
| jbernard | hemna: thanks! | 15:02 |
| hemna | ok I'll poke at it | 15:03 |
| rosmaita | simondodsley: jbernard: i have a reservation about https://review.opendev.org/c/openstack/cinder/+/945062 , see what you think | 15:08 |
| rosmaita | eharney: you too ^^ | 15:08 |
| eharney | i submitted a follow-up patch that fixes that, it's in the relation chain on that patch | 15:09 |
| rosmaita | cool, problem solved | 15:11 |
| jbernard | simondodsley: you can backport https://review.opendev.org/c/openstack/cinder/+/945062 | 15:19 |
| jbernard | im going to end the recorded meeting here, im not sure if this channel is used for other teams. And we can resume in our normal irc channel. thank you to everyone who participated, much appreciated | 15:30 |
| jbernard | #endmeeting | 15:30 |
| opendevmeet | Meeting ended Wed Mar 26 15:30:11 2025 UTC. Information about MeetBot at http://wiki.debian.org/MeetBot . (v 0.1.4) | 15:30 |
| opendevmeet | Minutes: https://meetings.opendev.org/meetings/cinder/2025/cinder.2025-03-26-14.10.html | 15:30 |
| opendevmeet | Minutes (text): https://meetings.opendev.org/meetings/cinder/2025/cinder.2025-03-26-14.10.txt | 15:30 |
| opendevmeet | Log: https://meetings.opendev.org/meetings/cinder/2025/cinder.2025-03-26-14.10.log.html | 15:30 |
Generated by irclog2html.py 2.17.3 by Marius Gedminas - find it at https://mg.pov.lt/irclog2html/!