| *** jbernard_ is now known as jbernard | 14:49 | |
| carloss | #startmeeting manila | 15:00 |
|---|---|---|
| opendevmeet | Meeting started Thu Jul 10 15:00:15 2025 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 |
| vhari | hi | 15:00 |
| haixin | o/ | 15:00 |
| carloss | courtesy ping: dviroel Sai ashrodri gireesh | 15:00 |
| gireesh | o/ | 15:02 |
| carloss | alright, let's get started | 15:04 |
| carloss | #link https://wiki.openstack.org/wiki/Manila/Meetings (meeting agenda) | 15:05 |
| kpdev | +1 | 15:05 |
| carloss | #topic Announcements | 15:05 |
| carloss | Schedule and Deadlines | 15:06 |
| carloss | #link https://releases.openstack.org/flamingo/schedule.html (Flamingo release schedule) | 15:06 |
| carloss | we're in R-12, aka week after the hack-a-ton | 15:07 |
| Sai | o/ | 15:07 |
| carloss | and it also means that we're 3 weeks away from feature proposal freeze | 15:07 |
| carloss | and new driver deadline | 15:07 |
| carloss | and 4 weeks away from the manila mid cycle | 15:08 |
| carloss | we should target all the collab review sessions to this mid cycle | 15:08 |
| carloss | I think we're in a good shape so far with features and the release | 15:09 |
| vhari | +1 | 15:10 |
| carloss | that's all in terms of announcement | 15:10 |
| carloss | s/announcement/announcements | 15:10 |
| carloss | do you have an announcement to share? | 15:10 |
| vhari | I would like to invite everyone to check out the hack-a-ton kanban | 15:11 |
| vhari | https://tree.taiga.io/project/gouthampacha-openstack-manila-osc-integration-with-python-manilaclient/kanban | 15:11 |
| vhari | in case something looks interesting :) | 15:12 |
| carloss | #topic Hack-a-ton follow up | 15:12 |
| carloss | ^ good timing for that topic | 15:12 |
| vhari | sry carloss .. | 15:12 |
| carloss | so we had one change merging and 3~4 changes being proposed in total | 15:12 |
| carloss | vhari: no problem at all | 15:12 |
| carloss | I was going for it anyways | 15:12 |
| vhari | excited to share the contributions with everyone | 15:12 |
| carloss | just wanted to group things | 15:12 |
| carloss | vhari: didn't mean to cut you, please go ahead :D | 15:13 |
| vhari | all set .. ty :) | 15:13 |
| carloss | nice, thanks | 15:15 |
| carloss | so the Kanban currently has all issues and their statuses seem to be up to date, so please take a look at the proposed changes if you haven't had the chance yet | 15:17 |
| carloss | I am hoping we can continue to close the changes in the upcoming weeks | 15:18 |
| carloss | and that we get some nice coverage of the CLI | 15:18 |
| carloss | alright, that's all for $topic | 15:19 |
| carloss | let's switch to review focus | 15:20 |
| carloss | #topic Review Focus | 15:20 |
| carloss | a little bird told me we don't have new bugs or revisit bugs today, so we can spend some more time in the changes from the etherpad | 15:21 |
| carloss | #link https://etherpad.opendev.org/p/manila-flamingo-review-focus | 15:21 |
| carloss | #link https://review.opendev.org/c/openstack/manila/+/950361 ([DOC] Update Ceph Testing Matrix) | 15:22 |
| carloss | we had some reviews on this change, thanks kpdev and nileesh | 15:22 |
| carloss | I've just posted the vote as well, will ask other Red Hatters to look at it too | 15:23 |
| carloss | we also have | 15:23 |
| carloss | #link https://review.opendev.org/c/openstack/manila/+/929965 (929965: [DOC] Add export location metadata documentation) | 15:23 |
| carloss | which is also doc related, but this time metadata API ref docs | 15:24 |
| carloss | which is also quite important | 15:24 |
| carloss | I'll take a look at it but we'd need more eyes | 15:24 |
| carloss | vhari haixin: could you please take a look? | 15:24 |
| vhari | +1 | 15:24 |
| carloss | ty | 15:25 |
| carloss | #link https://review.opendev.org/c/openstack/manila/+/946552 (946552: NetApp - Preserve custom snapshot policies) | 15:25 |
| carloss | had some reviews, we had Goutham reviewing it and myself as well | 15:25 |
| carloss | I can go add the vote but would be nice if all reviewers involved in the discussion that happened could ack the change | 15:26 |
| carloss | being Goutham, gireesh and myself | 15:26 |
| haixin | ok | 15:26 |
| carloss | I will circle back | 15:26 |
| carloss | haixin: ty | 15:26 |
| carloss | #link https://review.opendev.org/c/openstack/manila/+/934676 ([Netapp] Add support for prune deleted volumes) | 15:27 |
| carloss | needs more reviewers, I can take a look at it | 15:27 |
| carloss | #link https://review.opendev.org/c/openstack/manila/+/941084 (941084: NetApp: delete vlan even if ipspace is reused) | 15:28 |
| carloss | this is ready to be reviewed | 15:29 |
| carloss | gireesh Sai: could you please take a look? | 15:30 |
| gireesh | sure, I requested Anoop to review this | 15:30 |
| gireesh | I will also check this | 15:30 |
| Sai | Gireesh/Anoop covering it from NetApp side for this. Thanks carloss. | 15:31 |
| carloss | thank you | 15:31 |
| carloss | #linkn https://review.opendev.org/c/openstack/manila/+/952821 (NetApp - Update ensure_shares()) | 15:32 |
| carloss | this one also needs some eyes if possible | 15:32 |
| gireesh | sure, wil check this also | 15:33 |
| carloss | thanks | 15:33 |
| carloss | #link https://review.opendev.org/c/openstack/python-manilaclient/+/950414 (Add option to override API endpoint) | 15:33 |
| carloss | this has a discussion going on | 15:33 |
| carloss | haixin: could you please circle back to the comment thread? | 15:33 |
| carloss | #link https://review.opendev.org/c/openstack/manila/+/951087 (951087: [NetApp] Add config option for aes-encryption for cifs shares) | 15:34 |
| carloss | I see gireesh and Goutham reviewed it at some point. gireesh, could you please circle back? | 15:35 |
| carloss | I'll also take a look | 15:35 |
| gireesh | sure | 15:35 |
| haixin | sure | 15:36 |
| carloss | #link https://review.opendev.org/c/openstack/manila/+/951087/comments/302d9d15_fc98e692 (just noticed this discussion) | 15:36 |
| carloss | #undo | 15:36 |
| opendevmeet | Removing item from minutes: #link https://review.opendev.org/c/openstack/manila/+/951087/comments/302d9d15_fc98e692 | 15:36 |
| carloss | #link https://review.opendev.org/c/openstack/manila/+/951087/comments/302d9d15_fc98e692 (Goutham's query on enabling by default) | 15:36 |
| carloss | kpdev: have you looked at this? | 15:36 |
| carloss | are there implications of turning this on by default on existing deployments? | 15:37 |
| carloss | also, considered the different versions of ontap? | 15:37 |
| kpdev | by default its false | 15:37 |
| kpdev | this means it will no-op for existing deployments. | 15:38 |
| kpdev | admin must set config option to True to enable it | 15:38 |
| kpdev | the way to enable changes as per ONTAP version and that is implemented in PR | 15:38 |
| kpdev | but config option is kept default as False | 15:38 |
| carloss | got it, I see the version check there | 15:40 |
| carloss | makes sense | 15:40 |
| carloss | could you please respond to that comment from Goutham to close the loop on this? | 15:40 |
| kpdev | ok, since i implemted i marked resolved. i will add comment now | 15:41 |
| carloss | kpdev: thank you | 15:42 |
| carloss | next up | 15:42 |
| carloss | #link https://review.opendev.org/c/openstack/manila/+/952888 ([NetApp] guard already existing cifs access) | 15:43 |
| carloss | has my +2, gireesh Anoop_Shukla Sai, could you please review this change? | 15:43 |
| Anoop_Shukla_ | Sure.. | 15:43 |
| Sai | Sure. | 15:43 |
| carloss | thank you | 15:43 |
| carloss | #link https://review.opendev.org/c/openstack/manila/+/952580 (Netapp - Enable logical space reporting) | 15:44 |
| carloss | same to ^ please | 15:44 |
| carloss | and | 15:44 |
| carloss | #link https://review.opendev.org/c/openstack/manila/+/952690 (Add extra spec 'netapp:max_files_multiplier') | 15:44 |
| carloss | on the changes proposed to the Dell driver, we reviewed but there's a list of comments to the first change in line of being merged | 15:45 |
| carloss | and last but not least, share encryption | 15:45 |
| carloss | #link https://review.opendev.org/q/topic:%22bp/share-encryption%22 (Share encryption series) | 15:46 |
| kpdev | For share encyption, the implemetation is already there. Addressed comments from haixin and carlos. The user input today is both href and href_uuid. Haixin suggested to have only UUID and put barbican endpoint in conf file. This is not as per spec, so waiting on thoughts from carlos. app_cred_id field is not changed since I believe its small and simple. The tempest test PR is also raised convering share create positive test and negative test | 15:46 |
| carloss | kpdev Sai: thanks for updating the changes | 15:47 |
| carloss | kpdev: thanks for the detailed update. I'll get back to the change | 15:47 |
| Sai | Thank you carloss !! | 15:47 |
| carloss | I was also talking about the UUID because I thought it could be more intuitive for users | 15:47 |
| carloss | easier than the href honestly | 15:47 |
| carloss | we can have both no problem | 15:47 |
| carloss | it'd be something like accepting share name and UUID | 15:48 |
| kpdev | yes | 15:48 |
| carloss | I'll take a look at the python-manilaclient changes as well | 15:48 |
| carloss | the client release happens earlier, so the early we get everything sorted, the better | 15:48 |
| kpdev | otherwise end user everytime need to get href, and parse uuid out of it to provide share create option | 15:48 |
| kpdev | for tempest also we are in good share, just need review on manila. Sai will then help to run tempest locally with netapp driver. | 15:49 |
| kpdev | *good shape | 15:49 |
| carloss | I'll take a look at the tempest change | 15:50 |
| Sai | > for tempest also we are in good share, just need review on manila. Sai will then help to run tempest locally with netapp driver. | 15:50 |
| Sai | ++ | 15:50 |
| Sai | Thank you kpdev for updating the tests! | 15:50 |
| carloss | ++ | 15:51 |
| carloss | alright, that's all I had for $topic | 15:51 |
| carloss | is there any other change you'd like to bring up? | 15:51 |
| carloss | taking silence as no | 15:53 |
| carloss | let's wrap up | 15:53 |
| carloss | thanks for participating | 15:53 |
| carloss | have a good one | 15:54 |
| carloss | see you in #openstack-manila | 15:54 |
| carloss | #endmeeting | 15:54 |
| opendevmeet | Meeting ended Thu Jul 10 15:54:15 2025 UTC. Information about MeetBot at http://wiki.debian.org/MeetBot . (v 0.1.4) | 15:54 |
| opendevmeet | Minutes: https://meetings.opendev.org/meetings/manila/2025/manila.2025-07-10-15.00.html | 15:54 |
| opendevmeet | Minutes (text): https://meetings.opendev.org/meetings/manila/2025/manila.2025-07-10-15.00.txt | 15:54 |
| opendevmeet | Log: https://meetings.opendev.org/meetings/manila/2025/manila.2025-07-10-15.00.log.html | 15:54 |
Generated by irclog2html.py 4.0.0 by Marius Gedminas - find it at https://mg.pov.lt/irclog2html/!