*** dviroel|ruck is now known as dviroel|ruck|Afk | 00:05 | |
*** dviroel|ruck|Afk is now known as dviroel|ruck | 11:16 | |
*** dasm|off is now known as dasm | 13:43 | |
rosmaita | #startmeeting cinder | 14:00 |
---|---|---|
opendevmeet | Meeting started Wed Mar 16 14:00:18 2022 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 |
tosky | o/ | 14:00 |
simondodsley | hi | 14:00 |
whoami-rajat | Hi | 14:00 |
eharney | hi | 14:00 |
hemna | yough | 14:00 |
geguileo | hi! o/ | 14:00 |
yuval | hey | 14:01 |
rosmaita | hello everyone | 14:01 |
rosmaita | #link https://etherpad.opendev.org/p/cinder-yoga-meetings | 14:01 |
jungleboyj | o/ | 14:02 |
rosmaita | light agenda today, so plenty of time for open discussion later | 14:02 |
rosmaita | #topic announcements | 14:02 |
rosmaita | Festival of XS Reviews on Friday, 1400-1600 UTC | 14:02 |
rosmaita | you can find info about the festival here: http://eavesdrop.openstack.org/#Cinder_Festival_of_XS_Reviews | 14:03 |
fabiooliveira | o/ | 14:03 |
rosmaita | which reminds me that i didn't send out a reminder yet | 14:03 |
rosmaita | #action rosmaita festival of reviews reminder to ML | 14:03 |
whoami-rajat | We've a national festival on friday (Holi) so won't be able to attend... | 14:03 |
rosmaita | ok, that's a good excuse | 14:03 |
whoami-rajat | (probably anyone from India won't be around on friday) | 14:04 |
whoami-rajat | :) | 14:04 |
jungleboyj | rosmaita: I am not sure if I will be able to attend. My son has a college visit. | 14:04 |
rosmaita | ok, that's also a good excuse, though not as good as Rajat's | 14:04 |
rosmaita | :D | 14:04 |
jungleboyj | :-) | 14:04 |
rosmaita | enriquetaso also won't be there | 14:05 |
rosmaita | i have the time blocked off, so i am planning to be there | 14:05 |
rosmaita | next item | 14:05 |
rosmaita | final cinder RC must be released by Thursday next week | 14:05 |
rosmaita | not much more to say about that, i think everyone knows what it means | 14:06 |
rosmaita | RC-1 was released on Monday | 14:06 |
rosmaita | (that's "cinder release candidate #1" | 14:06 |
rosmaita | the stable/yoga branch has now been cut | 14:06 |
rosmaita | bugfixes must merge to master and then be backported to stable/yoga | 14:06 |
jungleboyj | ++ | 14:07 |
rosmaita | that's the case for everything (cinder, os-brick, cinderclient) other than cinderlib | 14:07 |
rosmaita | cinderlib master remains yoga development until it has its yoga release at zed R-15 | 14:07 |
rosmaita | which reminds me, i could use some eyes on https://review.opendev.org/c/openstack/cinderlib/+/833781 | 14:07 |
simondodsley | so things we only want in Zed can now be put up in master, correct? | 14:08 |
rosmaita | that is correct | 14:08 |
simondodsley | cool | 14:08 |
rosmaita | so at this point, only release-critical changes will be merged to stable/yoga | 14:09 |
rosmaita | so let's take a quick look at what we have | 14:09 |
rosmaita | #link https://etherpad.opendev.org/p/cinder-yoga-release-critical-changes | 14:09 |
rosmaita | so, first the good news | 14:09 |
rosmaita | the transition-to-alembic patches were all merged | 14:09 |
hemna | I hope that works when upgrading a real deployment :) | 14:10 |
rosmaita | we don't have any db migrations for yoga, but probably will soon | 14:10 |
rosmaita | so, https://docs.openstack.org/cinder/latest/contributor/database-migrations.html | 14:10 |
rosmaita | that explains how to use the new stuff | 14:10 |
rosmaita | hemna: me too, i'll be putting up a patch very early in Zed to do a minor migration so we can make sure it works as expected | 14:11 |
rosmaita | we have the advantage that nova is about 1.5 cycles ahead of us on this, and the bugs have been mostly shaken out | 14:11 |
rosmaita | anyway, i want to thank stephenfin for working on this for cinder | 14:12 |
rosmaita | we will declare Stephen Finucane a "Hero of Cinder" at the PTG | 14:12 |
jungleboyj | ++ | 14:12 |
whoami-rajat | also there's a new grenade job that checks migration from tock -> tock (or it's tick->tick) but it checks skipping one intermediate release | 14:12 |
rosmaita | as i mention on the etherpad, stephenfin has also posted a series of patches so that cinder will no longer use a deprecated oslo.db feature | 14:12 |
rosmaita | we will aim to merge those into Z quickly after the final RC | 14:13 |
rosmaita | ok, now the not-so-good news | 14:13 |
rosmaita | there is still some work ongoing for known issues in Xena | 14:13 |
rosmaita | the good thing is that i don't think any of them are destabilizing | 14:14 |
rosmaita | they are fairly localized | 14:14 |
rosmaita | so, we will consider them release-critical | 14:14 |
rosmaita | other than that, if anyone finds a bug that looks like it needs to be addressed before release, please put it on the etherpad so we can be aware | 14:15 |
rosmaita | and decide whether it really is release-critical or not | 14:15 |
rosmaita | any questions? | 14:15 |
rosmaita | ok, next item | 14:17 |
rosmaita | Zed virtual PTG 4-8 April | 14:17 |
rosmaita | #link https://etherpad.opendev.org/p/zed-ptg-cinder-planning | 14:17 |
rosmaita | usual deal, you need to register even though it's free | 14:17 |
rosmaita | whoami-rajat is doing the organization this time | 14:18 |
whoami-rajat | o/ | 14:18 |
rosmaita | if you look at the etherpad, there are a bunch of driver topics that i added | 14:18 |
rosmaita | it's looking like maybe we'll have a drivers' day | 14:18 |
whoami-rajat | ah, they weren't there the last time i checked | 14:19 |
rosmaita | so if you are a driver maintainer, and have something we should discuss, please add it | 14:19 |
whoami-rajat | looks like a drivers' day is needed | 14:19 |
whoami-rajat | (it could also be called Brian's day) | 14:19 |
rosmaita | btw, i will be happy to let anyone else who's interested take those over | 14:19 |
rosmaita | the PTG is coming up fast, so please spend a bit of time thinking about it | 14:20 |
geguileo | driver's day++ | 14:20 |
rosmaita | hemna: i didn't see your recomputing available space topic on there yet | 14:20 |
jungleboyj | Oooh, that sounds like an exciting one. | 14:21 |
rosmaita | that doesn't necessarily have to be on drivers day, though i imagine there may be some reporting issues that it would be good to get input on | 14:22 |
simondodsley | I won't be able to attend, but I'm going to add a topic for discussion if that is ok. | 14:22 |
geguileo | jungleboyj: was that a sarcastic comment? XD | 14:22 |
geguileo | simondodsley: nvme related? | 14:22 |
rosmaita | simondodsley: that's fine | 14:22 |
jungleboyj | No ... I have a feeling that it will be somewhat contentious. :-) | 14:22 |
rosmaita | though you'll need to get someone to agree to lead the discussion for you | 14:22 |
geguileo | jungleboyj: probably :-) | 14:22 |
jungleboyj | Might be interesting. | 14:23 |
rosmaita | since we're not in a hurry today, i will pause to give people time to look at the etherpad and add comments | 14:23 |
simondodsley | I'll add the topic and if someone want s to lead it that would be great, but if not it can be dropped for another session | 14:24 |
rosmaita | #link https://etherpad.opendev.org/p/zed-ptg-cinder-planning | 14:24 |
geguileo | I'll probably add a couple of topics: Current nvme-of connector issues, drivers reporting their storage_protocol, quota update | 14:25 |
rosmaita | geguileo: ++ | 14:25 |
whoami-rajat | If you've added a topic and have a preferred time and won't be attending the whole PTG, you can add the time (UTC) suited for your TZ (only in the available time slot) | 14:27 |
whoami-rajat | geguileo++ | 14:27 |
geguileo | simondodsley: are there leftovers on ALL successful runs or just some? | 14:29 |
simondodsley | i'd have to check, but it feels like all | 14:29 |
rosmaita | just a reminder about the PTG schedule for cinder: the Cinder team will meet in the 13:00-17:00 UTC blocks on Tuesday 5th April through Friday 8th April 2022. | 14:29 |
rosmaita | wow, we do have a lot of topics for drivers' day! | 14:31 |
geguileo | simondodsley: that'd be the best case scenario, because going into the system we can tell the test that left it behind | 14:31 |
rosmaita | ok, the etherpad is available 24 hours a day, so please add items as they occur to you | 14:31 |
geguileo | and the new field that tells us if it's a temporary resource may help as well | 14:31 |
rosmaita | ok, final announcement | 14:32 |
rosmaita | reminder about the cinder resource count survey | 14:32 |
rosmaita | #link http://lists.openstack.org/pipermail/openstack-discuss/2022-March/027660.html | 14:32 |
rosmaita | we are trying to get geguileo some data to determine what direction his quotas improvement plans should take | 14:33 |
rosmaita | so please tell your customers | 14:33 |
rosmaita | and if you run a cloud, please fill it out yourself! | 14:33 |
geguileo | please, pretty please | 14:33 |
rosmaita | that's all from me, anyone else have an announcement? | 14:33 |
rosmaita | actually, if sofia were here, she would remind us about outreachy | 14:34 |
rosmaita | i believe the deadline for proposals is early next week | 14:34 |
rosmaita | i think this email has details: | 14:34 |
rosmaita | #link http://lists.openstack.org/pipermail/openstack-discuss/2022-February/027239.html | 14:34 |
rosmaita | #topic reminder: no "naked" rechecks | 14:35 |
rosmaita | Gorka added some documentation about this: | 14:35 |
rosmaita | #link https://docs.openstack.org/cinder/latest/contributor/gerrit.html#ci-job-rechecks | 14:36 |
rosmaita | the basic idea is that we want to do a bit of troubleshooting before rechecking | 14:36 |
rosmaita | because a failure in any test that has 'volume' in the name is immediately ascribed to cinder, whether it's a cinder failure or not | 14:36 |
rosmaita | i guess that's all i have to day about that | 14:37 |
whoami-rajat | geguileo, found out a typo if you would like to fix "without locking at the errors" locking -> looking | 14:38 |
* geguileo is the king of typos | 14:38 | |
rosmaita | that's a good typo given that geguileo is our race conditions expert | 14:38 |
whoami-rajat | :D true | 14:39 |
rosmaita | #topic open discussion | 14:39 |
jungleboyj | :-) | 14:40 |
eharney | i have this cool patch that just removes a bunch of dead code, if someone wants an easy review: https://review.opendev.org/c/openstack/cinder/+/800704 | 14:43 |
geguileo | eharney: that one has a couple of +2 already... | 14:43 |
eharney | indeed | 14:43 |
eharney | last week's meeting got it this far, maybe today's can really do it | 14:43 |
rosmaita | eharney: i think you should feel free to +W it | 14:44 |
rosmaita | or maybe jungleboyj will | 14:45 |
geguileo | eharney: why do we need this change? https://review.opendev.org/c/openstack/cinder/+/800704/3/cinder/volume/drivers/remotefs.py | 14:45 |
rosmaita | looks like he and e0ne had a race condition, both thought they were putting the first +2 on it | 14:45 |
jungleboyj | rosmaita: I can. | 14:45 |
rosmaita | ty | 14:45 |
eharney | geguileo: good question | 14:45 |
eharney | geguileo: i wrote this last summer, so lemme go find out... | 14:45 |
geguileo | I know how that feels | 14:46 |
jungleboyj | *sad trombone* | 14:46 |
rosmaita | i wonder if that's a pylint thing | 14:47 |
yuval | I can bring up an idea | 14:48 |
geguileo | eharney: looks like it's irrelevant, it's merging now XD | 14:48 |
yuval | what about moving out unittest from using tox to use docker | 14:48 |
yuval | our* | 14:49 |
eharney | tox works pretty well | 14:49 |
rosmaita | yuval: can you say some more about what you're thinking there? | 14:50 |
yuval | my exp with tox - first was that its not working - every env I had to install it I had to debug certain bug, missing dependency libs | 14:50 |
yuval | just a docker with the minimum req to run the unittests | 14:51 |
yuval | zuul running py36 and py39 in my machine I am running py38 | 14:52 |
yuval | I would like to run exactly the same tests | 14:52 |
yuval | docker solves that | 14:52 |
yuval | is there someone here which is also active in kolla project? | 14:53 |
whoami-rajat | geguileo, eharney i can see those variables not used anywhere so maybe signifying that those are unused (_<var> instead of __) | 14:53 |
geguileo | whoami-rajat: makes sense | 14:53 |
eharney | whoami-rajat: that is the convention of what the leading underscore means, yes -- pylint etc. actually check based on this | 14:53 |
geguileo | thanks | 14:53 |
whoami-rajat | np | 14:54 |
whoami-rajat | ack | 14:54 |
rosmaita | yuval: i use pyenv to give me multiple pythons beyond what's available in my operating system distro | 14:55 |
rosmaita | don't know if that would help you | 14:56 |
rosmaita | ok, two-minute warning ... anyone else have something to mention? | 14:58 |
rosmaita | guess not ... thanks for attending, and don't forget about the festival of reviews on friday | 14:59 |
rosmaita | #endmeeting | 14:59 |
opendevmeet | Meeting ended Wed Mar 16 14:59:48 2022 UTC. Information about MeetBot at http://wiki.debian.org/MeetBot . (v 0.1.4) | 14:59 |
opendevmeet | Minutes: https://meetings.opendev.org/meetings/cinder/2022/cinder.2022-03-16-14.00.html | 14:59 |
opendevmeet | Minutes (text): https://meetings.opendev.org/meetings/cinder/2022/cinder.2022-03-16-14.00.txt | 14:59 |
opendevmeet | Log: https://meetings.opendev.org/meetings/cinder/2022/cinder.2022-03-16-14.00.log.html | 14:59 |
whoami-rajat | thanks! | 14:59 |
jungleboyj | Thank you! | 15:00 |
*** dviroel|ruck is now known as dviroel|ruck|lunch | 16:02 | |
*** dviroel|ruck|lunch is now known as dviroel|ruck | 17:15 | |
*** dviroel|ruck is now known as dviroel|ruck|afk | 20:03 |
Generated by irclog2html.py 2.17.3 by Marius Gedminas - find it at https://mg.pov.lt/irclog2html/!