Wednesday, 2020-05-27

« Tuesday, 2020-05-26 Index Thursday, 2020-05-28 »
*** dave-mccowan has quit IRC00:19
*** dave-mccowan has joined #openstack-cinder00:20
*** brinzhang has joined #openstack-cinder00:21
*** carloss has quit IRC00:27
*** brinzhang has quit IRC00:30
*** dave-mccowan has quit IRC00:30
*** Liang__ has joined #openstack-cinder00:45
*** dave-mccowan has joined #openstack-cinder01:22
*** rosmaita has quit IRC01:40
*** rosmaita has joined #openstack-cinder01:53
*** sapd1 has joined #openstack-cinder02:21
*** andrebeltrami has quit IRC02:27
*** mvkr has quit IRC02:27
*** mvkr has joined #openstack-cinder02:45
*** rcernin has quit IRC02:47
*** rcernin has joined #openstack-cinder02:49
*** mvkr has quit IRC02:55
*** mvkr has joined #openstack-cinder02:55
openstackgerrithjy proposed openstack/cinder master: Change the original drive path  https://review.opendev.org/73106403:08
*** psachin has joined #openstack-cinder03:29
openstackgerritAlan Bishop proposed openstack/cinder stable/train: 3PAR - Fix renaming volume after migration  https://review.opendev.org/73107003:59
*** Xuchu has quit IRC04:11
*** Lucas_Gray has joined #openstack-cinder04:21
*** dave-mccowan has quit IRC04:23
*** vishalmanchanda has joined #openstack-cinder05:06
*** psachin has quit IRC05:16
*** Xuchu has joined #openstack-cinder05:20
*** Xuchu has quit IRC05:22
*** knomura has joined #openstack-cinder05:32
*** ianychoi has quit IRC05:34
*** knomura has quit IRC05:38
*** udesale has joined #openstack-cinder05:40
*** ykarel|away is now known as ykarel05:44
*** dpawlik has joined #openstack-cinder06:05
*** dpawlik has quit IRC06:12
*** dpawlik has joined #openstack-cinder06:12
*** Lucas_Gray has quit IRC06:28
*** jawad_axd has quit IRC06:41
*** jawad_axd has joined #openstack-cinder06:42
*** ociuhandu has joined #openstack-cinder06:47
*** ociuhandu has quit IRC07:06
*** Liang__ is now known as LiangFang07:26
*** tosky has joined #openstack-cinder07:26
*** rcernin has quit IRC07:35
*** baba-tunde has joined #openstack-cinder07:38
*** baba-tunde has quit IRC07:42
*** baba-tunde has joined #openstack-cinder07:42
*** knomura has joined #openstack-cinder07:50
*** spatel has joined #openstack-cinder07:52
openstackgerritwangyu proposed openstack/cinder master: Fix TypeError when downloading an image  https://review.opendev.org/73110107:53
*** knomura has quit IRC07:54
*** spatel has quit IRC07:57
*** ociuhandu has joined #openstack-cinder08:07
*** martinkennelly has joined #openstack-cinder08:11
*** TheJulia_ has joined #openstack-cinder08:12
*** mnaser_ has joined #openstack-cinder08:13
*** TheJulia has quit IRC08:20
*** brtknr has quit IRC08:20
*** mnaser has quit IRC08:20
*** bcm has quit IRC08:20
*** melwitt has quit IRC08:20
*** TheJulia_ is now known as TheJulia08:20
*** mnaser_ is now known as mnaser08:20
*** brinzhang has joined #openstack-cinder08:21
*** zenkuro has quit IRC08:23
*** bcm has joined #openstack-cinder08:26
*** tkajinam has quit IRC08:27
*** brtknr has joined #openstack-cinder08:29
*** brinzhang has quit IRC08:30
*** rcernin has joined #openstack-cinder08:42
*** rcernin has quit IRC08:42
*** ociuhandu has quit IRC08:51
*** ociuhandu has joined #openstack-cinder08:59
openstackgerritIvan Pchelintsev proposed openstack/cinder stable/train: Add support for VxFlex OS 3.5 to VxFlex OS driver  https://review.opendev.org/72382509:12
openstackgerritIvan Pchelintsev proposed openstack/cinder stable/train: Add support for VxFlex OS 3.5 to VxFlex OS driver  https://review.opendev.org/72382509:19
openstackgerritIvan Pchelintsev proposed openstack/cinder stable/train: Add OpenStack volume replication v2.1 support in VxFlex OS driver  https://review.opendev.org/72382609:20
openstackgerritIvan Pchelintsev proposed openstack/cinder stable/train: Add support for volume migration in VxFlex OS driver  https://review.opendev.org/72382709:20
openstackgerritIvan Pchelintsev proposed openstack/cinder stable/train: Add revert to snapshot support in VxFlex OS driver  https://review.opendev.org/72382809:20
*** brinzhang has joined #openstack-cinder09:32
brinzhang#openstack-cyborg09:32
brinzhang#join #openstack-qa09:33
*** ykarel is now known as ykarel|lunch09:35
*** threestrands has quit IRC09:39
openstackgerrithjy proposed openstack/cinder master: Change the original drive path  https://review.opendev.org/73106409:41
*** ociuhandu has quit IRC09:47
*** ociuhandu has joined #openstack-cinder09:48
*** ociuhandu has quit IRC09:49
*** ociuhandu has joined #openstack-cinder09:49
*** LiangFang has quit IRC09:50
*** xinliang has joined #openstack-cinder09:55
*** Xuchu has joined #openstack-cinder09:58
*** brinzhang has quit IRC10:01
*** lpetrut has joined #openstack-cinder10:02
*** xinliang has quit IRC10:29
*** udesale_ has joined #openstack-cinder10:33
*** udesale has quit IRC10:36
*** ykarel|lunch is now known as ykarel10:36
openstackgerritwangyu proposed openstack/cinder master: Use 'visibility' instead of 'is_public' when only use Glance API v2  https://review.opendev.org/73017810:49
*** knomura has joined #openstack-cinder10:50
*** knomura has quit IRC10:55
*** sapd1 has quit IRC11:01
*** e0ne has joined #openstack-cinder11:03
*** lvbin02 has joined #openstack-cinder11:08
*** knomura has joined #openstack-cinder11:09
*** lvbin01 has quit IRC11:10
*** lvbin02 is now known as lvbin0111:10
*** spatel has joined #openstack-cinder11:29
openstackgerritGorka Eguileor proposed openstack/cinder master: NetApp: Support custom igroups  https://review.opendev.org/73093611:31
openstackgerritGorka Eguileor proposed openstack/cinder master: NetApp: Fix terminate_connection on unmapped vol  https://review.opendev.org/73116711:31
*** spatel has quit IRC11:34
*** carloss has joined #openstack-cinder11:41
*** ociuhandu has quit IRC11:46
openstackgerritMerged openstack/cinder-specs master: Re-target image encryption spec to Victoria  https://review.opendev.org/72957411:54
openstackgerritMerged openstack/cinder-specs master: Move volume local cache spec to V release  https://review.opendev.org/72947311:56
*** martinkennelly has quit IRC11:58
*** ykarel is now known as ykarel|afk11:59
*** baba-tunde has quit IRC12:03
*** baba-tunde has joined #openstack-cinder12:03
*** baba-tunde has quit IRC12:03
*** jv_ has quit IRC12:04
*** baba-tunde has joined #openstack-cinder12:08
*** elod_ has joined #openstack-cinder12:08
*** jv_ has joined #openstack-cinder12:08
*** elod has quit IRC12:09
*** baba-tunde has quit IRC12:11
*** elod_ is now known as elod12:12
*** baba-tunde has joined #openstack-cinder12:13
*** baba-tunde has quit IRC12:13
*** baba-tunde has joined #openstack-cinder12:14
openstackgerritMerged openstack/cinder-specs master: Fix pygments style  https://review.opendev.org/72981112:14
*** baba-tunde has quit IRC12:15
*** baba-tunde has joined #openstack-cinder12:15
*** ociuhandu has joined #openstack-cinder12:19
*** hemna has quit IRC12:23
*** hemna has joined #openstack-cinder12:24
hemnamorning12:25
*** brinzhang has joined #openstack-cinder12:27
openstackgerritIvan Kolodyazhny proposed openstack/cinder master: Switch to newer openstackdocstheme and reno versions  https://review.opendev.org/72827212:28
*** ociuhandu has quit IRC12:30
*** benj_ has quit IRC12:36
*** andrebeltrami has joined #openstack-cinder12:39
*** enriquetaso has joined #openstack-cinder12:39
*** ykarel|afk is now known as ykarel12:41
openstackgerritMerged openstack/cinder master: Cap jsonschema 3.2.0 as the minimal version  https://review.opendev.org/73094212:47
*** sapd1 has joined #openstack-cinder12:53
openstackgerritMerged openstack/python-brick-cinderclient-ext stable/ussuri: Fix hacking min version to 3.0.1  https://review.opendev.org/72898312:57
openstackgerritBrian Rosmaita proposed openstack/os-brick stable/ocata: DNM: check status of Ocata jobs  https://review.opendev.org/73119313:02
*** knomura has quit IRC13:03
*** eharney has joined #openstack-cinder13:05
openstackgerritBrian Rosmaita proposed openstack/os-brick stable/pike: DNM: Check status of Pike jobs  https://review.opendev.org/73119613:05
*** ociuhandu has joined #openstack-cinder13:06
*** e0ne_ has joined #openstack-cinder13:14
*** e0ne has quit IRC13:15
openstackgerritMerged openstack/os-brick master: Switch to newer openstackdocstheme and reno versions  https://review.opendev.org/72999513:15
*** lpetrut has quit IRC13:18
openstackgerritMerged openstack/python-cinderclient master: Add py38 package metadata  https://review.opendev.org/73025713:18
*** ociuhandu has quit IRC13:27
openstackgerritMerged openstack/python-cinderclient stable/ussuri: Fix hacking min version to 3.0.1  https://review.opendev.org/72897913:44
*** ociuhandu has joined #openstack-cinder13:47
*** Liang__ has joined #openstack-cinder13:50
*** jmlowe has quit IRC13:57
*** martinkennelly has joined #openstack-cinder13:57
rosmaitaCourtesy reminder: Cinder meeting in #openstack-meeting-alt at 1400 UTC13:59
rosmaitajungleboyj rosmaita smcginnis tosky whoami-rajat m5z e0ne geguileo eharney walshh_ jbernard lseki sfernand rajinir ^^13:59
geguileorosmaita: thanks13:59
rosmaitanp13:59
toskyenriquetaso: I think you may want to add your name to that list, right? ^^ ?14:00
*** e0ne_ has quit IRC14:00
*** Liang__ is now known as LiangFang14:00
enriquetasotosky++ yep14:00
*** e0ne has joined #openstack-cinder14:01
enriquetasodone tosky :)14:02
*** abdysn has joined #openstack-cinder14:04
abdysnhi all14:04
abdysni was wondering, can tgtadm be used with centos8?14:05
eharneyabdysn: should use LIO instead of tgtadm14:06
abdysneharney, thanks for the fast reply, so basically, it is not supported?14:07
*** jmlowe has joined #openstack-cinder14:31
*** abdysn has quit IRC14:39
*** martinkennelly has quit IRC14:46
*** jawad_axd has quit IRC14:47
*** martinkennelly has joined #openstack-cinder14:48
*** jmlowe has quit IRC14:50
*** jmlowe has joined #openstack-cinder14:55
*** jmlowe has quit IRC15:01
eharneyso the thing with https://review.opendev.org/#/c/697335/ ...15:01
eharneyi'm pretty sure lib/glance line 192 is not really the "right" way to accomplish setting up this config15:01
eharneyi just didn't know how else to do it15:01
eharneyit at least needs a check added to ensure that you aren't pip installing glance_store15:02
eharneybecause it won't even work if that is the case15:02
eharneywhoami-rajat: enriquetaso: rosmaita: ^15:02
rosmaitaeharney: looking15:03
whoami-rajathmm15:03
*** LiangFang has quit IRC15:05
eharney"/opt/stack/glance_store" is also likely supposed to be referenced by some variable instead of hard-coded15:08
*** melwitt has joined #openstack-cinder15:08
*** rishabhhpe has joined #openstack-cinder15:09
*** e0ne has quit IRC15:09
toskyaren't there other places where that code is touched, or everything else glance_store it is just installed from pip?15:09
toskys/everything/everywhere/15:09
*** e0ne has joined #openstack-cinder15:09
*** hemna has quit IRC15:09
*** sapd1 has quit IRC15:10
eharneytosky: which code?15:10
whoami-rajateharney, but that's the path for devstack always so it should work ?15:10
*** ociuhandu has quit IRC15:10
eharneywhoami-rajat: i think the general convention is that it would be $DEST/glance_store or so15:10
*** ociuhandu has joined #openstack-cinder15:11
toskyother devstack code15:11
whoami-rajateharney,  i mean the variable will be good to have but not important right?15:11
whoami-rajateharney, ack15:11
eharneyand /opt/stack/glance_store/ won't exist for a lot of jobs15:11
rosmaitawhoami-rajat and enriquetaso: that was a whole bunch of reviews that got mentioned at the end of the meeting -- can you put together an etherpad listing the order things need to go? sounds like we want a glance_store change before we merge the main cinder nfs encryption patch15:11
*** lvbin02 has joined #openstack-cinder15:11
rosmaitait will get some focus for reviewers15:11
whoami-rajatok rosmaita15:12
eharneyi don't know if this patch needs to work when glance_store is installed from pip or not15:12
*** lvbin01 has quit IRC15:12
*** lvbin02 is now known as lvbin0115:12
*** jmlowe has joined #openstack-cinder15:13
enriquetasoi think the nfs encrypt patch is not directly related with the glance as cinder backend... I confused in the meeting and that's why i link the glance patch :P15:14
enriquetasoLet me create an etherpad15:14
clarkbeharney: you should look at how other services do it. I believe $server/etc/foo.bar is a common pattern and devstack consumes them from there in places15:15
*** ociuhandu has quit IRC15:15
enriquetasohttps://etherpad.opendev.org/p/glance_support_for_Cinder_store15:17
rosmaitaenriquetaso: thanks -- i think we may need to make sure that glance isn't using an encrypted volume_type15:17
clarkbeharney: https://opendev.org/openstack/devstack/src/branch/master/lib/neutron#L165-L190 for example15:17
eharneyclarkb: what i'm not sure about here is that i need to copy files out of glance_store, which could be pip installed (as a library) or could be in /opt/stack/glance_store/15:18
eharneysince it's a lib, i think that means that there isn't a normal "service" dir to reference15:18
*** jmlowe has quit IRC15:18
clarkbeharney: well it can also be in /some/other/path/glance_store and devstack handles this case. I think in the devstack case though you wouldn't support installing from pip15:18
clarkbeharney: ya I would treat it as a service beacuse you need to configure it like one15:19
eharneyi think it's installed from pip by default currently15:19
eharneyhmm ok15:19
clarkbreally libs shouldn't need to be configured separately like that15:19
clarkbconsuming applications should handle configuration for their libraries15:19
*** jmlowe has joined #openstack-cinder15:19
clarkbif glance_store doesn't work like that I think we've essentially created a service15:19
eharneywell, the configuration is for the service, but the configuration comes from the library15:19
clarkbright thats backwards :)15:20
*** ociuhandu has joined #openstack-cinder15:20
clarkbif you really want glance_store to be a library behind the scenes I would move the configuration up into the consuming applications15:20
eharneyi'm just a cinder guy  o:)15:21
eharneybut, yes, worth thinking about15:21
clarkband cinder is consuming glance_store right? To me that implies it is cinders responsibility as the application to sort out configuration15:21
enriquetasorosmaita, not sure what else to add to the etherpad https://etherpad.opendev.org/p/glance_support_for_Cinder_store15:22
eharneyno15:22
eharneyglance_store is consuming cinderclient and os-brick15:22
clarkbeharney: something is importing glance_store right?15:22
eharneyclarkb: yes, glance15:22
clarkbgot it, in that case it seems to me that if we want glance_store to be a library we can import and run from any installation location we'd move the configuration into the consuming application which is glance15:23
clarkband it looks like that is already the case?15:24
clarkbhttps://opendev.org/openstack/devstack/src/branch/master/lib/glance#L186 seems to start roughly there15:24
*** ociuhandu has quit IRC15:24
eharneyright, which is around where my patch is adding things15:25
clarkbwhat I would suggest is the the content you want from the glance_store repo move into the glance/etc content instead15:25
openstackgerritMerged openstack/cinder stable/queens: VMAX Driver Queens - FC live migration failure  https://review.opendev.org/72353215:25
clarkbthen you don't have to worry about where glance_store is installed from, you're just editing glance configs15:25
openstackgerritMerged openstack/cinder master: Stop invoking Python 2  https://review.opendev.org/72262215:25
openstackgerritMerged openstack/cinder master: Fix api-ref for GET snapshot response  https://review.opendev.org/72830115:25
openstackgerritMerged openstack/cinder master: Make py3 default tox target flexible  https://review.opendev.org/72693815:25
openstackgerritMerged openstack/cinder stable/rocky: [stable only] Add warning about rbd_keyring_conf  https://review.opendev.org/72816315:26
*** ykarel is now known as ykarel|away15:27
eharneyclarkb: makes sense, though i don't know what the implications of moving it might be15:27
*** e0ne_ has joined #openstack-cinder15:28
*** e0ne has quit IRC15:28
clarkbanother options which gets complicated due to python packaging is to include the configs in the sdists and wheels if they aren't already. Then extract them out using pkg resources or similar15:28
rosmaitaenriquetaso: sorry i was unclear, what i was looking for was: make your NFS encryption patch the focus, and list what needs to merge first (for example, glance_store not allowing an encrypted volume to be used) and what would follow (the devstack and other patches)15:30
*** hemna has joined #openstack-cinder15:32
rishabhhpeHi .. can anyone please tell me how to invoke consistency group test case i had already enabled it in tempest.conf and also added extra spec for that but still tempest is not picking CG cases15:33
*** e0ne_ has quit IRC15:45
enriquetasoeharney, just to double check because I'm a bit confused now:  Do we need Glance support for Cinder store before the NFS encryption patch ? I thought the patches were independent15:45
*** e0ne_ has joined #openstack-cinder15:46
*** hemna has quit IRC15:48
enriquetasorosmaita, as far as I understand the etherpard should be something like this https://etherpad.opendev.org/p/nfs_encryption_volume_support15:51
openstackgerritMartin Kennelly proposed openstack/cinder master: Mark RSD driver as unsupported  https://review.opendev.org/71151215:51
rosmaitaenriquetaso: ty, i'll take a look15:51
*** rishabhhpe has quit IRC16:02
*** e0ne_ has quit IRC16:03
*** e0ne has joined #openstack-cinder16:03
*** hemna has joined #openstack-cinder16:04
*** e0ne_ has joined #openstack-cinder16:08
*** e0ne has quit IRC16:08
openstackgerritGorka Eguileor proposed openstack/cinder master: Fix leave volume mapped on attach failure  https://review.opendev.org/73126016:09
openstackgerritGorka Eguileor proposed openstack/cinder master: Fix leftover devices when copying data  https://review.opendev.org/73126116:09
*** sapd1 has joined #openstack-cinder16:17
*** hemna has quit IRC16:24
eharneyenriquetaso: we need a patch in glance_store to just fail when encrypted volumes show up -- should be a pretty small change16:26
*** hemna has joined #openstack-cinder16:26
*** udesale_ has quit IRC16:32
*** enriquetaso has quit IRC16:42
*** e0ne_ has quit IRC16:45
*** ociuhandu has joined #openstack-cinder16:49
openstackgerritArthur Nascimento Santos proposed openstack/cinder master: NetApp ONTAP: Fix extend volume for iSCSI/FCP  https://review.opendev.org/72978216:51
*** ociuhandu has quit IRC16:54
*** e0ne has joined #openstack-cinder16:56
openstackgerritMerged openstack/cinder-tempest-plugin master: Add a basic CentOS 8 job (LVM/lio/barbican)  https://review.opendev.org/68504817:14
*** enriquetaso has joined #openstack-cinder17:17
*** ianychoi has joined #openstack-cinder17:27
*** dpawlik has quit IRC17:59
openstackgerritMerged openstack/cinder master: Move macrosan unit tests to driver directory  https://review.opendev.org/72600018:08
*** e0ne has quit IRC18:18
*** lvbin02 has joined #openstack-cinder18:19
*** lvbin01 has quit IRC18:21
*** lvbin02 is now known as lvbin0118:21
*** baba-tunde has quit IRC18:36
*** sapd1 has quit IRC18:43
openstackgerritBrian Rosmaita proposed openstack/cinder stable/ocata: Cap stestr  https://review.opendev.org/73129718:50
openstackgerritBrian Rosmaita proposed openstack/cinder stable/ocata: Cap stestr  https://review.opendev.org/73129718:53
*** lvbin02 has joined #openstack-cinder18:53
*** lvbin01 has quit IRC18:54
*** lvbin02 is now known as lvbin0118:54
rosmaitasmcginnis: if you have a minute, can you look at ^^ ?  not sure about the ordering in test-requirements.txt18:54
rosmaitaor anyone else who knows about the ordering of requirements files ^^18:55
*** ociuhandu has joined #openstack-cinder19:27
*** spatel has joined #openstack-cinder19:29
*** jawad_axd has joined #openstack-cinder19:33
rosmaitasmcginnis: nm, that doesn't work19:34
*** e0ne has joined #openstack-cinder19:38
*** ociuhandu has quit IRC19:39
smcginnisrosmaita: I think we need to get https://review.opendev.org/726983 to land yet.19:39
smcginnisrosmaita: Then dirk's patch should be able to land: https://review.opendev.org/#/c/725213/319:39
smcginnisrosmaita: Or https://review.opendev.org/#/c/718202/19:40
smcginnisrosmaita: Or now yours https://review.opendev.org/#/c/731312/119:40
rosmaitaguess there's a theme19:40
*** Lucas_Gray has joined #openstack-cinder19:42
rosmaitasmcginnis: looking through the comments on https://review.opendev.org/#/c/718202/, looks like this may never get resolved19:48
toskyshould I simply +w this one, or would anyone prefer to kick https://review.opendev.org/#/c/725734/ ?19:48
rosmaitatosky: go ahead19:50
openstackgerritLuigi Toscano proposed openstack/cinder-tempest-plugin master: DNM test the newer ceph  https://review.opendev.org/72976620:10
rosmaitasmcginnis: i think you may need to drop the legacy-tempest-dsvm-neutron-full on https://review.opendev.org/#/c/726983/, or it's never going to pass20:25
smcginnisEither drop or include the stestr cap as part of it.20:26
smcginnisHoping for the latter, but we shall see.20:26
rosmaitaok, let's see what happens20:27
*** jawad_axd has quit IRC20:47
*** gnufied has joined #openstack-cinder20:50
*** e0ne has quit IRC20:55
*** ociuhandu has joined #openstack-cinder21:04
*** enriquetaso has quit IRC21:09
*** spatel has quit IRC21:10
*** ociuhandu has quit IRC21:11
*** spatel has joined #openstack-cinder21:26
*** eharney has quit IRC21:40
openstackgerritMerged openstack/cinder-tempest-plugin master: Add stable branch jobs on the plugins master gate  https://review.opendev.org/72573421:57
*** spatel has quit IRC22:00
*** vishalmanchanda has quit IRC22:03
*** martinkennelly has quit IRC22:28
*** rosmaita has left #openstack-cinder22:46
*** tkajinam has joined #openstack-cinder22:52
*** rcernin has joined #openstack-cinder22:59
*** spatel has joined #openstack-cinder23:08
*** spatel has quit IRC23:13
*** Lucas_Gray has quit IRC23:17
*** tosky has quit IRC23:30
« Tuesday, 2020-05-26 Index Thursday, 2020-05-28 »

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