Friday, 2017-09-29

*** ntpttr_laptop has quit IRC00:01
*** catintheroof has quit IRC00:06
*** yangyapeng has quit IRC00:18
*** yangyapeng has joined #openstack-cinder00:18
*** yangyapeng has quit IRC00:23
*** alkhodos has joined #openstack-cinder00:26
*** rcernin has quit IRC00:29
openstackgerritMerged openstack/cinder-specs master: Report backend state in service list  https://review.openstack.org/25292100:30
openstackgerritMerged openstack/cinder-specs master: Inspection Mechanism For Capacity Limited Host  https://review.openstack.org/30886900:30
*** zhurong has joined #openstack-cinder00:37
openstackgerritTommyLike proposed openstack/cinder-specs master: Update backup's size when backup is created  https://review.openstack.org/50614100:38
*** itlinux has joined #openstack-cinder00:38
*** ganso has quit IRC00:45
openstackgerritGhanshyam Mann proposed openstack/cinder master: Fix description for volume_type object in API Ref  https://review.openstack.org/42546100:45
*** erlon has quit IRC00:53
*** wes_dillingham has quit IRC00:56
*** gnufied has quit IRC00:58
*** crushil has joined #openstack-cinder01:04
*** wes_dillingham has joined #openstack-cinder01:10
*** mriedem_dinner is now known as mriedem01:11
*** yangyape_ has joined #openstack-cinder01:17
*** Apoorva has quit IRC01:25
*** Shunli has joined #openstack-cinder01:27
*** stakeda has joined #openstack-cinder01:27
*** sbezverk has joined #openstack-cinder01:31
*** markvoelker has joined #openstack-cinder01:38
*** itlinux has quit IRC01:39
*** stakeda has quit IRC01:46
*** chhavi has joined #openstack-cinder01:46
*** zhurong has quit IRC01:50
*** chhavi has quit IRC01:51
*** zhurong has joined #openstack-cinder01:59
*** markstur_ has quit IRC02:04
*** alkhodos has quit IRC02:08
*** lhx__ has joined #openstack-cinder02:12
*** markvoelker has quit IRC02:13
*** dave-mcc_ has quit IRC02:13
*** rcernin has joined #openstack-cinder02:20
*** itlinux has joined #openstack-cinder02:24
*** wes_dillingham has quit IRC02:27
*** spotz has quit IRC02:29
*** lbragstad has joined #openstack-cinder02:31
*** spotz has joined #openstack-cinder02:37
*** itlinux has quit IRC02:50
*** armax has quit IRC03:04
*** markstur has joined #openstack-cinder03:06
*** markstur has quit IRC03:07
*** markstur has joined #openstack-cinder03:07
*** markvoelker has joined #openstack-cinder03:10
*** zhurong has quit IRC03:12
*** gnufied has joined #openstack-cinder03:13
*** gnufied has quit IRC03:23
openstackgerritTommyLike proposed openstack/cinder master: [policy in code] Add support for qos and quota resources  https://review.openstack.org/50809103:24
*** gkadam has joined #openstack-cinder03:26
*** markvoelker has quit IRC03:42
lhx__patrickeast,  I think the spec should be udpated :-) https://review.openstack.org/#/c/408830/03:48
lhx__jbernard, will you propose a failback spec base on ^?03:54
*** lbragstad has quit IRC04:00
*** links has joined #openstack-cinder04:02
*** arnewiebalck_ has joined #openstack-cinder04:18
*** abhitechie has quit IRC04:19
*** zhurong has joined #openstack-cinder04:20
*** lhx__ has quit IRC04:30
*** lhx__ has joined #openstack-cinder04:31
*** lhx__ has quit IRC04:32
*** lhx__ has joined #openstack-cinder04:32
*** yangyape_ has quit IRC04:33
*** yangyapeng has joined #openstack-cinder04:33
*** markvoelker has joined #openstack-cinder04:39
*** stakeda has joined #openstack-cinder04:40
*** abhitechie has joined #openstack-cinder04:44
*** psachin has joined #openstack-cinder04:55
*** markvoelker has quit IRC05:12
*** Shunli has quit IRC05:15
*** lhx__ has quit IRC05:15
*** rcernin has quit IRC05:15
*** lhx__ has joined #openstack-cinder05:15
*** lhx__ has quit IRC05:22
*** armax has joined #openstack-cinder05:23
*** arnewiebalck_ has quit IRC05:24
*** pcaruana has joined #openstack-cinder05:24
*** stvnoyes has quit IRC05:26
*** pgadiya has joined #openstack-cinder05:27
*** yangyapeng has quit IRC05:27
*** yangyapeng has joined #openstack-cinder05:27
*** stvnoyes has joined #openstack-cinder05:29
*** pcaruana has quit IRC05:29
*** lpetrut has joined #openstack-cinder05:35
*** yangyapeng has quit IRC05:39
*** markstur has quit IRC05:40
*** yangyapeng has joined #openstack-cinder05:41
*** yangyape_ has joined #openstack-cinder05:44
*** yangyapeng has quit IRC05:44
*** ChubYann has quit IRC05:50
*** rcernin has joined #openstack-cinder05:52
*** yangyape_ has quit IRC05:53
*** josecastroleon has quit IRC06:00
*** yangyapeng has joined #openstack-cinder06:01
*** stakeda has quit IRC06:01
*** yangyape_ has joined #openstack-cinder06:02
*** lpetrut has quit IRC06:02
*** yangyapeng has quit IRC06:06
*** lpetrut has joined #openstack-cinder06:07
*** markvoelker has joined #openstack-cinder06:09
*** masber has joined #openstack-cinder06:15
*** crushil has quit IRC06:19
*** jmlowe has quit IRC06:20
*** lhx__ has joined #openstack-cinder06:28
*** lpetrut has quit IRC06:38
*** markvoelker has quit IRC06:43
*** yangyape_ has quit IRC06:51
*** yangyapeng has joined #openstack-cinder06:51
*** wewe0901 has joined #openstack-cinder06:54
*** vivsoni_ has joined #openstack-cinder06:58
*** vivsoni has quit IRC06:59
*** zengyingzhe__ has quit IRC07:02
*** zengyingzhe__ has joined #openstack-cinder07:03
*** pgadiya has quit IRC07:04
*** pcaruana has joined #openstack-cinder07:04
*** lpetrut has joined #openstack-cinder07:11
*** yangyapeng has quit IRC07:13
*** yangyapeng has joined #openstack-cinder07:14
*** gk__ has joined #openstack-cinder07:17
*** yangyape_ has joined #openstack-cinder07:24
*** lpetrut has quit IRC07:27
*** yangyapeng has quit IRC07:27
*** lpetrut has joined #openstack-cinder07:27
*** masber has quit IRC07:31
*** yangyapeng has joined #openstack-cinder07:32
*** yangyap__ has joined #openstack-cinder07:34
*** yangyapeng has quit IRC07:34
*** yangyape_ has quit IRC07:35
lhx__geguileo, around?07:37
*** lpetrut has quit IRC07:39
*** markvoelker has joined #openstack-cinder07:40
*** yangyapeng has joined #openstack-cinder07:44
*** yangyap__ has quit IRC07:44
*** yangyapeng has quit IRC07:44
*** yangyapeng has joined #openstack-cinder07:45
*** yangyape_ has joined #openstack-cinder07:49
*** yangyapeng has quit IRC07:49
*** armax has quit IRC07:57
*** chenying_ has quit IRC08:00
*** chenying_ has joined #openstack-cinder08:01
geguileolhx__: hi08:04
*** yangyapeng has joined #openstack-cinder08:07
lhx__geguileo, I ran into a error when installing with enabled devstack-plugin-ceph08:07
lhx__do you know if there is some something wrong?08:08
lhx__http://paste.openstack.org/show/622282/08:08
geguileoI haven't tried it in a while, but there shouldn't be anything wrong08:08
geguileowhat went wrong?08:09
*** yangyape_ has quit IRC08:09
*** markvoelker has quit IRC08:12
lhx__geguileo, there is not ceph.conf under /etc/ceph08:16
geguileoand stack didn't report any errors?  what's the backend in cinder.conf?  did that same local.conf work before?08:17
lhx__geguileo, this error occurs while installing devstack08:21
lhx__http://paste.openstack.org/show/622286/08:21
lhx__this is my local.conf08:22
geguileolhx__: ok, so it's not strange that you don't have ceph conf files if devstack failed08:23
geguileolhx__: what error did you get?08:23
*** e0ne has joined #openstack-cinder08:24
lhx__geguileo, I checked the cinder.conf, but it's bit weird that it dd not set backend. http://paste.openstack.org/show/622287/08:25
lhx__geguileo, my devstack didn't finish installation with this error: http://paste.openstack.org/show/622282/08:26
geguileolhx__: have you tried changing you enabling pluging with: enable_plugin ceph https://github.com/openstack/devstack-plugin-ceph08:27
geguileolhx__: oh, forget it, apparently they have changed it08:28
lhx__geguileo, yep :)08:28
lhx__geguileo, I also used ceph08:28
lhx__geguileo, hah, it's just a copy of your configurations08:29
geguileoit's strange that you overwrite CINDER_ENABLED_BACKENDS, but the conf looks right08:31
openstackgerrithuangjianbo proposed openstack/cinder master: Add support for striped volume in cinder.brick.local_dev.LVM  https://review.openstack.org/50843508:31
geguileoso I don't see why it should fail, you'll probably have to go through the logs from the stack to find out08:31
*** yangyapeng has quit IRC08:36
*** yangyapeng has joined #openstack-cinder08:36
*** alexchadin has joined #openstack-cinder08:37
*** sbezverk has quit IRC08:38
*** belmoreira has joined #openstack-cinder08:38
gk__Hi everyone, can a core member takes a look at this patch https://review.openstack.org/#/c/461665/16? It has already a +2. Thank you in advance!08:39
lhx__geguileo, thanks anyway, I will scan the track again08:42
geguileolhx__: good luck08:42
lhx__:-D08:42
*** gkadam has quit IRC08:51
openstackgerritVivek Soni proposed openstack/cinder master: testing -- DO NOT MERGE  https://review.openstack.org/50844708:54
*** gkadam has joined #openstack-cinder08:55
*** gkadam has quit IRC09:00
*** mriedem has quit IRC09:01
*** gkadam has joined #openstack-cinder09:01
*** markvoelker has joined #openstack-cinder09:09
openstackgerritVivek Soni proposed openstack/cinder master: testing - DONT MERGE  https://review.openstack.org/50845609:12
*** gkadam_ has joined #openstack-cinder09:21
*** gkadam has quit IRC09:24
*** markvoelker has quit IRC09:43
*** alexchadin has quit IRC09:47
*** alexchadin has joined #openstack-cinder09:48
*** abhinavtechie has joined #openstack-cinder10:02
*** abhitechie has quit IRC10:02
*** yuval has quit IRC10:06
*** jgriffith has quit IRC10:08
*** wN has quit IRC10:08
*** yuval has joined #openstack-cinder10:09
*** wN has joined #openstack-cinder10:14
*** wN has quit IRC10:14
*** wN has joined #openstack-cinder10:14
*** jgriffith has joined #openstack-cinder10:14
*** abhinavtechie has quit IRC10:18
*** abhinavtechie has joined #openstack-cinder10:18
*** aagrawal has joined #openstack-cinder10:20
*** aagrawal has quit IRC10:22
*** abhinavtechie has quit IRC10:22
*** abhitechie has joined #openstack-cinder10:22
*** masber has joined #openstack-cinder10:28
*** abhinavtechie has joined #openstack-cinder10:30
*** abhitechie has quit IRC10:32
*** masber has quit IRC10:32
*** abhitechie has joined #openstack-cinder10:35
*** abhinavtechie has quit IRC10:35
*** yangyapeng has quit IRC10:36
*** zhurong has quit IRC10:36
*** yangyapeng has joined #openstack-cinder10:37
*** yangyapeng has quit IRC10:38
openstackgerritGorka Eguileor proposed openstack/cinder master: Kaminario K2: Add non discovery iSCSI multipath  https://review.openstack.org/50814110:39
*** markvoelker has joined #openstack-cinder10:40
*** alexchadin has quit IRC10:41
*** abhitechie has quit IRC10:42
*** abhitechie has joined #openstack-cinder10:43
*** abhinavtechie has joined #openstack-cinder10:46
*** abhitechie has quit IRC10:47
*** abishop has joined #openstack-cinder10:47
*** abhinavtechie has quit IRC10:49
*** aagrawal has joined #openstack-cinder10:49
*** aagrawal has quit IRC10:51
*** aagrawal has joined #openstack-cinder10:51
*** aagrawal has quit IRC10:59
*** abhitechie has joined #openstack-cinder11:00
*** alexchadin has joined #openstack-cinder11:10
*** markvoelker has quit IRC11:12
*** sapd_ has quit IRC11:17
*** sapd_ has joined #openstack-cinder11:17
*** sapd_ has quit IRC11:17
*** sapd_ has joined #openstack-cinder11:18
*** edmondsw has quit IRC11:21
*** adisky has quit IRC11:23
*** sapd_ has quit IRC11:23
*** sapd_ has joined #openstack-cinder11:24
*** alexchadin has quit IRC11:24
*** tpsilva has joined #openstack-cinder11:28
*** alexchadin has joined #openstack-cinder11:30
*** yangyapeng has joined #openstack-cinder11:35
*** yangyapeng has quit IRC11:36
*** yangyapeng has joined #openstack-cinder11:36
openstackgerritGorka Eguileor proposed openstack/cinder-specs master: Provisioning Improvements  https://review.openstack.org/49011611:40
*** alexchadin has quit IRC11:42
*** alexchadin has joined #openstack-cinder11:43
openstackgerritGorka Eguileor proposed openstack/cinder master: Run backup compression on native thread  https://review.openstack.org/50751011:44
*** alexchadin has quit IRC11:52
*** armax has joined #openstack-cinder11:53
*** alkhodos has joined #openstack-cinder11:55
*** liverpooler has joined #openstack-cinder11:58
*** abhitechie has quit IRC12:00
lhx__e0ne, smcginnis , I saw the bp couldn't get started in newton. Should I move it to queens spec to undertake?12:02
lhx__https://review.openstack.org/#/c/270601/312:02
*** sapd__ has joined #openstack-cinder12:03
*** sapd__ has quit IRC12:03
*** sapd_ has quit IRC12:03
*** sapd__ has joined #openstack-cinder12:04
openstackgerritEvgeny Antyshev proposed openstack/cinder master: Add ploop to parallels naming conversion  https://review.openstack.org/50719312:06
*** markvoelker has joined #openstack-cinder12:09
*** alexchadin has joined #openstack-cinder12:10
*** edmondsw has joined #openstack-cinder12:13
*** alkhodos has quit IRC12:19
*** MVenesio has joined #openstack-cinder12:20
*** alkhodos has joined #openstack-cinder12:21
*** wes_dillingham has joined #openstack-cinder12:24
e0nelhx__, smcginnis : I think we should to move the spec to Queens if it's not started yet12:25
*** alkhodos has quit IRC12:26
*** alkhodos has joined #openstack-cinder12:26
*** gnufied has joined #openstack-cinder12:27
lhx__e0ne, hah, if smcginnis also agree, I will go head :)12:28
*** markvoelker has quit IRC12:29
*** markvoelker has joined #openstack-cinder12:29
*** 07IAA8DSW has joined #openstack-cinder12:29
*** 5EXAACMRJ has joined #openstack-cinder12:29
*** alkhodos has quit IRC12:31
*** 07IAA8DSW has quit IRC12:33
*** 5EXAACMRJ has quit IRC12:34
*** alkhodos has joined #openstack-cinder12:35
*** jmlowe has joined #openstack-cinder12:35
*** hemna_ has joined #openstack-cinder12:35
*** hoonetorg has joined #openstack-cinder12:35
*** mchlumsky has joined #openstack-cinder12:35
*** tonygunk has joined #openstack-cinder12:41
*** ganso has joined #openstack-cinder12:42
jbernardlhx__: i missed your first statement12:44
jbernardlhx__: based on which link?12:44
smcginnislhx__: Yes, I believe that work still needs to be done. That would be great if you moved that to Queens and worked on it.12:46
lhx__smcginnis, thanks for your confirm12:47
lhx__jbernard, will contact with you later, this link: https://review.openstack.org/#/c/408830/12:49
jbernardlhx__: yes12:50
lhx__jbernard, I think you can push you thought in that spec before you propose yours.12:52
jbernardlhx__: sure12:53
*** armax has quit IRC12:54
*** MVenesio has quit IRC12:55
*** mriedem has joined #openstack-cinder12:57
*** catintheroof has joined #openstack-cinder13:03
openstackgerritEric Young proposed openstack/cinder master: ScaleIO Driver - adding cache and refactoring tests.  https://review.openstack.org/50247313:05
*** lhx__ has quit IRC13:06
*** alkhodos has quit IRC13:10
*** lbragstad has joined #openstack-cinder13:15
*** gnufied has quit IRC13:16
*** efried is now known as fried_rice13:21
*** gnufied has joined #openstack-cinder13:21
*** eharney has joined #openstack-cinder13:22
*** stephenfin is now known as finucannot13:30
*** dansmith is now known as superdan13:34
*** psachin has quit IRC13:36
*** wanghao has joined #openstack-cinder13:36
*** lhx_ has joined #openstack-cinder13:39
*** links has quit IRC13:39
*** dustins has joined #openstack-cinder13:40
*** gouthamr has joined #openstack-cinder13:44
*** Dinesh_Bhor has quit IRC13:48
*** takedakn has joined #openstack-cinder13:56
*** sbezverk has joined #openstack-cinder13:59
*** crushil has joined #openstack-cinder14:03
*** takedakn has quit IRC14:03
*** wanghao has quit IRC14:06
*** amoralej is now known as amoralej|off14:06
*** amoralej|off is now known as amoralej|lunch14:06
*** alexchadin has quit IRC14:09
tommylikehusmcginnis:  need a professor to teach me why this patch keep failing :https://review.openstack.org/#/c/506976/14:13
smcginnistommylikehu: It's just due to the zuul migration.14:14
smcginnistommylikehu: We'll just have to wait a bit yet.14:14
tommylikehusmcginnis:  ok, for which reason we add both jenkins and zuul?14:14
smcginnistommylikehu: They are still working through some zuulv3 upgrade issues. Hopefully we can just recheck later.14:14
smcginnistommylikehu: Yeah, not quite all the way there yet.14:15
*** hemna_ has quit IRC14:15
jungleboyjsmcginnis:  Is Zuul replacing Jenkins?14:17
smcginnisjungleboyj: Well, technically it already did. We were running zuulv2, but now they are upgrading to v3 which is more bigger. New account and all, so we get the comments from zuul instead of jenkins.14:18
jungleboyjsmcginnis:  Ok.  I know that Zuul was always behind it but I meant to ask if we are going to see 'zuul' voting now instead of 'jenkins'14:19
smcginnisjungleboyj: Yeah, I believe so.14:20
smcginnisAnd some day it might actually vote something other than -1. :)14:20
*** mriedem1 has joined #openstack-cinder14:20
*** tonygunk has quit IRC14:21
jungleboyjsmcginnis: You are talking craziness!14:21
*** tonygunk has joined #openstack-cinder14:21
smcginnis:)14:21
*** kfarr has joined #openstack-cinder14:22
*** mriedem has quit IRC14:23
e0nejungleboyj, smcginnis: do we really want CI on Fridays? :)14:26
jungleboyje0ne:  Nah!14:26
e0ne:)14:26
*** takedakn has joined #openstack-cinder14:31
*** wewe0901 has quit IRC14:34
*** takedakn has quit IRC14:34
*** mriedem1 is now known as mriedem14:43
*** amoralej|lunch is now known as amoralej14:49
*** e0ne has quit IRC14:51
openstackgerritDavid Paterson proposed openstack/cinder master: Volume backup not supported  https://review.openstack.org/50723514:52
*** markstur has joined #openstack-cinder14:55
*** tonygunk has quit IRC14:56
*** tonygunk has joined #openstack-cinder14:57
openstackgerritEric Young proposed openstack/cinder master: ScaleIO Driver - adding cache and refactoring tests.  https://review.openstack.org/50247315:01
*** rcernin has quit IRC15:01
*** tonygunk has quit IRC15:03
*** hemna_ has joined #openstack-cinder15:08
geguileotommylikehu: hey, did you actually test this patch?  https://review.openstack.org/#/c/463906/15:11
geguileotommylikehu: ok, apparently it wasn't your fault... it was smcginnis15:13
*** belmoreira has quit IRC15:14
tommylikehugeguileo:  oh, what's wrong?15:14
smcginnisgeguileo: I see your name on there, not mine. :)15:14
geguileosmcginnis: you broke our os-brick gate with this patch: https://review.openstack.org/#/c/505420/15:14
geguileosmcginnis: that patch worked15:14
geguileosmcginnis: and then this one broke it https://review.openstack.org/#/c/505420/15:14
smcginnisgeguileo: Really? WHere/15:14
geguileosmcginnis: probably our Cinder tempest tests are not running in-tree tempest15:14
smcginnisOK, in meeting. Will be back soon.15:15
geguileoeharney: ^ can you confirm?15:15
tommylikehugeguileo:  any error log link on hand?15:16
geguileotommylikehu: I'll fix it, I know what's broken15:17
tommylikehugeguileo:  cool, thanks15:18
*** dave-mccowan has joined #openstack-cinder15:22
openstackgerritGorka Eguileor proposed openstack/cinder master: Fix Tempest Volume Revert test  https://review.openstack.org/50854115:22
geguileotommylikehu: smcginnis ^ that's the issue15:22
geguileoand here's the tempest error: http://logs.openstack.org/53/507653/1/check/gate-tempest-dsvm-full-lio-src-os-brick-ubuntu-xenial/a582289/console.html#_2017-09-28_10_44_00_98520015:23
geguileobut it's only affecting os-brick because we must not be running cinder in-tree tempests tests on Cinder gates15:23
smcginnisBack. Looking at that patch now.15:24
eharneythe lio job runs the in-tree tests, most jobs don't15:24
smcginnisgeguileo: So min_microversion is something in BaseVolumeTest?15:25
geguileosmcginnis: I believe it is, I kind of remember looking into it during the original review15:25
smcginnisgeguileo: How did that work before the constants change if that value wasn't set?15:26
geguileosmcginnis: because cinder is not running in-tree tempest tests15:27
geguileosmcginnis: so you can write whatever you want in those tests, it will always pass in Cinder15:27
smcginnisSo that patch didn't break it. It was never set up right to check MVs.15:27
geguileoand when it merges it breaks OS-Brick15:27
geguileogates that do run them15:27
openstackgerritDavid Paterson proposed openstack/cinder master: Volume backup not supported  https://review.openstack.org/50723515:28
tommylikehusmcginnis:  https://review.openstack.org/#/c/505420/4/cinder/tests/tempest/api/volume/test_volume_revert.py15:29
smcginnistommylikehu: Oh weird. Oops. :]15:30
geguileosmcginnis: you can see that the os-brick gate was working on th 22nd when the last patch was merged, then that microversion patch got merged in Cinder and no more os-brick patches get +115:30
*** chlong has quit IRC15:35
*** esker has joined #openstack-cinder15:37
*** tonygunk has joined #openstack-cinder15:46
*** zzzeek has quit IRC15:58
*** esker has quit IRC15:58
*** zzzeek has joined #openstack-cinder15:59
*** xyang1 has joined #openstack-cinder16:00
*** sbezverk has quit IRC16:04
*** arnewiebalck_ has joined #openstack-cinder16:14
*** jmlowe has quit IRC16:20
*** itlinux has joined #openstack-cinder16:23
*** pcaruana has quit IRC16:26
*** arnewiebalck_ has quit IRC16:35
*** edmondsw has quit IRC16:37
*** itlinux has quit IRC16:44
*** itlinux has joined #openstack-cinder16:47
openstackgerritEvgeny Antyshev proposed openstack/cinder master: Vzstorage: fix for create_cloned_volume  https://review.openstack.org/50856916:48
*** rtjure has quit IRC16:49
jungleboyjsmcginnis:  You around?16:49
smcginnisjungleboyj: Yep16:50
jungleboyjsmcginnis:  Hey, so looking at a spec that will also require a change in Horizon.  How have we handled that in the past?16:51
smcginnisjungleboyj: Which one is that?16:51
jungleboyjsmcginnis: https://review.openstack.org/#/c/319831/16:51
smcginnisjungleboyj: Well, the Cinder work doesn't16:52
jungleboyjThere would need to be a new check box added in the 'Transfer' dialog.16:52
smcginnis"require" a change in horizon, but they would need to make an update to take advantage of that.16:52
jungleboyjsmcginnis:  Ok, so we don't worry about that?16:52
smcginnisSo nothing that should hold us up, but would be a good courtesy to let robcresswell or someone there know.16:53
jungleboyjsmcginnis:  Ok.  I just had that thought because I was playing with that the other day.16:54
jungleboyjsmcginnis:  Wish we had a process for making sure those things happen.16:54
*** shaner has quit IRC16:54
smcginnisjungleboyj: Would require a little coordination with horizon, but we could try to have something started in Horizon before approving in Cinder. But that could get messy.16:55
jungleboyjI just approved it and added a note about Horizon.  I will try to remember to follow up on that.16:56
*** itlinux has quit IRC17:02
openstackgerritJohn Griffith proposed openstack/cinder-specs master: Add shared_targets column to volume table  https://review.openstack.org/50767017:10
*** gouthamr has quit IRC17:10
*** lhx_ has quit IRC17:21
*** dustins has quit IRC17:22
*** gouthamr has joined #openstack-cinder17:33
*** edmondsw has joined #openstack-cinder17:37
*** markstur_ has joined #openstack-cinder17:40
*** sage has quit IRC17:41
openstackgerritMerged openstack/cinder-specs master: Transfer snapshots with volumes  https://review.openstack.org/31983117:41
*** jmlowe has joined #openstack-cinder17:41
*** dave-mccowan has quit IRC17:41
*** sage has joined #openstack-cinder17:42
*** markstur has quit IRC17:42
*** rtjure has joined #openstack-cinder17:53
*** alkhodos has joined #openstack-cinder17:53
*** david-lyle has quit IRC17:56
*** david-lyle has joined #openstack-cinder17:56
openstackgerritDavid Paterson proposed openstack/cinder master: Dell EMC SC and PS volume backup not supported  https://review.openstack.org/50723518:02
*** hoonetorg has quit IRC18:06
*** alkhodos has quit IRC18:07
*** boris_42_ has joined #openstack-cinder18:15
*** alkhodos has joined #openstack-cinder18:16
*** lpetrut has joined #openstack-cinder18:26
openstackgerritMerged openstack/cinder-specs master: Add shared_targets column to volume table  https://review.openstack.org/50767018:28
*** markstur_ has quit IRC18:31
SwansonJohnny capitalization.18:34
openstackgerritDavid Paterson proposed openstack/cinder master: Dell EMC SC and PS volume backup not supported  https://review.openstack.org/50723518:37
*** tonygunk has quit IRC18:38
*** alkhodos2 has joined #openstack-cinder18:40
*** alkhodos has quit IRC18:40
openstackgerritDavid Paterson proposed openstack/cinder master: Dell EMC SC and PS volume backup not supported  https://review.openstack.org/50723518:43
*** dustins has joined #openstack-cinder18:49
*** markstur has joined #openstack-cinder18:52
*** markstur has quit IRC18:57
*** lbragstad has quit IRC19:03
*** vivsoni_ has quit IRC19:03
*** vivsoni_ has joined #openstack-cinder19:04
*** lbragstad has joined #openstack-cinder19:07
*** markstur has joined #openstack-cinder19:14
*** markstur has quit IRC19:18
*** catintheroof has quit IRC19:20
*** catintheroof has joined #openstack-cinder19:20
*** catintheroof has quit IRC19:24
*** e0ne has joined #openstack-cinder19:30
*** crushil has quit IRC19:34
*** ChubYann has joined #openstack-cinder19:34
abishopkfarr: hey, was hoping to ask a couple questions about https://review.openstack.org/46825219:36
abishopkfarr: specifically, what's the thinking on how users will migrate from the old conf key manager to barbican?19:37
SwansonLooking forward to Dell's new office design.  https://www.mcsweeneys.net/articles/our-open-plan-office-failed-so-were-moving-to-a-towering-panopticon19:45
smcginnisThere will be a Jamba Juice.19:47
SwansonThis sounds terrifying and dystopian. Is it? No.19:48
*** rarora has joined #openstack-cinder19:50
*** wes_dillingham has quit IRC19:57
*** tonygunk has joined #openstack-cinder20:08
kfarrabishop to clarify, that patch isn't actually removing conf key manager, though I did reference wanting to remove it20:10
*** xyang1 has quit IRC20:11
abishopkfarr: yeah, eharney and I paniced for a moment when we thought all the code had been removed!20:11
kfarrabishop in the future, if we do remove conf key manager, a user would have to copy their key from the config file, store in it barbican (presumably that's the preferred key manager)20:11
kfarrand update the encryption_key_id field on the volume to the barbican uuid20:12
abishopkfarr: totally get the desire to move away from legacy mgr and onward to barbican20:12
abishopyeah, that's what I'm looking at, but the devil is in the details20:12
abishopthe concern is we need to provide a "gentle" migration path, and minimize glitches and downtime20:12
abishopI've been incubating an idea I want to run by you...20:13
patrickeastanyone seen 404 errors with the volume revert tempest tests? like so http://openstack-logs.purestorage.com/PureISCSIDriver-tempest-dsvm-xenial-aio-multipath-chap/3122/logs/testr_results.html.gz20:13
abishopI was playing with having cinder's keymgr.API() (which the patch removes) provide a thin wrapper that would provide legacy support when it is asked to get or delete a key with all-zeros id20:14
abishopkfarr: those requests would be diverted from the barbican backend, and redirected to the legacy conf.key_manager20:14
abishopkfarr: the intent is to make this transparent to the upper layers20:15
*** e0ne has quit IRC20:15
*** e0ne has joined #openstack-cinder20:15
*** e0ne has quit IRC20:16
abishopkfarr: this would allow "things to work" with barbican as the key manager in situatinos where user is upgrading to barbican, and still has all-zeros key IDs in various dbases20:16
*** e0ne has joined #openstack-cinder20:16
*** e0ne has quit IRC20:16
*** e0ne has joined #openstack-cinder20:17
kfarrabishop technically that change would have to go into nova, since the key isn't retrieved until nova tries to attach the volume20:17
abishopkfarr: thoughts?20:17
abishopwell, cinder and nova20:17
*** e0ne has quit IRC20:17
*** e0ne has joined #openstack-cinder20:17
*** e0ne has quit IRC20:18
abishopkfarr: which has me wondering if castellan (as the common entrypoint) could help with the migration problem20:18
*** e0ne has joined #openstack-cinder20:18
*** e0ne has quit IRC20:19
kfarrabiship Ok, so I am a user who created a bunch of encrypted volumes using the conf key manager, but now I see the light and I want to use barbican (or I am forced to install barbican because conf key manager is going away)20:21
abishopkfarr: http://paste.openstack.org/show/622347/ is a hack to cinder's old keymgr that gives you idea20:21
kfarrand I want this to have the least amount of hassle possible20:21
kfarrand it's too much of a hassle to update the encryption_key_id in the database?20:22
abishopkfarr: exactly! in spite of warnings about old key manager, there are users (enterprise) who have deployments with many volumes encrypted that way20:23
abishopupdating the dbase is conceptually easy, but may be tricky in practice20:23
abishopfor example, the encryption_key_id can be stored in various metadata tables, so knowing where to hunt down all the instances might be difficult20:24
abishopthe update process might also take a non-trivial amount of time, so how do we ensure uptime until all instaces are migrated?20:25
Swansonjungleboyj, That make sense? https://review.openstack.org/#/c/507235/620:25
kfarrabishop as a user, am I going to want to move away from conf key manager eventually?  am I going to have a bunch of old volumes laying around that are using that conf key even though my new ones use barbican?20:26
*** tonygunk has quit IRC20:27
abishopI expect the user will _want_ to move away as quickly as possible, but that also needs to go as smoothly as possible20:27
kfarrabishop I want to say maybe it would be better to provide some sort of a script that will do that for them?  but then I also don't know the scope of how many places the encryption_key_id is replicated throughout the database20:28
abishopkfarr: primary concern is reducing risk during migration period20:28
kfarrabishop I'm not sure what the openstack-y way to do this is20:28
kfarrabishop yeah I gotcha20:28
*** dustins has quit IRC20:29
abishopkfarr: the process I'm shooting for will be the compromise that allows us to fully deprecate the old without abandoning users who get caught in the middle20:29
kfarrabishop right right20:31
abishopkfarr: right now, AFAIK both cinder and nova maintain an _identical_ implementation for the .get() function that uses the fixed key20:31
kfarrabishop yes but I don't think get() is ever called in cinder, get() is called in nova20:31
abishopproposal I'm now incubating is maybe castellan could help during the period when migration is desireable, but potentially risky20:31
abishopOh, get() is called in cinder ;-)20:32
abishopin fact, _any_ code that calls get() would benefit from a common solution20:32
abishop(common, meaning castellan)20:32
kfarrabishop if we merged code into castellan to help with this, would that transition code eventually get removed?20:32
abishopyes, I think that's completely reasonable20:33
abishopwe just need to help customers get all the way across the (migration) desert20:33
kfarrabishop I like the idea of making it easy for the user, though the main question on my mind is once the barbican driver in castellan has the workaround to get the key from conf if the uuid is 000s, then is that volume stuck using conf key manager?  Would the user have to clone the volume so that it starts using barbican?20:35
kfarrabishop also, I just remembered, there is one get() call in castellan, it's when cloning a volume20:36
*** markstur has joined #openstack-cinder20:36
abishopkfarr: yeah, and cinder does this to clone in a couple places20:36
kfarrwhen cloning a volume, the key data is retrieved and stored again as a new key, and the new uuid is associated with the volume clone20:36
abishopyep20:36
abishopthis is done for backups, too20:37
abishopkfarr: cinder can add code to look for existing all-zero key IDs, and migrate it into barbican (updating its dbase to barbican ID)20:38
abishopso the goal is to eventually hunt down and purge (migrate) all all-zero ids.20:39
kfarrohhhhh this is so messy20:39
abishopyeah, which is why eharney put his thumb on your review so we could have this discussion before anything merged20:40
abishop(yeah, we know the nova piece already has)20:40
abishopkfarr: obviously we won't solve this today, but wanted to raise the point so we can have further discussions20:40
kfarrabishop, which way do you think is better: 1. adding the code to cinder/nova to automatically migrate the key or 2. adding the workaround code into castellan and the key stays in the conf file indefinitely20:41
kfarrabishop is it better to just never remove conf key manager?20:41
abishopkfarr: well, how about a little of both? have cinder/nova start migrating their key, but have workaround in castellan there to provide the saftey net during migration20:42
abishopeventually deprecate castellan workaround code once we can be sure users feel safe knowing things are fully migrated20:42
kfarrabishop ok thanks for the discussion20:43
abishopkfarr: in fact, final user goal would be to remove fixed_key from cinder and nova conf files (that's the security risk we're trying to eliminate)20:43
abishopkfarr: but user won't want to do that until they know it's safe, and castellan workaround might be key (no pun) to ensuring that safety20:44
abishopI'll let you mull this, and we can pick it up again20:44
kfarrabishop ok sounds good to me20:44
*** KeithMnemonic has quit IRC20:45
abishopthanks for your time and consideration!20:45
kfarrabishop would be nice to not have to revert that nova patch that removed the deprecated options20:45
kfarrabishop you too!20:45
abishopkfarr: final thought: castellan workaround might allow nova patch to stand as-is!20:46
kfarrabishop yup :)20:47
*** jmlowe has quit IRC20:50
*** armax has joined #openstack-cinder21:00
*** wxy has quit IRC21:13
*** wes_dillingham has joined #openstack-cinder21:13
*** edmondsw has quit IRC21:20
*** mchlumsky has quit IRC21:21
*** amoralej has quit IRC21:22
*** amoralej has joined #openstack-cinder21:29
*** esker has joined #openstack-cinder21:32
*** esker has quit IRC21:32
*** esker has joined #openstack-cinder21:33
*** esker has quit IRC21:34
*** esker has joined #openstack-cinder21:35
*** hemna_ has quit IRC21:38
*** esker has quit IRC21:42
openstackgerritJohn Griffith proposed openstack/cinder master: Switch from ostestr to stestr  https://review.openstack.org/50862421:44
*** hoonetorg has joined #openstack-cinder21:50
jgriffithHmmm... /me wonders if zuul 3 might somehow adjust timing of things such that we deal with a bunch of intermittent failures?21:50
*** esker has joined #openstack-cinder21:50
jgriffithI wouldn't think that would be the case, but seems odd21:50
jgriffith"seems odd"  == the amount of red I'm seeing now21:51
smcginnisjgriffith: I'm thinking they aren't so intermittent.21:51
jgriffithyeah, actually now that you mention it21:52
smcginnisjgriffith: The jobs and zuul are all a mess right now.21:52
jgriffithI'd poke into infra and offer to help out, but I doubt I'd be much of an asset21:53
jgriffithwould take them longer to bring me up to speed on things than to work on it21:53
smcginnisSame here. Been bugging them every once and awhile, but mostly trying to not interupt them.21:54
smcginnisThere's a chance we may need to update some of the cinder jobs, but I'm hoping they just get everything sorted out.21:54
*** lpetrut has quit IRC21:58
*** shaner has joined #openstack-cinder21:58
*** tpsilva has quit IRC22:05
*** armax has quit IRC22:26
*** gouthamr has quit IRC22:29
*** esker has quit IRC22:34
*** gk__ has quit IRC22:41
jgriffithWell shit, this backend_id thing is like whack-o-mole with all the little *gotchas*23:29
jgriffithService initialization (I forgot) is a bit of a monster23:29
jgriffithhoped to have this done today but looks like it's going to take another morning of trickery I'm afraid23:29
*** lbragstad has quit IRC23:35
*** savvas_ has joined #openstack-cinder23:37
*** savvas has quit IRC23:39
*** fried_rice is now known as efried_thbagh23:40
*** zhurong has joined #openstack-cinder23:52
*** abishop has quit IRC23:58

Generated by irclog2html.py 2.15.3 by Marius Gedminas - find it at mg.pov.lt!