*** bauzas_ is now known as bauzas | 03:39 | |
*** bauzas_ is now known as bauzas | 05:29 | |
*** bauzas_ is now known as bauzas | 08:03 | |
*** VrindaDhakad is now known as vdhakad | 10:59 | |
*** bauzas_ is now known as bauzas | 13:53 | |
carloss | #startmeeting manila | 15:01 |
---|---|---|
opendevmeet | Meeting started Thu Aug 1 15:01:09 2024 UTC and is due to finish in 60 minutes. The chair is carloss. Information about MeetBot at http://wiki.debian.org/MeetBot. | 15:01 |
opendevmeet | Useful Commands: #action #agreed #help #info #idea #link #topic #startvote. | 15:01 |
opendevmeet | The meeting name has been set to 'manila' | 15:01 |
carthaca | hi | 15:01 |
vhari | o/ | 15:01 |
haixin | o/ | 15:01 |
carloss | courtesy ping: dviroel gouthamr msaravan pulluri ashrodri | 15:01 |
gouthamr | o/ | 15:02 |
ccokeke[m] | hello | 15:02 |
gireesh | hi | 15:02 |
msaravan | Hi | 15:03 |
carloss | o/ hello everyone | 15:04 |
carloss | let's get started with our meeting agenda for today | 15:04 |
carloss | #link https://wiki.openstack.org/wiki/Manila/Meetings#Next_meeting (Meeting agenda) | 15:04 |
carloss | #topic Announcements | 15:04 |
carloss | Schedule and Deadlines | 15:04 |
carloss | #link https://releases.openstack.org/dalmatian/schedule.html (Dalmatian release schedule) | 15:05 |
carloss | so this week is our feature proposal freeze | 15:05 |
carloss | and it means: all new Manila features must be proposed and substantially completed, with unit, functional and integration tests by the end of the week. Collaborative review sessions must be proposed at this timeline, in order to speed up the review process | 15:05 |
carloss | on the collaborative review sessions: as we discussed in the last week, we will be covering them during the mid cycle | 15:06 |
carloss | for feature proposal freeze, I see a couple of features submitted already: | 15:08 |
carloss | - Share encryption | 15:10 |
carloss | - Pass metadata updates to the share drivers | 15:10 |
carloss | - Export location metadata | 15:10 |
carloss | - Ensure shares APIs | 15:10 |
carloss | - Manage/unmanage for cephfs drivers | 15:10 |
carloss | - [NetApp] Custom storage efficiency policy support for Manila share | 15:10 |
carloss | am I missing a feature in this list? | 15:10 |
carloss | and is there a feature you are still willing to propose that would qualify for feature proposal freeze? | 15:10 |
carloss | I'd recommend pushing the code to gerrit by tomorrow, but if you'd like some time, please let me know and we can talk exceptions | 15:10 |
msaravan | Yes, support for backup_types via API | 15:11 |
carloss | I think most of the features on this list (except for APIs ensure shares) already have some unit tests. I am willing to update unit tests for that change by tomorrow | 15:12 |
msaravan | carloss : We need some time for this, and I'll ping you and provide updates | 15:12 |
gireesh | Snaplock support for Manila, I am working on that and will raise the patch by next week | 15:13 |
carloss | msaravan, gireesh: thank you | 15:13 |
gouthamr | gireesh msaravan: we have a feature proposal freeze to estimate review bandwidth that's necessary during these last few weeks | 15:14 |
gouthamr | are these two features you mention small/well-tested? | 15:15 |
gouthamr | the list carloss shared is pretty large | 15:15 |
gireesh | Snaplock support is small and will be well tested | 15:16 |
gouthamr | if we can defer anything meaningfully to Epoxy, i'd encourage it | 15:16 |
gouthamr | Epoxy is a SLURP release | 15:16 |
msaravan | Ack, @gouthamr : we understand that. Yes, it would be well tested. We all are working closely on the design part and testing part.. to make sure we are not missing anything. | 15:16 |
msaravan | @gouthamr : We understand your point.. if it takes more time, and if we find it complex, we'll update you and we can take a call as you suggested. | 15:17 |
gouthamr | sure, I don't mean to sound like a stuck tape - these deadlines have existed for a while :) am not in favor of a feature freeze exception.. | 15:18 |
gouthamr | (feature freeze is Milestone-3) | 15:18 |
ashrodri | for export location metadata, will push up tempest patch soon for functional testing. | 15:19 |
msaravan | Ack.. | 15:19 |
carloss | ++ on the measuring the review bandwidth and the importance to have the features on gerrit | 15:22 |
carloss | thanks for sharing your thoughts | 15:23 |
carloss | agree with deferring something to Epoxy in case we need, as it is a SLURP release | 15:25 |
carloss | and for the amount of features we already have, we'll need as many reviewers as possible | 15:26 |
carloss | so please submit the code to gerrit as soon as you can | 15:28 |
carloss | and ++ on trying to avoid feature freeze exceptions on this cycle | 15:30 |
carloss | still on announcements, and something similar | 15:31 |
carloss | our mid-cycle | 15:32 |
carloss | #link https://lists.openstack.org/archives/list/openstack-discuss@lists.openstack.org/thread/5UMH5BI55STHTMEV4W7JHWAOTURFMX3U/ (Mid-cycle announcement) | 15:32 |
carloss | I sent the email announcing it to the mailing list | 15:32 |
carloss | and it will be as we spoke in the last week: two days, two hours each day | 15:32 |
carloss | Aug 12 and 13 | 15:32 |
carloss | from 14 to 16 UTC | 15:33 |
carloss | on the first day we will focus on updates from what we discussed on the PTG | 15:33 |
carloss | and the second one we will use for collab review sessions | 15:33 |
carloss | I believe that due to the amount of features we currently have, we will need more time to collab reviews, so maybe we can get started with them on the first day, in case we have some time | 15:34 |
carloss | #link https://etherpad.opendev.org/p/manila-dalmatian-mid-cycle-notes (Mid cycle etherpad) | 15:34 |
carloss | please add a topic for the feature you are proposing under the collaborative reviews topic | 15:34 |
carloss | is there something else you'd like to add? | 15:35 |
carloss | or an announcement to share with us today? | 15:35 |
carloss | taking silence as no | 15:37 |
carloss | #topic Review focus | 15:38 |
carloss | #link https://etherpad.opendev.org/p/manila-dalmatian-review-focus (Review focus etherpad) | 15:38 |
carloss | Please add all the features and patches you are proposing to the review focus etherpad | 15:39 |
carloss | and let's try to get back using it again. It can be a great source of identifying changes that can use your eyes | 15:39 |
carloss | so I'll get back to updating it daily, and would be nice if all reviewers could look at it a couple of times every week | 15:40 |
carloss | and check if there's a change you can help with reviews | 15:40 |
carloss | for now, the review focus will naturally be shifting to features, because we are getting close to the feature freeze :) | 15:41 |
carloss | that's all I had for $topic ... Is there a change you'd like to ask for our attention today? | 15:41 |
kpdev | for feature freeze I would like to review and merge two things ;; deferred deletion and passing metadata to backend | 15:43 |
carloss | ack, thanks kpdev - I think we got a couple of reviews on deferred deletion already, so seems closer | 15:45 |
carloss | the metadata one seems close too. can you do the collab review session for it? | 15:45 |
kpdev | ok, will add for metadata | 15:46 |
carloss | thank you! | 15:47 |
carloss | #topic Bug Triage (vhari) | 15:47 |
vhari | o/ all | 15:48 |
vhari | let's dive in | 15:48 |
vhari | #link https://bugs.launchpad.net/manila/+bug/2073883 | 15:48 |
carloss | we discussed something similar lately | 15:49 |
carloss | let me refresh my memory :) | 15:49 |
carloss | that's a different thing: allow scheduling to disabled manila-share host Edit | 15:50 |
vhari | ack, and force delete option | 15:50 |
carloss | tricky one because host down is different from disabled, and we might not be able to reach it | 15:50 |
carloss | not sure if we had someone looking into it in the past though | 15:51 |
gouthamr | we did i think | 15:52 |
gouthamr | sorry; my search-fu isn't working | 15:53 |
gouthamr | https://bugs.launchpad.net/bugs/1867030 | 15:56 |
gouthamr | so we created this fix: | 15:57 |
gouthamr | #link https://review.opendev.org/c/openstack/manila/+/911694 | 15:57 |
gouthamr | ccokeke[m] worked on this during his internship application | 15:57 |
carloss | ah, nice | 15:57 |
carloss | so it's a duplicated bug | 15:57 |
gouthamr | this was reported against yoga | 15:58 |
carloss | yep... we would need to start backporting it | 15:58 |
carloss | this feels a lot like an RFE | 15:58 |
gouthamr | it is | 15:58 |
ccokeke[m] | gouthamr: yes i did | 15:58 |
ccokeke[m] | during my contribution stage | 15:59 |
vhari | ccokeke[m]++ | 16:00 |
carloss | the fix apparently doesn't show any risks of regression, so I believe we should be safe to backport | 16:00 |
vhari | gouthamr, good find | 16:00 |
carloss | nothing related to API versions or changing returns | 16:00 |
carloss | so I'd say: let's close this bug and say the fix is already in and add some links | 16:01 |
carloss | and we can start with the backports | 16:01 |
* carloss time check | 16:01 | |
carloss | let's wrap up | 16:02 |
carloss | thank you for participating | 16:02 |
carloss | let's get back to #openstack-manila | 16:02 |
carloss | #endmeeting | 16:02 |
opendevmeet | Meeting ended Thu Aug 1 16:02:40 2024 UTC. Information about MeetBot at http://wiki.debian.org/MeetBot . (v 0.1.4) | 16:02 |
opendevmeet | Minutes: https://meetings.opendev.org/meetings/manila/2024/manila.2024-08-01-15.01.html | 16:02 |
opendevmeet | Minutes (text): https://meetings.opendev.org/meetings/manila/2024/manila.2024-08-01-15.01.txt | 16:02 |
opendevmeet | Log: https://meetings.opendev.org/meetings/manila/2024/manila.2024-08-01-15.01.log.html | 16:02 |
*** bauzas_ is now known as bauzas | 20:03 | |
*** dwhite48 is now known as dwhite4 | 21:37 | |
*** pots3 is now known as pots | 21:37 |
Generated by irclog2html.py 2.17.3 by Marius Gedminas - find it at https://mg.pov.lt/irclog2html/!