Wednesday, 2022-03-09

*** dviroel|afk is now known as dviroel|out00:46
opendevreviewDataCore SANsymphony CI proposed openstack/cinder master: Add Datacore Software volume driver
opendevreviewDataCore SANsymphony CI proposed openstack/cinder master: Fix Datacore driver NameError
opendevreviewDataCore SANsymphony CI proposed openstack/cinder master: Fix pep8/flake8 violations in Datacore volume driver
opendevreviewDataCore SANsymphony CI proposed openstack/cinder master: Add unit test and documentation for DataCore volume driver
opendevreviewRajat Dhasmana proposed openstack/cinder-tempest-plugin master: Add multiattach tests
opendevreviewRajat Dhasmana proposed openstack/cinder-tempest-plugin master: Add multiattach tests
*** brinzhang_ is now known as brinzhang10:39
stephenfineharney: If so, we can probably just bump it even higher. Nova has OS_TEST_TIMEOUT=160 (seconds), so with TIMEOUT_SCALING_FACTOR=4 that's over 10 minutes. I could just set TIMEOUT_SCALING_FACTOR=10 and do the same11:16
stephenfineharney: That doesn't answer the question of why it's so damn slow for MySQL, but given the issue occurs in nova too, I suspect the answer is either MySQL is slow for this stuff or alembic is11:17
stephenfin(neither of which we can really fix)11:17
*** dviroel|out is now known as dviroel11:19
yuvalreviewed by me:,
yuvalcores can you please review:
rosmaitayuval: thanks for doing reviews!13:54
abishoprosmaita: do you know when cinder is scheduled to cut its first yoga release?13:54
rosmaitaabishop: tomorrow13:54
abishopwell that worked out nicely :D13:54
rosmaitadepends on what you mean by 'nicely' :D13:55
abishopan answer that I like... I'm holding a tripleo patch related to cinder endpoints until we have a release that includes the code13:56
abishopthe code that removes the need for project_id13:56
rosmaitaah, ok13:56
abishopdidn't know how long I needed to continue to hold the patch13:56
rosmaitaCourtesy reminder: Cinder meeting in #openstack-meeting-alt at 1400 UTC13:57
rosmaitajungleboyj rosmaita smcginnis tosky whoami-rajat m5z e0ne geguileo eharney walshh_ jbernard sfernand enriquetaso hemna fabiooliveira yuval ^^13:57
geguileorosmaita: thanks13:58
rosmaitait's the least i can do!13:58
*** sean-k-mooney1 is now known as sean-k-mooney14:02
*** hemna7 is now known as hemna14:28
opendevreviewGorka Eguileor proposed openstack/cinder master: Pure: Fix replication in A/A
geguileosimondodsley: ^14:38
*** hemna5 is now known as hemna14:47
dc_arunkvcore, I made a mistake by committing some logic changes as part of pep8 fix14:53
dc_arunkv. Is it ok if I fix the commit message to include those details or should I revert those logic changes. Thanks.14:53
rosmaitadc_arunkv: it would be better to split them up14:59
opendevreviewMerged openstack/cinder master: db: Increase timeout for migration tests
enriquetaso#startmeeting cinder_bs15:01
opendevmeetMeeting started Wed Mar  9 15:01:21 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:01
opendevmeetThe meeting name has been set to 'cinder_bs'15:01
enriquetasoWelcome to Cinder Bug Meeting15:01
enriquetasoCinder has 2 bugs reported this week.15:01
enriquetaso#topic [powerstore driver]: no volume caching after 35 volumes.15:02
enriquetasoThe driver needs to add a "SnapshotLimitReached" to invalidate cache on the limit. Looks valid and needs a patch. 15:02
enriquetasoMoving on.15:03
enriquetaso#topic Don't require control-location for creating encrypted volume types.15:03
enriquetasoSummary: We shouldn't require "control-location" on encrypted type creation -- just default to "front-end" if one is not supplied.15:03
enriquetasoThe full discussion and work is going on the patch.15:03
enriquetaso#link 15:03
eharneythis doesn't have a patch proposed for it15:03
eharneybut, basically, we require specifying control-location when creating an encrypted type, even though everyone always wants "front-end", so we should do that by default15:04
eharney(and probably drop support for back-end IMO but that's a whole additional project)15:04
enriquetasoOK, I've misunderstood the bp15:05
enriquetasothanks eharney for clarifying 15:06
dc_arunkvThanks rosmaita, I will do it.15:07
rosmaitathat's a widening change, so won't require a new microversion15:07
rosmaitadc_arunkv: ty15:07
rosmaitathe "widening change" was about the control-location15:07
enriquetasoif we would like to drop support for back-end, should we create a new bp or use that one eharney ?15:07
eharneythat bug is just about setting a default control-location behavior15:08
eharneyi don't know if we need a bp15:08
enriquetasosorry, not a bp, a new bug..15:08
eharneyi'm not sure if any drivers attempt to implement backend encryption any more, at one point i think one did15:08
eharneyi'll have to look around at that15:09
enriquetaso#action: drop support for back-end ? 15:09
enriquetasoMoving on..15:10
enriquetaso#topic Open Discussion15:10
rosmaitai can't believe i reviewed that patch 2 years ago15:10
enriquetasothanks for attending!15:12
enriquetasooh, i'll be on PTO, so no meeting next week!15:13
rosmaitaok, enjoy your PTO15:13
opendevmeetMeeting ended Wed Mar  9 15:14:03 2022 UTC.  Information about MeetBot at . (v 0.1.4)15:14
opendevmeetMinutes (text):
*** dviroel is now known as dviroel|lunch15:14
opendevreviewMerged openstack/cinder master: fix 'huawei dorado v6' FC initiator can be added to host
opendevreviewMerged openstack/cinder master: RBD driver: clean up _trash_purge exception handling
opendevreviewMerged openstack/cinder master: mypy: RBD driver
*** dviroel|lunch is now known as dviroel16:42
opendevreviewAntoine Eiche proposed openstack/python-cinderclient stable/ussuri: Add user and project domain name parameters in HTTPClient
jamesbensonMorning all, I'm failing on cinder refstack tests with a `tempest.lib.exceptions.UnexpectedResponseCode: Unexpected response code received` on about 12 cinder tests (and about 9 network tests).  They all have the same error. Any idea's?16:56
ManishLaddha_Hello everyone, can you please review a minor document change:, this change is planned for Yoga from SVf driver17:01
rosmaitajamesbenson: do you have a link to any logs? or at least a list of the failing tests17:05
jamesbensonlet me know if this works:
jamesbensonsorry for the multiline link there... gitlab is where we do our testing17:09
rosmaitaok, link works17:11
jamesbensonFWIW: tempest.api.compute.servers.test_multiple_create.MultipleCreateTestJSON always fails when run with other tests for us, so that one can be ignored.17:13
opendevreviewEric Harney proposed openstack/cinder master: mypy: annotate remotefs
rosmaitajamesbenson: at a quick glance, looks like the unexpected code you are getting is a 504 (Gateway Timeout) ... most of the failures are during setup and teardown, so i suspect some kind of networking problem17:19
jamesbenson@rosmaita: hmm, okay.  These hosts are all virsh VM's on the same host, so I feel like it shouldn't be network.  But I definitely don't claim to be a virsh expert.  Not sure if you know more about that. Could it be something with TLS not working properly?  We have TLS enabled on internal/external/backend.17:24
jamesbensonrosmaita: We have been running these tests for a while and only relatively recently have we been having these errors.17:24
rosmaitajamesbenson: you're going to have to dig into the logs a bit to see where the timeout is happening 17:27
jamesbensonalrighty, thanks :-) 17:29
rosmaitanot much help, i know, but good luck!17:29
jamesbensonOne other piece of info.17:32
jamesbensonThose failures are on a 3-node Xena deploy.  We do nightly tests also on an all-in-one Wallaby, those seem to always pass.17:32
gmannwhoami-rajat: rosmaita can you please check this patch, adding irrelevant-files for newly added grenade-skip-level job
whoami-rajatgmann, I'm unable to find much info regarding what this job does (even in the dependent patch)17:43
gmannwhoami-rajat: ah sorry, this is patch added the job definition and depends-on added to run for cinder gate
gmannso it already run on cinder gate as non voting and my patch 831438 just add the irrelevant-files so that it would not run on doc/test only changes17:45
whoami-rajatack, so it is testing kind of a fast forward upgrade skipping one release in between17:46
gmannyeah but not exactly the FFU -
opendevreviewEric Harney proposed openstack/cinder master: mypy: set no_implicit_optional
opendevreviewEric Harney proposed openstack/cinder master: mypy:
opendevreviewEric Harney proposed openstack/cinder master: mypy: ceph backup driver
opendevreviewMerged openstack/cinder master: db: Preserve API signatures
*** hemna6 is now known as hemna20:29
opendevreviewBrian Rosmaita proposed openstack/cinder master: Use functools with require_context wrappers
opendevreviewBrian Rosmaita proposed openstack/cinder master: Use functools with require_context wrappers
opendevreviewBrian Rosmaita proposed openstack/cinder master: Use functools with require_context wrappers
rosmaitayes, somebody forgot to run pep820:45
eharneyrosmaita: put some pep in your step(s)20:46
*** dviroel is now known as dviroel|out21:29
opendevreviewMerged openstack/cinder master: Hitachi: Add Target Port Assignment for VSP Driver

Generated by 2.17.3 by Marius Gedminas - find it at!