Thursday, 2021-01-21

« Wednesday, 2021-01-20 Index Friday, 2021-01-22 »
*** hoonetorg has joined #openstack-cinder00:01
*** sfernand has quit IRC00:10
*** sapd1 has joined #openstack-cinder00:26
*** benfelin has quit IRC00:36
*** caiqilong has joined #openstack-cinder00:55
*** benfelin has joined #openstack-cinder01:02
*** sapd1 has quit IRC01:11
*** knomura has joined #openstack-cinder01:17
*** knomura has quit IRC01:22
*** dave-mccowan has quit IRC01:24
caiqilongrosmaita: Hi, I have added newly patchset fixed the issues smcginnis proposed. If you have time, please review at: "https://review.opendev.org/c/openstack/cinder/+/767290"01:30
*** baojg has joined #openstack-cinder01:36
*** hamalq has quit IRC01:40
*** enriquetaso has quit IRC01:56
*** martinkennelly has quit IRC02:08
*** benfelin has quit IRC02:11
*** lbragstad_ is now known as lbragstad02:15
*** GirishChilukuri has joined #openstack-cinder02:25
caiqilongWhen I run this test: "cinder_tempest_plugin.scenario.test_volume_encrypted.TestEncryptedCinderVolumes.test_boot_cloned_encrypted_volume".02:59
caiqilongThere is an error when running cmd: "cryptsetup isLuks --verbose /dev/sdd".02:59
caiqilongThe error message was: "'Command failed with code -1 (wrong or missing parameters).\n'"02:59
caiqilongIs there any idea?02:59
*** jmlowe has joined #openstack-cinder03:36
*** zenkuro has quit IRC04:01
*** ajitha has joined #openstack-cinder04:24
*** manoj_kumar_kata has quit IRC04:58
*** m75abrams has joined #openstack-cinder05:10
*** udesale has joined #openstack-cinder05:22
openstackgerrithaobing1 proposed openstack/cinder master: Delete quotas when deleting volume type  https://review.opendev.org/c/openstack/cinder/+/76859205:28
*** zzzeek has quit IRC05:41
*** zzzeek has joined #openstack-cinder05:42
*** whoami-rajat__ has joined #openstack-cinder05:56
*** vishalmanchanda has joined #openstack-cinder05:58
openstackgerrithaobing1 proposed openstack/cinder master: Delete quotas when deleting volume type  https://review.opendev.org/c/openstack/cinder/+/76859206:07
*** rcernin has quit IRC06:07
*** GirishChilukuri has quit IRC06:28
openstackgerritRajat Dhasmana proposed openstack/cinder-tempest-plugin master: Add Snapshot data integrity test  https://review.opendev.org/c/openstack/cinder-tempest-plugin/+/70249506:55
openstackgerritRajat Dhasmana proposed openstack/cinder-tempest-plugin master: Add Snapshot data integrity test  https://review.opendev.org/c/openstack/cinder-tempest-plugin/+/70249507:02
openstackgerritRajat Dhasmana proposed openstack/cinder-tempest-plugin master: Add Snapshot data integrity test  https://review.opendev.org/c/openstack/cinder-tempest-plugin/+/70249507:04
*** lpetrut has joined #openstack-cinder07:10
*** tkajinam_ has joined #openstack-cinder07:19
*** tkajinam has quit IRC07:20
*** GirishChilukuri has joined #openstack-cinder07:55
*** e0ne has joined #openstack-cinder07:58
*** ociuhandu has joined #openstack-cinder08:00
*** GirishChilukuri has quit IRC08:00
*** ociuhandu has quit IRC08:05
*** rpittau|afk is now known as rpittau08:11
*** tosky has joined #openstack-cinder08:16
*** GirishChilukuri has joined #openstack-cinder08:28
openstackgerritMasahiro Okuri proposed openstack/cinder master: created s3 cinder backup driver  https://review.opendev.org/c/openstack/cinder/+/74656108:34
*** mokuri has joined #openstack-cinder08:41
*** mokuri has quit IRC08:48
*** brinzhang has quit IRC08:55
*** brinzhang has joined #openstack-cinder08:55
*** brinzhang has quit IRC08:57
*** brinzhang has joined #openstack-cinder08:57
*** ociuhandu has joined #openstack-cinder08:58
*** ociuhandu has quit IRC09:04
*** ociuhandu has joined #openstack-cinder09:10
*** ociuhandu has quit IRC09:14
openstackgerritVenkata krishna Thumu proposed openstack/cinder master: [SVF] Update volume replication properties  https://review.opendev.org/c/openstack/cinder/+/77156309:26
*** martinkennelly has joined #openstack-cinder09:29
*** lpetrut_ has joined #openstack-cinder09:34
*** udesale_ has joined #openstack-cinder09:35
*** tosky has quit IRC09:36
*** tosky_ has joined #openstack-cinder09:36
*** lpetrut has quit IRC09:37
*** udesale has quit IRC09:37
*** tosky_ is now known as tosky09:43
*** GirishChilukuri has quit IRC09:54
*** zoharm has joined #openstack-cinder09:58
openstackgerritZohar Mamedov proposed openstack/cinder master: Add KIOXIA KumoScale NVMeOF driver  https://review.opendev.org/c/openstack/cinder/+/76857410:04
*** raghavendrat has joined #openstack-cinder10:09
*** ociuhandu has joined #openstack-cinder10:18
openstackgerritGirish Chilukuri proposed openstack/cinder master: [SVF]:Storwize hyperswap volume is not deleting.  https://review.opendev.org/c/openstack/cinder/+/77178310:25
*** GirishChilukuri has joined #openstack-cinder10:48
*** benfelin has joined #openstack-cinder10:49
*** dsariel has joined #openstack-cinder10:56
*** raghavendrat has quit IRC11:06
*** udesale__ has joined #openstack-cinder11:12
*** udesale_ has quit IRC11:15
*** rosmaita has quit IRC11:19
*** rcernin has joined #openstack-cinder11:23
*** e0ne has quit IRC11:28
*** rosmaita has joined #openstack-cinder11:32
*** baojg has quit IRC11:41
*** rcernin has quit IRC11:57
caiqilongrosmaita: Hi, I have added newly patchset fixed the issues smcginnis proposed. If you have time, please review at: "https://review.opendev.org/c/openstack/cinder/+/767290"12:10
*** ociuhandu has quit IRC12:49
*** raghavendrat has joined #openstack-cinder13:05
*** raghavendrat has left #openstack-cinder13:06
*** tosky has quit IRC13:06
*** tosky has joined #openstack-cinder13:06
*** benfelin has quit IRC13:11
*** GirishChilukuri has quit IRC13:22
*** ociuhandu has joined #openstack-cinder13:23
*** enriquetaso has joined #openstack-cinder13:37
*** knomura has joined #openstack-cinder13:44
*** sapd1 has joined #openstack-cinder13:47
*** zenkuro has joined #openstack-cinder14:03
*** raghavendrat has joined #openstack-cinder14:18
raghavendrathi core reviewers, this is regarding https://review.opendev.org/#/c/752934/14:20
raghavendratCI have passed. It has one +2. Request if someone can have look & share feedback. Thanks14:22
*** m75abrams has quit IRC14:24
caiqilongrosmaita: Hi, I have added newly patchset fixed the issues smcginnis proposed. If you have time, please review at: "https://review.opendev.org/c/openstack/cinder/+/767290"14:26
rosmaitacaiqilong: ack14:26
*** vishalmanchanda has quit IRC14:38
*** dave-mccowan has joined #openstack-cinder14:40
*** brinzhang has quit IRC15:01
*** brinzhang has joined #openstack-cinder15:01
*** rosmaita has left #openstack-cinder15:02
*** raghavendrat has left #openstack-cinder15:03
*** brinzhang has quit IRC15:04
*** brinzhang has joined #openstack-cinder15:04
*** rosmaita has joined #openstack-cinder15:06
*** masterpe has quit IRC15:19
*** irclogbot_2 has quit IRC15:21
*** irclogbot_0 has joined #openstack-cinder15:22
*** masterpe has joined #openstack-cinder15:28
*** knomura has quit IRC15:39
*** lpetrut_ has quit IRC15:42
openstackgerritLee Yarwood proposed openstack/cinder-tempest-plugin master: test_volume_encrypted: Stop creating an image for each instance  https://review.opendev.org/c/openstack/cinder-tempest-plugin/+/77183515:43
openstackgerritLee Yarwood proposed openstack/cinder-tempest-plugin master: zuul: Add nova-ceph-multistore to gate  https://review.opendev.org/c/openstack/cinder-tempest-plugin/+/77183615:43
*** rosmaita has quit IRC15:52
*** rosmaita has joined #openstack-cinder15:54
*** GirishChilukuri has joined #openstack-cinder15:54
hemnarosmaita: CI is passing - https://review.opendev.org/c/openstack/cinder/+/66710816:04
rosmaitahemna: excellent!16:04
jungleboyjWoohoo!16:06
hemna:)16:16
caiqilongcongratulations, :)16:18
*** udesale__ has quit IRC16:24
*** GirishChilukuri has quit IRC16:25
*** e0ne has joined #openstack-cinder16:29
*** lpetrut has joined #openstack-cinder16:42
*** ociuhandu_ has joined #openstack-cinder16:49
*** lpetrut has quit IRC16:50
*** ociuhandu has quit IRC16:53
*** ociuhandu_ has quit IRC16:54
*** dave-mccowan has quit IRC17:31
*** manoj_kumar_kata has joined #openstack-cinder17:31
openstackgerritHelen Walsh proposed openstack/cinder master: PowerMax Driver - Offline r1 promotion fixes  https://review.opendev.org/c/openstack/cinder/+/75229517:32
*** hamalq has joined #openstack-cinder17:33
*** lpetrut has joined #openstack-cinder17:40
*** rpittau is now known as rpittau|afk17:42
lpetrutrosmaita: hi, good news: we've started the os-brick Windows Ceph job: https://review.opendev.org/c/openstack/os-brick/+/71840317:42
lpetrutos-brick-ceph http://cloudbase-ci.com/os-brick/718403/17/ceph/ : SUCCESS in 1h 02m 03s17:42
*** knomura has joined #openstack-cinder17:51
lpetrutit would be great if we could have another round of reviews17:51
*** rcernin has joined #openstack-cinder17:54
*** lpetrut has quit IRC17:55
*** knomura has quit IRC17:56
*** rcernin has quit IRC17:59
*** e0ne has quit IRC18:33
openstackgerritSimon Dodsley proposed openstack/cinder master: Pure Storage: Add volume prefix option and check lengths  https://review.opendev.org/c/openstack/cinder/+/76940518:39
openstackgerritSimon Dodsley proposed openstack/cinder master: Pure Storage: Add volume prefix option and check lengths  https://review.opendev.org/c/openstack/cinder/+/76940518:39
openstackgerritHelen Walsh proposed openstack/cinder master: PowerMax Driver - RDF checks  https://review.opendev.org/c/openstack/cinder/+/76810518:46
openstackgerritSofia Enriquez proposed openstack/cinder master: Log information about the Ceph v2 clone API  https://review.opendev.org/c/openstack/cinder/+/75484918:52
hemnalpetrut nice19:00
*** sapd1 has quit IRC19:02
openstackgerritLance Bragstad proposed openstack/cinder master: Introduce basic plumbing for protection tests with ddt  https://review.opendev.org/c/openstack/cinder/+/77187519:05
openstackgerritWalt proposed openstack/cinder master: [vmware] validate storage profile at startup  https://review.opendev.org/c/openstack/cinder/+/72618019:05
openstackgerritWalt proposed openstack/cinder stable/train: API: os-reset_status notification fix  https://review.opendev.org/c/openstack/cinder/+/71542119:06
*** sapd1 has joined #openstack-cinder19:06
openstackgerritLance Bragstad proposed openstack/cinder master: Introduce basic plumbing for protection tests with ddt  https://review.opendev.org/c/openstack/cinder/+/77187519:06
openstackgerritLance Bragstad proposed openstack/cinder master: Introduce basic plumbing for protection tests with ddt  https://review.opendev.org/c/openstack/cinder/+/77187519:07
*** MrClayPole has quit IRC19:27
*** MrClayPole has joined #openstack-cinder19:28
*** martinkennelly has quit IRC19:38
openstackgerritFernando Ferraz Silva proposed openstack/cinder stable/queens: NetApp SolidFire: Fix duplicate volume when API response is lost  https://review.opendev.org/c/openstack/cinder/+/76427819:43
*** dave-mccowan has joined #openstack-cinder19:51
*** zoharm has quit IRC20:00
*** manoj_kumar_kata has quit IRC20:12
*** enriquetaso has quit IRC20:14
*** irclogbot_0 has quit IRC20:14
*** enriquetaso has joined #openstack-cinder20:14
*** tosky has quit IRC20:14
*** tosky has joined #openstack-cinder20:15
*** irclogbot_1 has joined #openstack-cinder20:16
*** caiqilong has quit IRC20:52
*** ajitha has quit IRC20:56
*** whoami-rajat__ has quit IRC21:04
lbragstado/ hey folks - curious if anyone is around to discuss some potential changes necessary for testing the RBAC changes proposed to cinder21:12
lbragstadi'm attempting to write protection tests that should make it easier to review the policy changes, but i noticed the NoAuthMiddleware is used by default in the tests (which seems completely fine)21:24
lbragstadNoAuthMiddleware also appears to have an opinion about what a context object should look like21:25
lbragstadso 1.) i think using NoAuthMiddleware is going to have an impact on how we write protection tests for cinder (if we choose to use it)21:25
lbragstad2.) if anyone is using cinder standalone - is it safe to assume the new policies are going to make NoAuthMiddleware's representation of the context object obsolete21:26
toskyI can't help too much on this as I don't know that part, but are you suggesting to drop the usage of NoAuthMiddleware?21:27
toskyand use the real middleware directly?21:28
lbragstadi don't think so - because it sounds like people use cinder in standalone, which would require some sort of middleware that doesn't care about validating tokens against keystone21:30
lbragstadi think my real question is - do we evolve NoAuthMiddleware to work with the new policies and for the new protection tests?21:31
lbragstad(which could have implications on clients using cinder deployed with NoAuthMiddleware)21:32
rosmaitai think we need to research this a bit21:35
rosmaitai think smcginnis and jungleboyj may know the most about how people use cinder in standalone mode21:35
lbragstadyeah - that makes sense21:42
*** rcernin has joined #openstack-cinder21:54
*** rcernin has quit IRC21:59
*** rcernin has joined #openstack-cinder22:38
jungleboyjI wouldn't say that I know the most.  :-)22:40
jungleboyjlbragstad:  I think you are right that standalone requires some middleware that doesn't require keystone.22:42
jungleboyjSo, we still need something that will work in that way.  Not sure what changes you are referring to as far as policies are concerned.22:43
lbragstadjungleboyj https://review.opendev.org/q/project:openstack/cinder+topic:secure-rbac22:43
lbragstadthose are the proposed policy changes22:43
jungleboyjhemna: ^^ I think you worked on setting this up originally.  Do you have input?22:45
lbragstadgranted - these changes are deprecating the old policies and replacing them with new policies22:45
lbragstadso - i imagine the obsolete context objects from NoAuthMiddleware will continue to work for some period of time22:46
lbragstadand i'm saying that because i'm assuming people who are using noauth strategies aren't overriding their policies with custom overrides22:47
jungleboyjI don't think that people who are doing standalone are doing much with policies.  Probably just mostly trying to get things to work without more trouble than necessary.22:53
lbragstadsure22:53
lbragstadhttp://paste.openstack.org/show/801850/ is another option (it's a similar technique that ironic uses)22:53
lbragstadonly call enforcement with the proper inputs if keystone is actually the auth_strategy22:54
lbragstadotherwise just return True22:54
lbragstadin which case, it might not really matter what NoAuthMiddleware does or how it builds a context object (unless other parts of cinder use it in ways i'm not aware of)22:54
*** rcernin has quit IRC22:56
*** rcernin has joined #openstack-cinder22:57
openstackgerritSimon Dodsley proposed openstack/cinder master: Pure Storage: Add volume prefix option and check lengths  https://review.opendev.org/c/openstack/cinder/+/76940522:57
*** brinzhang has quit IRC23:00
jungleboyjlbragstad: Ok.  Would you be able to bring this up in our next meeting?  I would like to get input from geguileo hemna e0ne as they might have stronger opinions on the way forward.23:01
lbragstadsounds good - next wednesday at 8 CST?23:02
*** knomura has joined #openstack-cinder23:04
*** knomura has quit IRC23:08
jungleboyjlbragstad: Yes!23:12
lbragstadsounds good23:13
openstackgerritLance Bragstad proposed openstack/cinder master: Introduce basic plumbing for protection tests with ddt  https://review.opendev.org/c/openstack/cinder/+/77187523:19
openstackgerritLance Bragstad proposed openstack/cinder master: Introduce basic plumbing for protection tests with ddt  https://review.opendev.org/c/openstack/cinder/+/77187523:20
lbragstadrosmaita ^ that works for me locally - it takes a different approach to middleware for now23:21
*** ociuhandu has joined #openstack-cinder23:24
*** ociuhandu has quit IRC23:29
*** brinzhang has joined #openstack-cinder23:34
*** lemko has quit IRC23:51
*** lemko6 has joined #openstack-cinder23:51
« Wednesday, 2021-01-20 Index Friday, 2021-01-22 »

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