Thursday, 2022-07-14

*** dviroel|rover is now known as dviroel|out00:14
*** dviroel__ is now known as dviroel|rover11:15
*** dasm|off is now known as dasm12:53
carloss#startmeeting manila15:00
opendevmeetMeeting started Thu Jul 14 15:00:35 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
haixino/15:00
carlosscourtesy ping: vkmc dviroel felipe_rodrigues ecsantos vhari fabiooliveira gouthamr15:00
felipe_rodrigueso/15:01
caiquemello[m]o/15:01
ecsantos[m]o/15:01
vhario/15:01
nahimsouza[m]o/15:01
sfernandhi15:01
dviroel|roverhey o/15:01
carlosshello everyone! :)15:03
carlosssmaller audience today, but let's get started15:04
carlossit is great to have you here!15:04
carlossI have been hitting some issues while trying to authenticate in the openstack wiki to change the content of the manila meeting agenda15:04
carlossso I did not manage do update the next meeting part with today's topics15:04
carlossbut I will update that as soon as it is working again15:04
carloss#topic Announcements15:05
carlossSchedule and Deadlines:15:05
carloss#link https://releases.openstack.org/zed/schedule.html (Zed Release Schedule)15:05
carlossthis week is z-2 week!15:05
carlossif you have an open change/bug you would like to get merged at this milestone, there is still a bit of time15:06
carlossalso, from the agenda we can notice that our new driver deadline is in two weeks15:07
carlossI saw felipe_rodrigues review on the new Macrosan driver15:07
carloss#link https://review.opendev.org/c/openstack/manila/+/825429 (Add Macrosan Manila Driver)15:07
carlossI was looking at it - upstream CI is now failing due to unit test and pep8 issues15:07
carlossI have also checked the CI output, as mentioned during the last week's meeting15:08
carlossthe CI is not skipping all tests anymore15:08
carlossbut I saw a bunch of tests failing. I think the author still needs some time to investigate this further15:08
carlossI'll ping in the change and ask if there are any questions to clarify15:08
carlossthat's all I had for announcements15:09
carlossdo you have an announcement you would like to share with us today?15:09
* carloss taking silence as a no15:11
carloss#topic Review Focus15:11
carloss#link https://etherpad.opendev.org/p/manila-zorilla-review-focus (Review Focus Etherpad)15:11
carlosswe have had some movement on the changes of this etherpad15:11
carlossI took the opportunity to add some updates on the status of the changes15:11
carlossashrodri's metadata API seems to have felipe_rodrigues +1, but is waiting for some time for more reviewers15:12
carlossI'll get back to that possibly today :)15:12
carlossalso, there are some OSC changes looking good to go15:13
carloss#link https://review.opendev.org/c/openstack/python-manilaclient/+/816401 (OSC security services commands)15:13
carloss#link https://review.opendev.org/c/openstack/python-manilaclient/+/805107 (OSC share server commands)15:14
carlossgouthamr dviroel|rover vkmc - I think some of you have already had a pass in one of this changes in the past15:14
carlosscould you please take a look at it when you have some time available?15:14
carlossalso, thank you haixin and maaritamm for the reviews on these changes :)15:16
carlossis there any other change you would like to highlight in today's meeting?15:17
carloss#topic Bug Triage15:19
carloss#link https://etherpad.openstack.org/p/manila-bug-triage-pad-new (Bug Triage etherpad)15:19
carlossvhari, floor is yours :)15:20
vharicarloss, ty :)15:21
vhari#link https://bugs.launchpad.net/manila/+bug/1981626 15:21
vhariwe are looking for minor triage 15:22
carlossoh, that's me :p15:22
carlossso I found this bug while testing one of the OSC changes15:22
carlossapparently it is there for a long time already15:22
vharihaha :)15:22
carlossI kind of explained it in the comments15:22
carlosswe are kind of only displaying the share networks attached to a security service in the detailed view, but we should be displaying it on the show as well15:23
carlossthe bug for me was weird because while testing, I realized that when I showed security services using the name, share networks was one of the fields15:23
carlossbut when I used the ID instead, it did not display15:24
carlossso after me and gouthamr digging deeper, we found out that when we use the name, we will use a generic search15:25
carlossand in this case the generic search will be a detailed list followed by a filter15:25
carlossthe idea is to have all methods having the same output15:26
carloss#link https://bugs.launchpad.net/manila/+bug/1981626/comments/2 - here I showed a possible solution15:26
carlossadding a new microversion and from the new version on, we'll be always adding the share networks in the view15:26
carlossinstead of adding it in the API15:26
carlossand use the versioned methods as we use for all other operations15:27
carlossthat's an overview of this bug15:27
carlossthere is a missing information in the API here of course, but we did not have anyone complaining about this so far15:28
carlossso I would be inclined to low/medium tops15:28
vharicarloss++  good catch a dormant issue 15:29
carlossmy pleasure vhari 15:29
carlossI think it is possibly a LHF - fix does not seem to complex15:29
carlossthoughts?15:30
felipe_rodrigues+1 LHF15:32
ashrodrisame15:33
carlossanyone willing to take this bug? :)15:33
carlossseems good for onboarding and understanding APIs and versioning15:34
caiquemello[m]+1 i think you can assign luizsantos to this15:34
caiquemello[m]It's going to be a good bug for him to start15:35
felipe_rodrigues+115:35
carlossgreat caiquemello[m] :D15:35
carlossthanks - do you happen to have the lp id? or luizsantos[m] could assign https://bugs.launchpad.net/manila/+bug/1981626 to himself :)15:36
carlossz-3 milestone?15:36
caiquemello[m]I can assign him15:36
vharicarloss, assigned :)15:37
carlossvhari: thanks15:37
carlosscaiquemello[m] thanks15:37
vharicarloss, all set with this bug ... and it's a wrap for bugs15:37
carlossgreat, thank you very much vhari 15:38
vharithe floor it back to you 15:38
carlossthanks :D15:38
carloss#topic Open Discussion15:38
carlossit's been a while since we had some time for open discussion :p15:39
carloss🦗15:41
carlossI think we can get some minutes back to our day15:41
carlosslet's switch back to #openstack-manila15:41
carlossthank you all for attending15:41
carloss#endmeeting15:42
opendevmeetMeeting ended Thu Jul 14 15:42:04 2022 UTC.  Information about MeetBot at http://wiki.debian.org/MeetBot . (v 0.1.4)15:42
opendevmeetMinutes:        https://meetings.opendev.org/meetings/manila/2022/manila.2022-07-14-15.00.html15:42
opendevmeetMinutes (text): https://meetings.opendev.org/meetings/manila/2022/manila.2022-07-14-15.00.txt15:42
opendevmeetLog:            https://meetings.opendev.org/meetings/manila/2022/manila.2022-07-14-15.00.log.html15:42
*** dviroel|rover is now known as dviroel|rover|brb16:35
sfernandwhoami-rajat: thanks! 16:52
*** dviroel|rover|brb is now known as dviroel|rover17:35
*** dasm is now known as Guest501317:59
*** Guest5013 is now known as dasm18:01
*** dasm is now known as dasm|off21:23
*** dviroel|rover is now known as dviroel|rover|afk21:59

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