Tuesday, 2021-05-18

« Monday, 2021-05-17 Index Wednesday, 2021-05-19 »
openstackgerritMerged openstack/devstack-plugin-ceph master: Enable testing Ceph master  https://review.opendev.org/c/openstack/devstack-plugin-ceph/+/77215700:22
*** brinzhang_ has quit IRC00:23
*** sapd1_x has quit IRC00:27
openstackgerrithaixin proposed openstack/cinder master: Fix RBD create encrypted volume from snapshot  https://review.opendev.org/c/openstack/cinder/+/78462301:02
*** enriquetaso has quit IRC01:12
openstackgerritRajat Dhasmana proposed openstack/cinder-tempest-plugin master: Add multiattach data integrity test  https://review.opendev.org/c/openstack/cinder-tempest-plugin/+/79094202:11
*** zenkuro has quit IRC03:01
*** zenkuro has joined #openstack-cinder03:02
openstackgerrithaixin proposed openstack/cinder master: Fix RBD create encrypted volume from snapshot  https://review.opendev.org/c/openstack/cinder/+/78462303:21
*** GirishChilukuri has joined #openstack-cinder03:23
*** GirishChilukuri has quit IRC03:27
*** psachin has joined #openstack-cinder03:38
*** hamalq has quit IRC04:27
*** GirishChilukuri has joined #openstack-cinder04:36
*** GirishChilukuri has quit IRC04:53
*** GirishChilukuri has joined #openstack-cinder04:54
*** udesale has joined #openstack-cinder05:25
*** vishalmanchanda has joined #openstack-cinder05:32
*** psachin has quit IRC05:47
*** psachin has joined #openstack-cinder05:51
*** jawad_axd has joined #openstack-cinder06:13
*** GirishChilukuri_ has joined #openstack-cinder06:31
*** GirishChilukuri has quit IRC06:34
*** m75abrams has joined #openstack-cinder06:59
*** GirishChilukuri_ has quit IRC07:17
*** GirishChilukuri_ has joined #openstack-cinder07:18
*** sapd1_x has joined #openstack-cinder07:24
*** rpittau|afk is now known as rpittau07:27
*** JaySee has quit IRC07:44
*** tosky has joined #openstack-cinder07:46
*** ociuhandu has joined #openstack-cinder07:49
*** GirishChilukuri_ has quit IRC08:43
*** GirishChilukuri has joined #openstack-cinder08:44
*** sapd1_x has quit IRC08:49
*** ociuhandu has quit IRC08:55
*** ociuhandu has joined #openstack-cinder08:56
*** ociuhandu has quit IRC08:57
*** ociuhandu has joined #openstack-cinder08:57
*** strigazi has joined #openstack-cinder08:59
*** tkajinam has quit IRC09:02
*** tkajinam has joined #openstack-cinder09:03
openstackgerritGirish Chilukuri proposed openstack/cinder master: [SVF]:Fix multiple lshost calls during attach.  https://review.opendev.org/c/openstack/cinder/+/77262309:23
*** sapd1_x has joined #openstack-cinder09:36
*** ociuhandu has quit IRC09:56
*** ociuhandu has joined #openstack-cinder09:57
*** ociuhandu has quit IRC09:58
*** ociuhandu has joined #openstack-cinder09:58
*** GirishChilukuri has quit IRC10:02
*** GirishChilukuri has joined #openstack-cinder10:03
*** zenkuro has quit IRC10:04
*** zenkuro has joined #openstack-cinder10:05
*** manoj_katari has joined #openstack-cinder10:22
*** GirishChilukuri has quit IRC10:23
*** GirishChilukuri has joined #openstack-cinder10:24
openstackgerritRajat Dhasmana proposed openstack/cinder-tempest-plugin master: Import from tempest stable manager  https://review.opendev.org/c/openstack/cinder-tempest-plugin/+/79094210:35
*** ociuhandu has quit IRC10:56
*** ociuhandu has joined #openstack-cinder10:57
*** ociuhandu has quit IRC11:00
*** ociuhandu has joined #openstack-cinder11:00
*** ociuhandu has quit IRC11:00
*** ociuhandu has joined #openstack-cinder11:01
*** ociuhandu has quit IRC11:02
*** ociuhandu has joined #openstack-cinder11:02
*** ociuhandu has quit IRC11:02
*** ociuhandu has joined #openstack-cinder11:03
*** whoami-rajat has joined #openstack-cinder11:06
manoj_katariHi all, while generating release notes, i see this error11:07
manoj_katariERROR: InvocationError for command could not find executable reno11:07
openstackgerritRajat Dhasmana proposed openstack/cinder-tempest-plugin master: Add multiattach tests  https://review.opendev.org/c/openstack/cinder-tempest-plugin/+/79192111:07
manoj_katarii have reno binary installed11:07
manoj_kataridid anyone face this issue ?11:08
*** ociuhandu has quit IRC11:08
*** ociuhandu has joined #openstack-cinder11:23
*** ociuhandu has quit IRC11:25
*** ociuhandu has joined #openstack-cinder11:25
*** GirishChilukuri_ has joined #openstack-cinder11:32
*** GirishChilukuri has quit IRC11:36
*** GirishChilukuri has joined #openstack-cinder11:36
*** ociuhandu has quit IRC11:36
*** ociuhandu has joined #openstack-cinder11:37
*** GirishChilukuri_ has quit IRC11:39
*** carloss has joined #openstack-cinder11:41
*** Luzi has joined #openstack-cinder11:42
*** ociuhandu has quit IRC11:42
*** ociuhandu has joined #openstack-cinder11:45
*** ociuhandu has quit IRC11:51
*** fabiooliveira has joined #openstack-cinder11:55
*** ociuhandu has joined #openstack-cinder12:04
openstackgerritGorka Eguileor proposed openstack/cinder master: LVM: Added NVMe TCP support for the nvmet target  https://review.opendev.org/c/openstack/cinder/+/79192912:05
openstackgerritRajat Dhasmana proposed openstack/cinder-tempest-plugin master: Add multiattach tests  https://review.opendev.org/c/openstack/cinder-tempest-plugin/+/79192112:09
tobias-urdinrosmaita: abishop could we have another look on https://review.opendev.org/c/openstack/cinder/+/761113 see if it's ok to get in to xena12:11
tobias-urdinlet me know if something needs changing :)12:11
*** ociuhandu has quit IRC12:27
*** GirishChilukuri has quit IRC12:27
*** sapd1_x has quit IRC12:27
*** ociuhandu has joined #openstack-cinder12:28
rosmaitatobias-urdin: ack12:28
*** GirishChilukuri has joined #openstack-cinder12:32
*** ociuhandu has quit IRC12:33
*** ociuhandu has joined #openstack-cinder12:39
*** GirishChilukuri has quit IRC12:39
*** manoj_katari has quit IRC12:40
*** GirishChilukuri has joined #openstack-cinder12:40
openstackgerritBrian Rosmaita proposed openstack/python-cinderclient master: Remove v2 support from the shell  https://review.opendev.org/c/openstack/python-cinderclient/+/79183412:42
*** enriquetaso has joined #openstack-cinder12:48
*** snapdeal has joined #openstack-cinder13:01
*** Luzi has quit IRC13:24
*** ociuhandu has quit IRC13:29
*** carloss has quit IRC13:31
*** GirishChilukuri has quit IRC13:34
*** mnasiadka has quit IRC13:35
*** mnasiadka has joined #openstack-cinder13:37
*** coreycb has quit IRC13:37
*** ociuhandu has joined #openstack-cinder13:38
*** carloss has joined #openstack-cinder13:38
*** coreycb has joined #openstack-cinder13:38
*** udesale has quit IRC13:45
*** sapd1_x has joined #openstack-cinder13:48
*** bluex has quit IRC13:52
*** bhagyashris has quit IRC13:52
*** bluex has joined #openstack-cinder13:53
*** bhagyashris has joined #openstack-cinder13:54
openstackgerritEric Harney proposed openstack/cinder master: mypy: backup  https://review.opendev.org/c/openstack/cinder/+/78719513:56
*** openstack has joined #openstack-cinder14:11
*** ChanServ sets mode: +o openstack14:11
openstackgerritkatari manoj kumar proposed openstack/cinder master: [SVF]:Fix add volumes to GMCV group  https://review.opendev.org/c/openstack/cinder/+/78409214:12
openstackgerritkatari manoj kumar proposed openstack/cinder master: [SVF]:Fix add volumes to GMCV group  https://review.opendev.org/c/openstack/cinder/+/78409214:16
*** snapdeal has quit IRC14:40
*** ricolin has quit IRC14:41
*** psachin has quit IRC14:46
*** GirishChilukuri has joined #openstack-cinder14:49
*** GirishChilukuri has quit IRC14:53
*** ociuhandu has quit IRC15:34
*** ociuhandu has joined #openstack-cinder15:40
*** ociuhandu has quit IRC15:45
*** tosky_ has joined #openstack-cinder15:49
*** tosky has quit IRC15:52
*** ociuhandu has joined #openstack-cinder15:55
*** ociuhandu has quit IRC16:00
*** m75abrams has quit IRC16:02
*** ociuhandu has joined #openstack-cinder16:11
*** ociuhandu has quit IRC16:14
*** ociuhandu has joined #openstack-cinder16:14
*** ociuhandu has quit IRC16:14
*** __ministry has joined #openstack-cinder16:17
*** __ministry has quit IRC16:17
*** ociuhandu has joined #openstack-cinder16:19
*** ociuhandu has quit IRC16:20
*** GirishChilukuri has joined #openstack-cinder16:26
*** ociuhandu has joined #openstack-cinder16:30
gmannrosmaita: if you can add reno and microversion doc update, I will +A it - https://review.opendev.org/c/openstack/tempest/+/79020116:31
gmannceph_iscsi on devstack is +A16:32
gmannnow on v2 removal things16:32
gmannrosmaita: on v2 removal, do you have patch up in cinder side or discussion ref on removal plan in Xena which we can use for removal from devstack/tempest side.16:35
gmannI still see endpoint support in https://github.com/openstack/cinder/blob/ba27068fc0cc8a70e2a2e2fcfe6ab9896dab1908/etc/cinder/api-paste.ini#L916:35
*** ociuhandu has quit IRC16:36
*** ociuhandu has joined #openstack-cinder16:42
*** rpittau is now known as rpittau|afk16:45
*** ociuhandu has quit IRC16:49
*** ociuhandu has joined #openstack-cinder16:49
*** udesale has joined #openstack-cinder16:58
rosmaitagmann: i'm working on the patch, here's the discussion of removal plan: https://wiki.openstack.org/wiki/CinderXenaPTGSummary#Removing_the_Block_Storage_API_v217:04
gmannrosmaita: thanks, I will push the Tempest patch soon which can verify things will work on top of your devstack patch.17:05
rosmaitagmann: ty17:05
*** ociuhandu has quit IRC17:14
*** udesale has quit IRC17:21
tosky_gmann, rosmaita: shouldn't volume_{min,max}_microversion be used instead of {min,max}_microversion in https://review.opendev.org/c/openstack/tempest/+/790201/2/tempest/api/volume/test_volume_transfers.py ?17:38
*** GirishChilukuri has quit IRC17:38
rosmaitatosky_: don't know, i just copied it from somewhere else17:39
rosmaitatosky: looks like all the tempest/api/volume tests use that pattern, for example, https://opendev.org/openstack/tempest/src/branch/master/tempest/api/volume/test_volumes_backup.py#L167-L16817:42
tosky_rosmaita: but see tempest/api/compute/admin/test_volumes_negative.py17:43
tosky_let me see17:43
*** ikla has quit IRC17:45
rosmaitatosky_: they may need to do that in the compute test, since they also need to set their own using min_microversion17:45
tosky_rosmaita: iirc the parameter without prefix sets it for all clients, but I may be mistaken17:46
tosky_so maybe it's better to set it correctly in any case17:46
gmannrosmaita: tosky_ yeah  {min,max}_microversion  work fine.17:46
gmannvolume_{min,max}_microversion is used in the integration test where test need to mention more than once service microversion like compute+ volume17:47
rosmaitaok, that makes sense17:47
tosky_but wouldn't it be better if we consistently used the more precise variable?17:47
tosky_because I remember at least one case of being puzzled by using the wrong one17:47
gmanncurrently  {min,max}_microversion only will work for volume API tests. but we can make it consistent with volume_{min,max}_microversion in API test too but that is separate change17:48
tosky_and things were not working because something changed also in nova behavior that shouldn't have17:48
tosky_I see17:48
gmannwe need to change volume base class logic for that as min,max was since starting and volume_min.max we did once we started integration test for microversion17:49
gmannor more place may be17:49
gmannrosmaita: btw, i think this is all we need to do in tempest for v2-removal - https://review.opendev.org/c/openstack/tempest/+/79202517:49
rosmaitagmann: nice, that's not too much17:50
gmannonce this tempest-cinder-v2-api job pass then we are good17:50
gmannyeah17:50
rosmaitagmann: i will update the commit message in my devstack patch as you suggested17:51
gmannrosmaita: +1, let's wait for Tempest patch result and then you can remove this var too - https://review.opendev.org/c/openstack/devstack/+/791842/1/lib/tempest#46817:52
rosmaitagmann: ack17:53
rosmaitagmann: i think my updating the commit message in my patch made your tempest patch fail17:55
gmannok did recheck17:56
*** hamalq has joined #openstack-cinder18:05
*** lpetrut has joined #openstack-cinder18:10
*** lpetrut has quit IRC18:10
openstackgerritHironori Shiina proposed openstack/cinder-tempest-plugin master: Add a job for S3 backup driver  https://review.opendev.org/c/openstack/cinder-tempest-plugin/+/77208518:23
openstackgerritGoutham Pacha Ravi proposed openstack/devstack-plugin-ceph master: Remove devstack-gate test hooks  https://review.opendev.org/c/openstack/devstack-plugin-ceph/+/79203719:05
*** whoami-rajat has quit IRC19:16
*** tosky_ is now known as tosky19:25
*** zenkuro has quit IRC19:58
gmannrosmaita: its working in tempest patch and we are good to remove this var from lib/tempest - https://review.opendev.org/c/openstack/devstack/+/791842/1/lib/tempest20:05
rosmaitagmann: do you want me to do that now and push a new patch set?20:06
gmannrosmaita: yeah, I can +2 after that20:06
rosmaitaok, i just scrolled down and saw your other comment about the later lines, will remove that too20:06
gmanncool20:06
*** ociuhandu has joined #openstack-cinder20:27
*** ociuhandu has quit IRC20:35
*** dave-mccowan has quit IRC22:17
*** dave-mccowan has joined #openstack-cinder22:21
*** rcernin has joined #openstack-cinder22:36
*** rcernin has quit IRC22:44
*** rcernin has joined #openstack-cinder22:45
*** fabiooliveira has quit IRC22:45
*** tosky has quit IRC23:05
*** masterpe has quit IRC23:27
*** hamalq has quit IRC23:37
*** masterpe has joined #openstack-cinder23:55
« Monday, 2021-05-17 Index Wednesday, 2021-05-19 »

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