Thursday, 2025-07-10

*** jbernard_ is now known as jbernard14:49
carloss#startmeeting manila15:00
opendevmeetMeeting 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
opendevmeetUseful Commands: #action #agreed #help #info #idea #link #topic #startvote.15:00
opendevmeetThe meeting name has been set to 'manila'15:00
vharihi15:00
haixino/15:00
carlosscourtesy ping: dviroel Sai ashrodri gireesh15:00
gireesho/15:02
carlossalright, let's get started15:04
carloss#link https://wiki.openstack.org/wiki/Manila/Meetings (meeting agenda)15:05
kpdev+115:05
carloss#topic Announcements15:05
carlossSchedule and Deadlines15:06
carloss#link https://releases.openstack.org/flamingo/schedule.html (Flamingo release schedule)15:06
carlosswe're in R-12, aka week after the hack-a-ton15:07
Saio/15:07
carlossand it also means that we're 3 weeks away from feature proposal freeze15:07
carlossand new driver deadline15:07
carlossand 4 weeks away from the manila mid cycle15:08
carlosswe should target all the collab review sessions to this mid cycle15:08
carlossI think we're in a good shape so far  with features and the release15:09
vhari+1 15:10
carlossthat's all in terms of announcement15:10
carlosss/announcement/announcements15:10
carlossdo you have an announcement to share?15:10
vhariI would like to invite everyone to check out the hack-a-ton kanban15:11
vharihttps://tree.taiga.io/project/gouthampacha-openstack-manila-osc-integration-with-python-manilaclient/kanban15:11
vhariin case something looks interesting :)15:12
carloss#topic Hack-a-ton follow up15:12
carloss^ good timing for that topic15:12
vharisry carloss .. 15:12
carlossso we had one change merging and 3~4 changes being proposed in total15:12
carlossvhari: no problem at all15:12
carlossI was going for it anyways15:12
vhariexcited to share the contributions with everyone 15:12
carlossjust wanted to group things15:12
carlossvhari: didn't mean to cut you, please go ahead :D15:13
vhariall set .. ty :)15:13
carlossnice, thanks15:15
carlossso 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 yet15:17
carlossI am hoping we can continue to close the changes in the upcoming weeks15:18
carlossand that we get some nice coverage of the CLI15:18
carlossalright, that's all for $topic15:19
carlosslet's switch to review focus15:20
carloss#topic Review Focus15:20
carlossa 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 etherpad15:21
carloss#link https://etherpad.opendev.org/p/manila-flamingo-review-focus15:21
carloss#link  https://review.opendev.org/c/openstack/manila/+/950361 ([DOC] Update Ceph Testing Matrix)15:22
carlosswe had some reviews on this change, thanks kpdev and nileesh15:22
carlossI've just posted the vote as well, will ask other Red Hatters to look at it too15:23
carlosswe also have15:23
carloss#link  https://review.opendev.org/c/openstack/manila/+/929965 (929965: [DOC] Add export location metadata documentation)15:23
carlosswhich is also doc related, but this time metadata API ref docs15:24
carlosswhich is also quite important15:24
carlossI'll take a look at it but we'd need more eyes15:24
carlossvhari haixin: could you please take a look?15:24
vhari+115:24
carlossty15:25
carloss#link https://review.opendev.org/c/openstack/manila/+/946552 (946552: NetApp - Preserve custom snapshot policies)15:25
carlosshad some reviews, we had Goutham reviewing it and myself as well15:25
carlossI can go add the vote but would be nice if all reviewers involved in the discussion that happened could ack the change15:26
carlossbeing Goutham, gireesh and myself15:26
haixinok15:26
carlossI will circle back15:26
carlosshaixin: ty15:26
carloss#link https://review.opendev.org/c/openstack/manila/+/934676 ([Netapp] Add support for prune deleted volumes)15:27
carlossneeds more reviewers, I can take a look at it15:27
carloss#link https://review.opendev.org/c/openstack/manila/+/941084 (941084: NetApp: delete vlan even if ipspace is reused)15:28
carlossthis is ready to be reviewed15:29
carlossgireesh Sai: could you please take a look?15:30
gireeshsure,  I requested Anoop to review this 15:30
gireeshI will also check this 15:30
SaiGireesh/Anoop covering it from NetApp side for this. Thanks carloss.15:31
carlossthank you15:31
carloss#linkn https://review.opendev.org/c/openstack/manila/+/952821 (NetApp - Update ensure_shares())15:32
carlossthis one also needs some eyes if possible15:32
gireeshsure, wil check this also 15:33
carlossthanks15:33
carloss#link https://review.opendev.org/c/openstack/python-manilaclient/+/950414 (Add option to override API endpoint)15:33
carlossthis has a discussion going on15:33
carlosshaixin: 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
carlossI see gireesh and Goutham reviewed it at some point. gireesh, could you please circle back?15:35
carlossI'll also take a look15:35
gireeshsure15:35
haixinsure15:36
carloss#link https://review.opendev.org/c/openstack/manila/+/951087/comments/302d9d15_fc98e692 (just noticed this discussion)15:36
carloss#undo15:36
opendevmeetRemoving item from minutes: #link https://review.opendev.org/c/openstack/manila/+/951087/comments/302d9d15_fc98e69215:36
carloss#link https://review.opendev.org/c/openstack/manila/+/951087/comments/302d9d15_fc98e692 (Goutham's query on enabling by default)15:36
carlosskpdev: have you looked at this?15:36
carlossare there implications of turning this on by default on existing deployments?15:37
carlossalso, considered the different versions of ontap?15:37
kpdevby default its false15:37
kpdevthis means it will no-op for existing deployments.15:38
kpdevadmin must set config option to True to enable it15:38
kpdevthe way to enable changes as per ONTAP version and that is implemented in PR15:38
kpdevbut config option is kept default as False15:38
carlossgot it, I see the version check there15:40
carlossmakes sense15:40
carlosscould you please respond to that comment from Goutham to close the loop on this?15:40
kpdevok, since i implemted i marked resolved. i will add comment now15:41
carlosskpdev: thank you15:42
carlossnext up15:42
carloss#link https://review.opendev.org/c/openstack/manila/+/952888 ([NetApp] guard already existing cifs access)15:43
carlosshas my +2, gireesh Anoop_Shukla Sai, could you please review this change?15:43
Anoop_Shukla_Sure..15:43
SaiSure. 15:43
carlossthank you15:43
carloss#link https://review.opendev.org/c/openstack/manila/+/952580 (Netapp - Enable logical space reporting)15:44
carlosssame to ^ please15:44
carlossand15:44
carloss#link https://review.opendev.org/c/openstack/manila/+/952690 (Add extra spec 'netapp:max_files_multiplier')15:44
carlosson the changes proposed to the Dell driver, we reviewed but there's a list of comments to the first change in line of being merged15:45
carlossand last but not least, share encryption15:45
carloss#link https://review.opendev.org/q/topic:%22bp/share-encryption%22 (Share encryption series)15:46
kpdevFor 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 test15:46
carlosskpdev Sai: thanks for updating the changes15:47
carlosskpdev: thanks for the detailed update. I'll get back to the change15:47
SaiThank you carloss !! 15:47
carlossI was also talking about the UUID because I thought it could be more intuitive for users15:47
carlosseasier than the href honestly15:47
carlosswe can have both no problem15:47
carlossit'd be something like accepting share name and UUID15:48
kpdevyes15:48
carlossI'll take a look at the python-manilaclient changes as well15:48
carlossthe client release happens earlier, so the early we get everything sorted, the better15:48
kpdevotherwise end user everytime need to get href, and parse uuid out of it to provide share create option15:48
kpdevfor 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 shape15:49
carlossI'll take a look at the tempest change15: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
SaiThank you kpdev for updating the tests! 15:50
carloss++15:51
carlossalright, that's all I had for $topic15:51
carlossis there any other change you'd like to bring up?15:51
carlosstaking silence as no15:53
carlosslet's wrap up15:53
carlossthanks for participating15:53
carlosshave a good one15:54
carlosssee you in #openstack-manila15:54
carloss#endmeeting15:54
opendevmeetMeeting ended Thu Jul 10 15:54:15 2025 UTC.  Information about MeetBot at http://wiki.debian.org/MeetBot . (v 0.1.4)15:54
opendevmeetMinutes:        https://meetings.opendev.org/meetings/manila/2025/manila.2025-07-10-15.00.html15:54
opendevmeetMinutes (text): https://meetings.opendev.org/meetings/manila/2025/manila.2025-07-10-15.00.txt15:54
opendevmeetLog:            https://meetings.opendev.org/meetings/manila/2025/manila.2025-07-10-15.00.log.html15:54

Generated by irclog2html.py 4.0.0 by Marius Gedminas - find it at https://mg.pov.lt/irclog2html/!