Wednesday, 2025-07-09

*** mhen_ is now known as mhen02:03
jbernard#startmeeting cinder14:01
opendevmeetMeeting started Wed Jul  9 14:01:17 2025 UTC and is due to finish in 60 minutes.  The chair is jbernard. Information about MeetBot at http://wiki.debian.org/MeetBot.14:01
opendevmeetUseful Commands: #action #agreed #help #info #idea #link #topic #startvote.14:01
opendevmeetThe meeting name has been set to 'cinder'14:01
jbernard#topic roll call14:01
jbernardo/14:01
Saio/14:01
harsho/14:01
Luzio/14:01
rosmaitao/14:01
hvlcchao1o/14:02
gireesho/14:03
jbernardwelcome everyone14:05
jbernard#topic annoucements14:05
tosky_o/14:05
hvlcchao1o/14:05
jbernardmembership (openinfra) renewal14:05
jbernard#link https://openinfra.org/a/renew-membership14:06
jbernardthis seems to be important14:06
harshdone :)14:06
jbernardthere was a mail earlier this week, deadline is in about 10 days14:06
rosmaitai couldn't find the email, but it was easy to renew with the link14:06
jbernardmail subject is "Action Required: Renew Your OpenInfra Individual Membership by July 19"14:07
rosmaitaword on the street is that if your foundation membership had already lapsed, you won't get an email14:07
jbernardahhj14:07
jbernardso in summary, fixup your memberships :)14:08
rosmaita:D14:08
jbernard#topic followup14:09
jbernardsimondodsley asked about missing sdk commands in teh last meeting14:09
Vivek__o/14:09
jbernard#link https://etherpad.opendev.org/p/cinder-openstacksdk-gaps14:09
jbernardi think rajat would probably know best but I haven't gotten any feedback yet14:10
* jungleboyj sneaks in late14:10
jbernardrosmaita: ^ any insights that i may have missed?14:10
rosmaitanot really, what was the source of that etherpad?14:11
jbernard#link https://docs.google.com/spreadsheets/d/1yetPti2XImRnOXvvJH48yQogdKDzRWDl-NYbjTF67Gg/edit?pli=1&gid=1463660729#gid=146366072914:11
jbernardrosmaita: i thinkm simon created it, listing the ones he noticed were missing14:11
rosmaitaok, just curious14:12
jbernardi haven't had any time to look at the sdk/cli recently14:12
jbernardin summary, this is still something i need to followup on14:12
jbernard#topic image encryption tempest tests14:14
jbernard#link https://review.opendev.org/c/openstack/barbican-tempest-plugin/+/95269914:14
jbernardthis is very nice to have14:14
Luzimhen finished his work on this14:15
jbernardwe would like to land the image encryption patch14:15
jbernard#link https://review.opendev.org/c/openstack/cinder/+/92629814:15
jbernardi was able to test it sucessfully locally14:15
jbernardbut I think now others can review without having to do that, as we now have coverage in the gate14:16
jbernardLuzi: am I accurate about that^ ?14:16
Luziam i don't know about patches that depend on each other14:17
Luzii mean the tempest patch clearly depends on the cinder patch, the cinder patch currently does not depend on the tempest patch14:17
Luziis that correct?14:18
Luzibtw: the glance patch is needed for this too14:18
jbernardthat's what im seeing also14:18
jbernardbasically, we'd like to get this code in sooner as opposed to later in the cycle, so please review if you have the time14:19
jbernardLuzi: what is the glance status?14:19
Luzihttps://review.opendev.org/c/openstack/glance/+/926295 there is 2x +114:20
jbernard#topic ibm driver phone-home14:22
jbernard#link https://review.opendev.org/c/openstack/cinder/+/95182914:22
jbernardim not sure who added this14:22
Vivek__hi jbernard14:22
Vivek__This is vivek, Im part of IBM SVf cinder driver team. I have recently started contributing to the community.14:23
jbernardVivek__: welcome14:23
Vivek__One of my team mate uploaded the patch 14:23
Vivek__thanks 14:23
jbernardI took a look at this recently, and i think brian did as well14:24
rosmaitayep14:24
Vivek__We can discuss on setting the callhome as disabled 14:24
jbernardim not personally a fan of phone-home, but the enabled-by-default is difficult for me14:24
Vivek__We are enabling Callhome support since we are not capturing any sensitive data. The metadata collected includes the OS version, Cinder driver version, deployment type, PBHA status, and replication feature usage. 14:25
rosmaitaIt's good that it's not sensitive, but i think it should be "off" by default14:26
Vivek__This information helps us understand which features are actively being used by customers, allowing us to better prioritize future enhancements for our driver.14:26
rosmaitaYes, and i imagine that most of your customers will decide to opt-in14:27
rosmaitabut it would be good to give them a choice14:27
jungleboyjrosmaita:  ++14:27
rosmaitaare you thinking that if you ship with it "off", no one will bother to turn it on, and hence you won't get any data?14:28
Vivek__Yes, we have introduced a toggle switch in conf to turn it "off".14:28
jbernardone issue is that the toggle as written has no discoverability, you have to read the code to know how to do that14:29
rosmaitayes, i talked about that in my comment also ... this needs to be defined like a "real" option14:30
harshHi.. i am Vivek's colleague. his connection seems to have disconnected.14:31
harshrosmaita : yes. thats why we want to ship it with enabled. so that if a customer is unsure about data sharing, can turn it off.14:31
harshjbernard : yes, as there is no readibility except from the code, we want to ship it as enabled and update our driver document : https://docs.openstack.org/cinder/latest/configuration/block-storage/drivers/ibm-storwize-svc-driver.html14:31
rosmaitaharsh: i think it needs to be documented like any other option14:32
harshcorrect. we will add it in the documents. 14:32
rosmaitanot just the documentation, it should show up in the sample config file14:33
jungleboyj++14:33
rosmaitasee my comment from yesterday on the patch14:33
harshwe dont want to miss out on the customer info to ma14:33
jbernardi am personally against on-by-default, i think packagers/vendors can set it differently, but the upstream/opensource version should not default to enabled14:33
rosmaitapersonally, i do not like the opt-out model ... i suggest that you ship with it off by default, but all your sample docs can show turning it on14:34
harshto enhance the driver as per the features that are actually being utiliised by the user.14:34
Vivek__sorry my chat session got disconnected due to some network issue.14:35
harshokay.. thanks for your suggestions rosmaita and jbernard . I will take your inputs to my team 14:35
rosmaitai guess a key issue here is, do we have any other examples of a vendor call-home feature in openstack that is shipped "on" by default14:35
harshWe don't have that info as of now. anyone else has any idea ?14:36
jungleboyjI am not aware of any.14:36
rosmaitai am not aware of anything in cinder, but neutron and manila have third party backends too14:36
rosmaitai'm mentioning it because i think without a precedent, it will be a difficult sales job to the community to have this thing "on" by default14:37
harshi understand14:38
rosmaitaok14:38
harshwe try to find out if there is any precedent.. 14:38
Vivek__Netapp have activeIQ and Dell have Unisphere / Telemetry, we may need to check how they are enabling it 14:39
rosmaitathat would be good info to have14:39
rosmaitaanyone here today from netapp or dell?14:39
Anoop_ShuklaYes. NetApp telemetry is pushed to ONTAP via EMS and when we collect ASUP data, traces of telemetry can be mined out.14:42
Anoop_ShuklaASUP - AutoSupport14:43
Vivek__So support for NetApp telemetry is set to enabled or disabled while driver init ?14:44
jbernard#link https://github.com/openstack/cinder/blob/master/cinder/volume/drivers/netapp/utils.py#L57014:44
Anoop_ShuklaAs far as I know, customers can toggle ASUP on ONTAP storage or be on dark sites. Cinder/Manila would always post EMS to ONTAP.14:46
harshSo cinder is on-by-default as i understand..14:47
Vivek__Similarly we are setting it to "on" in order we can capture the metadata and derive the feature details.14:47
jbernardok, i at least need some time to look at netapp and think on things, lets revisit this in the next meeting.  we may need to take a vote, or elevate this to the TC for input, more thought is needed14:51
jbernardi do understand the argument14:52
jbernard#action sdk gaps14:52
harshyes sure.. thankyou :) we can have a followup in the next meeting.14:52
jbernard#action call home defaults followup14:52
Vivek__thanks 14:53
jbernard#topic powerscale driver rename in manilla14:53
jbernard#link https://review.opendev.org/c/openstack/manila/+/95273614:53
jbernardim not clear on how this relates to cinder, can someone take this?14:54
jbernardi suspect a similar rename is needed in cinder14:55
jbernardwhich shouldn't be a problem14:55
jbernardso this may not need discussion14:55
rosmaitahopefully not, these renames are a real PITA14:56
gireeshhttps://www.irccloud.com/pastebin/AuzWFywF/14:58
gireeshThere are couple of review pending for NetApp driver please have a look into those 14:59
gireeshhttps://www.irccloud.com/pastebin/Fov4BdmM/14:59
jbernardwe're nearly at time, last call15:00
jbernardok, thanks everyone, i know there are reviews waiting, we've been getting to some of them, still many to go but we're makign progress, appreciate the patience15:01
jbernard#endmeeting15:01
opendevmeetMeeting ended Wed Jul  9 15:01:28 2025 UTC.  Information about MeetBot at http://wiki.debian.org/MeetBot . (v 0.1.4)15:01
opendevmeetMinutes:        https://meetings.opendev.org/meetings/cinder/2025/cinder.2025-07-09-14.01.html15:01
opendevmeetMinutes (text): https://meetings.opendev.org/meetings/cinder/2025/cinder.2025-07-09-14.01.txt15:01
opendevmeetLog:            https://meetings.opendev.org/meetings/cinder/2025/cinder.2025-07-09-14.01.log.html15:01
jayaanand_also https://review.opendev.org/c/openstack/cinder/+/94496415:01
jungleboyjThanks!15:01
jayaanand_https://review.opendev.org/c/openstack/cinder/+/92848615:01
jayaanand_thank you!15:01
harshthankyou :)15:02
jbernardjayaanand_: fix zuul on the second one15:02
jayaanand_sure... thank you!15:03
jbernardjayaanand_: if a reviewer looks at your patch and it's failing zuul, they're likely to move to another one15:03
jayaanand_ok15:03
*** vhari_ is now known as vhari19:03

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