*** dviroel|rover is now known as dviroel|out | 00:14 | |
*** dviroel__ is now known as dviroel|rover | 11:15 | |
*** dasm|off is now known as dasm | 12:53 | |
carloss | #startmeeting manila | 15:00 |
---|---|---|
opendevmeet | Meeting 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 |
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 |
carloss | courtesy ping: vkmc dviroel felipe_rodrigues ecsantos vhari fabiooliveira gouthamr | 15:00 |
felipe_rodrigues | o/ | 15:01 |
caiquemello[m] | o/ | 15:01 |
ecsantos[m] | o/ | 15:01 |
vhari | o/ | 15:01 |
nahimsouza[m] | o/ | 15:01 |
sfernand | hi | 15:01 |
dviroel|rover | hey o/ | 15:01 |
carloss | hello everyone! :) | 15:03 |
carloss | smaller audience today, but let's get started | 15:04 |
carloss | it is great to have you here! | 15:04 |
carloss | I have been hitting some issues while trying to authenticate in the openstack wiki to change the content of the manila meeting agenda | 15:04 |
carloss | so I did not manage do update the next meeting part with today's topics | 15:04 |
carloss | but I will update that as soon as it is working again | 15:04 |
carloss | #topic Announcements | 15:05 |
carloss | Schedule and Deadlines: | 15:05 |
carloss | #link https://releases.openstack.org/zed/schedule.html (Zed Release Schedule) | 15:05 |
carloss | this week is z-2 week! | 15:05 |
carloss | if you have an open change/bug you would like to get merged at this milestone, there is still a bit of time | 15:06 |
carloss | also, from the agenda we can notice that our new driver deadline is in two weeks | 15:07 |
carloss | I saw felipe_rodrigues review on the new Macrosan driver | 15:07 |
carloss | #link https://review.opendev.org/c/openstack/manila/+/825429 (Add Macrosan Manila Driver) | 15:07 |
carloss | I was looking at it - upstream CI is now failing due to unit test and pep8 issues | 15:07 |
carloss | I have also checked the CI output, as mentioned during the last week's meeting | 15:08 |
carloss | the CI is not skipping all tests anymore | 15:08 |
carloss | but I saw a bunch of tests failing. I think the author still needs some time to investigate this further | 15:08 |
carloss | I'll ping in the change and ask if there are any questions to clarify | 15:08 |
carloss | that's all I had for announcements | 15:09 |
carloss | do you have an announcement you would like to share with us today? | 15:09 |
* carloss taking silence as a no | 15:11 | |
carloss | #topic Review Focus | 15:11 |
carloss | #link https://etherpad.opendev.org/p/manila-zorilla-review-focus (Review Focus Etherpad) | 15:11 |
carloss | we have had some movement on the changes of this etherpad | 15:11 |
carloss | I took the opportunity to add some updates on the status of the changes | 15:11 |
carloss | ashrodri's metadata API seems to have felipe_rodrigues +1, but is waiting for some time for more reviewers | 15:12 |
carloss | I'll get back to that possibly today :) | 15:12 |
carloss | also, there are some OSC changes looking good to go | 15: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 |
carloss | gouthamr dviroel|rover vkmc - I think some of you have already had a pass in one of this changes in the past | 15:14 |
carloss | could you please take a look at it when you have some time available? | 15:14 |
carloss | also, thank you haixin and maaritamm for the reviews on these changes :) | 15:16 |
carloss | is there any other change you would like to highlight in today's meeting? | 15:17 |
carloss | #topic Bug Triage | 15:19 |
carloss | #link https://etherpad.openstack.org/p/manila-bug-triage-pad-new (Bug Triage etherpad) | 15:19 |
carloss | vhari, floor is yours :) | 15:20 |
vhari | carloss, ty :) | 15:21 |
vhari | #link https://bugs.launchpad.net/manila/+bug/1981626 | 15:21 |
vhari | we are looking for minor triage | 15:22 |
carloss | oh, that's me :p | 15:22 |
carloss | so I found this bug while testing one of the OSC changes | 15:22 |
carloss | apparently it is there for a long time already | 15:22 |
vhari | haha :) | 15:22 |
carloss | I kind of explained it in the comments | 15:22 |
carloss | we 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 well | 15:23 |
carloss | the 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 fields | 15:23 |
carloss | but when I used the ID instead, it did not display | 15:24 |
carloss | so after me and gouthamr digging deeper, we found out that when we use the name, we will use a generic search | 15:25 |
carloss | and in this case the generic search will be a detailed list followed by a filter | 15:25 |
carloss | the idea is to have all methods having the same output | 15:26 |
carloss | #link https://bugs.launchpad.net/manila/+bug/1981626/comments/2 - here I showed a possible solution | 15:26 |
carloss | adding a new microversion and from the new version on, we'll be always adding the share networks in the view | 15:26 |
carloss | instead of adding it in the API | 15:26 |
carloss | and use the versioned methods as we use for all other operations | 15:27 |
carloss | that's an overview of this bug | 15:27 |
carloss | there is a missing information in the API here of course, but we did not have anyone complaining about this so far | 15:28 |
carloss | so I would be inclined to low/medium tops | 15:28 |
vhari | carloss++ good catch a dormant issue | 15:29 |
carloss | my pleasure vhari | 15:29 |
carloss | I think it is possibly a LHF - fix does not seem to complex | 15:29 |
carloss | thoughts? | 15:30 |
felipe_rodrigues | +1 LHF | 15:32 |
ashrodri | same | 15:33 |
carloss | anyone willing to take this bug? :) | 15:33 |
carloss | seems good for onboarding and understanding APIs and versioning | 15:34 |
caiquemello[m] | +1 i think you can assign luizsantos to this | 15:34 |
caiquemello[m] | It's going to be a good bug for him to start | 15:35 |
felipe_rodrigues | +1 | 15:35 |
carloss | great caiquemello[m] :D | 15:35 |
carloss | thanks - do you happen to have the lp id? or luizsantos[m] could assign https://bugs.launchpad.net/manila/+bug/1981626 to himself :) | 15:36 |
carloss | z-3 milestone? | 15:36 |
caiquemello[m] | I can assign him | 15:36 |
vhari | carloss, assigned :) | 15:37 |
carloss | vhari: thanks | 15:37 |
carloss | caiquemello[m] thanks | 15:37 |
vhari | carloss, all set with this bug ... and it's a wrap for bugs | 15:37 |
carloss | great, thank you very much vhari | 15:38 |
vhari | the floor it back to you | 15:38 |
carloss | thanks :D | 15:38 |
carloss | #topic Open Discussion | 15:38 |
carloss | it's been a while since we had some time for open discussion :p | 15:39 |
carloss | 🦗 | 15:41 |
carloss | I think we can get some minutes back to our day | 15:41 |
carloss | let's switch back to #openstack-manila | 15:41 |
carloss | thank you all for attending | 15:41 |
carloss | #endmeeting | 15:42 |
opendevmeet | Meeting ended Thu Jul 14 15:42:04 2022 UTC. Information about MeetBot at http://wiki.debian.org/MeetBot . (v 0.1.4) | 15:42 |
opendevmeet | Minutes: https://meetings.opendev.org/meetings/manila/2022/manila.2022-07-14-15.00.html | 15:42 |
opendevmeet | Minutes (text): https://meetings.opendev.org/meetings/manila/2022/manila.2022-07-14-15.00.txt | 15:42 |
opendevmeet | Log: https://meetings.opendev.org/meetings/manila/2022/manila.2022-07-14-15.00.log.html | 15:42 |
*** dviroel|rover is now known as dviroel|rover|brb | 16:35 | |
sfernand | whoami-rajat: thanks! | 16:52 |
*** dviroel|rover|brb is now known as dviroel|rover | 17:35 | |
*** dasm is now known as Guest5013 | 17:59 | |
*** Guest5013 is now known as dasm | 18:01 | |
*** dasm is now known as dasm|off | 21:23 | |
*** dviroel|rover is now known as dviroel|rover|afk | 21:59 |
Generated by irclog2html.py 2.17.3 by Marius Gedminas - find it at https://mg.pov.lt/irclog2html/!