Tuesday, 2016-03-22

*** salv-orl_ has joined #openstack-cinder00:01
*** salv-orlando has quit IRC00:04
*** salv-orl_ has quit IRC00:04
*** yangyapeng has quit IRC00:05
*** garthb has quit IRC00:08
*** sasukeh has joined #openstack-cinder00:10
*** kprabhu has quit IRC00:16
*** kprabhu has joined #openstack-cinder00:16
*** laughterwym has joined #openstack-cinder00:49
*** cheneydc has joined #openstack-cinder00:56
*** dims has joined #openstack-cinder00:57
*** mtanino has quit IRC01:10
*** salv-orlando has joined #openstack-cinder01:10
*** laughterwym has quit IRC01:15
*** mriedem has quit IRC01:17
*** apoorvad has quit IRC01:20
*** Lee1092 has joined #openstack-cinder01:22
*** salv-orlando has quit IRC01:26
*** laughterwym has joined #openstack-cinder01:27
*** salv-orlando has joined #openstack-cinder01:28
*** EinstCrazy has joined #openstack-cinder01:28
*** bardia has quit IRC01:30
*** EinstCra_ has joined #openstack-cinder01:41
*** EinstCr__ has joined #openstack-cinder01:42
*** EinstCrazy has quit IRC01:42
*** salv-orlando has quit IRC01:43
*** baojg has joined #openstack-cinder01:43
*** akshai has quit IRC01:44
*** laughterwym has quit IRC01:45
*** EinstCra_ has quit IRC01:46
*** laughterwym has joined #openstack-cinder01:48
*** kprabhu has quit IRC01:57
*** ircuser-1 has joined #openstack-cinder02:02
*** panda has quit IRC02:05
*** panda has joined #openstack-cinder02:06
*** knikolla has quit IRC02:15
*** bardia has joined #openstack-cinder02:16
*** iceyao has joined #openstack-cinder02:18
jamielennoxis there a simple backend that drops into a folder i can setup with, i just want it for testing02:19
*** houming has joined #openstack-cinder02:24
*** dims has quit IRC02:28
*** LZ has joined #openstack-cinder02:38
*** salv-orlando has joined #openstack-cinder02:39
*** Fdaisuke has joined #openstack-cinder02:43
*** mylu has quit IRC02:43
*** gouthamr has quit IRC02:44
smcginnisjamielennox: LVM is probably the most straightforward if you don't have anything else to use.02:44
jamielennoxsmcginnis: yea, unfortunately for reasons i don't think i can use LVM - even in testing02:45
jamielennoxthough maybe i can make a loopback file02:45
*** iceyao has quit IRC02:45
jamielennoxi can try that, i was just hoping for something i could point at a folder and let someone else fix properly later02:46
*** mylu has joined #openstack-cinder02:46
*** iceyao has joined #openstack-cinder02:46
smcginnisjamielennox: That could be useful for sure.02:47
smcginnisjamielennox: I don't think we have anything that easy right now.02:47
*** mylu has quit IRC02:48
*** mylu has joined #openstack-cinder02:48
*** akshai has joined #openstack-cinder02:49
*** salv-orlando has quit IRC02:50
*** esker has quit IRC02:50
*** akshai has quit IRC02:53
*** iceyao has quit IRC03:06
*** iceyao has joined #openstack-cinder03:06
*** salv-orlando has joined #openstack-cinder03:07
*** akshai has joined #openstack-cinder03:10
*** kprabhu has joined #openstack-cinder03:15
*** 20WAALIJF has joined #openstack-cinder03:17
*** mylu has quit IRC03:19
*** links has joined #openstack-cinder03:19
*** salv-orlando has quit IRC03:24
*** IlyaG has quit IRC03:27
*** 20WAALIJF has quit IRC03:28
*** haomaiwang has joined #openstack-cinder03:30
*** laughterwym has quit IRC03:41
*** laughterwym has joined #openstack-cinder03:41
*** laughterwym has joined #openstack-cinder03:42
openstackgerritwanghao proposed openstack/cinder: Unmanaging vol/snap reduces quota incorrectly  https://review.openstack.org/23243603:45
*** laughterwym has quit IRC03:46
*** haomaiwang has quit IRC04:01
*** haomaiwang has joined #openstack-cinder04:01
*** aswadr_ has joined #openstack-cinder04:01
*** adisky has joined #openstack-cinder04:02
*** akshai has quit IRC04:06
*** chlong|wfh has quit IRC04:12
openstackgerritzhangguoqing proposed openstack/cinder: Use assertIn and assertNotIn  https://review.openstack.org/29346104:19
*** salv-orlando has joined #openstack-cinder04:23
*** chlong has joined #openstack-cinder04:25
*** IlyaG has joined #openstack-cinder04:27
*** bardia has quit IRC04:28
*** bardia has joined #openstack-cinder04:29
*** salv-orlando has quit IRC04:30
*** salv-orlando has joined #openstack-cinder04:32
*** IlyaG has quit IRC04:32
*** laughterwym has joined #openstack-cinder04:37
*** laughterwym has quit IRC04:40
*** laughterwym has joined #openstack-cinder04:40
*** daneyon_ has quit IRC04:42
*** daneyon has joined #openstack-cinder04:43
*** shyama has quit IRC04:50
*** shyama has joined #openstack-cinder04:50
*** klkumar has joined #openstack-cinder04:51
*** salv-orlando has quit IRC04:52
*** daneyon has quit IRC04:57
*** salv-orlando has joined #openstack-cinder04:59
*** haomaiwang has quit IRC05:01
*** haomaiwang has joined #openstack-cinder05:01
*** dave-mccowan has quit IRC05:03
*** salv-orl_ has joined #openstack-cinder05:04
*** salv-orl_ has quit IRC05:06
*** jwcroppe has quit IRC05:06
*** salv-orlando has quit IRC05:09
*** kprabhu has quit IRC05:09
openstackgerritAngus Lees proposed openstack/os-brick: Trivial rootwrap -> privsep replacement  https://review.openstack.org/27722405:12
*** houming has quit IRC05:12
*** klkumar has quit IRC05:17
*** _alastor_ has quit IRC05:25
*** houming has joined #openstack-cinder05:27
*** IlyaG has joined #openstack-cinder05:28
*** bardia has quit IRC05:28
*** bardia has joined #openstack-cinder05:30
*** lprice has joined #openstack-cinder05:31
*** IlyaG has quit IRC05:33
*** klkumar has joined #openstack-cinder05:34
*** bardia has quit IRC05:34
*** houming has quit IRC05:36
openstackgerritAbhishek Kekane proposed openstack/cinder: DB purge command fails due to FK constraint  https://review.openstack.org/29108305:36
itlinuxhello cinder experts, I have installed a simple env, using packstack and I get the same issue all the time .. unable to retrieve volume limits etc.. any suggestions on that? Thank you05:38
*** houming has joined #openstack-cinder05:42
*** Poornima has joined #openstack-cinder05:46
*** sasukeh has quit IRC05:56
*** baojg has quit IRC05:56
*** baojg has joined #openstack-cinder05:57
*** haomaiwang has quit IRC06:01
*** haomaiwang has joined #openstack-cinder06:01
*** ankit_ag has joined #openstack-cinder06:07
*** salv-orlando has joined #openstack-cinder06:07
*** sgotliv has joined #openstack-cinder06:08
openstackgerritaditi sharma proposed openstack/python-cinderclient: Adds missing user message  https://review.openstack.org/26954806:16
*** Poornima has quit IRC06:21
*** Poornima has joined #openstack-cinder06:22
*** salv-orlando has quit IRC06:24
*** ChubYann has quit IRC06:25
*** nkrinner has joined #openstack-cinder06:29
DuncanTjamielennox: The NFS backend sort of sounds like what you want, it just drops volume image files in a folder06:32
*** bardia has joined #openstack-cinder06:35
*** salv-orlando has joined #openstack-cinder06:35
*** sasukeh has joined #openstack-cinder06:41
*** salv-orlando has quit IRC06:41
*** sasukeh has quit IRC06:46
*** anshul has joined #openstack-cinder06:46
*** sasukeh has joined #openstack-cinder06:50
*** Fdaisuke_ has joined #openstack-cinder06:54
*** klkumar has quit IRC06:55
*** Fdaisuke has quit IRC06:55
*** haomaiwang has quit IRC07:01
*** haomaiwang has joined #openstack-cinder07:01
*** jwcroppe has joined #openstack-cinder07:04
*** yangyapeng has joined #openstack-cinder07:15
*** sgotliv has quit IRC07:20
*** liverpooler has joined #openstack-cinder07:22
*** tesseract has joined #openstack-cinder07:28
*** tesseract is now known as Guest9125107:28
*** klkumar has joined #openstack-cinder07:33
*** rcernin has joined #openstack-cinder07:39
*** arecknag has joined #openstack-cinder07:41
*** bardia has quit IRC07:45
*** salv-orlando has joined #openstack-cinder07:46
*** ebalduf has quit IRC07:48
*** sasukeh has quit IRC07:48
*** salv-orlando has quit IRC07:48
*** sasukeh has joined #openstack-cinder07:50
*** anshul has quit IRC07:53
*** salv-orlando has joined #openstack-cinder07:53
*** belmoreira has joined #openstack-cinder07:56
*** salv-orlando has quit IRC07:59
*** haomaiwang has quit IRC08:01
*** haomaiwang has joined #openstack-cinder08:01
*** salv-orlando has joined #openstack-cinder08:03
*** anshul has joined #openstack-cinder08:08
*** salv-orlando has quit IRC08:14
*** salv-orlando has joined #openstack-cinder08:16
*** markvoelker has quit IRC08:16
*** salv-orlando has quit IRC08:25
*** sasukeh has quit IRC08:25
*** sasukeh has joined #openstack-cinder08:26
*** permalac has quit IRC08:33
*** openstackgerrit has quit IRC08:33
*** openstackgerrit has joined #openstack-cinder08:34
*** Fdaisuke_ has quit IRC08:40
*** _degorenko|afk is now known as degorenko08:41
*** Julien-zte has joined #openstack-cinder08:43
*** Julien-zte has quit IRC08:45
*** Julien-zte has joined #openstack-cinder08:48
*** sgotliv has joined #openstack-cinder08:49
*** sasukeh has quit IRC08:50
*** jordanP has joined #openstack-cinder08:52
*** Julien-zte has quit IRC08:53
*** sasukeh has joined #openstack-cinder08:53
*** sasukeh has quit IRC08:53
*** sasukeh has joined #openstack-cinder08:54
*** haomaiwang has quit IRC09:01
*** 92AAALWH6 has joined #openstack-cinder09:01
*** lpetrut has joined #openstack-cinder09:16
*** markvoelker has joined #openstack-cinder09:17
*** ndipanov has quit IRC09:18
*** ndipanov has joined #openstack-cinder09:18
*** ndipanov has quit IRC09:18
*** ndipanov has joined #openstack-cinder09:19
*** lpetrut has quit IRC09:21
*** salv-orlando has joined #openstack-cinder09:22
*** e0ne has joined #openstack-cinder09:22
*** klkumar has quit IRC09:24
*** klkumar has joined #openstack-cinder09:26
openstackgerritWenjun Wang proposed openstack/cinder: quota-defaults lists info with wrong tenant-id  https://review.openstack.org/29489509:28
openstackgerritWenjun Wang proposed openstack/cinder: quota-defaults lists info with wrong tenant-id  https://review.openstack.org/29489509:29
openstackgerritWenjun Wang proposed openstack/cinder: quota-defaults lists info with wrong tenant-id  https://review.openstack.org/29489509:30
*** salv-orlando has quit IRC09:39
*** sasukeh has quit IRC09:44
*** sdague has joined #openstack-cinder09:45
*** sasukeh has joined #openstack-cinder09:47
*** dims has joined #openstack-cinder09:51
*** sasukeh has quit IRC09:51
*** markvoelker has quit IRC09:52
*** bjornar has joined #openstack-cinder09:53
*** EinstCr__ has quit IRC10:00
*** iceyao has quit IRC10:00
*** 92AAALWH6 has quit IRC10:01
*** 14WAAJIS3 has joined #openstack-cinder10:01
*** cheneydc has quit IRC10:02
*** dims has quit IRC10:04
*** yangyapeng has quit IRC10:04
openstackgerritaditi sharma proposed openstack/python-cinderclient: Add missing user message  https://review.openstack.org/26954810:12
*** dims has joined #openstack-cinder10:12
*** tpsilva has joined #openstack-cinder10:13
*** lpetrut has joined #openstack-cinder10:16
*** lpetrut has quit IRC10:17
*** lpetrut has joined #openstack-cinder10:17
bjornar__Seems lxml 3.6.0 failed to build (wheel) due to missing -I/usr/include/libxml2 switch10:17
openstackgerritGorka Eguileor proposed openstack/cinder: Add new API job distribution features  https://review.openstack.org/29426310:31
*** salv-orlando has joined #openstack-cinder10:33
*** 14WAAJIS3 has quit IRC10:36
*** 7F1AAKFVO has joined #openstack-cinder10:36
*** 7F1AAKFVO has quit IRC10:36
*** haomaiwang has joined #openstack-cinder10:36
*** haomaiwang has quit IRC10:37
sheelankit_ag: hi10:39
ankit_agsheel: hi10:39
sheelankit_ag: regarding https://review.openstack.org/#/c/269548/10:39
sheelankit_ag: for else after exception10:40
ankit_agsheel: yes10:40
sheelankit_ag: may be we can skip this change for now being very small change and seems no impact of this10:40
sheelankit_ag: like we have to change other places as well for same to maintain consistency everywhere10:40
sheelankit_ag: but this seems no big bug just an improvement, only small improvement10:41
sheelankit_ag: what you say?10:41
ankit_agsheel: Yes I am ok with either way but using else block looks more appropriate here10:42
sheelankit_ag: yes, you are absolutely right10:42
sheelankit_ag: my point was only that we have to change multiple places for this10:42
sheelankit_ag: which may be ignored for now10:42
sheelankit_ag: as seems no benefit as such(only 2 line of code in whole try block)  :)10:43
ankit_agsheel: okay I will opt out my -1 from the patch10:43
sheelankit_ag: yes, may be +1 for fix :)10:43
ankit_agin favour of consistency10:43
sheelankit_ag: thank you10:44
ankit_agsheel: no problem, happy to help with the patch10:46
sheelankit_ag:  :)10:46
*** klkumar has quit IRC10:55
*** TheBall has quit IRC11:03
e0ne,,11:07
*** mvk_ has joined #openstack-cinder11:12
*** atuvenie has joined #openstack-cinder11:12
*** TheBall has joined #openstack-cinder11:15
*** mvk has quit IRC11:15
*** vbel has joined #openstack-cinder11:18
vbelhi  everyone, is it possible to use nfs storage that already is mounted to hosts for instances and live migration? I know that existing nfs drivers work by mounting shares from nova-computes11:19
*** Guest17490 has quit IRC11:30
*** gcb has joined #openstack-cinder11:36
DuncanTvbel: Is it possible to use the same share for cinder volumes? I don't see why not, the mounting logic doesn't seem to prevent you11:37
*** wilson has joined #openstack-cinder11:37
*** sgotliv has quit IRC11:37
*** wilson is now known as Guest697911:38
*** laughterwym has quit IRC11:38
*** TheBall is now known as The_Ball11:39
*** jwcroppe has quit IRC11:40
*** baojg has quit IRC11:41
*** iceyao has joined #openstack-cinder11:42
*** dencaval has joined #openstack-cinder11:42
*** houming has quit IRC11:44
*** jwcroppe_ has joined #openstack-cinder11:48
*** markvoelker has joined #openstack-cinder11:48
*** sgotliv has joined #openstack-cinder11:51
*** cdelatte has quit IRC11:52
*** yangyapeng has joined #openstack-cinder11:59
*** salv-orl_ has joined #openstack-cinder12:01
*** EinstCrazy has joined #openstack-cinder12:03
vbelDuncanT, thanks12:04
*** salv-orlando has quit IRC12:05
*** mvk_ has quit IRC12:07
*** alyson_ has joined #openstack-cinder12:10
*** cdelatte has joined #openstack-cinder12:13
*** klkumar has joined #openstack-cinder12:17
*** delattec has joined #openstack-cinder12:18
*** smoriya_ has quit IRC12:18
*** delattec has quit IRC12:21
*** cdelatte has quit IRC12:21
*** markvoelker has quit IRC12:21
*** dave-mccowan has joined #openstack-cinder12:22
*** jwcroppe_ has quit IRC12:22
*** raildo-afk is now known as raildo12:30
*** mtanino has joined #openstack-cinder12:30
*** diablo_rojo has joined #openstack-cinder12:31
*** mmm has joined #openstack-cinder12:35
*** sgotliv has quit IRC12:36
*** rlrossit has joined #openstack-cinder12:47
*** markvoelker has joined #openstack-cinder12:47
*** klkumar has quit IRC12:51
*** klkumar has joined #openstack-cinder12:53
openstackgerritWilson Liu proposed openstack/cinder: Huawei: Failed to get admin_metadata  https://review.openstack.org/29577612:55
*** laughterwym has joined #openstack-cinder12:56
*** laughterwym has quit IRC12:58
*** mriedem has joined #openstack-cinder12:59
*** ociuhandu has quit IRC12:59
*** klkumar has quit IRC13:01
*** links has quit IRC13:01
*** dims_ has joined #openstack-cinder13:02
*** cknight has joined #openstack-cinder13:02
*** dims has quit IRC13:02
openstackgerritWilson Liu proposed openstack/cinder: Huawei: Failed to get admin_metadata  https://review.openstack.org/29577613:03
*** akerr has joined #openstack-cinder13:06
*** bswartz has quit IRC13:08
*** Guest6979 has quit IRC13:08
*** Guest6979 has joined #openstack-cinder13:09
*** laughterwym has joined #openstack-cinder13:10
*** timcl has joined #openstack-cinder13:13
*** bswartz has joined #openstack-cinder13:13
*** mtanino has quit IRC13:17
*** knikolla has joined #openstack-cinder13:22
*** gouthamr has joined #openstack-cinder13:23
*** takedakn has joined #openstack-cinder13:26
*** mvk_ has joined #openstack-cinder13:27
*** takedakn has quit IRC13:31
*** mylu has joined #openstack-cinder13:33
*** klkumar has joined #openstack-cinder13:33
*** jwcroppe has joined #openstack-cinder13:34
*** mylu has quit IRC13:34
*** erlon has joined #openstack-cinder13:34
*** markvoelker_ has joined #openstack-cinder13:34
*** markvoelker has quit IRC13:36
*** porrua has joined #openstack-cinder13:37
*** markvoelker has joined #openstack-cinder13:38
*** ankit_ag has quit IRC13:40
*** markvoelker_ has quit IRC13:40
*** eharney has joined #openstack-cinder13:41
*** vbel has quit IRC13:42
*** mylu has joined #openstack-cinder13:43
*** ametts has joined #openstack-cinder13:44
*** mylu has quit IRC13:46
*** anshul has quit IRC13:46
*** Thelo has joined #openstack-cinder13:46
TheloHi, Do the storage solutions that use iscsi protocol also require a driver and a CI?13:47
smcginnisThelo: Yes. iSCSI is the data path, but there still needs to be a control path that uses vendor specific APIs to manage the storage.13:50
Thelosmcginnis: Thanks13:50
smcginnisThelo: No problem. Are you planning another driver?13:50
*** liverpooler has quit IRC13:52
*** ociuhandu has joined #openstack-cinder13:52
*** rhagarty has joined #openstack-cinder13:54
*** Julien-zte has joined #openstack-cinder13:55
*** dustins has joined #openstack-cinder13:55
*** mylu has joined #openstack-cinder13:57
*** mylu has quit IRC13:58
*** edtubill has joined #openstack-cinder14:00
*** mylu has joined #openstack-cinder14:01
*** mtanino has joined #openstack-cinder14:01
Thelosmcginnis: not yet ( I still have to fix my CI first when I am back from vacation), but I am preparing a presentation for the Taiwanese industry to share our experience with the cinder driver & CI.14:02
*** yangyapeng has quit IRC14:02
openstackgerritMerged openstack/cinder: NexentaStor5 iscsi: resize volume if cloned image is larger  https://review.openstack.org/29530414:04
openstackgerritMerged openstack/cinder: Enable functional test for volume creation  https://review.openstack.org/29419514:04
*** baojg has joined #openstack-cinder14:05
*** links has joined #openstack-cinder14:05
*** yangyapeng has joined #openstack-cinder14:06
smcginnisThelo: Nice!14:07
*** jwcroppe has quit IRC14:08
*** laughterwym has quit IRC14:11
*** laughterwym has joined #openstack-cinder14:12
*** ebalduf has joined #openstack-cinder14:12
*** martyturner has joined #openstack-cinder14:12
*** kprabhu has joined #openstack-cinder14:14
sheelsmcginnis: hi14:14
sheelsmcginnis: are we open with newton now?14:15
smcginnissheel: It is technically open on master now.14:15
smcginnissheel: But we should still focus on closing out on Mitaka before we put too much focus on Newton.14:15
*** laughterwym has quit IRC14:16
sheelsmcginnis: ah yes..agree...14:16
smcginnis;)14:16
sheelsmcginnis: thanks14:16
*** dims_ has quit IRC14:19
*** kprabhu has quit IRC14:20
*** laughterwym has joined #openstack-cinder14:22
*** laughterwym has quit IRC14:27
*** Poornima has quit IRC14:27
*** openstackgerrit has quit IRC14:33
*** baojg has quit IRC14:34
*** openstackgerrit has joined #openstack-cinder14:34
*** jgregor has joined #openstack-cinder14:34
*** baojg has joined #openstack-cinder14:35
*** jungleboyj has joined #openstack-cinder14:35
*** baojg has quit IRC14:36
*** baumann has joined #openstack-cinder14:36
scottdae0ne: I am thinking of adding some functional tests to the cinderclient. Had you done any work on this?14:37
smcginnisReminder to everyone - https://etherpad.openstack.org/p/mataka-rc-backports-needed14:37
e0nescottda: we've got some tests and job for it14:38
e0nescottda: https://github.com/openstack/python-cinderclient/tree/master/cinderclient/tests/functional14:39
e0nescottda: we setup full devstack to run these tests14:39
scottdae0ne: I see them. I don't know why I missed them before.14:39
e0nescottda: I'm sorry, need to go away if nothing important14:40
e0nescottda: I'll be available in ~2 hours14:40
scottdae0ne: Nope, nothing besides that. Thanks.14:40
e0nescottda: you're welcome :)14:40
*** mylu has quit IRC14:41
*** e0ne has quit IRC14:41
*** wN has joined #openstack-cinder14:42
*** wN has joined #openstack-cinder14:42
*** mylu has joined #openstack-cinder14:42
*** links has quit IRC14:42
*** lprice has quit IRC14:49
yuriy_n17geguileo: Hi! Very sorry for disturbing. Please review https://review.openstack.org/#/c/281511 whenever you have free time. I'm interested in your opinion on the race in that case.14:49
*** lprice has joined #openstack-cinder14:50
*** lprice1 has joined #openstack-cinder14:50
sheelsmcginnis: I have added one entry on this etherpad14:51
*** martyturner has quit IRC14:52
sheel https://etherpad.openstack.org/p/mataka-rc-backports-needed14:52
*** baojg has joined #openstack-cinder14:53
smcginnissheel: The client one doesn't need to be backported since stable branches for those are mainly for historical reasons.14:54
*** lprice has quit IRC14:54
smcginnissheel: But the cinder one looks OK to backport. There aren't any translatable string changes, and it's small enough.14:54
smcginnissheel: Go ahead and cherry pick it to stable/mitaka.14:54
sheelsmcginnis: oh great...give me sec and i will be done14:54
sheelsmcginnis: thanks14:54
sheelsmcginnis: oops merge conflict..14:56
*** Swanson has quit IRC14:58
*** IlyaG has joined #openstack-cinder14:59
*** atuvenie has quit IRC15:01
*** JoseMello has joined #openstack-cinder15:01
geguileoyuriy_n17: Yep, that's a racing nightmare15:02
geguileoyuriy_n17: Only solution is using transactions with serializable isolation15:02
geguileoyuriy_n17: Which is a performance kill15:02
sheeldulek: hi15:03
yuriy_n17geguileo: Thanks a lot.15:03
geguileoyuriy_n17: I improved cinder's metadata update some time ago to minimize races but didn't remove them completely because I didn't want to use that isolation level15:04
*** IlyaG has quit IRC15:04
*** jwcroppe has joined #openstack-cinder15:05
*** baojg has quit IRC15:05
*** Thelo has quit IRC15:06
*** iceyao has quit IRC15:07
duleksheel: I need to head home, I should be available in about an hour. Sorry.15:09
sheeldulek: its ok...chill..15:10
*** crose has joined #openstack-cinder15:11
*** Julien-zte has quit IRC15:11
*** kprabhu has joined #openstack-cinder15:16
*** garthb has joined #openstack-cinder15:17
DuncanTgeguileo: yuriy_n17: There's hopefully going to be some advice on using etags issued shortly, which might enable us to provide a race-free metadata API....15:22
geguileoDuncanT: That would be awesome!15:22
yuriy_n17geguileo, DuncanT: Please look at http://paste.openstack.org/show/491467/15:23
DuncanTgeguileo: As my comments on that review indicate, I'm not particularly sure how etags are supposed to work in certain cases, however I can definitely see how they can help in general, and the specific cases look solvable15:23
DuncanTyuriy_n17: I'd like to put a hold on any solution to this until we have some guidance for the API working group - it isn't going to be merged before Newton now, and the issues concerned are fairly subtle15:25
DuncanTyuriy_n17: I think this is worth getting right, and I think the solution will affect our update API(s) too, which suffer the same race15:27
*** adisky has quit IRC15:30
yuriy_n17Duncan: Thanks for review.15:30
*** diablo_rojo has quit IRC15:31
DuncanTyuriy_n17: Your spotting that there was a problem has opened a nice discussion about this issue across all openstack APIs, and I've learnt a bit about etags in the process, so thank you :-) It won't be quick to get an agreement on how to fix I suspect, but hopefully the end result will be solid15:35
yuriy_n17DuncanT: Ok;)15:37
DuncanTyuriy_n17: Keep breaking things! It's where our best bugs get found15:37
*** jwcroppe has quit IRC15:39
*** mmm has quit IRC15:40
*** nkrinner has quit IRC15:42
openstackgerritSheel Rana proposed openstack/cinder: Updated doc string  https://review.openstack.org/29591015:43
kprabhuHi guys, could somebody have a look at this bug and confirm this https://bugs.launchpad.net/cinder/+bug/156053915:44
openstackLaunchpad bug 1560539 in Cinder "type-access list should show some information for a public volume type" [Undecided,New] - Assigned to Karthik Prabhu Vinod (karthik-prabhu-vinod)15:44
kprabhulooking for suggestions too15:44
*** dims has joined #openstack-cinder15:45
*** timcl has quit IRC15:47
*** timcl has joined #openstack-cinder15:47
openstackgerritMerged openstack/cinder-specs: List volumes/snapshots for manage-existing  https://review.openstack.org/28282915:47
sheelkprabhu: hi15:48
sheelkprabhu: i think type-access is specific for private volume types only15:48
sheelkprabhu: why we want to work this for public volume type15:48
sheelalso currently proper details are shown if this command is run against public volume type "ERROR: Failed to get access list for public volume type."15:49
mtaninosheel: agree! I just wrote a comment.15:51
sheelmtanino: oopss, I think we wrote comments in parallel..15:51
mtaninolol15:52
sheelmtanino: 👍15:52
openstackgerritWalter A. Boring IV (hemna) proposed openstack/cinder: Remove local_dev from cinder  https://review.openstack.org/26075615:52
sheelmtanino: only 10 nano second difference ..  you are damn fast15:53
*** mylu has quit IRC15:53
sheelmtanino: lol15:53
*** mylu has joined #openstack-cinder15:53
*** EinstCrazy has quit IRC15:53
mtaninosheel:  might be using fast internet!!15:54
*** dims_ has joined #openstack-cinder15:55
*** dims has quit IRC15:55
sheelmtanino: hahaha... wish i had been in japan...15:56
*** darosale has joined #openstack-cinder16:00
kprabhumtanino,sheel : makes sense...thanks for reviewing the bug.16:01
openstackgerritMerged openstack/python-cinderclient: Add docstrings for chessecake methods  https://review.openstack.org/29006816:01
sheelkprabhu: you are welcome..16:01
mtaninokprabhu: Thank you for your evaluation :)16:02
*** IlyaG has joined #openstack-cinder16:02
openstackgerritEric Harney proposed openstack/cinder: Docs: Add genconfig to devref toc  https://review.openstack.org/29592916:06
*** arecknag has quit IRC16:08
*** apoorvad has joined #openstack-cinder16:09
*** sasukeh has joined #openstack-cinder16:10
*** mylu has quit IRC16:16
*** mylu has joined #openstack-cinder16:19
*** mylu has quit IRC16:19
*** mylu has joined #openstack-cinder16:19
*** sasukeh has quit IRC16:20
*** e0ne has joined #openstack-cinder16:21
*** bswartz has quit IRC16:21
*** belmoreira has quit IRC16:25
*** daneyon has joined #openstack-cinder16:26
*** mylu has quit IRC16:26
hemnasmcginnis, jungleboyj so the os-brick privsep patch seems to work now in Nova16:27
*** atuvenie has joined #openstack-cinder16:27
*** bjornar has quit IRC16:28
*** Yogi1 has joined #openstack-cinder16:30
jungleboyjhemna: That is good.16:31
*** mylu has joined #openstack-cinder16:32
*** mylu has quit IRC16:35
*** jwcroppe has joined #openstack-cinder16:36
*** breitz has quit IRC16:36
duleksheel: I've answered your comment on 278605. I guess that's what you wanted to ask about?16:38
*** earlephilhower has joined #openstack-cinder16:39
sheeldulek: ah but still i think we should stick with current implementation16:39
sheeldulek:  reason being its ok to keep API check at single level instead of taking that further to sub functions16:40
sheeldulek: writing different function is the original approach which was documented as well16:40
sheeldulek: I am not saying any approach wrong, just doing a tradeoff between code complexity with code refactoring16:41
* dulek is looking into the code once again.16:43
*** _alastor_ has joined #openstack-cinder16:44
*** akshai has joined #openstack-cinder16:45
*** dims_ has quit IRC16:46
duleksheel: I think what was documented in microversions docs is more about API methods, than utility methods and that's utility.16:48
duleksheel: If we end up with a situation when multiple methods will look better than multiple parameters - fine, let's do that refactoring then.16:48
duleksheel: But right now the more I look on that code, the more I think that it's breaking DRY rule.16:49
sheeldulek: haha16:49
sheeldulek: ok, no issue we can go with your suggestion, i just put my part...16:49
*** haomaiwang has joined #openstack-cinder16:50
sheeldulek: just to dig more for better approach16:50
sheeldulek: thanks for suggestions...16:51
dulekThanks for working on that!16:51
sheeldulek: :)16:52
*** akshai has quit IRC16:52
*** akshai has joined #openstack-cinder16:52
*** akshai_ has joined #openstack-cinder16:54
*** haomaiwang has quit IRC16:55
*** akshai has quit IRC16:58
*** kfarr has joined #openstack-cinder16:58
*** sgotliv has joined #openstack-cinder17:01
*** Guest91251 has quit IRC17:02
*** dims has joined #openstack-cinder17:02
*** timcl1 has joined #openstack-cinder17:03
*** timcl has quit IRC17:04
*** klkumar has quit IRC17:04
*** klkumar has joined #openstack-cinder17:06
*** precurse has left #openstack-cinder17:09
*** jwcroppe has quit IRC17:09
*** haplo37 has joined #openstack-cinder17:11
*** dims has quit IRC17:13
*** degorenko is now known as _degorenko|afk17:15
*** diablo_rojo has joined #openstack-cinder17:16
*** Swanson has joined #openstack-cinder17:17
SwansonHoly moly I have a network again!  Weeee!17:17
*** bjornar has joined #openstack-cinder17:18
*** bswartz has joined #openstack-cinder17:20
scottdaIs c-bak supposed to be running in Devstack?17:23
*** akerr is now known as akerr_away17:24
scottdahmm.. or is Swift supposed to be running in Devstack?17:26
*** akerr_away is now known as akerr17:27
*** krtaylor has quit IRC17:27
*** mriedem1 has joined #openstack-cinder17:28
openstackgerritScott DAngelo proposed openstack/python-cinderclient: api-microversion support for python-cinderclient  https://review.openstack.org/24816317:30
sheelscottda: yes17:30
*** klkumar has quit IRC17:30
sheelscottda: you have to set enabled_services for swift17:30
scottdasheel: OK, so it's not running by default17:30
sheelscottda: yes17:30
*** mriedem has quit IRC17:31
sheelscottda: we need to change stackrc and then again stack.sh17:31
scottdasheel: OK, thanks. That explains it :)17:31
sheelscottda: welccome..17:31
*** mriedem has joined #openstack-cinder17:31
*** kfarr has quit IRC17:32
*** earlephilhower has quit IRC17:32
*** mriedem has quit IRC17:32
*** mriedem has joined #openstack-cinder17:33
*** mriedem1 has quit IRC17:33
*** rlrossit has quit IRC17:34
openstackgerritNate Potter proposed openstack/cinder: Added more options while uploading volume as image  https://review.openstack.org/23061717:35
*** Swanson has quit IRC17:36
*** bardia has joined #openstack-cinder17:37
*** Swanson has joined #openstack-cinder17:39
*** rlrossi__ has joined #openstack-cinder17:39
*** krtaylor has joined #openstack-cinder17:41
*** arch-nemesis has joined #openstack-cinder17:42
*** krotscheck has quit IRC17:43
*** EinstCrazy has joined #openstack-cinder17:44
*** akerr has quit IRC17:46
*** EinstCrazy has quit IRC17:46
*** EinstCrazy has joined #openstack-cinder17:46
*** EinstCrazy has quit IRC17:47
*** jwcroppe has joined #openstack-cinder17:48
*** EinstCrazy has joined #openstack-cinder17:48
*** shyama has quit IRC17:49
*** lpetrut has quit IRC17:49
*** shyama has joined #openstack-cinder17:49
*** jordanP has quit IRC17:49
*** EinstCrazy has quit IRC17:50
*** EinstCrazy has joined #openstack-cinder17:50
*** EinstCrazy has quit IRC17:53
*** ebalduf_ has joined #openstack-cinder17:53
*** EinstCrazy has joined #openstack-cinder17:53
*** EinstCrazy has quit IRC17:54
*** EinstCrazy has joined #openstack-cinder17:55
*** akerr has joined #openstack-cinder17:56
*** gcb has quit IRC17:56
*** gcb has joined #openstack-cinder17:57
*** rlrossi__ has quit IRC17:58
*** salv-orlando has joined #openstack-cinder18:02
*** rlrossit_ has joined #openstack-cinder18:03
*** panda has quit IRC18:04
*** panda has joined #openstack-cinder18:05
*** salv-orl_ has quit IRC18:05
*** akerr has quit IRC18:05
*** akerr has joined #openstack-cinder18:06
*** dencaval has quit IRC18:09
*** EinstCrazy has quit IRC18:10
*** EinstCrazy has joined #openstack-cinder18:10
*** angela-s has joined #openstack-cinder18:11
*** dencaval has joined #openstack-cinder18:15
*** EinstCrazy has quit IRC18:15
*** akerr has quit IRC18:15
*** EinstCrazy has joined #openstack-cinder18:16
*** ChubYann has joined #openstack-cinder18:16
*** akerr has joined #openstack-cinder18:16
*** angela-s has quit IRC18:17
*** EinstCrazy has quit IRC18:19
*** boris-42 has joined #openstack-cinder18:21
*** markus_z has joined #openstack-cinder18:21
*** angela-s has joined #openstack-cinder18:22
openstackgerritVincent Hou proposed openstack/cinder: Storwize: do the correct update after failover  https://review.openstack.org/29257018:25
ildikovscottda: hi18:26
scottdaildikov: Hi. I was wondering if we were set to schedule a cinder-nova meeting to discuss API changes?18:27
ildikovI've just got back from the other side of the ocean18:27
scottdano problem, just checking. Which ocean?18:28
e0nedoes anybody uses LIO target?18:28
*** akerr has quit IRC18:31
*** akerr has joined #openstack-cinder18:31
e0neI can't get working LIO target with devstack out of the box:(18:34
hemnaheh18:34
thingeee0ne: last time I tried was around august and didn't have problems18:35
thingeea lot has changed since then of course.18:35
e0nethingee: you're lucky!18:35
thingeeand there is a CI job that is actively using it18:35
e0nethingee: in my envs, destack didn't install targetcli/lio-utils packages and didn't start the service18:36
e0neI know about job, that's why I'm surprised a bit18:36
e0necan't find what I have to do except add 'CINDER_ISCSI_HELPER=lioadm' to my devstack conf18:37
*** tongli has joined #openstack-cinder18:37
ildikovscottda: sorry, back now18:39
ildikovscottda: I was in the US for a conference18:40
ildikovscottda: so regarding the meeting I wonder what now as I did not get too many responses to my mail18:40
openstackgerritScott DAngelo proposed openstack/python-cinderclient: Add docs for running tests  https://review.openstack.org/29600018:40
eharneye0ne: targetcli is not a requirement18:41
scottdaildikov: I think it's OK to schedule. We could also ping the people on the list to make sure that they know about it. It can be hard to get people in sync for a meeting.18:41
scottdaBut it might be better to postpone 1 week, since it's getting close to the meeting time.18:41
e0neeharney: ok, what is required packages for it?18:41
eharneye0ne: python-rtslib18:41
eharneye0ne: but that's already in cinder reqs18:42
ildikovscottda: so I can send out a reminder mail with the ML in cc, so that everyone is aware of the thing18:42
scottdaildikov: Sure. And we could meet anyway, and see who shows up. We should get this ball rolling.18:43
ildikovscottda: ok, then let's do it next week18:43
ildikovscottda: ok, I'm getting lost, are we still going for next week? :)18:43
e0neeharney: without  targetcli or may be lio-utils, attach doesn not work on my env. nova can't login to portal18:44
eharneye0ne: does Cinder succeed in creating a portal?18:44
e0neyes18:44
eharneywhat does the failure look like?18:44
e0neI didn't see any errors in cinder18:44
e0nenova failed to connect to the portal (timeout) and return volume to 'available' status18:45
eharneyhmm18:45
e0neand nobody was listening 3260 port18:46
eharneyright18:46
e0nebut nova tried to connect to it18:46
*** lpetrut has joined #openstack-cinder18:46
eharneyyou can run "cinder-rtstool get_targets" to verify part of it, iirc18:48
e0neit works18:48
eharneydoes it show one?18:49
e0neyes, but now I've got installed targetcli18:49
*** haomaiwang has joined #openstack-cinder18:49
scottdaildikov: Sure, let's shoot for next week.18:50
e0neeharney: if you give me 10 minutes, I'll try it on a fresh devstack18:50
eharneye0ne: sure, let me know what you find18:50
eharneye0ne: beware of port conflicts w/ tgtd service running18:51
e0neeharney: should I do anything else, except adding CINDER_ISCSI_HELPER=lioadm to config?18:51
eharneye0ne: i don't think so18:51
ildikovscottda: ok, I will send out the follow mail accordingly18:51
ildikovscottda: should we come up with an agenda now?18:51
hemnaildikov, Agenda: fix the API.18:52
smcginnis:)18:52
e0neeharney: that's  could be a root cause of my issues. i have to stop tgtd before devstack run18:52
hemnaglad I can help18:52
eharneyquite possible18:53
ildikovhemna: lol :) are you volunteering?18:53
hemnahey now, I dunno if I'd go THAT far!18:53
hemna:)18:53
*** IlyaG has quit IRC18:54
*** haomaiwang has quit IRC18:54
ildikovhemna: :)18:56
*** mylu has joined #openstack-cinder18:58
scottdaildikov: I think we can start with going over the etherpad: Goals & Issues. We can see where things lead from there.19:01
e0neeharney: do you know, is any performance testing results to compare tgtd and lio?19:02
eharneye0ne: i don't have any, but the general theory is that LIO will perform better19:02
*** ircuser-1 has quit IRC19:02
e0neeharney: I've done some testing. LIO is faster and don't eat CPU19:03
ildikovscottda: Sounds good, my thinking as well19:03
e0neeharney: but I have to check my test lab and results19:03
e0nebefore sharing it19:03
eharneye0ne: right, should be far less data being transferred between userspace and kernel space19:03
*** mtanino has quit IRC19:04
*** mylu has quit IRC19:07
*** mylu has joined #openstack-cinder19:07
*** aswadr_ has quit IRC19:10
*** ebalduf has quit IRC19:14
*** ociuhandu has quit IRC19:16
*** ametts has quit IRC19:17
*** akerr has quit IRC19:26
*** mylu has quit IRC19:27
*** mylu has joined #openstack-cinder19:30
openstackgerritMerged openstack/cinder: Dell SC: Error results moved to json  https://review.openstack.org/29480419:32
*** Lee1092 has quit IRC19:32
*** porrua has quit IRC19:33
*** mylu has quit IRC19:34
*** ociuhandu has joined #openstack-cinder19:35
openstackgerritAngela Smith proposed openstack/cinder: Fix Brcd lookup service to use defined southbound protocol  https://review.openstack.org/29425819:39
ildikovscottda: I sent out a follow up mail, I will start ping people tomorrow to ensure we'll have participants next week :)19:40
openstackgerritVincent Hou proposed openstack/cinder: Storwize: update the retype after replication v2.1  https://review.openstack.org/29260819:41
*** mylu has joined #openstack-cinder19:42
openstackgerritVincent Hou proposed openstack/cinder: Storwize: update the retype after replication v2.1  https://review.openstack.org/29260819:43
*** mylu has quit IRC19:44
openstackgerritVincent Hou proposed openstack/cinder: Clean up the prior implementation of replication  https://review.openstack.org/29246719:45
*** mylu has joined #openstack-cinder19:47
openstackgerritNate Potter proposed openstack/cinder: Added more options while uploading volume as image  https://review.openstack.org/23061719:47
*** ircuser-1 has joined #openstack-cinder19:51
*** timcl1 has quit IRC19:52
*** mylu has quit IRC19:52
*** jwcroppe has quit IRC19:54
*** kprabhu has quit IRC19:56
smcginnisFresh batch of shenanigans: https://bugs.launchpad.net/cinder/+bugs?field.tag=create-volume-from-snapshot19:56
*** david-lyle_ has joined #openstack-cinder19:56
*** akerr has joined #openstack-cinder19:57
*** kfarr has joined #openstack-cinder19:57
*** david-lyle has quit IRC19:57
*** pck_ has joined #openstack-cinder19:58
*** ekarlso- has quit IRC19:58
*** jwcroppe has joined #openstack-cinder19:58
*** salv-orlando has quit IRC19:58
*** Yogi1 has quit IRC19:58
*** pck has quit IRC19:58
*** pots has quit IRC19:58
*** sborkows has quit IRC19:58
*** akerr_ has joined #openstack-cinder19:59
*** sborkows has joined #openstack-cinder19:59
*** mylu has joined #openstack-cinder20:01
*** david-lyle_ is now known as david-lyle20:02
*** akerr has quit IRC20:03
*** ekarlso- has joined #openstack-cinder20:11
e0neeharney: here is my c-vol logs http://paste.openstack.org/show/491508/20:13
e0neeharney: and cinder-rtstool  output: http://paste.openstack.org/show/491509/20:14
openstackgerritDanny Al-Gaaf proposed openstack/cinder: Datera: resize volume if cloned image is larger  https://review.openstack.org/29491720:15
e0neeharney: oops, I forgot to stop tgt:(20:16
*** atuvenie has quit IRC20:16
eharneye0ne: ahh20:16
e0neone more try...20:16
openstackgerritDanny Al-Gaaf proposed openstack/cinder: rbd: resize volume not cloned but full copy  https://review.openstack.org/29548020:16
*** mvk_ has quit IRC20:18
*** atuvenie has joined #openstack-cinder20:22
*** akerr_ is now known as akerr_away20:22
*** IlyaG has joined #openstack-cinder20:22
*** mylu has quit IRC20:25
*** lpetrut has quit IRC20:27
openstackgerritTom Swanson proposed openstack/cinder: Dell SC: Checking volume size in create_volume_from_snapshot  https://review.openstack.org/29604020:28
*** mylu has joined #openstack-cinder20:30
SwansonI for one am happy that the IBM GPFS CI let me know that it can't merge.20:30
SwansonTwice.20:30
eharneyyeeeeeaaaahhh20:30
smcginnisStill?20:31
smcginnisI thought they disabled that account.20:31
smcginnisOr rather accounts.20:31
eharneysmcginnis: didn't we figure out how to ask people to disable that reporting?20:31
*** sdague has quit IRC20:31
SwansonNo disabling HAL. It knows how to work around things.20:31
smcginniseharney: Yeah, pretty easy layout.yaml change I thought.20:31
SwansonI'm sorry Jay, I'm afraid I can't merge that.20:32
*** jungleboyj has quit IRC20:38
jgriffithGPFS FTW!!!20:38
jgriffith\0/20:38
diablo_rojojgriffith: ?20:40
*** mylu has quit IRC20:40
*** atuvenie has quit IRC20:40
jgriffithdiablo_rojo: the GPFS CI telling us things can't merge20:40
jgriffith:)20:40
diablo_rojojgriffith: But its happy now?20:41
smcginnisdiablo_rojo: It's happily not merging.20:41
smcginnis:)20:41
jgriffithsmcginnis: :)20:41
jgriffithdiablo_rojo: see the scroll back... I dunno20:41
diablo_rojojgriffith: Ah my bad. Got it now :)20:42
jgriffithdiablo_rojo: :)  Got to keep a sense of humor20:43
jgriffithdiablo_rojo: even if it's not a "good" sense of humor :)20:43
Swansondiablo_rojo, I want a personal apology! At least a full second of my time was wasted!20:43
smcginnisdiablo_rojo: I'll kick him for you.20:43
diablo_rojojgriffith: Of course :) I for one love puns though I know many people consider them to be ridiculous :)20:44
Swansondiablo_rojo, I'm never getting that second back!20:44
diablo_rojosmcginnis: Thanks. I'll lend you my nerf gun ;)20:44
diablo_rojoSwanson: My deepest apologies ;)20:44
Swansondiablo_rojo, :) Thank you!20:45
diablo_rojoSwanson: Anytime :)20:45
e0neeharney: you were right. it was an issue with running tgtd and devstack:(20:46
diablo_rojojgriffith: Did you ever get a chance to give more thought to which statuses are valid for a volumes's replication status?20:46
*** mylu has joined #openstack-cinder20:46
eharneye0ne: good to know20:46
e0neeharney: thanks for the help20:46
jgriffithdiablo_rojo: hurumph.20:46
jgriffithdiablo_rojo: thought we did that :(20:46
jgriffithdiablo_rojo: as in... for a volume right now it's a noop20:47
jgriffithdiablo_rojo: because we're basing it on the entire backend for now20:47
jgriffithdiablo_rojo: but maybe patrickeast aorourke or Swanson had a differing opinion20:47
diablo_rojojgriffith: Okay so as of now I should not bother making that field into an enum versioned object ?20:47
jgriffithdiablo_rojo: so it's unfortunate that the field is still there because currently we're not actually using it20:48
jgriffithdiablo_rojo: I would say that we should either come up with a plan to either:20:48
jgriffith1. use it and decide on the enums20:48
jgriffith2. Mark it with N/A since 2.1 ignores it anyway20:48
diablo_rojojgriffith: So...would it make sense to go through and pull all of the instances of that field getting set out?20:48
jgriffithdiablo_rojo: maybe... let me look20:49
*** cknight has quit IRC20:49
*** jwcroppe has quit IRC20:49
jgriffithdiablo_rojo: hmm... that's squiriled away in more places than I remembered :(20:50
jgriffithdiablo_rojo: including 3par whcih updated20:50
jgriffithand dell20:50
jgriffithSwanson: ^^20:50
Swansonwha?20:50
diablo_rojojgriffith: So its still used in cheesecake?20:50
jgriffithdiablo_rojo: so *sadly* another case of "some chose to use it, some didn't" :(20:51
diablo_rojojgriffith: Sounds like I can't yank it then.20:51
jgriffithdiablo_rojo: correct20:51
jgriffithdiablo_rojo: so at this point best answer I *think* is to make sure everybody updates and uses it20:51
Swansonjgriffith, this the replication_driver_data thing or whatever?20:51
jgriffithdiablo_rojo: myself and patrickeast I believe are the only two20:51
jgriffithSwanson: no.. it's the replication_status field in the Volume object20:52
diablo_rojojgriffith: So to the enum versioned object we go.20:52
jgriffithdiablo_rojo: :)20:52
jgriffithdiablo_rojo: enjoy!!!20:52
diablo_rojojgriffith: Which brings me to the original question again..20:52
diablo_rojojgriffith: We've come full circle :)20:52
jgriffithdiablo_rojo: sorry  :)20:52
*** e0ne has quit IRC20:53
*** jwcroppe has joined #openstack-cinder20:53
jgriffithdiablo_rojo: "replicating, null, error" IMHO20:53
jgriffithdiablo_rojo: but let me see what people created :)20:53
diablo_rojojgriffith: There was a glimmer of hope this would be easy, but as usual it was dashed on spikes.20:53
*** wanghao has quit IRC20:53
diablo_rojojgriffith: That would be wonderfully short and sweet20:53
aorourkejgriffith, we use 'enabled', 'failed-over' and 'error' i believe20:53
jgriffithdiablo_rojo: ok... so here's what we have:20:53
*** wanghao has joined #openstack-cinder20:54
jgriffithenabled, disabled, failed-over, error20:54
jgriffithand crazy ibm has "copying" ?20:54
patrickeastFwiw I don't use that field, we do change the volume fields to error when they are left behind, but that's it20:54
jgriffithOh.. and of course... you also have "None"20:54
jgriffithpatrickeast: yeah... me too20:54
jgriffithpatrickeast: but given that others chose to use it, we should probably go ahead and populate it20:55
diablo_rojojgriffith: I'll talk to Vincent about those..20:55
Swansonjgriffith, I believe someone told me that I needed to use it so I used it. I don't think I'm married to it.20:55
jgriffithpatrickeast: otherwise it gets kinda confusing/inconsitent20:55
patrickeastMeh, its not used anywhere20:55
jgriffithSwanson: it's easier IMHO for patrickeast and I to implement it than to try and get everybody else to remove it20:55
patrickeastDoesn't really make sense to set it20:55
jgriffithpatrickeast: well....20:55
Swansonjgriffith, Yeah, it was useful for none replicated volumes.20:55
smcginnisIf it's not used anywhere, probably easier to remove usage from the drivers that have it rather than adding it where it's not needed.20:56
*** salv-orlando has joined #openstack-cinder20:56
Swansonnon20:56
diablo_rojoIt is inconsistent20:56
jgriffithpatrickeast: funny, because in my code I have a "TODO(JDG): WTF should I do on a failover to let user know they're screwed"20:56
aorourkejgriffith, patrickeast  i chose to use that value because i need to perform certain actions against replicated volumes only. It made more sense to use that than to check the volume_type since i already had the value set in the volume20:56
diablo_rojoMaybe we get rid of those in Newton?20:56
Swansonsmcginnis, In a perfect world it is made consistent with my usage.20:56
aorourkebecause we can still have non-replicated volumes on a replication enabled host20:56
smcginnis:)20:57
jgriffithdiablo_rojo: aorourke Swanson patrickeast My opinion is that maybe theyr'e useful, if for nothing else but to convey status20:57
jgriffithaorourke: yes, I agree20:57
* jgriffith votes we keep them and use them20:57
diablo_rojojgriffith: I can get behind that, but I think we should have an agreed upon set of valid statuses20:57
jgriffithbut I'll leave it up to the rest of ya'll20:57
*** jwcroppe has quit IRC20:58
jgriffithdiablo_rojo: yes, agreed...and I think they should be ' "", replicating, failed-over , error'20:58
jgriffithI don't see any value in all the crazy permutations20:58
*** raildo is now known as raildo-afk20:58
smcginnisYeah, if they serve some purpose, let's push to use them consistently.20:58
aorourkejgriffith, those make sense to me20:59
smcginnisI'm sure that will make Swanson happy. :)20:59
jgriffithpatrickeast: violent objections?20:59
diablo_rojojgriffith: smcginnis Works for me. Yeah. There dont need to be that many.20:59
jgriffithdiablo_rojo: you can fix the IBM drivers :)20:59
patrickeastI'm ok with us using them if there is some value in it, but it was a point of contention on v1 replication where there were a bunch of statuses that didn't make sense for all vendors20:59
patrickeastAnnnd they probably shouldn't be user faci g20:59
diablo_rojojgriffith: Yes yes, I will talk to Vincent about it.20:59
aorourkewhen do we want to get this in? Is it a pressing issue or can it wait?20:59
patrickeastDancing*20:59
jgriffithpatrickeast: agreed, that's why I think the proposed set is reasonable20:59
patrickeastOmg, lol20:59
patrickeastFacing**21:00
jgriffithaorourke: yesterday21:00
jgriffithpatrickeast: ?21:00
diablo_rojoaorourke: I'm fairly sure it can wait.21:00
jgriffithpatrickeast: ohh... "shouldn't be used for user facing"21:00
jgriffith:)21:00
diablo_rojoaorourke: I've just been working on making fields into enum versioned objects and replication status for volumes is next on the list.21:00
patrickeastIirc for v1 replication that status was viewable by a tenant doing like show commands21:01
* smcginnis picture patrickeast doing the cabbage patch21:01
patrickeastHaha21:01
jgriffithOk, well I've given my opinion... I'll let others discuss and decide :)21:01
*** IlyaG has quit IRC21:01
diablo_rojojgriffith: Thank you. You've been most helpful :) I'll buy you some cheesecake and a shot of fireball in Austin21:02
jgriffithdiablo_rojo: haha :)  I don't know about helpful, but I'll take the CheeseCake and the Fireball!21:02
*** dustins has quit IRC21:03
diablo_rojoSwanson: patrickeast aorourke So the agreed upon status are: replicating, failed-over, error...anything else?21:05
*** mylu has quit IRC21:05
*** mylu has joined #openstack-cinder21:06
Swansondiablo_rojo, I use enabled.  I don't use replicating.21:06
aorourkediablo_rojo, and NULL or '' or something to show it is not replicated21:06
Swansondisabled is in there, too.  (I think if you retype to a non replicating type.)21:06
aorourkeI am using enabled as well, but can switch if that is the consensus21:07
diablo_rojoOkay so enabled, disabled, replicating, error, failed-over, and null. Agreed?21:07
SwansonSo replicating means what?21:08
*** haomaiwa_ has joined #openstack-cinder21:08
diablo_rojoUhh...I thought someone was actively using that status?21:08
aorourkewe dont need enabled/disabled if we use replicating21:08
aorourkereplicating = volume is being replicated. null = not being replicated.21:08
aorourkeerror = bad, failed-over .... failed over21:09
Swansonaight21:09
diablo_rojoaorourke: Got it. So more people are using enabled and disabled at this point so it would make more sense to use those.21:09
aorourkediablo_rojo, less for me to change :) but if we use those i don't think we need replicating.21:10
*** IlyaG has joined #openstack-cinder21:10
*** crose has quit IRC21:10
diablo_rojoSounds good. Shortens my list :) Enabled, disabled, failed-over, and error. Muy bien?21:10
Swansondiablo_rojo, works for me.21:11
*** gouthamr has quit IRC21:11
diablo_rojopatrickeast: ^^21:11
*** dims has joined #openstack-cinder21:11
*** haomaiwa_ has quit IRC21:12
* jgriffith agrees 21:13
*** haplo37 has quit IRC21:13
patrickeastI can live with that21:25
diablo_rojoSpectacular :)21:25
*** cknight has joined #openstack-cinder21:25
*** diablo_rojo has quit IRC21:30
*** diablo_rojo has joined #openstack-cinder21:31
*** mylu has quit IRC21:32
*** cknight has quit IRC21:33
*** diablo_rojo has quit IRC21:37
*** cknight has joined #openstack-cinder21:37
*** cknight1 has joined #openstack-cinder21:38
*** JoseMello has quit IRC21:39
*** baumann has left #openstack-cinder21:39
*** atuvenie has joined #openstack-cinder21:41
*** jgregor has quit IRC21:41
*** cknight has quit IRC21:42
*** tongli has quit IRC21:42
*** akerr_away has quit IRC21:42
*** kfarr has quit IRC21:44
*** gouthamr has joined #openstack-cinder21:47
*** ebalduf_ has quit IRC21:48
*** openstackgerrit has quit IRC21:48
*** openstackgerrit has joined #openstack-cinder21:49
*** e0ne has joined #openstack-cinder21:50
openstackgerritChristopher J Schaefer proposed openstack/python-cinderclient: Adding tox support for bandit  https://review.openstack.org/29608221:50
*** edtubill has quit IRC21:51
openstackgerritDanny Al-Gaaf proposed openstack/cinder: Hitachi HBSD: extend volume if larger than snapshot  https://review.openstack.org/29608321:51
*** darosale has quit IRC21:52
*** mylu has joined #openstack-cinder21:53
*** Julien-zte has joined #openstack-cinder21:53
*** jwcroppe has joined #openstack-cinder21:54
*** rlrossit_ has quit IRC21:55
*** mriedem has quit IRC21:58
*** jwcroppe has quit IRC22:02
*** panda has quit IRC22:04
*** panda has joined #openstack-cinder22:05
*** eharney has quit IRC22:06
*** ebalduf has joined #openstack-cinder22:07
*** gcb has quit IRC22:09
*** kfox1111 has joined #openstack-cinder22:09
*** mylu has quit IRC22:09
kfox1111ever seen this: http://pastebin.com/imkeTtra22:09
*** hogepodge has quit IRC22:12
kfox1111pip version bug?22:12
smcginniskfox1111: That looks familiar.22:12
smcginniskfox1111: What version of pip do you have installed?22:13
kfox1111http://eavesdrop.openstack.org/irclogs/%23openstack-dns/%23openstack-dns.2015-11-12.log.html makes reference to a bug...22:13
kfox1111yup. pip install --upgrade pip pbr setuptools fixed it.22:13
smcginniskfox1111: Oh wait, that's inside your venv,22:13
*** mylu has joined #openstack-cinder22:14
kfox1111sorry for the noise. :)22:14
smcginniskfox1111: Oh good. Glad you got it.22:14
openstackgerritDanny Al-Gaaf proposed openstack/cinder: Hitachi: extend volume if larger than snapshot  https://review.openstack.org/29608322:16
*** e0ne has quit IRC22:21
*** gcb has joined #openstack-cinder22:22
*** angela-s has quit IRC22:23
*** gcb has quit IRC22:23
*** mylu has quit IRC22:23
openstackgerritMerged openstack/cinder: Docs: Add genconfig to devref toc  https://review.openstack.org/29592922:27
*** IlyaG has quit IRC22:29
*** mylu has joined #openstack-cinder22:33
*** gouthamr_ has joined #openstack-cinder22:34
*** cknight1 has quit IRC22:35
*** gouthamr has quit IRC22:37
*** IlyaG has joined #openstack-cinder22:38
*** haomaiwang has joined #openstack-cinder22:39
*** hogepodge has joined #openstack-cinder22:39
*** ociuhandu has quit IRC22:40
*** haomaiwang has quit IRC22:43
*** knikolla has quit IRC22:49
*** gcb has joined #openstack-cinder22:56
*** sgotliv has quit IRC22:59
*** ociuhandu has joined #openstack-cinder22:59
*** apoorvad has quit IRC23:01
*** IlyaG has quit IRC23:06
*** apoorvad has joined #openstack-cinder23:06
*** IlyaG has joined #openstack-cinder23:07
openstackgerritMerged openstack/cinder: Remove extra get_active_zoneset query from Brocade CLI  https://review.openstack.org/26604423:08
*** dims_ has joined #openstack-cinder23:12
*** dims has quit IRC23:15
*** dims has joined #openstack-cinder23:15
*** dims_ has quit IRC23:18
*** dims_ has joined #openstack-cinder23:19
*** dims has quit IRC23:22
*** dims has joined #openstack-cinder23:23
*** mylu has quit IRC23:24
*** dims_ has quit IRC23:25
openstackgerritWalter A. Boring IV (hemna) proposed openstack/os-brick: Fix coverage generation  https://review.openstack.org/29610723:25
*** mylu has joined #openstack-cinder23:26
*** mylu has quit IRC23:28
*** IlyaG has quit IRC23:30
*** lprice1 has quit IRC23:30
*** mylu has joined #openstack-cinder23:35
*** kprabhu has joined #openstack-cinder23:37
*** knikolla has joined #openstack-cinder23:39
*** mylu has quit IRC23:40
*** mylu has joined #openstack-cinder23:41
*** markvoelker has quit IRC23:44
*** Julien-zte has quit IRC23:46
*** markvoelker has joined #openstack-cinder23:47
*** smoriya_ has joined #openstack-cinder23:48
*** sasukeh has joined #openstack-cinder23:58

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