Wednesday, 2023-04-05

*** chuanm0 is now known as chuanm04:59
*** chuanm9 is now known as chuanm05:46
*** chuanm8 is now known as chuanm10:20
whoami-rajat#startmeeting cinder14:00
opendevmeetMeeting started Wed Apr  5 14:00:02 2023 UTC and is due to finish in 60 minutes.  The chair is whoami-rajat. Information about MeetBot at http://wiki.debian.org/MeetBot.14:00
opendevmeetUseful Commands: #action #agreed #help #info #idea #link #topic #startvote.14:00
opendevmeetThe meeting name has been set to 'cinder'14:00
whoami-rajat#topic roll call14:00
rosmaitao/14:00
felipe_rodrigueshi14:00
MounikaHi14:00
nahimsouza[m]hi14:00
thiagoalvoravelhi14:01
whoami-rajatwe've new etherpad!14:01
whoami-rajat#link https://etherpad.opendev.org/p/cinder-bobcat-meetings14:01
crohmannwhoami-rajat: I already abused that :-)14:01
HelenaDantas[m]hi14:02
simondodsleyo/14:02
whoami-rajatstarting the cycle with a topic this big should be a good sign (at least I'm optimistic about it)14:02
lucasmoliveira059o/14:02
keerthivasansuresho/14:02
whoami-rajatwe've enough people, let's start the first meeting of 2023.2 Bobcat14:04
whoami-rajat#topic announcements14:04
whoami-rajatthese are some announcements from PTG, in case anyone missed it14:04
whoami-rajat2023.1 (Antelope) is released!14:04
whoami-rajat#link https://lists.openstack.org/pipermail/openstack-discuss/2023-March/032872.html14:05
jungleboyjo/14:05
whoami-rajatagain thanks everyone for your contributions and efforts!14:05
whoami-rajatnext, 2023.1 (Antelope) Project update in OpenInfra live14:05
caiquemello[m]o/14:05
whoami-rajat#link https://www.youtube.com/watch?v=YdLTUTyJ1eU14:05
whoami-rajatI along with other PTLs provided project updates for 2023.1 Antelope, if you're interest please take a look at the youtube video14:06
whoami-rajatnext, Welcome to first meeting of 2023.2 Bobcat cycle14:06
whoami-rajatAdd your name in the Courtesy reminder section if you would like to be notified about cinder meeting (Wednesday, 1400 UTC)14:06
whoami-rajatit's on L#2714:07
whoami-rajat#link https://etherpad.opendev.org/p/cinder-bobcat-meetings#L2714:07
rosmaitawe've never figured out a way to tell people who aren't here and don't want reminders any more to remove their names14:08
whoami-rajatwe had 2023.2 Bobcat PTG last week from tuesday to friday 1300-1700 UTC, here is the etherpad containing all the discussion minutes14:08
whoami-rajat#link https://etherpad.opendev.org/p/bobcat-ptg-cinder14:08
whoami-rajatrosmaita, true, but we ping them on #openstack-cinder so they can reply there or reach me out if they don't want to be notified anymore (that is my assumption)14:09
jungleboyj++14:09
jungleboyjI haven't seen anyone complaining about still getting pinged.  So, I don't think that is a big issue.14:09
whoami-rajatwelcome back jungleboyj , we missed you at the PTG14:10
jungleboyjThank you!  Sorry I had to miss it.14:10
whoami-rajatno problem, I'm sure there were valid reasons for it :)14:11
whoami-rajatso continuing with the announcements14:11
whoami-rajatwe've recordings of sessions for all 4 days in the cinder etherpad14:11
whoami-rajatIt's currently in bluejeans but will be uploaded to Cinder youtube channel soon14:12
jungleboyjWorking on that as we speak.14:12
whoami-rajatbut i don't think that should be a problem if you would like to watch a session you're interested in14:12
whoami-rajatgreat, thanks Jay!14:12
whoami-rajatthat's all the announcements I had14:13
whoami-rajatI'm currently working on the summary, it will be out soon before end of the week14:14
whoami-rajatlet's move to the topics then14:15
whoami-rajat#topic Cinder backup "improvements"14:15
whoami-rajatcrohmann, that's you14:15
whoami-rajatjust to mention we discussed your topics and there should be action items/discussion points in the etherpad14:15
crohmannYes. Thanks again. I took off the specs and now have the remaining "improvements" listed here14:16
crohmannOverall I am simply interested in getting cinder-backup work really well with object storage (via chunked driver). So my "little" list here is about the issues I currently see.14:17
crohmannIf you already did talk about anything I have on todays list, please mention that again and I gladly look at the recording / notes14:18
crohmann(notes I did look at though)14:19
whoami-rajatregarding the (How) to reduce memory footprint for "all" deployment tools, as done here14:20
whoami-rajatwe discussed that in the operator hour https://etherpad.opendev.org/p/march2023-ptg-operator-hour-cinder14:21
whoami-rajat#link https://etherpad.opendev.org/p/march2023-ptg-operator-hour-cinder14:21
crohmannbut itthat is more of a workaround14:22
crohmannto simply reduce the chunk size. I was more talking about the proposed and merged changes using MALLOC_ARENA_MAX malloc tuning..14:23
crohmannThe referenced changes already set this for devstack and tripleo. I was just wondering how to have other deployment tools apply the same.14:23
whoami-rajatoh, for that you will need to contact other deployment projects, probably you can write a mail to openstack-discuss mentioning all the deployment tools you would like to include these changes14:25
whoami-rajatsomething like, [kolla-ansible][deployment-project-2][...]... Imporve cinder backup performance14:25
crohmannYes. I certainly could do that. I was just wondering that this rather vital tuning paramter was something the cinder team would like to promote / package more prominently somehow.14:26
whoami-rajatack, maybe geguileo can take this question but as per my understanding, it did bring down the memory consumption in his testing14:27
geguileocrohmann: you could always submit a patch to the other deployment tools14:28
geguileocrohmann: or ask them to add it14:28
geguileothere are too many deployment tools out there14:29
crohmanngeguileo:  good point. I shall do so for at least openstack-ansible then. I was just wondering (not anymore) if these malloc tunrings are the way to go. But that seems to be the case. Maybe somebody has any thoughts about next bullet point (streaming io) ?14:29
geguileocrohmann: for Cinder that is the way to go, I don't know about other projects14:29
geguileothe reason to have multiple memory arenas in glibc is to remove bottlenecks in multithreaded applications14:30
geguileothat do a lot of memory allocation/free operations14:30
geguileobut we are running on Python, which already has its own memory management system14:30
geguileoso having additional per native thread memory arenas ends up creating memory problems14:31
geguileohigher peak consumption14:31
geguileohigh watermark issues14:31
geguileoconsidering the way cinder works, we wouldn't be really benefiting from those arenas, and they just create problems for us14:31
crohmannYeah. But thanks for clarifying this again. I did not want to promote adding these parameters without another consultation here.14:32
geguileoso I doubt openstack-ansible will be oposed to your patch14:32
crohmannI shall point them to this weekly meeting then :-)14:32
geguileocrohmann: if you have trouble with the reviewers you can ping me on IRC and I'll chime in that review14:32
geguileocrohmann: I recommend you pointing out how this has already been done in devstack and tripleo and show the links to the patches14:33
geguileothat usually helps to convince people14:33
whoami-rajatexample: *nova does it so we should too*14:34
whoami-rajatso that should address the second point14:35
crohmannI'll get to it soon. It's just unfortunate that options like these need to be done by deployment tooling sometimes and there is not easy way for you to simply include that in your release.14:35
whoami-rajatregarding the 3rd point, i think zaitcev brought up the discussion of streaming IO rather than using chunks14:36
whoami-rajatbut i might be wrong14:36
rosmaitai think you're right, it was Pete14:37
whoami-rajatok, good to know14:37
crohmannMaybe yes. But I never talked to him directly about it but send s few emails regarding cinder-backup improvement ideas to geguileo and zaitcev .14:37
crohmannCurrently the throughput using chunked driver powered cinder-backup drivers is a little sub-par. So maybe there is only so much one can do tuning the current design14:38
zaitcevThe large size of uncompressed chunk was something that jumped out, but I thought that I might be out of touch with memory sizes at typical modern systems.14:39
crohmannBut doing streaming io (including compression, ...) there memory foodprint should be much lower, right?14:40
crohmannzaitcev: You also said that doing multi-threading on the current chunking driver (https://review.opendev.org/c/openstack/cinder/+/779233) was not helping (to improve throughput). Maybe that change should be rejected alltogether then?14:41
zaitcevcrohmann: I highly doubt that I could say something like that. I don't have enough field experience to make such claims.14:43
zaitcevbut let's see14:43
crohmannAll I am saying is that cinder-backup via chunked driver currently is memory heavy and rather slow. That's unfortunate, but cause that hinders all of them drivers using the abstract chunking to be really a good replacement. So I am simply urging to find the bottlenecks to be able to use object storage for volume backups :-)14:45
Tony_SaadHey guys, I have a couple for review https://review.opendev.org/c/openstack/cinder/+/879067 https://review.opendev.org/c/openstack/os-brick/+/876284 https://review.opendev.org/c/openstack/cinder/+/874813/314:46
whoami-rajatTony_Saad, please add it to the review request section on the etherpad14:47
Tony_SaadAlso I have a question about https://bugs.launchpad.net/cinder/+bug/2003179 Password appears in plain text. This is a security vulnerability. Is there a way to merge security issues faster than bobcat?14:48
zaitcevI made a note to look into the performance of backups. Having a bug is a great help, because it adds it credibility and the data point that is not an artefact of my test VMs.14:48
whoami-rajatTony_Saad, please wait for the open discussion or add topics on etherpad before the cinder meeting if you would like something to be discussed, we're currently discussing an ongoing topic14:49
Tony_Saadopps sorry thought we were open discussion 14:49
crohmannzaitcev: Thanks. I shall retest what I sent via email a few weeks ago and raise a bug with some data14:49
whoami-rajatTony_Saad, no problem14:50
whoami-rajatcrohmann, regarding the last point, "What ever happend to "multi-backend" support for cinder-backup?"14:50
whoami-rajatI can see the spec was proposed by Ivan and he's not currently active anymore (though he shows up from time to time)14:50
whoami-rajatI don't remember if he implemented that functionality but that should be open for taking up if anyone would like to work on it14:51
crohmannSo is this still valid to start working on and then propose for e.g. Bobcat or Bobcat++ ?14:52
crohmannOr would this need further discussion / a resubmission of the spec?14:52
whoami-rajatresubmitting the spec should be a good start14:53
crohmannMay I simply do that (because I believe allowing for multiple backends is a good idea) or do you want to try and reach Ivan?14:53
zaitcevDo you guys have a link to that spec for "multi-backend"? The title sounds dubious because we certainly have many back-ends today.14:54
zaitcevMaybe needs renaming for clarity.14:54
zaitcevOh14:54
zaitcevOoooh14:54
whoami-rajatcrohmann, i think you can pursue it, I haven't seen Ivan around for last 2 cycles14:54
zaitcevHe probably means many back-ends simultaneously on the same Cinder backup service.14:54
whoami-rajatzaitcev, it's for backups, https://review.opendev.org/c/openstack/cinder-specs/+/71230114:54
crohmannzaitcev: Link is in the otherpad14:55
whoami-rajatcrohmann, the other topic you've added seems relevant for the glance team, "Glance support for multiple checksum types" or maybe i missed something there and we can discuss it?14:57
crohmannHaving this is not only about having some sort of a migration path ... but actually an invitation to "try" other backends or to offer e.g. "local backup" and "offsite" backups ..14:57
crohmannwhoami-rajat: Yes. Sorry - that should have gone to the glance list of topics.14:57
whoami-rajatcool14:57
whoami-rajatlet's quickly do open discussion to discuss Tony's topic14:58
whoami-rajat#topic open discussion14:58
whoami-rajatTony_Saad, do we have a fix proposed for that bug?14:58
Tony_Saadnot yet14:58
rosmaitaeharney left some suggestions in the bug14:59
whoami-rajatack, so I think it's upto the Dell team to propose a fix first then the cinder team can actively review->merge->release it14:59
Tony_Saadis it possible to do this before bobcat?15:00
whoami-rajatreading eharney's comment, I think geguileo already proposed a patch for it and it merged last cycle15:00
whoami-rajat#link https://review.opendev.org/c/openstack/os-brick/+/87183515:01
whoami-rajatok we're out of time, let's discuss this later in #openstack-cinder15:01
whoami-rajatthanks everyone for joining!15:01
whoami-rajat#endmeeting15:01
opendevmeetMeeting ended Wed Apr  5 15:01:35 2023 UTC.  Information about MeetBot at http://wiki.debian.org/MeetBot . (v 0.1.4)15:01
opendevmeetMinutes:        https://meetings.opendev.org/meetings/cinder/2023/cinder.2023-04-05-14.00.html15:01
opendevmeetMinutes (text): https://meetings.opendev.org/meetings/cinder/2023/cinder.2023-04-05-14.00.txt15:01
opendevmeetLog:            https://meetings.opendev.org/meetings/cinder/2023/cinder.2023-04-05-14.00.log.html15:01
Tony_Saadill look into this thanks!15:01
*** chuanm3 is now known as chuanm17:32
*** chuanm5 is now known as chuanm18:00
*** chuanm8 is now known as chuanm19:05
*** chuanm2 is now known as chuanm19:27
*** chuanm9 is now known as chuanm22:23

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