| carloss | #startmeeting manila | 15:00 |
|---|---|---|
| opendevmeet | Meeting started Thu Mar 12 15:00:09 2026 UTC and is due to finish in 60 minutes. The chair is carloss. Information about MeetBot at http://wiki.debian.org/MeetBot. | 15:00 |
| opendevmeet | Useful Commands: #action #agreed #help #info #idea #link #topic #startvote. | 15:00 |
| opendevmeet | The meeting name has been set to 'manila' | 15:00 |
| gouthamr | o/ | 15:00 |
| carloss | courtesy ping: dviroel gouthamr carthaca Sai gireesh Kumar_T | 15:00 |
| dviroel | o/ | 15:01 |
| carthaca | hi | 15:01 |
| * carloss waits a few minutes for some quorum | 15:04 | |
| carloss | let's get started | 15:05 |
| carloss | our meeting agenda for today: | 15:05 |
| carloss | #link https://wiki.openstack.org/wiki/Manila/Meetings (Meeting agenda) | 15:05 |
| carloss | #topic Announcements | 15:06 |
| carloss | Schedule and Deadlines | 15:06 |
| carloss | #link https://releases.openstack.org/gazpacho/schedule.html (Gazpacho Schedule) | 15:06 |
| gireesh | o/ | 15:06 |
| carloss | we're currently at R-3, meaning it is RC1 week | 15:07 |
| carloss | we made significant progress with changes merging and tests, all thanks to the hard work from the zorillas | 15:07 |
| carloss | I am planning to update the RC1 patch as soon as possible. I feel I am late for that already | 15:08 |
| carloss | but there was one more fix we were pursuing | 15:08 |
| carloss | that failed CI earlier today, and I caught a potential issue | 15:08 |
| carloss | thanks to unit tests | 15:08 |
| carloss | I'm referring to | 15:09 |
| carloss | #link https://review.opendev.org/c/openstack/manila/+/980281 (Fix broken super() call in snapshot show method ) | 15:09 |
| carloss | #link https://review.opendev.org/c/openstack/manila/+/968765 (Add JSON schema validation for show snapshot API) | 15:09 |
| carloss | these are nice to haves, but not must | 15:10 |
| carloss | I was also hoping we could get | 15:10 |
| carloss | #link https://review.opendev.org/c/openstack/manila/+/849015 (Remove privileged fields from networks APIs) | 15:10 |
| carloss | and | 15:10 |
| carloss | #link https://review.opendev.org/c/openstack/manila-tempest-plugin/+/979050 (Test share network privileged fields differently) | 15:10 |
| carloss | to merge | 15:10 |
| carloss | but let's get into it in a bit | 15:11 |
| carloss | all non-api facing changes that don't make it to RC1 could potentially be backported, so please keep that in mind | 15:11 |
| carloss | and we can request releases earlier in the next cycle | 15:11 |
| carloss | speaking of next cycle | 15:12 |
| carloss | PTG Planning etherpad: | 15:12 |
| carloss | #link https://etherpad.opendev.org/p/hibiscus-ptg-manila-planning (Hibiscus PTG planning etherpad) | 15:12 |
| carloss | I've created the PTG planning etherpad for the next cycle | 15:12 |
| carloss | and booked some time slots for our meeting | 15:12 |
| carloss | s/meeting/meetings | 15:12 |
| carloss | please feel free to start adding your topics to the PTG planning etherpad | 15:13 |
| carloss | PTG is in 1 month ~10 days from now | 15:13 |
| carloss | Apr 20th~24th 2026 | 15:13 |
| carloss | please save the date if you haven't already | 15:14 |
| carloss | last but not least - which I forgot to add to the angenda: | 15:15 |
| carloss | Bugsquash updates | 15:15 |
| carloss | #link https://etherpad.opendev.org/p/manila-gazpacho-bugsquash (Gazpacho bugsquash etherpad) | 15:15 |
| carloss | so far, 11 bugs fixed | 15:15 |
| carloss | and a couple of other enhancements getting merged | 15:16 |
| carloss | it's been a really productive time for us | 15:16 |
| carloss | thanks for participating and putting the time into organizing, reviewing, proposing fixes | 15:16 |
| carloss | all of it | 15:16 |
| carloss | one of the changes I mentioned earlier is also part of the bugsquash | 15:17 |
| carloss | we're still looking at some manila ui fixes as well | 15:18 |
| carloss | so please help us driving those changes to completion | 15:18 |
| carloss | and again, thanks for the collaboration | 15:19 |
| carloss | that's all I had in terms of announcements. Is there an announcement you'd like to share? | 15:19 |
| carloss | taking silence as no... | 15:20 |
| carloss | #topic Review Focus | 15:20 |
| carloss | I think our review focus etherpad today should be slightly different... | 15:20 |
| carloss | #link https://etherpad.opendev.org/p/manila-gazpacho-bugsquash (Bugsquash etherpad) | 15:20 |
| carloss | we're almost at RC1 and only looking at bug fixes at the moment, and we've been tracking them here ^ | 15:21 |
| carloss | first from the list is a fix I already linked earlier on the meeting | 15:22 |
| carloss | #link https://review.opendev.org/c/openstack/manila/+/849015 (Remove privileged fields from networks APIs) | 15:23 |
| carloss | #link https://review.opendev.org/c/openstack/manila-tempest-plugin/+/979050 (Test share network privileged fields differently) | 15:23 |
| carloss | we've been slow on the reviews of these but quite important change | 15:23 |
| carloss | s/change/changes | 15:23 |
| carloss | thanks for the review, dviroel++ | 15:23 |
| carloss | we've also been reviewing | 15:25 |
| carloss | #link https://review.opendev.org/c/openstack/manila/+/980281 (Fix broken super() call in snapshot show method ) | 15:25 |
| carloss | #link https://review.opendev.org/c/openstack/manila/+/968765 (Add JSON schema validation for show snapshot API) | 15:25 |
| carloss | #link https://review.opendev.org/c/openstack/manila-ui/+/977718 ( Add CSS blur effect to share access keys for security) | 15:26 |
| carloss | #link https://review.opendev.org/c/openstack/manila-ui/+/977416 (Cleanup ResizeForm: enforce min value and proper error handling) | 15:26 |
| gouthamr | the snapshot changes can/should be squashed | 15:27 |
| gouthamr | think https://review.opendev.org/c/openstack/manila-ui/+/977718 needs more work | 15:27 |
| gouthamr | it's feature-ish | 15:27 |
| gouthamr | so i don't mind us merging after this week | 15:28 |
| carloss | gouthamr: yep... I was both overconfident and naive while splitting the snapshot changes, hoping they'd work separately somehow. Will squash them | 15:28 |
| gouthamr | thank you | 15:28 |
| gouthamr | ty for the reviews, dviroel | 15:30 |
| carloss | so I think those would be the changes we're still pursuing and would like to get in today | 15:30 |
| carloss | > think https://review.opendev.org/c/openstack/manila-ui/+/977718 needs more work | 15:30 |
| carloss | and ack on this, gouthamr :) | 15:30 |
| gouthamr | this fix https://review.opendev.org/c/openstack/manila/+/849015 depends on a tempest plugin fix: https://review.opendev.org/c/openstack/manila-tempest-plugin/+/979050 | 15:30 |
| carloss | ++ | 15:31 |
| gouthamr | vhari_ looked at an earlier patchset, but there were more changes | 15:31 |
| gouthamr | carloss: since i updated the change, please ignore the "owner" if you look and decide to +2 :D | 15:31 |
| carloss | gouthamr: sure, was holding back because of that | 15:32 |
| carloss | :D | 15:32 |
| carloss | I agree with the change, will post my vote | 15:32 |
| carloss | just squashed the snapshot changes | 15:33 |
| carloss | CI should let us know in a few | 15:33 |
| carloss | ty for the reviews on the manila-tempest-plugin change, kpdev and dviroel | 15:34 |
| dviroel | ack, i will let you folks w+1 :) | 15:34 |
| carloss | just did | 15:34 |
| carloss | great, let's keep an eye on the other changes then and try getting them in | 15:35 |
| carloss | I'll try updating the RC1 patches by EOD for me | 15:35 |
| carloss | we did grant a feature freeze exception for resource locks in manila ui. I was reviewing the changes earlier today | 15:36 |
| carloss | re-reviewing :) | 15:36 |
| gouthamr | ++ | 15:36 |
| carloss | in the testing it seemed okay | 15:37 |
| carloss | I'm double checking workflows for now | 15:37 |
| carloss | I think the only thing I was missing in my testing was the possibility to lock a share/access rule during the creation | 15:39 |
| carloss | I did not check the admin panels yet for that though | 15:39 |
| gouthamr | ack, we could file a tech-debt bug for that and still get the rest in? | 15:40 |
| carloss | gouthamr: yes, I agree with that | 15:40 |
| gouthamr | ugh, mea culpa on https://review.opendev.org/c/openstack/manila-ui/+/963640 | 15:41 |
| carloss | np at all | 15:42 |
| gouthamr | i think this will hit us if horizon accepts https://review.opendev.org/c/openstack/horizon/+/960678 | 15:42 |
| gouthamr | i'll check if i can refresh these and if it's in the horizon team's priority to get in | 15:43 |
| carloss | great, thanks for checking | 15:43 |
| carloss | that is likely the list of review focus now at the end of the cycle | 15:44 |
| carloss | something else we're missing? | 15:44 |
| gouthamr | in manila, we could get this in: | 15:45 |
| gouthamr | https://review.opendev.org/c/openstack/manila/+/978165 | 15:45 |
| gouthamr | #link https://review.opendev.org/c/openstack/manila/+/978165 | 15:45 |
| carloss | hmm, true | 15:45 |
| carloss | mostly driver isolated | 15:45 |
| carloss | I agree | 15:46 |
| gouthamr | one more like that: https://review.opendev.org/c/openstack/manila/+/979671 | 15:46 |
| gouthamr | and this one before we branch: | 15:46 |
| gouthamr | #link https://review.opendev.org/c/openstack/manila/+/980175 | 15:46 |
| gouthamr | i had some comments on kpdev's ensure shares change: #link https://review.opendev.org/c/openstack/manila/+/952821 | 15:47 |
| carloss | three AIs for me | 15:47 |
| carloss | will look into it | 15:47 |
| gouthamr | kpdev: unsure if you were targeting rc1 for this | 15:47 |
| carloss | if we're all comfortable backporting, I'd be up for it | 15:51 |
| carloss | ensure_shares does impact a bunch of things though | 15:51 |
| carloss | we need to be careful | 15:52 |
| carloss | alright, I think that's a wrap for review focus... let's continue on ensure_shared in #openstack-manila if needed | 15:53 |
| gouthamr | +1 | 15:53 |
| carloss | #topic Bug triage | 15:53 |
| carloss | #link https://etherpad.opendev.org/p/manila-bug-triage-pad-new (Bug triage etherpad) | 15:54 |
| kpdev | @gouthamr: ensure_shares can wait and merged be after release. | 15:54 |
| carloss | kpdev: ack, thank you | 15:55 |
| carloss | first bug on the list is: | 15:55 |
| carloss | #link https://bugs.launchpad.net/manila/+bug/2142102 (App credentials's secret path is hardcoded to "/key-manager/v1/secrets" in Manila, causing failures with non-DevStack deployments Edit) | 15:55 |
| carloss | thanks for the bug report, Sai | 15:56 |
| carloss | this happens in environments outside devstack, which is where we did most of the upstream testing | 15:56 |
| carloss | gireesh Anoop_Shukla: can we assign this to a NetApp engineer? | 15:57 |
| carloss | this would also be a high priority bug IMO, as it breaks the encryption workflow entirely | 15:57 |
| carloss | thoughts? | 15:57 |
| gireesh | sure carloss | 15:58 |
| carloss | thank you | 15:59 |
| carloss | unfortunately we only had time to cover this bug today, let's try having some extra time to go through the backlog next week | 16:00 |
| carloss | thank you for participating | 16:00 |
| carloss | let's get back to #openstack-manila | 16:00 |
| carloss | #endmeeting | 16:00 |
| opendevmeet | Meeting ended Thu Mar 12 16:00:45 2026 UTC. Information about MeetBot at http://wiki.debian.org/MeetBot . (v 0.1.4) | 16:00 |
| opendevmeet | Minutes: https://meetings.opendev.org/meetings/manila/2026/manila.2026-03-12-15.00.html | 16:00 |
| opendevmeet | Minutes (text): https://meetings.opendev.org/meetings/manila/2026/manila.2026-03-12-15.00.txt | 16:00 |
| opendevmeet | Log: https://meetings.opendev.org/meetings/manila/2026/manila.2026-03-12-15.00.log.html | 16:00 |
| Anoop_Shukla | I missed the meeting today. Will catch up with the meeting chat. | 16:23 |
Generated by irclog2html.py 4.1.0 by Marius Gedminas - find it at https://mg.pov.lt/irclog2html/!