Wednesday, 2021-06-16

*** elodilles is now known as elodilles_afk08:10
opendevreviewMerged openstack/cinder master: image_utils: Remove unused _validate_file_format method
opendevreviewMerged openstack/cinderlib master: Remove reliance on embedded imports
opendevreviewHelen Walsh proposed openstack/cinder master: PowerMax Driver - Improve error handling around deletes
*** geguileo is now known as Guest237910:21
*** bhagyashris_ is now known as bhagyashris10:27
*** Guest2379 is now known as geguileo10:27
opendevreviewHelen Walsh proposed openstack/cinder master: PowerMax Driver - Improve error handling around deletes
opendevreviewEric Harney proposed openstack/cinder master: mypy: image_utils
*** elodilles_afk is now known as elodilles13:04
opendevreviewEric Harney proposed openstack/cinder-specs master: Snapshot attached volumes w/o force flag
opendevreviewEric Harney proposed openstack/cinder master: mypy: create_volume flows
opendevreviewSean McGinnis proposed openstack/cinder master: Address new hacking enforcement
opendevreviewEric Harney proposed openstack/os-brick master: encryptors: Remove workaround for bug #1633518
opendevreviewEric Harney proposed openstack/cinder master: Update decode_cipher doc text about encryption ciphers
rosmaitaCourtesy reminder: Cinder meeting in #openstack-meeting-alt at 1400 UTC13:58
rosmaitajungleboyj rosmaita smcginnis tosky whoami-rajat m5z e0ne geguileo eharney walshh_ jbernard lseki sfernand rajinir enriquetaso hemna ^^13:58
enriquetasothanks rosmaita 13:59
opendevreviewSean McGinnis proposed openstack/cinder master: Address new hacking enforcement
opendevreviewTom Barron proposed openstack/cinder master: Rename db implementation API extra specs method
opendevreviewTom Barron proposed openstack/cinder master: Don't drop group type extra specs at DB layer
*** geguileo is now known as Guest239614:32
*** Guest2396 is now known as geguileo14:36
opendevreviewIvan Pchelintsev proposed openstack/cinder master: Fix typo in Dell EMC Unity driver documentation
whoami-rajatrosmaita: hey, i was looking at gate failures on the etherpad, 3/4 issues are not related to cinder and the one related is lvdisplay (added c-vol error trace there) which is known?14:45
rosmaitawhoami-rajat: yeah, let's keep track -- if most of them aren't cinder, that would be good to know!14:50
enriquetaso#startmeeting cinder_bs15:00
opendevmeetMeeting started Wed Jun 16 15:00:32 2021 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:00
opendevmeetThe meeting name has been set to 'cinder_bs'15:00
enriquetasoHello, welcome to the cinder bug meeting.15:00
enriquetaso#topic bug_1: 'LVM Critical CI problems?'15:01
enriquetasoFrom cinder meeting early :15:01
enriquetaso<eharney> we should probably write a new bug for retrying all of the other lvm commands that can segfault that weren't covered by 1901783, since that bug already has backports spanning a few branches:15:01
enriquetasoWhat are the topics to cover in the new bug reports?- LVM delete volumes: we have . Should I reopen the bug or should I wait? 15:01
eharneythat bug already has patches shipped across a handful of branches which are helpful, so we shouldn't reopen it15:02
eharneywe need a broader bug to cover the other various lvm calls we make that also occasionally crash15:02
rosmaitaeharney: i lost my link to your patch15:03
eharneywell i wrote but that basically isn't helpful for this particular fix15:03
enriquetasocalls like create vol - create/delete snap - create/delete backup eharney ?15:03
eharneythe hope was that calling lvdisplay w/ --readonly would avoid where it crashes, but it crashes anyway15:04
eharneyenriquetaso: everything that calls lvdisplay from cinder/brick/local_dev/ to start15:04
eharneyalso calls to lvdisplay from os_brick/local_dev/lvm.py15:04
eharneythat's 7 methods or so just for lvdisplay15:05
eharneywe know this hits calls to "lvs" and "lvdisplay", i don't know if it also will hit "vgs" and "lvcreate" etc or not15:05
eharneyi think we can implement the retry for lvs/lvdisplay and see what else shows up15:06
rosmaitaso is the theory that "lvdisplay --readonly" works, we just don't have the readonly flag in enough places?15:06
eharneymy guess was that it would help, but it didn't help15:07
eharneyi just think we should do it for reasons unrelated to this bug so the patch is still up15:07
eharney(should probably remove the Related-Bug tag)15:07
rosmaitayeah, that would help15:07
eharneywhat i mean is, i saw lvdisplay also crash w/ the --readonly flag at some point after i submitted it15:08
rosmaitai agree it makes sense to use readonly if that's all we need15:08
enriquetaso#action(enriquetaso): check if this hits vgs and lvcreate as well.15:09
eharneyso we need to replicate for lvs/lvdisplay calls15:10
eharneyand then look at the lvm code and see what else calls get_lv_info15:10
opendevreviewEric Harney proposed openstack/cinder master: LVM: Use --readonly for lvdisplay in lv_has_snapshot
enriquetasoOK, sounds like a plan15:12
enriquetasoi can do the replication if nobody is working on that already 15:13
eharneyi think nobody is15:13
rosmaitanot me!15:14
enriquetaso#action(enriquetaso): replicate for lvs/lvdisplay calls and then  look at the lvm code and see what else calls get_lv_info15:14
enriquetasomoving on...15:14
enriquetasoCinder has 3 reported bugs related to documentation, there's two I'd like to talk about very quick:15:14
eharneyalso, elastic recheck queries for code: 139 ...15:14
enriquetasoshould I track them as well? I didn't understand that part in the cinder meeting earlier 15:15
eharneywell i think a few people have spent time rediscovering this issue where lvm tools crash15:16
eharneythe premise of the elastic recheck system is that we can avoid that by just identifying long-occurring known failures like this automatically15:16
rosmaitadid melanie have a link to a query in the original bug?15:18
eharneythere are a couple of queries in there that examine syslog for crashes but i suspect we actually want to catch it from the cinder volume log15:18
eharneyit will still crash and show up in syslog with our retry fixes15:19
eharneyso using those queries would result in unrelated failures being tagged as this...15:20
enriquetasoAny other considerations for this topic?15:21
enriquetasothanks eharney !15:23
enriquetasoand Brian15:23
eharneyi think it'd be useful to know if it only happens on certain platforms, but that's less important than patching it up probably15:23
enriquetaso#topic bug_2 'Still see v2 endpoints after disabling per documentation'15:25
enriquetasoBug description: 15:25
enriquetasoThe documentation states if you perform the following "Block Storage V2 API has been deprecated. To ensure Cinder does not use the V2 API, update enable_v2_api=false and enable_v3_api=true in your cinder.conf file". However after successful deployment, executing 'openstack catalog list' sourced as overcloudrc , shows that there are still v2 cinder endpoints.15:25
* enriquetaso couldn't find where in the documentation states that so I assumed is here: [1]
enriquetasorosmaita: Currently devstack doesn't show the v2 anymore, but I've tried to disable the v3 and it still shows it. Is there any way to disable it in cinder.conf?   15:26
eharneyturning off enable_v2_api isn't going to change whether a deployment tool creates endpoints for v215:26
opendevreviewRajat Dhasmana proposed openstack/cinder master: Fix: Schema validation for attachment create API
enriquetasooh ok15:26
eharneyso i assume this isn't a cinder bug15:27
enriquetasoso there's a misunderstanding here and the bug is invalid15:27
rosmaitayes, i think so, i'll put a comment on it15:27
opendevreviewRajat Dhasmana proposed openstack/python-cinderclient master: Make instance_uuid optional in attachment create
enriquetasoLast question:15:28
enriquetaso#topic bug_3: ' Block Storage API V3 (CURRENT) in cinder - wrong URL for backup-detail'15:28
enriquetasoBug description: API URL of Import a backup is wrong (
enriquetasoURL should be fixed to /v3/{project_id}/backups/import_record15:28
enriquetaso* I was going to reproduce it now but maybe a cinder member already knows if this is correct.15:29
eharneydo we support importing to an existing backup?15:30
eharneyi think the bug is correct if you look at the URLs in cinder/tests/unit/api/contrib/test_backups.py15:31
eharneyprobably a bad copy/paste from export_record15:32
enriquetasofrom usage: cinder backup-import <backup_service> <backup_url>, cinder doesn't allow specifying an existing backup15:33
eharneymakes sense15:33
enriquetaso#action(enriquetaso): patch for 1930526 and fix bad copy/paste15:34
enriquetaso#topic Open Discussion15:34
enriquetasoany other bugs to discuss today? :)15:35
rosmaitanothing from me, thanks sofia15:36
enriquetasothen.. That's all I have for today's meeting. Thank you!15:36
enriquetasoSee you next week15:36
opendevmeetMeeting ended Wed Jun 16 15:36:39 2021 UTC.  Information about MeetBot at . (v 0.1.4)15:36
opendevmeetMinutes (text):
geguileoeharney: rosmaita the NetApp patch has +1 from Zuul again, if you have the time to review it again:
rosmaitageguileo: ack15:40
zoharmhi all! requesting reviews of "NVMe agent" spec please (it differs from the spec that was approved for Wallaby) -
eharneyrosmaita: enriquetaso: fyi, saw this related fix go by:
opendevreviewMerged openstack/cinder master: volume api: Remove unused get_snapshot_metadata_value method
opendevreviewMerged openstack/cinder master: PowerMax Driver - Fix for create group from source
*** ricolin_ is now known as ricolin17:49

Generated by 2.17.2 by Marius Gedminas - find it at!