| rosmaita | #startmeeting cinder | 14:00 |
|---|---|---|
| opendevmeet | Meeting started Wed Oct 4 14:00:29 2023 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 |
| Saikumar | o/ | 14:00 |
| rosmaita | #topic roll call | 14:00 |
| raghavendrat | hi | 14:00 |
| rosmaita | Saikumar: you were slightly early :) | 14:00 |
| jungleboyj | o/ | 14:01 |
| thiagoalvoravel | o/ | 14:01 |
| simondodsley | o/ | 14:01 |
| felipe_rodrigues | o/ | 14:01 |
| caiquemello[m] | o/ | 14:01 |
| happystacker | o/ | 14:01 |
| jayaanand | o/ | 14:01 |
| jbernard | o/ | 14:01 |
| eharney | o/ | 14:02 |
| rosmaita | Rajat is caught up in a customer issue (i think i can say that, it's no secret that we have customers and they occasionally have issues) | 14:02 |
| rosmaita | #link https://etherpad.opendev.org/p/cinder-bobcat-meetings | 14:02 |
| rosmaita | welcome to the final meeting of the bobcat cycle! | 14:02 |
| rosmaita | which i guess brings us directly to | 14:02 |
| rosmaita | #topic announcements | 14:02 |
| rosmaita | 2023.2 (Bobcat) release is tomorrow | 14:03 |
| rosmaita | but we are already tagged for 23.0.0 from RC 2, i believe | 14:03 |
| rosmaita | thanks to everyone for their contributions, and for another successful development cycle | 14:04 |
| rosmaita | and on-time, too | 14:04 |
| rosmaita | next item | 14:05 |
| rosmaita | I put up a proposal to make Jon Bernard a stable core for cinder | 14:05 |
| rosmaita | #link https://lists.openstack.org/pipermail/openstack-discuss/2023-October/035259.html | 14:05 |
| rosmaita | so obviously i am in favor of the proposal | 14:06 |
| rosmaita | and i have seen only positive responses on the ML so far | 14:06 |
| rosmaita | i should say a few things about how openstack operates | 14:06 |
| rosmaita | when proposals are made on the ML, we have a "lazy consensus" model | 14:06 |
| rosmaita | so basically, we assume that everything is OK unless someone says otherwise | 14:07 |
| rosmaita | so probably Rajat will wait for about a week to make sure there are no objections, and then he will make Jon a stable core | 14:07 |
| jungleboyj | ++ | 14:08 |
| rosmaita | if you are wondering what a "stable core" is, we have a handy guide hidden in the cinder docs | 14:08 |
| zaitcev | I don't see why not. Jon doesn't seem like having a history of rubber-stamping dubious patches. | 14:08 |
| rosmaita | (so well hidden that i can't find it) | 14:08 |
| rosmaita | #link https://docs.openstack.org/cinder/latest/contributor/cinder-groups.html | 14:08 |
| rosmaita | that explains the various groups ^^ | 14:09 |
| rosmaita | also, since many of our contributors want to fix stuff and then backport fixes to older branches | 14:09 |
| rosmaita | here's a reminder of the way that works | 14:09 |
| rosmaita | #link https://docs.openstack.org/project-team-guide/stable-branches.html | 14:09 |
| rosmaita | (that's the openstack-wide policy) | 14:10 |
| rosmaita | #link https://docs.openstack.org/cinder/latest/contributor/backporting.html | 14:10 |
| rosmaita | (that's cinder-specific info about backports) | 14:10 |
| rosmaita | one more link, this is the general guide to the cinder project for contributors: | 14:11 |
| rosmaita | tiny.cc/cinder-info | 14:11 |
| rosmaita | ok, final announcement | 14:11 |
| simondodsley | Great to see all these listed in one place at last - now to put this list somewhere permanent | 14:12 |
| rosmaita | well, it's permanent, just not easily discoverable! | 14:12 |
| happystacker | godd idea! | 14:12 |
| simondodsley | that's really what i meant | 14:12 |
| rosmaita | which reminds me ... people sometimes ask about easy first contributions | 14:12 |
| rosmaita | remember that the cinder docs are in the code repository | 14:13 |
| rosmaita | so fixing documents is just as "real" a contribution as code fixes | 14:13 |
| rosmaita | and i got off track there, here is the final announcement: | 14:13 |
| rosmaita | Add topics for 2024.1 (Caracal) PTG | 14:13 |
| rosmaita | the virtual PTG will be held 23-27 October | 14:14 |
| rosmaita | #link https://etherpad.opendev.org/p/caracal-ptg-cinder-planning | 14:14 |
| rosmaita | Rajat is collecting topics ^^ | 14:14 |
| rosmaita | it might be a good use of time to do a doc review to maybe re-organize them, or to identify holes in our docs | 14:15 |
| rosmaita | (if anyone wants to lead such a session, add it to the etherpad!) | 14:15 |
| rosmaita | ok, that's all the official announcements ... anyone got anything else to announce? | 14:16 |
| jbernard | oh o/ our outreachy proposal got accepted (osc client gap) | 14:16 |
| rosmaita | nice! | 14:16 |
| jbernard | #link https://www.outreachy.org/outreachy-december-2023-internship-round/communities/openstack/reduce-openstack-client-gap/cfp/ | 14:17 |
| rosmaita | is there anything we can do to get potential outreachy participants interested? | 14:17 |
| rosmaita | maybe we need to mention in each meeting how exciting and important it is to address the osc / cinderclient gap | 14:18 |
| jbernard | im not sure sure honestly, applications are being accepted, so potential folks are encouraged to reach out | 14:18 |
| jbernard | i haven't heard anything yet, but it's only been a day | 14:18 |
| rosmaita | jbernard: will you be mentoring? do you need help? | 14:19 |
| jbernard | i will be, im happy to take it on and will ask for help if needed, you're certainly welcome to co-mentor, everythign helps | 14:20 |
| rosmaita | ok, so anyone interested in helping out, contact jbernard | 14:21 |
| rosmaita | thanks! | 14:21 |
| rosmaita | ok, on to the regular topics | 14:21 |
| rosmaita | looks like mostly review requests | 14:22 |
| rosmaita | Increase size of volume image metadata values | 14:22 |
| rosmaita | #link https://review.opendev.org/c/openstack/cinder/+/868485 | 14:22 |
| rosmaita | i think this one is in good shape, has some excellent tests, not sure what the holdup is | 14:22 |
| rosmaita | anyone have any questions about the patch? | 14:23 |
| rosmaita | it's got a +1 from zaitcev and he is not an easy reviewer | 14:24 |
| eharney | i've looked at it a few times, but i'm a bit confused about what's going on there when i look at it today | 14:25 |
| eharney | it doesn't change field lengths in our db model? | 14:25 |
| rosmaita | no, we already had the right size field | 14:25 |
| zaitcev | Well. I'm stickler to details but you can easily hoodwink me about the global architecture of Cinder.. | 14:25 |
| eharney | ah right | 14:25 |
| rosmaita | our schema validation was rejecting the "too long" values | 14:26 |
| rosmaita | we had discussed the mismatch between character count and byte count for utf-8 | 14:26 |
| eharney | yeah, i remember being ok with this whole thing a while back, but it's been long enough that i forgot the details, and the commit message doesn't really spell it out :/ | 14:26 |
| eharney | seems good though | 14:26 |
| rosmaita | and the author added custom schema validation to make sure the bytes will fit regardless of character count | 14:26 |
| rosmaita | ok, so eharney can we count on you taking a closer look in the next few days? | 14:27 |
| eharney | yes | 14:27 |
| rosmaita | excellent | 14:28 |
| drencrom | thanks folks! | 14:28 |
| rosmaita | ok, simondodsley i think you have the next 2 | 14:28 |
| simondodsley | Yep - first is just a bugfix backport | 14:28 |
| rosmaita | ok, i reviewed the patch to master, so i will take a look | 14:29 |
| simondodsley | second is more extensive. This one includes work from an abandoned patch by geguileo and a bug fix | 14:29 |
| simondodsley | it also enables a volume to be sync repl volume to be created during a failover event | 14:30 |
| simondodsley | We did them all as one as the interlock was a bit messy for multiple dependant patches | 14:31 |
| rosmaita | ok, sounds reasonable | 14:32 |
| rosmaita | any questions for Simon? | 14:33 |
| simondodsley | Thanks guys for the +2s and +W on the backport | 14:33 |
| rosmaita | next up, 2 patches from Dell ... happystacker is that you? | 14:34 |
| rosmaita | the patches look pretty straightforward | 14:35 |
| rosmaita | ok, looks like a job for stable cores since they are both backports | 14:37 |
| rosmaita | raghavendrat: any helpful comments about your patch? | 14:37 |
| rosmaita | #link https://review.opendev.org/c/openstack/cinder/+/887559 | 14:37 |
| raghavendrat | hi | 14:37 |
| raghavendrat | i have received one +2 ... Rajat had a comment. i replied in code review | 14:38 |
| raghavendrat | waiting for Rajat's response | 14:38 |
| rosmaita | was just looking, looks like you explained the issue and have some helpful comments in the code also | 14:38 |
| raghavendrat | this patch is one of the patches targeted for RC2 | 14:39 |
| happystacker | yes two backports | 14:39 |
| rosmaita | looks like we missed that, but we'll be doing another bobcat release in a month or so, i imagine | 14:40 |
| raghavendrat | ok | 14:40 |
| rosmaita | since Rajat already reviewed, i'm inclined to let him follow up, especially since you have a +2 from Walt | 14:41 |
| raghavendrat | fine. thanks for having a look | 14:41 |
| rosmaita | hopefully we can get this knocked out before the next meeting | 14:41 |
| jungleboyj | happystacker: Backports are taken care of. | 14:41 |
| rosmaita | jungleboyj: thanks! | 14:42 |
| jungleboyj | Welcome. :-) | 14:42 |
| rosmaita | final item on the list is from eharney | 14:42 |
| happystacker | thks guys! | 14:42 |
| rosmaita | #link https://review.opendev.org/c/openstack/cinder/+/888296 | 14:42 |
| eharney | yep | 14:43 |
| eharney | it's a bugfix | 14:44 |
| rosmaita | looks like an old bug, too | 14:45 |
| jungleboyj | :-) | 14:45 |
| rosmaita | oslo fix is from 2017! | 14:45 |
| happystacker | wow! | 14:45 |
| eharney | there's also a task for someone to work on us not duplicating code out of oslo.db there | 14:45 |
| eharney | but this was the quick fix | 14:45 |
| rosmaita | ok, i will commit to looking at this one | 14:46 |
| rosmaita | this is slightly off topic, but the fact that you had to recheck reminded me of something | 14:46 |
| rosmaita | i have seen a few weird openstack-tox-py310 failures | 14:47 |
| rosmaita | but they don't seem to be systematic, which is weird | 14:47 |
| rosmaita | because the error is | 14:47 |
| rosmaita | oslo_config.cfg.NoSuchOptError: no such option cinder_internal_tenant_project_id in group [DEFAULT] | 14:47 |
| rosmaita | which you'd think would happen on every run | 14:47 |
| rosmaita | so i'm not sure what's going on there | 14:48 |
| rosmaita | just want to give a heads up to anyone about to do a recheck | 14:48 |
| rosmaita | if you see that error, put a shout in the #openstack-cinder channel so we can keep an eye on it | 14:48 |
| zaitcev | The bug for that boolean thing makes sense. Even if making the request were pointless, it removes a traceback. | 14:49 |
| rosmaita | and that brings us to | 14:50 |
| rosmaita | #topic Open Discussion | 14:50 |
| eharney | the request actually makes sense, it's just that, i guess not too many people are using this functionality | 14:50 |
| rosmaita | anyone got anything they'd like to discuss? | 14:52 |
| rosmaita | guess not | 14:54 |
| rosmaita | ok, thanks for attending everybody ... don't forget to add topics to the PTG etherpad: | 14:54 |
| rosmaita | https://etherpad.opendev.org/p/caracal-ptg-cinder-planning | 14:54 |
| rosmaita | next meeting will be the first of the Caracal cycle, see you then! | 14:55 |
| rosmaita | #endmeeting | 14:55 |
| opendevmeet | Meeting ended Wed Oct 4 14:55:29 2023 UTC. Information about MeetBot at http://wiki.debian.org/MeetBot . (v 0.1.4) | 14:55 |
| opendevmeet | Minutes: https://meetings.opendev.org/meetings/cinder/2023/cinder.2023-10-04-14.00.html | 14:55 |
| opendevmeet | Minutes (text): https://meetings.opendev.org/meetings/cinder/2023/cinder.2023-10-04-14.00.txt | 14:55 |
| opendevmeet | Log: https://meetings.opendev.org/meetings/cinder/2023/cinder.2023-10-04-14.00.log.html | 14:55 |
| jungleboyj | Thanks! | 14:57 |
Generated by irclog2html.py 2.17.3 by Marius Gedminas - find it at https://mg.pov.lt/irclog2html/!