| *** bauzas_ is now known as bauzas | 01:12 | |
| *** bauzas_ is now known as bauzas | 07:36 | |
| *** bauzas_ is now known as bauzas | 08:57 | |
| *** whoami-rajat_ is now known as whoami-rajat | 13:56 | |
| jbernard | #startmeeting cinder | 14:00 |
|---|---|---|
| opendevmeet | Meeting started Wed Aug 7 14:00:50 2024 UTC and is due to finish in 60 minutes. The chair is jbernard. 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 |
| jbernard | #topic roll call | 14:00 |
| jbernard | o/ | 14:00 |
| simondodsley | o/ | 14:01 |
| yuval | 0/ | 14:01 |
| akawai | o/ | 14:01 |
| whoami-rajat | Hi | 14:01 |
| rosmaita | o/ | 14:01 |
| msaravan | hi | 14:02 |
| jbernard | #link https://etherpad.opendev.org/p/cinder-dalmatian-meetings | 14:02 |
| jungleboyj | o/ | 14:04 |
| jbernard | hello all, thanks for coming | 14:05 |
| raghavendrat | hi | 14:05 |
| jbernard | #topic annoucements | 14:05 |
| jbernard | https://releases.openstack.org/dalmatian/schedule.html | 14:06 |
| jbernard | we are fast approaching the third milestone (D-3) in a couple of weeks | 14:06 |
| jbernard | our midcycle is ptg is next week! | 14:07 |
| jbernard | #action jbernard to update the ptg etherpad | 14:07 |
| whoami-rajat | just a note, I will not be around the PTG time next week | 14:08 |
| whoami-rajat | have a conflict with a family event during evening | 14:08 |
| jbernard | whoami-rajat: ok, let us know if you need something raised or discussed | 14:08 |
| whoami-rajat | sure thanks jbernard | 14:08 |
| jbernard | aside from the ptg, the non-client library freeze will happen on Aug 22 | 14:09 |
| jbernard | and stable branches for those will be cut a bit before the rest of our repositories | 14:10 |
| rosmaita | we need to prioritize review of os-brick patches | 14:10 |
| jbernard | this means os-brick | 14:10 |
| jbernard | rosmaita: exactly | 14:10 |
| jbernard | in other words, if you have review cycles, os-brick needs some attention to make sure we get what we need | 14:11 |
| jbernard | the freeze for everything else is the week following | 14:12 |
| jbernard | so there is much to do :) | 14:12 |
| jbernard | in other news, we've seen some movement in review requests from last week | 14:13 |
| jbernard | i tried to get to as many as I could and several were crossed off the list | 14:13 |
| yuval | Thanks you! | 14:13 |
| jbernard | thanks to everyone that reviewed | 14:13 |
| whoami-rajat | do we have a list of os-brick patches that need attention? or maybe create an etherpad to track them? | 14:14 |
| whoami-rajat | the open patches list seems broad | 14:14 |
| whoami-rajat | #link https://review.opendev.org/q/project:openstack/os-brick+status:open | 14:15 |
| jbernard | whoami-rajat: not that I know of, i think that's a great idea | 14:15 |
| whoami-rajat | but a lot are in merge connflict | 14:15 |
| jbernard | yuval: i saw your brick patch, will review today | 14:15 |
| yuval | wait for it - I still have some issues | 14:15 |
| yuval | but thank you! | 14:15 |
| jbernard | #action jbernard to create etherpad for os-brick patch priority | 14:16 |
| jbernard | yuval: sure | 14:16 |
| jbernard | I'll send a link on the list later today | 14:16 |
| jbernard | that's all I have for annoucments, things are winding down toward the release so be mindful of the schedule as we move through august | 14:17 |
| jbernard | as always, if you feel something is being overlooked, please reach out | 14:18 |
| jbernard | #topic scheduler related unite test hardening | 14:18 |
| jbernard | rosmaita: ^ | 14:18 |
| rosmaita | thanks! | 14:18 |
| rosmaita | i was running unit tests in a weird environment recently and hit a bunch (around 284 failures) consistently | 14:19 |
| rosmaita | i traced it down to an issue with stevedore not locating the entry points in the cinder setup.cfg where we list the scheduler filters | 14:20 |
| rosmaita | which is not nice, but should not affect unit tests | 14:21 |
| rosmaita | since the unit tests are supposed to be isolated to test the code they are designed to test, | 14:21 |
| rosmaita | anyway, i put up a series of small patches to fix this, so that the tests are isolated correctly | 14:21 |
| rosmaita | they will be easy to review | 14:21 |
| rosmaita | (did i say they are small) | 14:22 |
| rosmaita | that's basically it | 14:22 |
| jbernard | rosmaita: are they under a specific topic? | 14:23 |
| jbernard | nevermind, i see the relation chain | 14:23 |
| rosmaita | just checking, yes, they are | 14:23 |
| rosmaita | https://review.opendev.org/q/topic:%22scheduler-tests%22 | 14:23 |
| whoami-rajat | we have festival of XS reviews next week but it's always good to even review them before that | 14:24 |
| rosmaita | :D | 14:24 |
| yuval | open discussion? | 14:26 |
| jbernard | #topic open discussion! | 14:27 |
| whoami-rajat | is anyone from storpool team around? | 14:27 |
| whoami-rajat | looks like not | 14:30 |
| whoami-rajat | I had a PTL question for jbernard and probably rosmaita | 14:30 |
| whoami-rajat | I wanted to backport this os-brick patch | 14:31 |
| whoami-rajat | #link https://review.opendev.org/c/openstack/os-brick/+/920516 | 14:31 |
| whoami-rajat | but it adds 2 new config options with suitable defaults | 14:31 |
| whoami-rajat | We haven't allowed it in the past for Cinder but since os-brick was a library, I'm not sure | 14:31 |
| whoami-rajat | s/was/is | 14:32 |
| jungleboyj | Hmmm, that is an interesting one. | 14:32 |
| jbernard | im not personally opposed, i wonder what the rules and precedence are. | 14:32 |
| jungleboyj | I am trying to remember what the rules are if they are config changes. | 14:33 |
| * jbernard is digging through old notes | 14:34 | |
| rosmaita | i guess we can ask elod, but i think worrying about that is a relic from the old days before oslo.config | 14:34 |
| rosmaita | i mean, they have sensible defaults, so it's not like an operator needs to do anything | 14:35 |
| rosmaita | and having them there allows for tuning | 14:35 |
| jungleboyj | Right. And I think that adding a config to fix a bug with sensible defaults should be ok. | 14:35 |
| rosmaita | otherwise, we would have to remove the options and hard-code the values | 14:35 |
| rosmaita | which would be pretty dumb, in my opinion | 14:35 |
| jbernard | i agree with all of that | 14:35 |
| jungleboyj | Reading the rules it says 'no incompatible config file changes'. | 14:36 |
| rosmaita | thanks, jungleboyj | 14:36 |
| jbernard | #action whoami-rajat to run his patch by elod | 14:36 |
| jbernard | sounds like it will be fine thought | 14:36 |
| rosmaita | sounds like we are OK here | 14:36 |
| jungleboyj | Let's do that to be safe, but I think this should be ok. | 14:36 |
| whoami-rajat | the initial patch did hardcode values but it only catered a small number of issues related to multipath, making it tune-able helps address broader issues like udev rules taking time to load | 14:37 |
| jungleboyj | Yeah, makes sense to have that be configurable. | 14:38 |
| whoami-rajat | thanks jungleboyj rosmaita and jbernard for your opinions, i will propose a backport and we can run it by elod | 14:38 |
| jungleboyj | ++ | 14:38 |
| jbernard | anyone else? yuval - you mentioned you have issues? | 14:39 |
| jbernard | we have a few more minutes if needed | 14:39 |
| yuval | I want to ask | 14:40 |
| yuval | if I run my setup with ACTIVE/ACTIVE | 14:40 |
| yuval | then the same setup - I want to disable the ACTIVE/ACTIVE | 14:40 |
| yuval | just removing the conf from the cinder.conf doesnt seems to do the trick | 14:41 |
| jbernard | can you elaborate, what is the setup, what behaviour are you expecting, and what are you actually seeing? | 14:42 |
| jbernard | also, is this a general issue, or one with your driver? this might be better in general irc or email form, i guess it depends on how much info is needed to be exchanged - if it's a bug though - we will want to capture it | 14:44 |
| yuval | I dont this its a bug just I dont fully understand how it works | 14:44 |
| yuval | I am setting the "cluster" the config for cinder | 14:44 |
| yuval | then I see my cinder backend is running as a part of a cluster | 14:45 |
| yuval | with a single backend | 14:45 |
| jbernard | someone correct me if i am wrong, but to move from A/A to A/P, you would need to modify your deployment to include something like haproxy, no? | 14:45 |
| yuval | I want to do some checks - afterwards run the same checks but not in a cluster | 14:45 |
| yuval | I am following this doc: https://docs.openstack.org/cinder/latest/contributor/high_availability.html | 14:46 |
| yuval | but have not see how to reverse it | 14:47 |
| yuval | other than destroy my setup and re-create without the "cluster" config | 14:47 |
| jbernard | personally, i don't know - i would need to try it in devstack and see for myself | 14:47 |
| simondodsley | i think the issue might be that volumes created under the cluster mode have the incorrect hostname in the DB after going back to non-cluster, so they will effectivley be unmanageable | 14:48 |
| yuval | simondodsley: thanks I will try to view the DB directly | 14:49 |
| simondodsley | i guess if that is the issue, then there needs to be a process, after disabling cluster, to clean the DB with a cinde manage? Or we could just say that reversion is not supported | 14:50 |
| yuval | yes I wonder if we have something like that | 14:51 |
| jbernard | yuval: probably not currently | 14:53 |
| yuval | got it | 14:53 |
| jbernard | ok, last call | 14:54 |
| jbernard | thanks everyone! | 14:55 |
| jbernard | #endmeeting | 14:55 |
| opendevmeet | Meeting ended Wed Aug 7 14:55:09 2024 UTC. Information about MeetBot at http://wiki.debian.org/MeetBot . (v 0.1.4) | 14:55 |
| opendevmeet | Minutes: https://meetings.opendev.org/meetings/cinder/2024/cinder.2024-08-07-14.00.html | 14:55 |
| opendevmeet | Minutes (text): https://meetings.opendev.org/meetings/cinder/2024/cinder.2024-08-07-14.00.txt | 14:55 |
| opendevmeet | Log: https://meetings.opendev.org/meetings/cinder/2024/cinder.2024-08-07-14.00.log.html | 14:55 |
| jungleboyj | Thank you! | 14:55 |
| yuval | Thank you | 14:56 |
| whoami-rajat | thanks! | 14:57 |
| *** bauzas_ is now known as bauzas | 18:15 | |
| *** bauzas_ is now known as bauzas | 18:26 | |
| *** bauzas_ is now known as bauzas | 19:36 | |
Generated by irclog2html.py 2.17.3 by Marius Gedminas - find it at https://mg.pov.lt/irclog2html/!