14:59:39 <carloss> #startmeeting manila 14:59:39 <opendevmeet> Meeting started Thu Aug 29 14:59:39 2024 UTC and is due to finish in 60 minutes. The chair is carloss. Information about MeetBot at http://wiki.debian.org/MeetBot. 14:59:39 <opendevmeet> Useful Commands: #action #agreed #help #info #idea #link #topic #startvote. 14:59:39 <opendevmeet> The meeting name has been set to 'manila' 14:59:56 <vhari> o/ 14:59:56 <carthaca> hi 15:00:21 <haixin> o/ 15:00:31 <gouthamr> o/ 15:00:35 <tspyderboy> hello all 15:00:35 <ccokeke[m]> hello 15:00:52 * carloss started the meeting 1 minute earlier than I should've, alarm set to 1 minute before the meeting tricked me, sorry :p 15:01:11 <carloss> courtesy ping: dviroel msaravan pulluri ashrodri 15:01:19 <ashrodri> o/ 15:01:36 <chuanm> hi 15:01:47 <gireesh> hi 15:02:08 <jayaanand> hi 15:03:41 <carloss> hello everyone 15:03:50 <carloss> and welcome 15:04:09 <kpdev> hi 15:04:17 <carloss> the meeting agenda for today: 15:04:27 <carloss> #link https://wiki.openstack.org/wiki/Manila/Meetings#Next_meeting 15:04:48 <carloss> a lot to cover today, so let's get started 15:04:55 <carloss> #topic Announcements 15:05:03 <carloss> Schedule and Deadlines 15:05:13 <carloss> #link https://releases.openstack.org/dalmatian/schedule.html (Dalmatian release schedule) 15:05:25 <carloss> so as you are very likely aware, today is the feature freeze day 15:05:33 <carloss> day/week 15:05:50 <carloss> we'll go into the features we are targeting in detail in a bit 15:06:24 <carloss> and that reminds me that our bugsquash is in two weeks :) 15:06:43 <carloss> I should send the email to openstack-discuss soon 15:07:10 <msaravan> Hi everyone..sorry for joining late 15:07:15 <carloss> o/ 15:07:34 <carloss> brief announcements session, as we will have a lot to talk about in this meeting 15:07:44 <carloss> do you have any other announcements you'd like to share with us today? 15:09:36 * carloss taking silence as no 15:09:58 <carloss> #topic Outreachy internship wrap up 15:10:06 <carloss> last week we briefly talked about this too 15:10:16 <carloss> but today we have our interns joining us 15:10:24 <carloss> gouthamr: would you like to cover this topic? :) 15:10:50 <gouthamr> sure; thanks carloss 15:11:08 <gouthamr> time flies 15:11:38 <gouthamr> this is one of those really nice times when we got an opportunity to host two Outreachy interns from one cohort 15:12:15 <gouthamr> ccokeke[m] and tspyderboy have both successfully completed their Outreachy internship on the OpenStack Manila UI project 15:12:47 <gouthamr> wanted to take a moment to congratulate them for the hard work they've put in 15:13:32 <gouthamr> and thank all of you for reviewing their patches, speaking with them 1x1, answering their questions and making them feel welcome! 15:14:49 <haixin> congratulate ! 15:14:57 <carloss> ccokeke[m] tspyderboy: thank you very much for your dedication and hard work. Your work is very valuable to this community 15:15:05 <carloss> and congrats on the successful internship 15:16:03 <gouthamr> #link https://review.opendev.org/q/owner:tspyderboy@gmail.com+-status:abandoned (tspyderboy's patches) 15:16:15 <ccokeke[m]> Thanks carloss: gouthamr: ashrodri: and every member of this great community. 15:16:27 <gouthamr> #link https://review.opendev.org/q/owner:ccokeke+-status:abandoned (ccokeke[m]'s patches) 15:16:31 <ashrodri> we thouroughly enjoyed mentoring you, and i hope to continue to see you in the channel :) thanks guys 15:17:26 <ccokeke[m]> ashrodri: Sure 15:17:33 <gouthamr> ++ i do hope both of you remain in touch, and continue to work on OpenStack! :) 15:17:49 <tspyderboy> It was a great learning experience working as an intern, hope I continue my pace with more contributions ahead. Openstack is the warmest community :) 15:18:10 <gouthamr> at the same time, if someone is looking to hire 'em, please reach out :) 15:19:31 <gouthamr> great; thanks also for keeping up with reviews, and seeing your patches to closure.. 15:19:40 <gouthamr> that's all we have for $topic 15:20:10 <carloss> gouthamr: thanks for bringing this up 15:21:23 <carloss> and tspyderboy ccokeke[m] thanks for joining us :D 15:21:28 <carloss> #topic Review focus (Feature freeze edition) 15:21:35 <carloss> #link https://etherpad.opendev.org/p/manila-dalmatian-review-focus (Dalmatian review focus etherpad) 15:21:52 <carloss> I've been keeping the review focus etherpad updated 15:22:12 <carloss> so let's go over the features we are targeting for feature freeze (today/tomorrow tops) 15:22:44 <carloss> first, let's start with what we already managed to merge: 15:23:13 <carloss> - Pass metadata to backend drivers 15:23:13 <carloss> - Fix share revert to snapshot for specific scenario 15:23:23 <carloss> thanks everyone for keeping up with the reviews on the changes 15:23:55 <carloss> now, let's go over what's still open 15:24:06 <carloss> #link https://review.opendev.org/c/openstack/manila/+/921313 (Update share/snapshot instance deferred deletion) 15:24:14 <gouthamr> ^ looking at this 15:24:35 <carloss> it is a big change, but I was considering we can also look at this as a bugfix 15:24:38 <carloss> I might be wrong there 15:24:42 <carloss> gouthamr: thank you :) 15:24:49 <gouthamr> it is a bug fix 15:25:21 <gouthamr> i'd assume we'll want to backport this; and we've had pretty good review attention here.. 15:25:29 <carloss> ++ 15:25:35 <carloss> that would make the deadline for this being rc1 15:25:43 <carloss> and we can look at backporting this change 15:25:59 <gouthamr> kpdev: no issues if this doesn't merge today; we can merge this as soon as possible, but, no need to rush it in 15:26:10 <carloss> gouthamr++ 15:26:12 <carloss> so I'll move it down the list, to the things that we are targeting for rc1 15:26:59 <carloss> #link https://review.opendev.org/q/topic:%22bp/metadata-for-share-resources%22+status:open+export (Export location metadata) 15:27:20 <carloss> ashrodri: thanks for updating the patches and addressing the reviewers' comments 15:27:49 <ashrodri> yep, working on addressing gouthams review on the api patch, and adding testing in the tempest one 15:28:01 <carloss> ack, thank you 15:28:07 <carloss> we must release the client this week, so I think the client for this won't make it 15:28:39 <carloss> but we can still manage to land the API change in case everything is sorted out by tomorrow 15:29:43 <gouthamr> +1 15:30:12 <ashrodri> sounds good 15:30:18 <carloss> I would say this change is not that far away from being merged. We could also consider a feature freeze exception for it, in case we need, but let's try to avoid that :) 15:30:28 <ashrodri> agreed 15:30:39 <carloss> thank you! 15:30:54 <carloss> next up: 15:31:04 <carloss> #link https://review.opendev.org/c/openstack/manila/+/924773 (Manage/unmanage for CephFS drivers) 15:31:33 <carloss> I am pursuing this change. Managed to enable CI testing and tests are passing 15:31:52 <carloss> comments have been addressed yesterday, and I also did some tests to ensure everything was working just fine 15:32:04 <carloss> I fixed another comment left by haixin earlier today 15:32:22 <carloss> more reviews would be welcome, and I also think this change is feasible for today/tomorrow 15:32:36 <gouthamr> +1 will look 15:32:45 <carloss> gouthamr haixin: thank you very much for the reviews 15:33:45 <carloss> kpdev thanks too, just saw your vote 15:34:24 <carloss> #link https://review.opendev.org/q/topic:%22bp/ensure-shares-api%22 (Ensure shares API) 15:34:43 <carloss> I am also proposing this change but we will also miss the client release for this 15:34:55 <carloss> change took too long to be updated and that's on me :) 15:35:13 <carloss> honestly, it's not that extensive in number of lines 15:35:23 <carloss> carthaca: thank you for the review on the change :) 15:36:11 <carloss> More feedback would be welcome. I see some reviewers in the focus etherpad: chuanm, ashrodri, haixin, gouthamr: could you please take a look? 15:36:39 <chuanm> sure, I will take a look 15:36:43 <haixin> sure 15:36:45 <carloss> tyvm 15:37:45 <carloss> I also would like to avoid feature freeze exceptions for it, so if there are many major changes to do, we can defer it 15:37:56 <gouthamr> ack 15:38:18 <carloss> thanks folks 15:38:33 <carloss> #link https://review.opendev.org/c/openstack/manila/+/926049 (NetApp: SnapLock support) 15:38:56 <carloss> gireesh: thanks for updating the change according to the feedback and making it a driver-only feature 15:39:52 <carloss> I see some reviewers on the list (myself included): msaravan, jayaanand, kpdev, gouthamr, vhari 15:39:53 <gireesh> thanks carloss, can you and Goutham can also review the patch 15:39:56 <carloss> can we have some eyes? :) 15:40:10 <carloss> gireesh: yep, intend to post a review today 15:40:24 <gireesh> ty 15:40:28 <jayaanand> i will take a look at it 15:40:37 <carloss> ty jayaanand 15:41:08 <carloss> next up... 15:41:09 <carloss> #link https://review.opendev.org/c/openstack/manila/+/925079 ([NetApp] Custom storage efficiency policy support for Manila share) 15:41:32 <carloss> there are some comments in the change 15:42:02 <carloss> jayaanand: should we continue targeting this to this release? 15:42:05 <jayaanand> thank you! gireesh and carloos for review. i didn't work on this patch for couple of weeks due to customer priority issue. this is netapp driver only patch 15:42:31 <jayaanand> also my setup is gone. need couple of days to complete. i can address review by monday next week 15:42:36 <jayaanand> yes 15:42:51 <jayaanand> customer is waiting for this feature 15:42:57 <carloss> so this would need a feature freeze exception 15:42:58 <jayaanand> we need in this relase 15:43:14 <jayaanand> yea, need exception 15:43:25 <carloss> ack... thank you for the updates 15:43:58 <jayaanand> thank you! carloss 15:45:24 <carloss> this is a driver-only feature, it's not that extensive in lines of code. I would be okay granting the exception for this, but would be nice to have some testing and some netappers feedback of course (gireesh is already covering the second part :)) 15:46:26 <carloss> would also be nice to have more reviews from people in the reviewers list for this 15:46:41 <carloss> gouthamr, saravanan, carthaca, kpdev - could you please take a look? 15:46:53 <gouthamr> when he has the patch, correct? 15:46:55 <kpdev> ack 15:46:57 <carloss> yep 15:47:00 <gouthamr> ack 15:47:12 <carthaca> ok 15:47:19 <carloss> thanks folks :) 15:47:33 <carloss> last but not least on the feature freeze list: 15:47:46 <carloss> #link https://review.opendev.org/c/openstack/manila/+/925902 (Allow scheduling to disabled manila-share host) 15:48:17 <carloss> not that big of a change too, I believe it is close 15:48:46 <carloss> gouthamr, ashrodri, gireesh: can I have your eyes on this change too? 15:49:02 <kpdev> yes, i have addressed comments from your earlier review, 15:49:21 <gireesh> sure 15:49:29 <carloss> thanks 15:49:53 <carloss> there are a couple of other things in the review focus etherpad, but a lot of them are targeted to RC1 or Epoxy 15:50:25 <carloss> so I'd encourage everyone to keep looking at the review focus etherpad and keeping up with the changes reviews 15:50:30 <carloss> that's all I had for $topic 15:50:37 <carloss> is there something else you'd like to bring up? 15:52:12 <carloss> cool 15:52:17 <carloss> we have some time for our next topic then 15:52:20 <carloss> #topic Bug Triage (vhari) 15:52:31 <vhari> ty carloss 15:52:40 <vhari> there were no new bugs this week 15:52:42 <carloss> #link https://etherpad.openstack.org/p/manila-bug-triage-pad-new (Bug Triage etherpad) 15:52:55 <carloss> \o/ 15:53:02 <vhari> :) 15:53:09 <vhari> for 22 days at that 15:53:22 <vhari> was planning to loop back on some stale bugs 15:53:46 <vhari> but in the interest of time .. we can defer to next sessions 15:53:57 <vhari> back to you carloss 15:54:07 <carloss> vhari: thank you! 15:54:12 <carloss> #topic Open Discussion 15:54:34 <carloss> well, 5 minutes left :) 15:56:43 <carloss> oh, silence :) 15:56:57 <vhari> if anyone is interested to take a look at stale bugs ^^ pls see the bug etherpad 09-05-24 15:57:17 <carloss> vhari++ 15:57:19 <vhari> ty for pitching in advance 15:57:44 <carloss> okay, so let's get back to reviewing changes and uploading patch sets 15:57:50 <carloss> thank you everyone for your efforts to make feature freeze happen 15:57:52 <carloss> let's get back to #openstack-manila 15:57:58 <carloss> have a great day! 15:58:00 <carloss> #endmeeting