Wednesday, 2022-06-22

*** dviroel|afk is now known as dviroel00:07
opendevreviewOpenStack Proposal Bot proposed openstack/cinder master: Imported Translations from Zanata
opendevreviewBrian Rosmaita proposed openstack/cinderlib master: [docs] add tox, zuul maintenance tasks
whoami-rajathemna, thanks, will take a look04:36
whoami-rajathemna, looks like there's a line from some other spec that needs to be removed, else LGTM04:53
opendevreviewArun KV proposed openstack/cinder master: Reintroduce DataCore driver
*** vishalmanchanda_ is now known as vishalmanchanda06:12
*** masayukig_ is now known as masayukig06:12
*** carloss_ is now known as carloss06:12
*** arne_wiebalck_ is now known as arne_wiebalck06:19
opendevreviewMerged openstack/cinderlib master: Update Ceph version on CI job
opendevreviewPeter Penchev proposed openstack/cinder master: Send the correct location URI to the Glance v2 API
opendevreviewPeter Penchev proposed openstack/cinder master: StorPool: drop _attach_volume() and _detach_volume()
opendevreviewPeter Penchev proposed openstack/cinder master: StorPool: really detach after copy to/from image.
opendevreviewPeter Penchev proposed openstack/cinder master: StorPool driver: remove the obsolete backup_volume()
opendevreviewPeter Penchev proposed openstack/cinder master: StorPool: create_cloned_volume() improvements
opendevreviewPeter Penchev proposed openstack/cinder master: Add the clone_across_pools driver capability
opendevreviewPeter Penchev proposed openstack/cinder master: StorPool: declare the clone_across_pools capability
Roamer`...argh... I meant to reorder those... sorry for the upcoming noise07:59
Roamer`nah, it's fine, the bot reported them in the wrong order08:00
opendevreviewMerged openstack/cinderlib master: Use stable/yoga override for CI jobs (conclusion)
geguileowhoami-rajat: finally figured out what's going on with the cinderlib stable gate failures !!!09:43
whoami-rajatgeguileo, oh nice!09:44
whoami-rajatis it an easy or a complex fix?09:45
geguileoI have to figure out how to fix it in the gate, locally is easy to fix, just edit a system file and make a change...09:46
geguileoIn file /usr/local/bin/privsep-helper it forces usage of the system's python with "#!/usr/bin/python3.6" so the privsep running ignores the virtual environment completely and uses libraries from the host system.09:46
geguileoIf we replace it with "#!/usr/bin/env python" then it works fine09:46
whoami-rajatoh cool, so the assumption was right that privsep is not using the venv09:48
whoami-rajatmaybe tosky has idea how/if we can make this change in a gate run09:48
whoami-rajatgeguileo, but isn't that an issue for all projects that privsep is using host packages and not venv ones, and we only faced the issue in cinderlib10:08
whoami-rajatmaybe it's about the usage and our tests that catch this problem10:08
geguileowhoami-rajat: exactly, that is the problem10:08
geguileothis happens for us because we are using the same run from tempest10:09
geguileoif the system didn't have the privsep package then it would be installed in the virtualenv and it would work10:09
geguileothere is an easy fix, which is modify the tox to force the installation of the package in the virtual env10:11
geguileobut that's not a proper fix, because then outside of the tox tests cinderlib could have the same issue when working inside virtual envs10:11
geguileoI'm exploring an alternative fix10:12
whoami-rajatack, thanks for looking into this10:13
toskywhoami-rajat: no, I have no idea :/10:46
whoami-rajatoh ...11:01
opendevreviewAlexander Malashenko proposed openstack/cinder master: Cinder displays the volume size provided by the driver, when creating the volume with enabled cache.
opendevreviewFrancesco Pantano proposed openstack/devstack-plugin-ceph master: Deploy with cephadm
*** dviroel__ is now known as dviroel11:28
opendevreviewAtsushi Kawai proposed openstack/cinder master: Hitachi and OEM: Support multi pool
*** mattia is now known as Guest293912:48
*** Guest2939 is now known as mattia12:51
*** mattia is now known as blmt12:52
opendevreviewGorka Eguileor proposed openstack/cinderlib master: Fix privsep issues inside virtual env
geguileowhoami-rajat: ^ The fix13:07
whoami-rajatgeguileo, looking13:48
geguileowhoami-rajat: thanks. Let me know if something is not clear enough13:49
whoami-rajatsure, as far as i understand, we are using a custom privsep-helper which is same as original but just runs inside venv13:51
whoami-rajatgeguileo, so are we using cinderlib master in cinder stable/wallaby and this patch will fix the gate situation or we would need to backport it till stable/wallaby?13:55
opendevreviewWalt proposed openstack/cinder-specs master: Add reporting of capacity factors
whoami-rajatCinder meeting in #openstack-meeting-alt at 1400 UTC13:58
whoami-rajatjungleboyj rosmaita smcginnis tosky whoami-rajat m5z e0ne geguileo eharney walshh_ jbernard sfernand enriquetaso hemna fabiooliveira yuval tobias-urdin13:58
jungleboyjwhoami-rajat:  Getting pulled into internal meetings today.  Will join when I can.13:58
whoami-rajatsure, no worries13:59
hemnathis may be a dumb question, but is there a mechanism to throttle/queue the number of backups being done at any time?14:00
geguileohemna: yes, we have the mechanism, whoami-rajat and I worked on it14:01
whoami-rajathemna, is this what you're looking for ?14:01
whoami-rajatyep, mostly geguileo but it's there14:02
geguileohemna: 'backup_max_operations'14:03
geguileothey are per node14:04
geguileoso if you have 3 backup services it will be able to do 3 * backup_max_operations14:04
geguileothe mechanism is not great because what's happening is that they are queued on the service14:04
geguileonot on the scheduler14:05
geguileoso there is room for improvement14:05
hemnaok thank you that's at least something.  I had 107 backups kicked off on 1 deployment within 13 minutes.  crushed14:09
hemnaah that's Victoria?  I'm still on train.....backport time.14:17
geguileoyou are the backport master  ;-)14:25
geguileohemna: I'm sure you'll be interested in making the changes I'm making to devstack and triple-o14:26
geguileohemna: they reduce the peak memory usage by 50% and the fix the high watermark memory issue14:26
geguileohemna: Just passing some env variables does that
*** dviroel is now known as dviroel|lunch14:59
tosky(another reminder for which enriquetaso raised last week IIRC, it allows us to set the minimal compatibility feature set for ceph; it is already supported by the cephadm patch, we need it for the current codebase)15:00
whoami-rajatenriquetaso, bug squad meeting?15:02
enriquetasothanks tos15:04
enriquetasoHey, no bug meeting today becuase i only  have one bug 15:08
enriquetasoregarding StorPool driver15:08
hillpdenriquetaso, I wanted to discuss
enriquetasooh cool!15:10
hillpdthere was a fix proposed here:
enriquetaso#startmeeting cinder_bs15:11
opendevmeetMeeting started Wed Jun 22 15:11:22 2022 UTC and is due to finish in 60 minutes.  The chair is enriquetaso. Information about MeetBot at
opendevmeetUseful Commands: #action #agreed #help #info #idea #link #topic #startvote.15:11
opendevmeetThe meeting name has been set to 'cinder_bs'15:11
enriquetaso#link 15:11
enriquetaso#topic  It's not possible to upload a volume that was build from an image back to glance, if multistore (glance) is enabled. 15:11
enriquetasoFix proposed to master
enriquetasogo on hillpd 15:12
hillpdthe last comment was on Mar 7th, and I wanted to know what's needed to help push this fix forward?15:13
enriquetasoI think whoami-rajat may be interested in ^ since it's using cinder as glance backend 15:13
enriquetasohillpd, since the bug it's Medium importance it would be nice to have a release note for it 15:14
enriquetasoI'll review it late this week (adding it to my TO Review List)15:14
hillpdack, I'll add a draft release note15:15
whoami-rajatenriquetaso, will take a look thansk15:16
hillpdI had one other fix that I was interested in discussing:
hillpdbut I don't know if this is the right forum15:17
*** blmt is now known as mattia15:19
*** mattia is now known as blmt15:20
enriquetasommm merge conflict, i don't think the team review it with that red flag there15:20
enriquetasoplease consider to rebase it15:20
hillpdokay, I can do that15:21
enriquetasoi'll review it as well and create a launchpad bug for it since I'm not sure if we plan to migrate to Story for now15:21
enriquetasomaybe it's a good question for next cinder meeting15:22
enriquetasoOK, any other bug to discuss hillpd ?15:22
hillpdnope, those were the only two I had to raise15:22
enriquetasothanks !15:28
enriquetaso#topic StorPool: detach volumes and snapshots after copying to/from an image.15:29
enriquetaso#link 15:29
enriquetasothere's a fix proposed to master15:30
enriquetasofeel free to review it!15:30
enriquetaso#topic open discussion15:30
enriquetasofeel free to proposed any bug now :)15:31
whoami-rajathillpd, I've proposed a similar patch like the one you mentioned for OSC
whoami-rajati think someone raised this issue on the ML15:32
hillpdah, yeah that fix addresses the same issue15:33
whoami-rajatack, i added some new tests since the volume arg was not tested at all15:34
hillpdinstead of rebasing the other patch, I'll review yours15:34
whoami-rajatcool, let me know if it fixes the issue (if you're facing this as well)15:35
whoami-rajati think i was able to reproduce it on master15:35
opendevreviewJan Hartkopf proposed openstack/cinder master: Ceph: add option to keep only last n snapshots per backup
whoami-rajatgeguileo, hey, can you update the patch, it's failing on pep8
geguileodarn, I forgot to run pep8 locally15:50
* geguileo facepalms15:50
whoami-rajatit's a one line change, hopefully we can get that patch in today and update the release patch15:51
whoami-rajati am thinking of proposing a DNM to stable wallaby to test if this fixes the gate15:52
opendevreviewGorka Eguileor proposed openstack/cinderlib master: Fix privsep issues inside virtual env
geguileowhoami-rajat: I find your lack of faith disturbing15:53
geguileokidding. It's a good idea15:53
whoami-rajat:D I've heard always, more testing is never bad15:54
opendevreviewRajat Dhasmana proposed openstack/cinder stable/wallaby: DNM: Test cinderlib fix for privsep
hemna50% ram reduction in devstack!?16:02
hemnais this something that can be done outside of devstack?!16:03
opendevmeetMeeting ended Wed Jun 22 16:14:27 2022 UTC.  Information about MeetBot at . (v 0.1.4)16:14
opendevmeetMinutes (text):
enriquetasoforgot to end the meeting 16:14
* enriquetaso facepalms16:14
*** dviroel|lunch is now known as dviroel16:31
opendevreviewWalt proposed openstack/cinder-specs master: Add reporting of capacity factors
opendevreviewRafael Weingartner proposed openstack/cinder master: Filter reserved image properties
whoami-rajatgeguileo, privsep failures on the cinderlib patch :(
enriquetasoone +2 left \o/18:41
opendevreviewGorka Eguileor proposed openstack/cinder master: Log when waiting to acquire coordinator lock
*** dviroel is now known as dviroel|afk19:48
*** dviroel|afk is now known as dviroel20:43
*** dviroel is now known as dviroel|afk21:22
*** hemna6 is now known as hemna23:37

Generated by 2.17.3 by Marius Gedminas - find it at!