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