*** dviroel|afk is now known as dviroel | 00:31 | |
*** dviroel is now known as dviroel|out | 00:35 | |
*** hemna8 is now known as hemna | 07:38 | |
*** dviroel|out is now known as dviroel | 11:15 | |
*** dasm|off is now known as dasm | 12:08 | |
gouthamr | #startmeeting manila | 15:01 |
---|---|---|
opendevmeet | Meeting started Thu Jan 13 15:01:00 2022 UTC and is due to finish in 60 minutes. The chair is gouthamr. Information about MeetBot at http://wiki.debian.org/MeetBot. | 15:01 |
opendevmeet | Useful Commands: #action #agreed #help #info #idea #link #topic #startvote. | 15:01 |
opendevmeet | The meeting name has been set to 'manila' | 15:01 |
vkmc | o/ | 15:01 |
ashrodri | o/ | 15:01 |
fabiooliveira | o/ | 15:01 |
vhari | o/ | 15:01 |
carloss | o/ | 15:01 |
nahimsouza[m] | o/ | 15:01 |
felipe_rodrigues | o/ | 15:01 |
gouthamr | courtesy ping: ganso dviroel tbarron felipe_rodrigues ecsantos | 15:01 |
ecsantos[m] | o/ | 15:01 |
dviroel | o/ | 15:02 |
sfernand | hi | 15:02 |
caiquemello[m] | o/ | 15:02 |
gouthamr | hello everyone o/ | 15:02 |
gouthamr | thank you for joining, here's our agenda for this meeting: https://wiki.openstack.org/wiki/Manila/Meetings#Next_meeting | 15:03 |
gouthamr | lets get started | 15:03 |
gouthamr | #topic Announcements | 15:03 |
gouthamr | pretty silent second week of 2022 | 15:04 |
gouthamr | we're trying to name the Zorilla cycle if you missed the ML post: | 15:04 |
gouthamr | #link http://lists.openstack.org/pipermail/openstack-discuss/2022-January/026620.html ([all] Nomination open for OpenStack "Z" Release Naming) | 15:04 |
gouthamr | our new driver deadline for yoga is at the end of next week | 15:05 |
gouthamr | #link https://releases.openstack.org/yoga/schedule.html | 15:06 |
vkmc | ++ whoever that proposed zorilla to the names | 15:06 |
gouthamr | and the feature proposal freeze is in two weeks (Jan 28) | 15:06 |
gouthamr | time's flying, so if you have featureful changes that you're working on that need to get merged in the yoga cycle, please pay attention to this deadline | 15:07 |
dviroel | two weeks of fun for some of you :) | 15:08 |
gouthamr | mental exercise :) | 15:08 |
felipe_rodrigues | this release is going so fast, just 2 weeks | 15:09 |
carloss | :D | 15:09 |
gouthamr | felipe_rodrigues: i know - its like we lost nov-dec; but the calendar in my head still thinks its 2019 | 15:10 |
gouthamr | anyone else got further announcements for today? | 15:10 |
gouthamr | #topic Changes to manila-stable-maint | 15:11 |
gouthamr | for a very long time, we've had a stable maintenance team that managed stable release branches | 15:12 |
gouthamr | so we'd propose members and the team would vet these members whether they'd adhere to the stable branch maintenance procedures | 15:13 |
gouthamr | before allowing them to +2/+W patches against stable branches of manila repositories | 15:14 |
gouthamr | which meant that this was a subset of manila-core | 15:14 |
gouthamr | there's a change that i'd like to discuss with this process: | 15:15 |
gouthamr | #link https://governance.openstack.org/tc/resolutions/20210923-stable-core-team.html (2021-09-23 Stable Core Team Process) | 15:15 |
gouthamr | given our proactive approach to backports, we need as many reviewers that we can use on stable branch changes | 15:16 |
gouthamr | and in the recent past, i've considered +1s from non-stable cores to mean a +2 mentally because we were spread too thin | 15:17 |
gouthamr | #link https://docs.openstack.org/project-team-guide/stable-branches.html#appropriate-fixes | 15:17 |
gouthamr | i think at this point, everyone on manila-*-core has read these policies | 15:18 |
gouthamr | and committed them to heart when reviewing/proposing changes to stable branches | 15:18 |
gouthamr | i'd like to know if this is a good time to merge the core and stable-core teams... | 15:18 |
gouthamr | i don't mean that we'd dissolve the manila*-stable-maint teams on gerrit | 15:19 |
gouthamr | but, i'd like to add all members from the regular manila*-core groups to the respective manila*-stable-maint teams | 15:19 |
carloss | that makes sense to me gouthamr... and we'd have more hands to get things moving on the stable branches | 15:20 |
gouthamr | as we add new members to the core team, we can still take care to maybe take a cycle's worth of vetting if the person hasn't done enough core reviews or needs time to understand the stable branch process | 15:20 |
dviroel | I'm ok with that. As long they are aware with https://docs.openstack.org/project-team-guide/stable-branches.html#appropriate-fixes as you mentioned gouthamr | 15:21 |
* dviroel sorry missing some stable reviews :( | 15:22 | |
gouthamr | dviroel you're not, i'm piling most reviews on you and carloss atm :D | 15:22 |
vhari | dviroel++ carloss++ | 15:23 |
gouthamr | thanks for your vote of confidence; any objections to this proposal? | 15:24 |
* gouthamr *crickets*... ahem.. *zorillas* | 15:25 | |
gouthamr | alright :) i'll make these changes | 15:27 |
gouthamr | any other thoughts about $topic? | 15:27 |
gouthamr | #topic Reviews needing attention | 15:28 |
gouthamr | #link https://etherpad.opendev.org/p/manila-yoga-review-focus | 15:28 |
gouthamr | alright, i think a couple of the open specs merged | 15:29 |
gouthamr | #link https://review.opendev.org/c/openstack/manila-specs/+/818854 (share transfer between project) | 15:29 |
* dviroel fixes year :) | 15:29 | |
gouthamr | ^ this one is waiting on vkmc, sfernand | 15:30 |
gouthamr | :P dviroel++ | 15:30 |
sfernand | hey sorry didn't have time to work on it yet | 15:30 |
sfernand | but I will prioritize this review for today | 15:30 |
vkmc | same, sorry about that | 15:31 |
gouthamr | thanks sfernand vkmc | 15:31 |
gouthamr | can i have some more eyes on a six removal patch? | 15:32 |
gouthamr | #link https://review.opendev.org/c/openstack/manila/+/821162 (remove usage of six library in driver layer and the corresponding test) | 15:33 |
fabiooliveira | i could take a look | 15:33 |
carloss | me too :) | 15:33 |
gouthamr | thanks fabiooliveira carloss | 15:33 |
carloss | anytime gouthamr | 15:34 |
gouthamr | need reviewers on a nondisruptive migration bugfix: | 15:34 |
gouthamr | #link https://review.opendev.org/c/openstack/manila/+/820071 (Force share server selection on nondisruptive migration) | 15:34 |
gouthamr | dviroel: can you help with this one in your copious free time? | 15:35 |
dviroel | gouthamr: sure, will do | 15:36 |
carloss | thanks much :) | 15:36 |
gouthamr | thanks dviroel :) | 15:36 |
carloss | also, thanks felipe_rodrigues and gouthamr for the reviews there | 15:36 |
gouthamr | #link https://review.opendev.org/c/openstack/python-manilaclient/+/788060 ([OSC] Add missing waiters) | 15:37 |
gouthamr | ^ felipe_rodrigues we have you on this one... could you please re-review? | 15:38 |
felipe_rodrigues | for sure! | 15:38 |
gouthamr | vkmc: this could use your review as well ^ | 15:38 |
gouthamr | kiwi_36: o/ i read your message on a related topic :) thanks for pushing to complete these 'waiters' -- sure maaritamm and i will appreciate the help! | 15:39 |
vkmc | ok :) | 15:39 |
kiwi_36 | gouthamr, you welcome | 15:39 |
gouthamr | thanks felipe_rodrigues vkmc | 15:40 |
felipe_rodrigues | yw | 15:41 |
gouthamr | vhari proposed spending some time doing a collab review - i think the time's ripe for one given we have this pad full at the moment | 15:41 |
vhari | gouthamr, ty | 15:41 |
vhari | lets look at this bug https://bugs.launchpad.net/manila/+bug/1920942 | 15:42 |
vhari | https://review.opendev.org/c/openstack/manila/+/820071 | 15:42 |
vhari | has +1 and +2s | 15:42 |
vhari | could use extra eye | 15:42 |
* gouthamr had thrown dviroel under this bus | 15:42 | |
carloss | :) | 15:42 |
vhari | :) | 15:42 |
* dviroel on it btw | 15:43 | |
vhari | gouthamr, hehe yeah we are in sync | 15:43 |
vhari | should we work on it here or move on to another one? | 15:43 |
gouthamr | yep - move on to the next one | 15:44 |
vhari | https://bugs.launchpad.net/manila/+bug/1924230 | 15:44 |
vhari | https://review.opendev.org/c/openstack/manila/+/821753 | 15:44 |
gouthamr | and https://review.opendev.org/c/openstack/manila/+/817112 | 15:45 |
gouthamr | this is a two part fix, and these stable backports could use some reviews.. although here both changes have merged in the main branch | 15:46 |
* gouthamr double checks https://review.opendev.org/c/openstack/manila/+/817112 --- seems like i've skipped stable/xena :( | 15:47 | |
dviroel | gouthamr: merged in Sep 16 | 15:48 |
gouthamr | ah, so part of stable/xena | 15:48 |
gouthamr | thanks :) panicked there for a moment | 15:49 |
dviroel | i'll review those replica update ones | 15:50 |
gouthamr | thanks dviroel | 15:50 |
gouthamr | thanks for bringing these up vhari | 15:52 |
gouthamr | we can move https://bugs.launchpad.net/manila/+bug/1924230 to fix released | 15:52 |
vhari | gouthamr, yw, ack | 15:53 |
gouthamr | did you want to chat about any other changes | 15:53 |
vhari | gouthamr, not atm .. almost out of time | 15:54 |
gouthamr | ack, do we have any bugs to triage? | 15:54 |
vhari | have one to share | 15:54 |
gouthamr | #topic Bug Triage (vhari) | 15:54 |
vhari | #link https://bugs.launchpad.net/manila/+bug/1957075 | 15:55 |
dviroel | oh | 15:55 |
vhari | we're looking for minor triage on this bug | 15:55 |
dviroel | felipe_rodrigues: what do you think? low? medium? | 15:57 |
gouthamr | does the ONTAP driver support "grow_shrink"? | 15:57 |
dviroel | felipe_rodrigues: will you take this one? | 15:57 |
gouthamr | how does that work with the "size" attribute from manila? | 15:57 |
dviroel | yeah, it is weird | 15:59 |
felipe_rodrigues | it sounds like a medium | 15:59 |
sfernand | agree | 15:59 |
felipe_rodrigues | the ontap driver lets the user grows more than manila size reports | 16:00 |
felipe_rodrigues | if the user is using the default behavior (grows more than the manila size), the promoted replica will lose it | 16:00 |
felipe_rodrigues | how could the user recover from it ? From manila it is not possible | 16:01 |
felipe_rodrigues | the user must go driectly to ontap | 16:01 |
dviroel | i understand the issue, but don't understand why they let it grow over maniza share size | 16:02 |
gouthamr | that'd be strange/unintended behavior felipe_rodrigues -- i recall thinking about autogrow when discussing "infinite" shares; but i thought the driver didn't "support" it | 16:02 |
felipe_rodrigues | thats the idea of flexvols, it is flexible volumes.. we could remove this behavior though | 16:03 |
felipe_rodrigues | I think that the default config | 16:03 |
felipe_rodrigues | we don't change it when creating | 16:04 |
gouthamr | alright, we can triage this further on #openstack-manila and offline; will you respond to the issue, felipe_rodrigues? | 16:04 |
felipe_rodrigues | Yeah, I'll talk to the reporter and see what it's the expected behavior for default volumes | 16:04 |
gouthamr | we can assign a priority, assignee and target if there's a confirmed bug here.. | 16:04 |
gouthamr | awesome thank you | 16:04 |
gouthamr | sorry we're over time.. | 16:04 |
gouthamr | thanks for bringing this up vhari | 16:05 |
gouthamr | and thank you all for attending! | 16:05 |
gouthamr | lets wrap up and chat on #openstack-manila | 16:05 |
gouthamr | #endmeeting | 16:05 |
opendevmeet | Meeting ended Thu Jan 13 16:05:34 2022 UTC. Information about MeetBot at http://wiki.debian.org/MeetBot . (v 0.1.4) | 16:05 |
opendevmeet | Minutes: https://meetings.opendev.org/meetings/manila/2022/manila.2022-01-13-15.01.html | 16:05 |
opendevmeet | Minutes (text): https://meetings.opendev.org/meetings/manila/2022/manila.2022-01-13-15.01.txt | 16:05 |
opendevmeet | Log: https://meetings.opendev.org/meetings/manila/2022/manila.2022-01-13-15.01.log.html | 16:05 |
dviroel | tks | 16:05 |
*** dviroel is now known as dviroel|lunch | 16:05 | |
felipe_rodrigues | tks | 16:07 |
*** dviroel|lunch is now known as dviroel | 17:00 | |
*** dviroel is now known as dviroel|out | 21:36 | |
*** dasm is now known as dasm|off | 21:45 |
Generated by irclog2html.py 2.17.3 by Marius Gedminas - find it at https://mg.pov.lt/irclog2html/!