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