Thursday, 2022-05-12

*** dviroel|afk is now known as dviroel00:00
*** dviroel is now known as dviroel|out00:24
*** dviroel|out is now known as dviroel00:59
*** dviroel is now known as dviroel|out01:24
*** dviroel_ is now known as dviroel11:33
*** dasm|off is now known as dasm13:15
carloss#startmeeting manila15:00
opendevmeetMeeting 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
opendevmeetUseful Commands: #action #agreed #help #info #idea #link #topic #startvote.15:00
opendevmeetThe meeting name has been set to 'manila'15:00
gouthamro/15:00
haixino/15:00
carlosscourtesy ping: ganso vkmc dviroel felipe_rodrigues ecsantos vhari fabiooliveira15:00
vharihi15:00
fabiooliveirao/15:00
ckolodo/15:00
vkmco/15:01
archanaservero/15:01
felipe_rodrigueshi15:01
carlossHello everyone! It's good to have you here :D15:01
carlossToday's meeting agenda:15:01
carloss#link https://wiki.openstack.org/wiki/Manila/Meetings#Next_meeting15:02
carlossbut before we start with the regular agenda items:15:02
carlossI would like to thank you for last week's bugsquash!15:02
gouthamr++ 15:02
carlosswe ended up with a couple of patches submitted as well as closure to some stale bugs15:02
dviroelo/15:02
carlosso/ dviroel :)15:03
carlosswe still have a couple of changes open for reviews... all of the changes on this list are targeted to z-115:03
carloss#link https://ethercalc.net/41lb87ry2kol15:03
* carloss moved the content to ethercalc.net just in case ethercalc.openstack.org went down in this week interval15:04
carlossturns out is still up but will be down in a couple of weeks :/15:04
gouthamrRIP ethercalc15:05
carlossRIP ethercalc.openstack.org - you've were a good friend15:05
carlossnow getting back to the regular agenda topics, starting with:15:06
carloss#topic Announcements15:06
gouthamrunreliable friend 15:06
carlosshahaha, ++ gouthamr :D15:06
fabiooliveirahaha15:06
caiquemello[m]0/15:06
carlossZ1 Milestone is right around the corner!15:07
carlossnow for realsies15:07
* carloss isn't willing to scare anyone today15:07
* fabiooliveira isn't he?15:07
carloss#link https://releases.openstack.org/zed/schedule.html15:07
carlossand with Z-1, we get to the second event we have scheduled for the Zed cycle!15:08
carlossour exciting hackaton is back next week! :D15:08
carlossManila Hackathon (May 16th - May 23rd)15:08
carlossover PTG we discussed it would be nice to host a hackaton, but the idea was to make it earlier in the cycle15:09
carlossthe date we came to a conclusion was the Z-1 milestone week15:09
carlossthe primary focus of this hackaton will be functional testing with OSC15:09
carlossmaaritamm 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/kanban15:10
gouthamrmaaritamm++ 15:11
carlossthe 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
carlossit's a huge list as you see15:11
carlosswe have few examples that can be used as basis for implementation, but I'll be working on more before next week15:12
carloss#link https://review.opendev.org/c/openstack/python-manilaclient/+/83683415:12
carlossmaaritamm++ - thank you for helping to gather the list and create the tasks :)15:12
carlossso in the date you can see a peculiarity: it starts in a Monday and ends in a Monday15:13
carlossyou may ask why15:13
carlossthis time the idea is to have a different format than we are used to15:13
carlossone of the reasons is that most Red Hatters will be off next friday (20th)15:13
carlossso we can wrap up on the following Monday15:14
carlossthe idea for dates I had is:15:14
carlossA kick-off on Monday, 15UTC15:14
carlossUse the next week's manila meeting slot for a mid-term checkpoint15:14
carlossAnd 30 more minutes on the 23rd (15-15:30 UTC) just to wrap up15:14
carlosshow does that sound to you?15:15
felipe_rodrigues+115:15
gouthamr+1 lets try to do some reviews at the meeting15:15
fabiooliveirasounds good15:15
gouthamrmidpoint meeting*15:15
caiquemello[m]+115:16
carloss++ gouthamr :)15:16
carlossgreat! so I'll send out an email to the mailing list today15:17
carlossthank you for the input15:17
carlosson to the next topic...15:18
carlossoops15:18
carlossbefore I move to the next topic :)15:18
vkmcmaaritamm++ :) always 15:18
carlossanyone else has an announcement today?15:18
* carloss taking silence as a no :)15:19
carlossnow moving to the next topic15:20
carloss#topic NDSU/Manila-UI internship wrap up15:20
carlosso/ ckolod, shkoh1, sdailey, JonahFisher15:20
ckolodo/15:20
gouthamr\o/ so we're at the close of the Capstone project internship at NDSU 15:20
gouthamrand the end of term for these guys, first off, congratulations!15:21
ckolodthank you! it's been a long semester15:21
gouthamrwe had numerous meetings and pair programming sessions through the last three months, and we enjoyed the mentoring experience!15:21
gouthamrthere 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
gouthamrmost of the work is complete, need to get zuul happy with some unit tests and such.. thanks for working on these ckolod shkoh1, sdailey, JonahFisher15:23
ckolodI've yet to finish cleaning up 840326 due to finals, but I'll get to it15:25
gouthamr++ 15:25
ckolodthanks for mentoring us along the way15:25
vkmc++! 15:25
gouthamryou're welcome.. i learned many manila-ui things afresh too15:25
carlossthank you ckolod, shkoh1, sdailey, JonahFisher for your efforts to implement this change and collaborating with us15:26
carloss> you're welcome.. i learned many manila-ui things afresh too15:26
carloss++++15:26
carlossI hope this has been a positive experience for you15:26
carlossI wish you all the best in your future and be sure that you'll always be welcome in this community!15:26
ckolodon behalf of the team, thank you carloss gouthamr vkmc ashrodri15:27
gouthamrindeed 15:27
gouthamrawesome, that's all i had about $topic carloss 15:29
carlossgreat, thank you! :)15:29
carlossokay, going to the next topic...15:30
carloss#topic Review Focus15:30
carloss#link https://etherpad.opendev.org/p/manila-zorilla-review-focus (Review Focus Etherpad)15:30
carlossI see a couple of changes in the list requiring some reviews15:31
carlossbut I would like to highlight some that have been up for a while today:15:31
carlossAdd config option reserved_share_extend_percentage15:31
carloss#link https://review.opendev.org/c/openstack/manila/+/83284315:31
carlossit is waiting for reviews and we also mentioned this change in last week's bugsquash15:32
carlosscan I have some volunteers?15:32
carloss:)15:32
felipe_rodrigueso/15:32
fabiooliveirai can take a look also15:33
felipe_rodrigueswe also need some core reviewers there15:34
gouthamri can take a look15:34
haixini will too :)15:34
carlossthank you felipe_rodrigues fabiooliveira gouthamr haixin :)15:35
carlossnext one I'd like to highlight is:15:35
carloss[OSC] Implement Share Server Commands15:35
carloss#link https://review.opendev.org/c/openstack/python-manilaclient/+/80510715:35
carlossthis had a couple of reviews in the past15:35
carlossthank you archanaserver for working on this change!15:35
carlossI had some time to work on this change over this week and posted some updates, as I promised in the PTG :)15:36
carlossaddressed reviewers' comments and also fixed other few issues I found15:36
carlossCI 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 again15:37
* fabiooliveira 👀15:37
carlossecsantos[m] fabiooliveira vkmc can I have your eyes on this again? :)15:37
fabiooliveirayes :)15:38
carlossmaaritamm/gouthamr too if possible :D15:38
vkmccarloss, aye15:38
carloss> maaritamm/gouthamr too if possible :D15:39
carlossas we'll need 2 +2s on that and I became a co-author15:39
carlosstyvm fabiooliveira vkmc 15:40
fabiooliveiramy pleasure, caloss15:40
gouthamrack, will take a look at this too15:40
carlossthank you gouthamr :)15:41
carlossthere 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
carlossokay, so those were the changes I wanted to highlight today15:41
carlossany other change you'd like to bring our attention to in this meeting? :)15:42
carlosstaking silence as a no15:43
carlossnext 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
carlossvhari, floor is yours15:43
vharity carloss 15:44
vhariso we had 4 new bugs 15:44
vhari1 was fixed, 2 are in progress :)15:44
vharicarthaca++ gouthamr++ 15:44
vhariand there is one new bug to triage for today 15:45
vhari#link https://bugs.launchpad.net/manila/+bug/197286715:45
gouthamrhmm, i do like the solution in https://review.opendev.org/c/openstack/neutron/+/770935 15:46
gouthamri thought the intent was different, with every new migration test, we're compounding the number of upgrades/downgrades we're doing15:47
gouthamrso bumping timeouts has been a bandaid15:47
gouthamrhttps://review.opendev.org/c/openstack/neutron/+/834746 is super cool too15:47
gouthamralthough i'll steal that idea for something else; i am not sure we need it for the db tests, uet15:48
gouthamryet*15:48
gouthamri think this is a medium... can be convinced if folks want to go to high because we're wasting CI resources15:49
carlossyes - we haven't been hitting this issue much lately15:49
carloss> can be convinced if folks want to go to high because we're wasting CI resources15:49
carloss++15:49
gouthamrperhaps one in ten test runs? 15:49
vkmc++15:50
gouthamrwhich may be annoying since these tests aren't run in isolation... every job has to be re-run15:50
carloss> perhaps one in ten test runs? 15:50
carlossyes, I would say so15:50
gouthamrany volunteers that want to evaluate https://review.opendev.org/c/openstack/neutron/+/770935 with our db migration test code?15:51
felipe_rodriguesit seems that the fix is given 15:51
felipe_rodriguesright ?15:51
carlossin a sense yes felipe_rodrigues - we'd need to adapt to our scenario15:51
gouthamrneutron and manila pioneered with these tests iirc, so the code's a lot similar15:52
gouthamrhttps://opendev.org/openstack/manila/src/branch/master/manila/tests/db/migrations/alembic/test_migration.py#L11815:52
felipe_rodriguesI see15:52
felipe_rodriguesI can try to fix it15:52
gouthamrthanks felipe_rodrigues 15:53
carlossfelipe_rodrigues++ thank you15:53
felipe_rodrigues:)15:53
vharity felipe_rodrigues++ 15:53
vharinext up 15:53
gouthamralso ralonsoh++ thanks for the pointers15:53
vhari#link https://bugs.launchpad.net/manila/+bug/197064915:54
gouthamrah looks familiar.. 15:54
vkmcfelipe_rodrigues++ yay15:54
gouthamrthe scheduler assumes netapp pools to always be thin provisioned... so it'll _always_ oversubscribe them (unless max_over_subscription_ratio = 1) 15:55
gouthamrbut, the creation request might be a thick provisioning request15:56
gouthamrso i think this confuses operators15:56
gouthamrwhy oversubscribe, and then potentially end up failing in the share manager?15:56
felipe_rodriguesyeah.. 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 issue15:57
gouthamroh?15:57
gouthamrfelipe_rodrigues: this bug comes from stable/train15:57
felipe_rodriguesI need to recheck the patch that we have related to the thin provisioning and the customer email.. 15:58
carlosshttps://review.opendev.org/c/openstack/manila/+/811417 - wouldn't it be this one felipe_rodrigues 15:58
felipe_rodriguesyou can add me15:58
gouthamrwould 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
gouthamrthe problem occurs when you have multiple pools15:58
felipe_rodriguesThanks carloss.. that one!15:59
gouthamrand 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 capacity15:59
felipe_rodriguesyeah it's a different problem here15:59
gouthamrtime check15:59
gouthamrcan 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++ medium16:00
carlosswe can add a topic for it on next weekly meeting too16:01
carlossif needed16:01
felipe_rodriguesplease, add me.. we'll discuss internally and provide a fix16:01
carlossalright folks, thank you for participating on this meeting today!16:01
carlosssee you in #openstack-manila16:01
gouthamrthanks carloss 16:01
carloss#endmeeting16:01
opendevmeetMeeting ended Thu May 12 16:01:45 2022 UTC.  Information about MeetBot at http://wiki.debian.org/MeetBot . (v 0.1.4)16:01
opendevmeetMinutes:        https://meetings.opendev.org/meetings/manila/2022/manila.2022-05-12-15.00.html16:01
opendevmeetMinutes (text): https://meetings.opendev.org/meetings/manila/2022/manila.2022-05-12-15.00.txt16:01
opendevmeetLog:            https://meetings.opendev.org/meetings/manila/2022/manila.2022-05-12-15.00.log.html16:01
*** dviroel is now known as dviroel|afk21:09
*** dasm is now known as dasm|off21:11

Generated by irclog2html.py 2.17.3 by Marius Gedminas - find it at https://mg.pov.lt/irclog2html/!