15:00:22 #startmeeting manila 15:00:25 Meeting started Thu Feb 13 15:00:22 2020 UTC and is due to finish in 60 minutes. The chair is gouthamr. Information about MeetBot at http://wiki.debian.org/MeetBot. 15:00:26 Useful Commands: #action #agreed #help #info #idea #link #topic #startvote. 15:00:27 o/ 15:00:28 The meeting name has been set to 'manila' 15:00:29 hi 15:00:31 hi 15:00:40 hi 15:00:50 hi 15:00:58 courtesy ping: xyang toabctl ganso vkmc amito jgrosso tbarron 15:01:11 o/ 15:01:14 \o/ 15:01:46 * gouthamr waits a minute for stragglers ... 15:01:50 hello 15:02:24 o/ 15:02:31 Howdie 15:02:58 Here's the agenda for today's meeting: 15:03:08 https://wiki.openstack.org/wiki/Manila/Meetings#Next_meeting 15:03:19 lets begin with.. 15:03:21 yo 15:03:23 #topic Announcements 15:03:45 First up, it's milestone-2 release week! 15:03:53 #link https://releases.openstack.org/ussuri/schedule.html (Ussuri schedule) 15:04:46 my quick math skills say we're over halfway through the Ussuri development cycle with this week 15:05:09 thanks for getting your specs in time, we had no new drivers to review this cycle 15:05:32 if you see any coming in, please gently tell them they will be deferred to the next release 15:05:53 we'll talk about specs that haven't merged in a few.. 15:06:31 We also made our initial release of python-manilaclient for this cycle 15:06:38 python-manilaclient 2.0.0 15:06:38 #link https://releases.openstack.org/ussuri/index.html#ussuri-python-manilaclient 15:07:03 the major version has been raised because we dropped support for python 2.7 15:07:33 i'll be tagging the initial release for manila-ui on similar lines... 15:07:55 the thing of note there is this review: 15:08:07 #link: https://review.opendev.org/#/c/706988/ (Raise python-manilaclient req to 1.29.0) 15:08:54 we'll be able to now raise the API version support in the UI beyond 2.33 (thanks maaritamm) 15:09:15 Next up, 15:09:24 (as is tradition) 15:09:37 Gate's currently broken 15:09:41 #link https://launchpad.net/bugs/1862833 15:09:42 Launchpad bug 1862833 in Manila "Sqlalchemy errors with subnet and group snapshot models" [High,In progress] - Assigned to Goutham Pacha Ravi (gouthamr) 15:10:11 This affects unit/functional testing on master for manila, and functional testing for the client 15:10:24 :( 15:10:49 we had some issues testing this patch yesterday; but those have been addressed thanks to changes to devstack and devstack-gate 15:11:29 so please review this patch; if CI agrees with it we'll merge it and then be able to get the other pending changes in 15:12:13 until then, do the gate a favor and don't 'recheck' patches - unless you want to depend on this fix itself 15:12:47 let's take discussion regarding this change to review 15:13:25 There's now a planning etherpad posted for the Victoria PTG 15:13:32 #link https://etherpad.openstack.org/p/vancouver-ptg-manila-planning (Victoria PTG Planning Etherpad) 15:13:59 #link http://lists.openstack.org/pipermail/openstack-discuss/2020-February/012445.html 15:14:31 Please bookmark it and add any topics that you anticipate to bring up 15:15:12 please also indicate if you're attending in person/remote 15:15:34 That's all i had for announcements this week 15:15:38 anyone else? 15:16:24 #topic Tracking our work 15:16:41 We've two unmerged specs that need your attention 15:16:49 #link https://review.opendev.org/#/c/700776/ (Update Create Share from Snapshot in Another Pool) 15:17:53 looks mostly ready, if previous reviewers are okay, we'll merge this 15:18:06 #link https://review.opendev.org/#/c/704793/ (Add lite spec to fix unlimited share replicas per share) 15:18:59 carthaca: ^^^ please review ^^^ 15:19:12 There's been a fresh patchset for this, so it needs a re-review. Please let carloss know if you disagree with anything, but i think this had had soak time 15:19:19 has had* 15:19:43 carloss: just double-checking, nice work 15:19:52 thanks, tbarron 15:19:52 +1 15:19:58 thanks gouthamr 15:20:04 carloss++ 15:20:33 ty dviroel 15:20:34 No reviews were added to this week's "Reviews needing attention" 15:21:08 are there any? i know it doesn't motivate reviewers when zuul's not voted 15:21:39 no rush but when gate clears, https://review.opendev.org/#/c/704246/ 15:22:12 cool, ty tbarron 15:22:28 #link https://review.opendev.org/#/c/704246 (Fix export location used for extend/shrink tests) 15:22:59 alright, moving on.. 15:23:05 #topic Bugs (vhari) 15:23:24 you're up, vhari 15:23:31 hi gouthamr .. have a couple to look at today 15:23:39 #link: https://etherpad.openstack.org/p/manila-bug-triage-pad-new (Bug Triage etherpad) 15:23:40 https://bugs.launchpad.net/manila/+bug/1635393 15:23:41 Launchpad bug 1635393 in Manila "Replace usage of 'retrying' with 'tenacity'" [Low,In progress] - Assigned to Boden R (boden) 15:24:58 what are next steps on this one.. 15:25:49 appears code review was abandoned 15:25:53 vhari: i'd be inclined to close it 15:26:15 or just leave it low prio 15:26:27 we can wishlist this one 15:26:28 retrying module is used in many other projects than maniia 15:26:34 manila 15:26:53 wishlist ok then? 15:26:58 ^ yes, and there are a lot that are using tenacity 15:26:59 cinder, ironic, kuryr 15:27:24 sure, i have no *objection* to converting to tenacity 15:27:45 cool, will do 15:27:48 next up 15:27:51 if someone wants to explore this alternative, manila-tempest-plugin seems like a nice place to start and then revive boden's patch 15:28:13 https://bugs.launchpad.net/manila/+bug/1836310 15:28:14 Launchpad bug 1836310 in Manila "Space savings from efficiency could be lost druing migrations" [Undecided,New] - Assigned to Carlos Eduardo (silvacarlose) 15:28:30 ^^ Assigned to silvacarlose - any updates to share? 15:28:35 that's me 15:28:49 o/ carloss 15:28:53 :) 15:28:56 I do not have any update yet 15:29:40 we might take a look again on this since we are working with volume moves 15:29:53 would encourage responding on the bug and letting the reporter know that you'll be looking at it, carloss/dviroel 15:30:10 gouthamr: ack 15:30:11 ack, gouthamr 15:30:20 kk, so that's a wrap for bugs today 15:30:27 oh nice, ty vhari 15:30:29 ty vhari 15:30:30 ty gouthamr 15:30:36 :) 15:30:48 #topic Open Discussion 15:31:36 I read an interesting ML post that i wanted to share with you all, regarding code reviews 15:31:44 #link https://lwn.net/ml/linux-fsdevel/20200202214620.GA20628@dread.disaster.area/ 15:32:22 this post by Dave Chinner neatly complements our own review ethos 15:33:27 oh, interesting... I've started reading it but haven't finished yet 15:33:35 promise it's worth a read, specially for how to address: "I don't know anything so my review is not going to be worth anything" - something i feel 90% of the time in open source 15:33:36 saw it on Twitter 15:33:56 haha, yeah - we relay it here for tbarron's benefi 15:34:02 t 15:34:18 cool, that' 15:34:29 is all i had 15:34:40 * gouthamr return key seems to have moved on my keyboard 15:34:57 :) 15:35:37 unless anyone else had anything more to discuss 15:36:01 going once... 15:36:31 okay, here's 24 minutes of your day returned. Thank you all for joining! 15:36:43 :D 15:36:46 ty gouthamr 15:36:48 #endmeeting