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