| *** dviroel|ourt is now known as dviroel | 11:45 | |
| *** dasm|off is now known as dasm | 13:50 | |
| carloss | #startmeeting manila | 15:00 |
|---|---|---|
| opendevmeet | Meeting started Thu Jan 5 15:00:19 2023 UTC and is due to finish in 60 minutes. The chair is carloss. Information about MeetBot at http://wiki.debian.org/MeetBot. | 15:00 |
| opendevmeet | Useful Commands: #action #agreed #help #info #idea #link #topic #startvote. | 15:00 |
| opendevmeet | The meeting name has been set to 'manila' | 15:00 |
| haixin | o/ | 15:00 |
| gouthamr | o/ | 15:00 |
| felipe_rodrigues | o/ | 15:00 |
| ashrodri | o/ | 15:01 |
| chuanm | hi | 15:01 |
| nahimsouza[m] | o/ | 15:01 |
| carthaca | hi | 15:01 |
| kpdev_ | hi | 15:01 |
| lucasmoliveira059 | \o | 15:01 |
| carloss | courtesy ping: vkmc dviroel | 15:01 |
| dviroel | hi o/ | 15:01 |
| MatheusAndrade[m] | o/ | 15:02 |
| carloss | o/ hello everyone | 15:03 |
| carloss | and happy new year! | 15:04 |
| carloss | it's good to be back | 15:05 |
| haixin | Is the Christmas break over? | 15:05 |
| carloss | haixin: for many of us, yes :) | 15:05 |
| carloss | some could have extended | 15:05 |
| carloss | today's meeting agenda: | 15:05 |
| carloss | #link https://wiki.openstack.org/wiki/Manila/Meetings#Next_meeting | 15:05 |
| thiagoalvoravel | hi | 15:05 |
| carloss | o. thiagoalvoravel | 15:06 |
| carloss | o/ * | 15:06 |
| HelenaDantas[m] | o/ | 15:06 |
| carloss | #topic Announcements | 15:06 |
| carloss | Schedule and Deadlines: | 15:07 |
| carloss | #link https://releases.openstack.org/antelope/schedule.html | 15:07 |
| carloss | so we are past spec freeze and this week is M-2 for Antelope | 15:07 |
| carloss | next week would be our new driver deadline but apparently we don't have candidates for this cycle | 15:08 |
| carloss | so I think we are good with that | 15:09 |
| carloss | next deadlines to track, I'd say: feature proposal freeze (in 2 weeks) | 15:10 |
| carloss | where featureful patches should be submitted with unit and functional tests to gerrit (also client changes if needed) | 15:11 |
| carloss | and m-3 is in 6 weeks, which is also our client merge deadline | 15:11 |
| carloss | next announcement is the PTG, which is in a few months from now | 15:12 |
| carloss | #link https://openinfra-ptg.eventbrite.com | 15:12 |
| carloss | please register if you intend to participate. I will create the topics etherpad soon and share it with you | 15:13 |
| carloss | on a side note: talk and forums submission for the next OpenInfra Summit in June is still open | 15:14 |
| carloss | so you can propose talks you might want to bring up in the CFP | 15:14 |
| carloss | #link https://openinfra.dev/summit/vancouver-2023 | 15:14 |
| carloss | ^ more info here | 15:14 |
| carloss | and last announcement on my side: | 15:15 |
| carloss | we have a name for the next release | 15:15 |
| carloss | 2023.2, aka Bobcat :D | 15:15 |
| carloss | #link https://lists.openstack.org/pipermail/openstack-discuss/2023-January/031666.html | 15:15 |
| carloss | the foundation shared the name yesterday | 15:16 |
| carloss | that's all I had for announcements | 15:16 |
| carloss | is there an announcement you would like to share with us today? | 15:16 |
| MatheusAndrade[m] | I have an announcement, we have a new member in Netapp Team | 15:19 |
| MatheusAndrade[m] | thiagoalvoravel: welcome ^^ | 15:19 |
| lucasmoliveira059 | welcome thiagoalvoravel ^^ | 15:19 |
| dviroel | thiagoalvoravel: welcome o/ | 15:19 |
| carloss | welcome thiagoalvoravel! glad to hear the Zorilla team is growing up | 15:20 |
| gouthamr | ++ welcome thiagoalvoravel | 15:20 |
| felipe_rodrigues | welcome :) | 15:20 |
| thiagoalvoravel | thank you everybody =) | 15:21 |
| carloss | moving on... | 15:22 |
| carloss | #topic Feedback on deleting service table entries (gouthamr/kpdev) | 15:22 |
| carloss | gouthamr kpdev_: floor is yours | 15:22 |
| gouthamr | thanks carloss | 15:22 |
| gouthamr | wanted to get some more eyes on a change to clean up the service table entries while bringing down a service | 15:24 |
| gouthamr | #link https://review.opendev.org/c/openstack/manila/+/859301 | 15:24 |
| kpdev_ | i think we can make changes as you suggested | 15:24 |
| gouthamr | this should be fine in deployments as is where services are deployed on unique hosts | 15:24 |
| kpdev_ | instead of deleting service entry during service stop, we mark it as STOPPED | 15:25 |
| kpdev_ | and add per service periodic cleanup task, which checks for STOPPED entries and delete those entries. | 15:25 |
| kpdev_ | if entry is marked as DOWN, it wont be deleted as its actual service down and not the stopped in previous iterations. | 15:26 |
| kpdev_ | wdyt, gouthamr | 15:27 |
| kpdev_ | in our use case, multiple manila-share backends deployed as PODS and so they does on different host during each deployment, so we want to delete stale entries of invalid/previous deployments. | 15:28 |
| gouthamr | kpdev_: that sounds okay.. do i understand correctly that you'd introduce a new column in the service table? | 15:28 |
| gouthamr | kpdev_: indeed, the use case is perfectly valid - trying to look out for another set of deployments where we configure multiple copies of the service across different hosts by overriding the "host" attribute to the same name | 15:29 |
| gouthamr | thereby making these services look the same, but still have H/A | 15:30 |
| kpdev_ | yes, i will add new column and periodic cleanup task | 15:30 |
| gouthamr | thanks, i like that solution | 15:32 |
| carloss | ++ I like it too | 15:32 |
| haixin | what is the different between your new column mark STPPPED and the original logic uses service_down_time (60s by default) to determine if the service is down | 15:32 |
| kpdev_ | STOPPED is service stopped and if its stopped more than periodic interval, its stale entry.. so should be deleted | 15:34 |
| kpdev_ | while DOWN, service down under current deployment | 15:34 |
| haixin | This means we need to develop a new api to flag the service as stopped? | 15:35 |
| gouthamr | haixin: won't be necessary; if services need to be taken down explicitly, the "disable" API would work fine.. | 15:37 |
| gouthamr | the "stopped" state is only an internal representation of the service | 15:37 |
| gouthamr | nothing should rely on it except this periodic cleanup task | 15:37 |
| haixin | got it, i will check this "disable" api. | 15:37 |
| gouthamr | if a service reports again within the report interval, we pop off the "stopped" state | 15:38 |
| haixin | sounds good | 15:39 |
| kpdev_ | this will mostly happen during service start i.e. move from STOPPED to UP | 15:40 |
| carloss | sounds great kpdev_ | 15:41 |
| gouthamr | kpdev_: we wouldn't wait for start time, if a service reports, we can simultaneously remove the stopped state.. | 15:41 |
| kpdev_ | yes | 15:41 |
| carloss | thanks kpdev_ gouthamr haixin :) | 15:42 |
| carloss | do you have something else to add? | 15:43 |
| gouthamr | ++.. thanks kpdev_ for this enhancement! | 15:43 |
| carloss | okay, moving to the next topic: | 15:44 |
| carloss | #topic Review Focus | 15:44 |
| carloss | #link https://etherpad.opendev.org/p/manila-antelope-review-focus | 15:45 |
| carloss | the etherpad was updated earlier today | 15:45 |
| carloss | the usual changes are there, some changes in the pad were merged and I removed them from the list | 15:45 |
| carloss | and added some more | 15:45 |
| carloss | stephenfin's changes for making the code compatible with sqlalchemy 2.0 | 15:46 |
| carloss | #link https://review.opendev.org/q/topic:sqlalchemy-20+project:openstack/manila | 15:46 |
| carloss | would be good to have some eyes on these changes | 15:46 |
| carloss | thanks gouthamr for the reviews already | 15:46 |
| carloss | I intend to review the changes, but would like to voluntell some people :) | 15:46 |
| carloss | felipe_rodrigues ashrodri MatheusAndrade[m]: could you please take a look when you have some time? :) | 15:47 |
| felipe_rodrigues | sure | 15:47 |
| MatheusAndrade[m] | sure | 15:47 |
| carloss | awesome, thanks! :) | 15:48 |
| carloss | is there any other change you would like to bring up during this meeting? | 15:49 |
| haixin | support share transfer between project need review | 15:50 |
| haixin | https://review.opendev.org/c/openstack/manila/+/843832 | 15:50 |
| kpdev_ | I will review | 15:50 |
| haixin | thanks :) | 15:50 |
| gouthamr | +1 me too, haixin can you rebase? should be a trivial one | 15:51 |
| carloss | ++, I intend to take a look as well | 15:51 |
| haixin | ok, i will rebase it tomorrow | 15:51 |
| gouthamr | thanks haixin | 15:51 |
| carloss | thanks! | 15:51 |
| carloss | and our last topic for today: | 15:52 |
| carloss | #topic Bug Triage | 15:52 |
| carloss | we have one bug we carried from last week: | 15:53 |
| carloss | #link https://bugs.launchpad.net/manila/+bug/2000171 | 15:53 |
| carloss | thanks for filing the bug carthaca | 15:53 |
| carloss | and thanks for working on the fix kpdev_ | 15:53 |
| carloss | #link https://review.opendev.org/c/openstack/manila/+/868486 | 15:54 |
| gouthamr | +1 ; medium, M-3? | 15:55 |
| carloss | yes, I'd say so | 15:55 |
| carloss | an RFE, good to have | 15:55 |
| carloss | some netappers eyes on this would be appreciated too :) | 15:56 |
| * carloss checks time | 15:58 | |
| felipe_rodrigues | sure, I will | 15:58 |
| carloss | that's all for bus on the list | 15:58 |
| carloss | thanks felipe_rodrigues | 15:58 |
| carloss | thank you for joining today's meeting! | 15:59 |
| carloss | let's get back to #openstack-manila | 15:59 |
| carloss | #endmeeting | 15:59 |
| opendevmeet | Meeting ended Thu Jan 5 15:59:32 2023 UTC. Information about MeetBot at http://wiki.debian.org/MeetBot . (v 0.1.4) | 15:59 |
| opendevmeet | Minutes: https://meetings.opendev.org/meetings/manila/2023/manila.2023-01-05-15.00.html | 15:59 |
| opendevmeet | Minutes (text): https://meetings.opendev.org/meetings/manila/2023/manila.2023-01-05-15.00.txt | 15:59 |
| opendevmeet | Log: https://meetings.opendev.org/meetings/manila/2023/manila.2023-01-05-15.00.log.html | 15:59 |
| gouthamr | thanks carloss | 15:59 |
| *** dviroel is now known as dviroel|lunch | 16:01 | |
| *** dviroel|lunch is now known as dviroel | 17:10 | |
| *** dviroel is now known as dviroel|afk | 21:19 | |
| *** dasm is now known as dasm|off | 22:33 | |
Generated by irclog2html.py 2.17.3 by Marius Gedminas - find it at https://mg.pov.lt/irclog2html/!