*** dviroel|afk is now known as dviroel | 00:00 | |
*** dviroel is now known as dviroel|out | 00:24 | |
*** dviroel|out is now known as dviroel | 00:59 | |
*** dviroel is now known as dviroel|out | 01:24 | |
*** dviroel_ is now known as dviroel | 11:33 | |
*** dasm|off is now known as dasm | 13:15 | |
carloss | #startmeeting manila | 15:00 |
---|---|---|
opendevmeet | Meeting started Thu May 12 15:00:07 2022 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 |
gouthamr | o/ | 15:00 |
haixin | o/ | 15:00 |
carloss | courtesy ping: ganso vkmc dviroel felipe_rodrigues ecsantos vhari fabiooliveira | 15:00 |
vhari | hi | 15:00 |
fabiooliveira | o/ | 15:00 |
ckolod | o/ | 15:00 |
vkmc | o/ | 15:01 |
archanaserver | o/ | 15:01 |
felipe_rodrigues | hi | 15:01 |
carloss | Hello everyone! It's good to have you here :D | 15:01 |
carloss | Today's meeting agenda: | 15:01 |
carloss | #link https://wiki.openstack.org/wiki/Manila/Meetings#Next_meeting | 15:02 |
carloss | but before we start with the regular agenda items: | 15:02 |
carloss | I would like to thank you for last week's bugsquash! | 15:02 |
gouthamr | ++ | 15:02 |
carloss | we ended up with a couple of patches submitted as well as closure to some stale bugs | 15:02 |
dviroel | o/ | 15:02 |
carloss | o/ dviroel :) | 15:03 |
carloss | we still have a couple of changes open for reviews... all of the changes on this list are targeted to z-1 | 15:03 |
carloss | #link https://ethercalc.net/41lb87ry2kol | 15:03 |
* carloss moved the content to ethercalc.net just in case ethercalc.openstack.org went down in this week interval | 15:04 | |
carloss | turns out is still up but will be down in a couple of weeks :/ | 15:04 |
gouthamr | RIP ethercalc | 15:05 |
carloss | RIP ethercalc.openstack.org - you've were a good friend | 15:05 |
carloss | now getting back to the regular agenda topics, starting with: | 15:06 |
carloss | #topic Announcements | 15:06 |
gouthamr | unreliable friend | 15:06 |
carloss | hahaha, ++ gouthamr :D | 15:06 |
fabiooliveira | haha | 15:06 |
caiquemello[m] | 0/ | 15:06 |
carloss | Z1 Milestone is right around the corner! | 15:07 |
carloss | now for realsies | 15:07 |
* carloss isn't willing to scare anyone today | 15:07 | |
* fabiooliveira isn't he? | 15:07 | |
carloss | #link https://releases.openstack.org/zed/schedule.html | 15:07 |
carloss | and with Z-1, we get to the second event we have scheduled for the Zed cycle! | 15:08 |
carloss | our exciting hackaton is back next week! :D | 15:08 |
carloss | Manila Hackathon (May 16th - May 23rd) | 15:08 |
carloss | over PTG we discussed it would be nice to host a hackaton, but the idea was to make it earlier in the cycle | 15:09 |
carloss | the date we came to a conclusion was the Z-1 milestone week | 15:09 |
carloss | the primary focus of this hackaton will be functional testing with OSC | 15:09 |
carloss | maaritamm has groomed some tasks for us and they are available here: | 15:10 |
carloss | #link https://tree.taiga.io/project/gouthampacha-openstack-manila-osc-integration-with-python-manilaclient/kanban | 15:10 |
gouthamr | maaritamm++ | 15:11 |
carloss | the new column holds the new tasks - we can distribute the workload over the next week... the idea is to get the most unit tests we can :) | 15:11 |
carloss | it's a huge list as you see | 15:11 |
carloss | we have few examples that can be used as basis for implementation, but I'll be working on more before next week | 15:12 |
carloss | #link https://review.opendev.org/c/openstack/python-manilaclient/+/836834 | 15:12 |
carloss | maaritamm++ - thank you for helping to gather the list and create the tasks :) | 15:12 |
carloss | so in the date you can see a peculiarity: it starts in a Monday and ends in a Monday | 15:13 |
carloss | you may ask why | 15:13 |
carloss | this time the idea is to have a different format than we are used to | 15:13 |
carloss | one of the reasons is that most Red Hatters will be off next friday (20th) | 15:13 |
carloss | so we can wrap up on the following Monday | 15:14 |
carloss | the idea for dates I had is: | 15:14 |
carloss | A kick-off on Monday, 15UTC | 15:14 |
carloss | Use the next week's manila meeting slot for a mid-term checkpoint | 15:14 |
carloss | And 30 more minutes on the 23rd (15-15:30 UTC) just to wrap up | 15:14 |
carloss | how does that sound to you? | 15:15 |
felipe_rodrigues | +1 | 15:15 |
gouthamr | +1 lets try to do some reviews at the meeting | 15:15 |
fabiooliveira | sounds good | 15:15 |
gouthamr | midpoint meeting* | 15:15 |
caiquemello[m] | +1 | 15:16 |
carloss | ++ gouthamr :) | 15:16 |
carloss | great! so I'll send out an email to the mailing list today | 15:17 |
carloss | thank you for the input | 15:17 |
carloss | on to the next topic... | 15:18 |
carloss | oops | 15:18 |
carloss | before I move to the next topic :) | 15:18 |
vkmc | maaritamm++ :) always | 15:18 |
carloss | anyone else has an announcement today? | 15:18 |
* carloss taking silence as a no :) | 15:19 | |
carloss | now moving to the next topic | 15:20 |
carloss | #topic NDSU/Manila-UI internship wrap up | 15:20 |
carloss | o/ ckolod, shkoh1, sdailey, JonahFisher | 15:20 |
ckolod | o/ | 15:20 |
gouthamr | \o/ so we're at the close of the Capstone project internship at NDSU | 15:20 |
gouthamr | and the end of term for these guys, first off, congratulations! | 15:21 |
ckolod | thank you! it's been a long semester | 15:21 |
gouthamr | we had numerous meetings and pair programming sessions through the last three months, and we enjoyed the mentoring experience! | 15:21 |
gouthamr | there are a couple of WIP patches: https://review.opendev.org/c/openstack/manila-ui/+/840326 and https://review.opendev.org/c/openstack/manila-ui/+/839232 | 15:22 |
gouthamr | most of the work is complete, need to get zuul happy with some unit tests and such.. thanks for working on these ckolod shkoh1, sdailey, JonahFisher | 15:23 |
ckolod | I've yet to finish cleaning up 840326 due to finals, but I'll get to it | 15:25 |
gouthamr | ++ | 15:25 |
ckolod | thanks for mentoring us along the way | 15:25 |
vkmc | ++! | 15:25 |
gouthamr | you're welcome.. i learned many manila-ui things afresh too | 15:25 |
carloss | thank you ckolod, shkoh1, sdailey, JonahFisher for your efforts to implement this change and collaborating with us | 15:26 |
carloss | > you're welcome.. i learned many manila-ui things afresh too | 15:26 |
carloss | ++++ | 15:26 |
carloss | I hope this has been a positive experience for you | 15:26 |
carloss | I wish you all the best in your future and be sure that you'll always be welcome in this community! | 15:26 |
ckolod | on behalf of the team, thank you carloss gouthamr vkmc ashrodri | 15:27 |
gouthamr | indeed | 15:27 |
gouthamr | awesome, that's all i had about $topic carloss | 15:29 |
carloss | great, thank you! :) | 15:29 |
carloss | okay, going to the next topic... | 15:30 |
carloss | #topic Review Focus | 15:30 |
carloss | #link https://etherpad.opendev.org/p/manila-zorilla-review-focus (Review Focus Etherpad) | 15:30 |
carloss | I see a couple of changes in the list requiring some reviews | 15:31 |
carloss | but I would like to highlight some that have been up for a while today: | 15:31 |
carloss | Add config option reserved_share_extend_percentage | 15:31 |
carloss | #link https://review.opendev.org/c/openstack/manila/+/832843 | 15:31 |
carloss | it is waiting for reviews and we also mentioned this change in last week's bugsquash | 15:32 |
carloss | can I have some volunteers? | 15:32 |
carloss | :) | 15:32 |
felipe_rodrigues | o/ | 15:32 |
fabiooliveira | i can take a look also | 15:33 |
felipe_rodrigues | we also need some core reviewers there | 15:34 |
gouthamr | i can take a look | 15:34 |
haixin | i will too :) | 15:34 |
carloss | thank you felipe_rodrigues fabiooliveira gouthamr haixin :) | 15:35 |
carloss | next one I'd like to highlight is: | 15:35 |
carloss | [OSC] Implement Share Server Commands | 15:35 |
carloss | #link https://review.opendev.org/c/openstack/python-manilaclient/+/805107 | 15:35 |
carloss | this had a couple of reviews in the past | 15:35 |
carloss | thank you archanaserver for working on this change! | 15:35 |
carloss | I had some time to work on this change over this week and posted some updates, as I promised in the PTG :) | 15:36 |
carloss | addressed reviewers' comments and also fixed other few issues I found | 15:36 |
carloss | CI is green, I did lots of manual testing and didn't catch other issues - but it would be good to have some more eyes looking into this again | 15:37 |
* fabiooliveira 👀 | 15:37 | |
carloss | ecsantos[m] fabiooliveira vkmc can I have your eyes on this again? :) | 15:37 |
fabiooliveira | yes :) | 15:38 |
carloss | maaritamm/gouthamr too if possible :D | 15:38 |
vkmc | carloss, aye | 15:38 |
carloss | > maaritamm/gouthamr too if possible :D | 15:39 |
carloss | as we'll need 2 +2s on that and I became a co-author | 15:39 |
carloss | tyvm fabiooliveira vkmc | 15:40 |
fabiooliveira | my pleasure, caloss | 15:40 |
gouthamr | ack, will take a look at this too | 15:40 |
carloss | thank you gouthamr :) | 15:41 |
carloss | there are a couple more in the queue... I'll recheck them and I can bring up in the next meetings or ping in the change itself :) | 15:41 |
carloss | okay, so those were the changes I wanted to highlight today | 15:41 |
carloss | any other change you'd like to bring our attention to in this meeting? :) | 15:42 |
carloss | taking silence as a no | 15:43 |
carloss | next topic is: | 15:43 |
carloss | #topic Bug Triage (vhari) | 15:43 |
carloss | #link https://etherpad.openstack.org/p/manila-bug-triage-pad-new (Bug Triage etherpad) | 15:43 |
carloss | vhari, floor is yours | 15:43 |
vhari | ty carloss | 15:44 |
vhari | so we had 4 new bugs | 15:44 |
vhari | 1 was fixed, 2 are in progress :) | 15:44 |
vhari | carthaca++ gouthamr++ | 15:44 |
vhari | and there is one new bug to triage for today | 15:45 |
vhari | #link https://bugs.launchpad.net/manila/+bug/1972867 | 15:45 |
gouthamr | hmm, i do like the solution in https://review.opendev.org/c/openstack/neutron/+/770935 | 15:46 |
gouthamr | i thought the intent was different, with every new migration test, we're compounding the number of upgrades/downgrades we're doing | 15:47 |
gouthamr | so bumping timeouts has been a bandaid | 15:47 |
gouthamr | https://review.opendev.org/c/openstack/neutron/+/834746 is super cool too | 15:47 |
gouthamr | although i'll steal that idea for something else; i am not sure we need it for the db tests, uet | 15:48 |
gouthamr | yet* | 15:48 |
gouthamr | i think this is a medium... can be convinced if folks want to go to high because we're wasting CI resources | 15:49 |
carloss | yes - we haven't been hitting this issue much lately | 15:49 |
carloss | > can be convinced if folks want to go to high because we're wasting CI resources | 15:49 |
carloss | ++ | 15:49 |
gouthamr | perhaps one in ten test runs? | 15:49 |
vkmc | ++ | 15:50 |
gouthamr | which may be annoying since these tests aren't run in isolation... every job has to be re-run | 15:50 |
carloss | > perhaps one in ten test runs? | 15:50 |
carloss | yes, I would say so | 15:50 |
gouthamr | any volunteers that want to evaluate https://review.opendev.org/c/openstack/neutron/+/770935 with our db migration test code? | 15:51 |
felipe_rodrigues | it seems that the fix is given | 15:51 |
felipe_rodrigues | right ? | 15:51 |
carloss | in a sense yes felipe_rodrigues - we'd need to adapt to our scenario | 15:51 |
gouthamr | neutron and manila pioneered with these tests iirc, so the code's a lot similar | 15:52 |
gouthamr | https://opendev.org/openstack/manila/src/branch/master/manila/tests/db/migrations/alembic/test_migration.py#L118 | 15:52 |
felipe_rodrigues | I see | 15:52 |
felipe_rodrigues | I can try to fix it | 15:52 |
gouthamr | thanks felipe_rodrigues | 15:53 |
carloss | felipe_rodrigues++ thank you | 15:53 |
felipe_rodrigues | :) | 15:53 |
vhari | ty felipe_rodrigues++ | 15:53 |
vhari | next up | 15:53 |
gouthamr | also ralonsoh++ thanks for the pointers | 15:53 |
vhari | #link https://bugs.launchpad.net/manila/+bug/1970649 | 15:54 |
gouthamr | ah looks familiar.. | 15:54 |
vkmc | felipe_rodrigues++ yay | 15:54 |
gouthamr | the scheduler assumes netapp pools to always be thin provisioned... so it'll _always_ oversubscribe them (unless max_over_subscription_ratio = 1) | 15:55 |
gouthamr | but, the creation request might be a thick provisioning request | 15:56 |
gouthamr | so i think this confuses operators | 15:56 |
gouthamr | why oversubscribe, and then potentially end up failing in the share manager? | 15:56 |
felipe_rodrigues | yeah.. If I remember correctly, there is a customer complaining in the past about it.. And we provide a patch (at least from my mind) covering this issue | 15:57 |
gouthamr | oh? | 15:57 |
gouthamr | felipe_rodrigues: this bug comes from stable/train | 15:57 |
felipe_rodrigues | I need to recheck the patch that we have related to the thin provisioning and the customer email.. | 15:58 |
carloss | https://review.opendev.org/c/openstack/manila/+/811417 - wouldn't it be this one felipe_rodrigues | 15:58 |
felipe_rodrigues | you can add me | 15:58 |
gouthamr | would you/anyone from netapp take a look and lets discuss what a good solution should be here.. i think this is more than a UX concern.. | 15:58 |
gouthamr | the problem occurs when you have multiple pools | 15:58 |
felipe_rodrigues | Thanks carloss.. that one! | 15:59 |
gouthamr | and the request ends up in the "wrong" pool because of this unneeded oversubscription calculation --- the "wrong" pool has no space, where as there are other pools with capacity | 15:59 |
felipe_rodrigues | yeah it's a different problem here | 15:59 |
gouthamr | time check | 15:59 |
gouthamr | can we tag it medium (because its a netapp issue as of now - other backends don't support "True/False" in the same pool), and assign to you, felipe_rodrigues? | 16:00 |
carloss | ++ on medium | 16:00 |
gouthamr | --time-check-- | 16:00 |
felipe_rodrigues | ++ medium | 16:00 |
carloss | we can add a topic for it on next weekly meeting too | 16:01 |
carloss | if needed | 16:01 |
felipe_rodrigues | please, add me.. we'll discuss internally and provide a fix | 16:01 |
carloss | alright folks, thank you for participating on this meeting today! | 16:01 |
carloss | see you in #openstack-manila | 16:01 |
gouthamr | thanks carloss | 16:01 |
carloss | #endmeeting | 16:01 |
opendevmeet | Meeting ended Thu May 12 16:01:45 2022 UTC. Information about MeetBot at http://wiki.debian.org/MeetBot . (v 0.1.4) | 16:01 |
opendevmeet | Minutes: https://meetings.opendev.org/meetings/manila/2022/manila.2022-05-12-15.00.html | 16:01 |
opendevmeet | Minutes (text): https://meetings.opendev.org/meetings/manila/2022/manila.2022-05-12-15.00.txt | 16:01 |
opendevmeet | Log: https://meetings.opendev.org/meetings/manila/2022/manila.2022-05-12-15.00.log.html | 16:01 |
*** dviroel is now known as dviroel|afk | 21:09 | |
*** dasm is now known as dasm|off | 21:11 |
Generated by irclog2html.py 2.17.3 by Marius Gedminas - find it at https://mg.pov.lt/irclog2html/!