Tuesday, 2016-11-15

*** alonma has joined #openstack-cinder00:01
jgriffithSwanson: huh?00:02
jgriffithSwanson: I'd highly recommend not doing that00:02
jgriffithSwanson: and I'm not sure how you could00:02
jgriffithSwanson: you need the worker tenant and all that00:02
jgriffithSwanson: which we don't build for you etc00:03
*** asselin has quit IRC00:05
Swansonjgriffith, just trying to figure out what the second volume that showed up is all about.00:05
jgriffithSwanson: what's the id of the volume?  Any chance you had a race and taskflow retried the create giving you a second volume?00:06
*** alonma has quit IRC00:06
Swansonjgriffith, ID's are different. Anywho, gotta roll. Sort it out in the morning. (After the logs have filled with crap.)00:10
*** dave-mccowan has joined #openstack-cinder00:17
*** Julien-zte has joined #openstack-cinder00:27
*** laughterwym has joined #openstack-cinder00:27
*** laughterwym has quit IRC00:27
*** laughterwym has joined #openstack-cinder00:28
*** lixiaoy1 has joined #openstack-cinder00:30
*** slayer has joined #openstack-cinder00:30
*** rajinir has quit IRC00:36
*** catintheroof has quit IRC00:47
patrickeastSwanson: its turned on in devstack by default, yea00:47
patrickeastjgriffith: ^00:47
*** catintheroof has joined #openstack-cinder00:48
*** slayer has quit IRC00:51
*** catintheroof has quit IRC00:52
*** Julien-zte has quit IRC00:58
*** Julien-zte has joined #openstack-cinder01:02
*** alonma has joined #openstack-cinder01:03
*** yangyapeng has joined #openstack-cinder01:04
*** alonma has quit IRC01:07
*** yangyapeng has quit IRC01:16
*** yangyapeng has joined #openstack-cinder01:22
*** cknight has joined #openstack-cinder01:28
*** zhangguoqing has joined #openstack-cinder01:30
*** yangyapeng has quit IRC01:31
openstackgerritwangxiyuan proposed openstack/cinder: Remove "service" from required_import_options  https://review.openstack.org/39715501:31
*** yangyapeng has joined #openstack-cinder01:32
*** cknight has quit IRC01:33
*** chris_morrell has joined #openstack-cinder01:38
openstackgerritwanghao proposed openstack/cinder-specs: Transfer snapshots with volumes  https://review.openstack.org/31983101:39
*** dave-mccowan has quit IRC01:45
*** huanxuan has joined #openstack-cinder01:49
*** markvoelker has quit IRC01:50
*** kfox1111 is now known as kfox1111_away01:51
openstackgerritxianming.mao proposed openstack/os-brick: Add developer docs url in README.rst(trivial)  https://review.openstack.org/37609101:56
*** gcb has joined #openstack-cinder02:00
*** chris_morrell has quit IRC02:02
*** asselin has joined #openstack-cinder02:03
*** victorhe has joined #openstack-cinder02:03
openstackgerritTommyLike proposed openstack/cinder-specs: Cinder volume revert to snapshot  https://review.openstack.org/31654002:13
*** alonma has joined #openstack-cinder02:13
*** alonma has quit IRC02:18
*** markvoelker has joined #openstack-cinder02:19
openstackgerritWang Shilong proposed openstack/os-brick: Add lustre mount type support  https://review.openstack.org/39747302:20
*** alonma has joined #openstack-cinder02:24
*** mnaser has quit IRC02:24
openstackgerritWang Shilong proposed openstack/cinder: Add cinder Driver for Lustre  https://review.openstack.org/39557202:27
*** alonma has quit IRC02:28
*** victorhe has quit IRC02:32
openstackgerritwangxiyuan proposed openstack/cinder: Delete the snapshot when the volume has no host  https://review.openstack.org/39748502:33
openstackgerritwangxiyuan proposed openstack/cinder: Delete the snapshot when the volume has no host  https://review.openstack.org/39748502:34
openstackgerritwangxiyuan proposed openstack/cinder: Delete the snapshot when the volume has no host  https://review.openstack.org/39748502:36
*** mnaser has joined #openstack-cinder02:36
openstackgerritwangxiyuan proposed openstack/cinder: Delete the snapshot when the volume has no host  https://review.openstack.org/39748502:37
*** namnh has joined #openstack-cinder02:37
openstackgerritxing-yang proposed openstack/cinder: WIP: Migrate consistency groups to groups  https://review.openstack.org/35035002:40
*** namnh has quit IRC02:40
*** namnh has joined #openstack-cinder02:40
*** alonma has joined #openstack-cinder02:44
*** asselin has quit IRC02:46
openstackgerritxing-yang proposed openstack/cinder: WIP: Migrate consistency groups to groups  https://review.openstack.org/35035002:46
*** alonma has quit IRC02:48
*** links has joined #openstack-cinder02:49
*** Lee1092 has joined #openstack-cinder02:55
*** bkopilov_ has quit IRC02:57
*** kaisers_ has joined #openstack-cinder02:59
*** bardia has quit IRC03:01
openstackgerritwangxiyuan proposed openstack/cinder: Skip more parameters when import backup record  https://review.openstack.org/39749503:03
*** kaisers_ has quit IRC03:04
*** namnh has quit IRC03:04
*** jwcroppe has quit IRC03:05
*** jwcroppe has joined #openstack-cinder03:05
*** jwcroppe has quit IRC03:06
*** sudipto has joined #openstack-cinder03:06
*** jamielennox is now known as jamielennox|away03:07
*** jwcroppe has joined #openstack-cinder03:08
*** GB21 has joined #openstack-cinder03:08
*** cknight has joined #openstack-cinder03:17
*** GB21 has quit IRC03:19
*** jamielennox|away is now known as jamielennox03:21
*** cknight has quit IRC03:21
*** cknight has joined #openstack-cinder03:28
openstackgerritJohnnyChou proposed openstack/cinder: Infortrend - Add MPIO support for iSCSI  https://review.openstack.org/39428403:31
*** cknight has quit IRC03:34
*** zengyingzhe has quit IRC03:36
*** zengyingzhe has joined #openstack-cinder03:37
*** udesale has joined #openstack-cinder03:42
*** nicolasbock has quit IRC03:54
*** alonma has joined #openstack-cinder03:55
*** kaisers_ has joined #openstack-cinder03:55
*** alonma has quit IRC03:59
*** kaisers_ has quit IRC03:59
*** alonma has joined #openstack-cinder04:05
openstackgerritDao Cong Tien proposed openstack/cinder: Stop using mox in unit/api/v1/ tests  https://review.openstack.org/35618504:08
*** psachin has joined #openstack-cinder04:09
*** alonma has quit IRC04:09
*** jamielennox is now known as jamielennox|away04:15
*** jamielennox|away is now known as jamielennox04:17
*** bkopilov_ has joined #openstack-cinder04:17
jgriffithpatrickeast: Whaaaa, no shit?04:35
jgriffithpatrickeast: Swanson my bad, sorry04:35
patrickeastjgriffith: haha, yea, my bad on that one.. I added it to devstack so it would be tested in the gate04:36
patrickeastSo gate lvm jobs are using it04:36
patrickeastAnd anyone else who doesn't disable it or implement clone image04:37
jgriffithpatrickeast: well there ya have it04:38
jgriffithpatrickeast: the only problem I have with it is the effect of a *real* cloud where you get like 10 of those requests at once04:38
jgriffithpatrickeast: but not a big deal; and not a problem with the impl itself at all04:39
patrickeastYea, if we had a more centralized place to orchestrate we could do smarter things04:39
jgriffithpatrickeast: like local functional tests perhaps?04:40
patrickeastFrom what I've seen so far though is other than poc's it's rare to boot a bunch of never before seen images04:40
*** jamielennox is now known as jamielennox|away04:40
jgriffithpatrickeast: yeah, the problem in my case is I have people doing like 10/20 at a time04:41
jgriffithpatrickeast: ie 10+ clones at a time04:41
jgriffithwhich does have some issues in my case :(04:41
patrickeastjgriffith: oh really?04:41
jgriffithyeah :(04:41
patrickeastjgriffith: hrm maybe we just need some sort of throttling04:41
jgriffithpatrickeast: well; it's case by case right04:42
patrickeastAlthough I guess it's back to the orchestration pieces then, especially in our brave new aa ha world04:42
jgriffithI put some things in on my side to deal with it, but then you run the risk of RPC timeout if you have enough clone calls04:42
patrickeastGotcha04:42
jgriffithpatrickeast: and that's the other piece that has me concerned BTW04:42
jgriffithpatrickeast: the AA piece04:43
patrickeastWell, the good news is that it still "just works" (after geguileo fixed the queries)04:43
patrickeastjgriffith: it still has the problem of getting duplicate images in the cache when parallel requests come in the first time04:44
patrickeastjgriffith: but it will keep going in that case as if they all got cache misses and theoretically the extras get evicted for space later as needed04:45
jgriffithpatrickeast: that is good04:45
jgriffithi mean that like "that IS good"04:45
*** sudipto has quit IRC04:52
*** shausy has joined #openstack-cinder04:59
*** laughter_ has joined #openstack-cinder05:00
*** diablo_rojo has joined #openstack-cinder05:01
*** jamielennox|away is now known as jamielennox05:03
*** laughterwym has quit IRC05:03
*** khamtamtun has joined #openstack-cinder05:04
*** vladani has joined #openstack-cinder05:09
*** pgadiya has joined #openstack-cinder05:15
*** alonma has joined #openstack-cinder05:15
*** cknight has joined #openstack-cinder05:17
*** laughter_ has quit IRC05:19
*** laughterwym has joined #openstack-cinder05:19
*** alonma has quit IRC05:20
*** khamtamtun has quit IRC05:21
*** cknight has quit IRC05:22
*** sudipto has joined #openstack-cinder05:23
*** pdeore has joined #openstack-cinder05:23
*** Poornima has joined #openstack-cinder05:25
*** Julien-zte has quit IRC05:34
*** Julien-zte has joined #openstack-cinder05:36
*** pbandark has joined #openstack-cinder05:43
*** alonmarx has joined #openstack-cinder05:45
*** kaisers_ has joined #openstack-cinder05:51
*** wanghao_ has joined #openstack-cinder05:53
*** vladani has quit IRC05:53
*** vladani has joined #openstack-cinder05:54
*** kaisers_ has quit IRC05:55
*** bhagyashris has quit IRC05:56
*** sandanar has joined #openstack-cinder05:56
openstackgerritwanghao proposed openstack/cinder: Backup project attribute support  https://review.openstack.org/25837405:56
*** bardia has joined #openstack-cinder06:04
*** bardia has quit IRC06:09
*** Poornima has quit IRC06:12
*** kwathore__ has joined #openstack-cinder06:14
*** kwathore_ has quit IRC06:14
*** alonma has joined #openstack-cinder06:16
*** khamtamtun has joined #openstack-cinder06:17
*** alonma has quit IRC06:20
*** bks has joined #openstack-cinder06:21
*** bardia has joined #openstack-cinder06:30
openstackgerritWalter A. Boring IV (hemna) proposed openstack/cinder: Added fczm decorators  https://review.openstack.org/39465506:33
*** Poornima has joined #openstack-cinder06:35
*** khamtamtun has quit IRC06:35
*** bardia has quit IRC06:35
*** mjura has joined #openstack-cinder06:43
*** khamtamtun has joined #openstack-cinder06:48
*** Poornima has quit IRC06:50
*** lpetrut has joined #openstack-cinder06:50
*** kaisers_ has joined #openstack-cinder06:51
*** Poornima has joined #openstack-cinder06:54
*** nadeem_ has joined #openstack-cinder06:59
*** stakeda has joined #openstack-cinder07:00
sudiptohi any one who can help here about scsi_id?07:01
*** nherciu has joined #openstack-cinder07:01
*** bardia has joined #openstack-cinder07:02
*** vladani_ has joined #openstack-cinder07:03
*** vladani has quit IRC07:05
*** cknight has joined #openstack-cinder07:06
*** nadeem_ has quit IRC07:06
*** gaurangt has joined #openstack-cinder07:09
*** bardia has quit IRC07:10
*** arecknag has joined #openstack-cinder07:11
*** ChubYann has quit IRC07:12
*** cknight has quit IRC07:12
*** alonma has joined #openstack-cinder07:17
*** bhagyashris has joined #openstack-cinder07:17
*** alonma has quit IRC07:21
*** rcernin has quit IRC07:23
openstackgerritwangxiyuan proposed openstack/cinder: Delete the snapshot when the volume has no host  https://review.openstack.org/39748507:24
openstackgerritLisaLi proposed openstack/cinder: Update detach_volume() with versionedobjects  https://review.openstack.org/26860907:25
openstackgerritLisaLi proposed openstack/cinder: Update attach_volume() with versionedobjects  https://review.openstack.org/26860807:25
*** nadeem_ has joined #openstack-cinder07:32
*** belmoreira has joined #openstack-cinder07:33
*** alonma has joined #openstack-cinder07:37
*** alonmarx has quit IRC07:38
*** ociuhandu has quit IRC07:39
*** alonma has quit IRC07:41
*** alonma has joined #openstack-cinder07:43
*** nadeem_ has quit IRC07:47
openstackgerritPony Chou proposed openstack/cinder: Add QNAP ES Storage Driver  https://review.openstack.org/30987207:49
*** kaisers_ has quit IRC07:54
*** nadeem_ has joined #openstack-cinder08:04
*** lpetrut has quit IRC08:06
openstackgerritTina Tang proposed openstack/cinder: Adds Unity Cinder Driver  https://review.openstack.org/39647708:10
*** diablo_rojo has quit IRC08:10
*** gaurangt has quit IRC08:14
*** chenying has quit IRC08:20
*** pcaruana has joined #openstack-cinder08:22
*** amoralej|off is now known as amoralej08:22
*** e0ne has joined #openstack-cinder08:24
*** e0ne has quit IRC08:25
*** lixiaoy1 has quit IRC08:27
*** vladani_ has quit IRC08:29
*** vladani_ has joined #openstack-cinder08:29
*** Zer0Byte has joined #openstack-cinder08:30
*** Zer0Byte has quit IRC08:31
*** zul has quit IRC08:33
*** laughterwym has quit IRC08:41
openstackgerritMichal Dulko proposed openstack/cinder: Add generic code for online data migrations  https://review.openstack.org/33039108:41
*** khushbu_ has joined #openstack-cinder08:42
*** asselin has joined #openstack-cinder08:42
*** zul has joined #openstack-cinder08:45
*** kaisers_ has joined #openstack-cinder08:47
*** kaisers_ has quit IRC08:48
*** kaisers_ has joined #openstack-cinder08:48
*** markvoelker has quit IRC08:49
*** yangyapeng has quit IRC08:52
*** cknight has joined #openstack-cinder08:56
*** gouthamr has joined #openstack-cinder08:57
*** yangyapeng has joined #openstack-cinder08:58
*** zzzeek has quit IRC09:00
*** yangyape_ has joined #openstack-cinder09:00
*** yangyapeng has quit IRC09:00
*** zzzeek has joined #openstack-cinder09:00
*** gouthamr has quit IRC09:03
*** chenying has joined #openstack-cinder09:03
*** cknight1 has joined #openstack-cinder09:07
*** gouthamr has joined #openstack-cinder09:08
*** yangyapeng has joined #openstack-cinder09:08
*** yangyape_ has quit IRC09:09
*** cknight has quit IRC09:10
*** khushbu_ has quit IRC09:11
*** pranalik has joined #openstack-cinder09:13
*** mvk has quit IRC09:13
*** gouthamr_ has joined #openstack-cinder09:15
*** mvk has joined #openstack-cinder09:15
*** pranalik has quit IRC09:15
*** openstackgerrit has quit IRC09:18
*** gouthamr has quit IRC09:18
*** openstackgerrit has joined #openstack-cinder09:19
*** clsacramento has joined #openstack-cinder09:21
openstackgerritwanghao proposed openstack/cinder: Backup project attribute support  https://review.openstack.org/25837409:21
*** alonma has quit IRC09:26
*** alonma has joined #openstack-cinder09:27
*** vladani_ has quit IRC09:29
*** vladani_ has joined #openstack-cinder09:29
*** alonma has quit IRC09:30
*** alonma_ has joined #openstack-cinder09:30
*** cknight1 has quit IRC09:30
*** kaisers_ has quit IRC09:31
openstackgerritwanghao proposed openstack/cinder: Backup project attribute support  https://review.openstack.org/25837409:31
*** cknight has joined #openstack-cinder09:33
*** gouthamr_ has quit IRC09:33
*** gouthamr has joined #openstack-cinder09:36
openstackgerritDao Cong Tien proposed openstack/cinder: Remove use of timeutils.set_time_override  https://review.openstack.org/39762109:41
*** stakeda has quit IRC09:44
*** udesale has quit IRC09:45
*** markvoelker has joined #openstack-cinder09:49
*** cknight has quit IRC09:49
*** cknight1 has joined #openstack-cinder09:50
*** asselin has quit IRC09:51
*** markvoelker has quit IRC09:55
*** udesale has joined #openstack-cinder09:56
*** gouthamr has quit IRC10:05
*** gouthamr has joined #openstack-cinder10:06
*** Poornima has quit IRC10:09
*** cknight1 has quit IRC10:10
*** cknight has joined #openstack-cinder10:10
*** vladani_ has quit IRC10:13
*** vladani_ has joined #openstack-cinder10:13
*** Julien-z_ has joined #openstack-cinder10:15
*** Julien-zte has quit IRC10:16
*** khushbu_ has joined #openstack-cinder10:17
*** e0ne has joined #openstack-cinder10:19
*** gouthamr has quit IRC10:20
openstackgerritBéla Vancsics proposed openstack/cinder: Reduced the complexity of the main() method  https://review.openstack.org/25091410:21
*** gouthamr has joined #openstack-cinder10:24
*** Poornima has joined #openstack-cinder10:24
*** khushbu_ has quit IRC10:26
*** alonma_ has quit IRC10:28
*** alonma has joined #openstack-cinder10:29
*** alonma has quit IRC10:33
*** laughterwym has joined #openstack-cinder10:33
*** gouthamr has quit IRC10:34
*** watanabe_isao has joined #openstack-cinder10:35
*** huanxuan has quit IRC10:36
*** laughterwym has quit IRC10:37
*** khamtamtun has quit IRC10:39
*** nikeshm has joined #openstack-cinder10:39
nikeshmhi10:39
nikeshmwhat is report_discard_supported used for10:39
nikeshmwhere to use this, in controller node or cinder-volume node10:40
*** cknight has quit IRC10:40
*** cknight has joined #openstack-cinder10:41
*** cknight has quit IRC10:41
nikeshmin case of multiple backends, where to use this10:42
*** gouthamr has joined #openstack-cinder10:44
*** gouthamr_ has joined #openstack-cinder10:47
*** alonma has joined #openstack-cinder10:49
*** gouthamr has quit IRC10:50
openstackgerritJohnnyChou proposed openstack/cinder: Infortrend - Add MPIO support for iSCSI  https://review.openstack.org/39428410:51
*** udesale has quit IRC10:53
*** alonma has quit IRC10:53
*** alonma has joined #openstack-cinder10:55
*** gouthamr_ has quit IRC10:58
*** lpetrut has joined #openstack-cinder10:59
*** alonma has quit IRC10:59
*** khamtamtun has joined #openstack-cinder11:01
*** Poornima has quit IRC11:02
*** Poornima has joined #openstack-cinder11:03
*** alonma has joined #openstack-cinder11:06
*** mvk has quit IRC11:16
*** khushbu_ has joined #openstack-cinder11:19
*** gouthamr has joined #openstack-cinder11:25
*** ntpttr has quit IRC11:30
*** laughterwym has joined #openstack-cinder11:31
*** kaisers_ has joined #openstack-cinder11:31
*** gouthamr has quit IRC11:31
*** ntpttr has joined #openstack-cinder11:31
*** kaisers_ has quit IRC11:35
*** houming has joined #openstack-cinder11:40
*** vladani_ has quit IRC11:40
*** vladani_ has joined #openstack-cinder11:40
*** pdeore has quit IRC11:42
*** nicolasbock has joined #openstack-cinder11:42
*** links has quit IRC11:43
*** mvk has joined #openstack-cinder11:47
*** houming has quit IRC11:49
openstackgerritMerged openstack/cinder: Drop unused parameter in class EntryCreateTask  https://review.openstack.org/38476611:50
*** markvoelker has joined #openstack-cinder11:51
openstackgerritMerged openstack/cinder: Remove unnecessary setUp and tearDown  https://review.openstack.org/37289511:51
*** khushbu_ has quit IRC11:52
*** markvoelker has quit IRC11:55
*** bks has quit IRC11:57
*** khushbu_ has joined #openstack-cinder12:03
*** shausy has quit IRC12:04
*** mars_ has joined #openstack-cinder12:10
*** mars_ has quit IRC12:10
*** mars_ has joined #openstack-cinder12:11
*** yangyapeng has quit IRC12:11
*** mars_ has quit IRC12:11
*** nikeshm has quit IRC12:13
*** watanabe_isao has quit IRC12:13
*** vladani__ has joined #openstack-cinder12:19
*** vladani_ has quit IRC12:19
*** gcb has quit IRC12:19
*** sudipto has quit IRC12:20
*** catintheroof has joined #openstack-cinder12:28
*** Julien-z_ has quit IRC12:31
*** bkopilov_ has quit IRC12:31
openstackgerritwanghao proposed openstack/cinder: Backup project attribute support  https://review.openstack.org/25837412:32
*** Poornima has quit IRC12:34
*** khushbu_ has quit IRC12:37
*** gouthamr has joined #openstack-cinder12:42
*** alonma has quit IRC12:47
*** alonma has joined #openstack-cinder12:48
*** nadeem_ has quit IRC12:48
*** kaisers_ has joined #openstack-cinder12:49
*** kaisers_ has quit IRC12:49
*** kaisers_ has joined #openstack-cinder12:49
*** khushbu_ has joined #openstack-cinder12:50
*** alonma has quit IRC12:52
*** khushbu_ has quit IRC12:55
*** alonma has joined #openstack-cinder12:56
*** e0ne has quit IRC12:56
*** e0ne has joined #openstack-cinder12:56
*** gouthamr has quit IRC12:59
*** pdeore has joined #openstack-cinder12:59
*** vladani__ has quit IRC13:01
*** dave-mccowan has joined #openstack-cinder13:03
*** vladani has joined #openstack-cinder13:04
*** sudipto has joined #openstack-cinder13:04
*** diablo_rojo_phon has joined #openstack-cinder13:07
*** xyang1 has joined #openstack-cinder13:07
*** ducttape_ has joined #openstack-cinder13:08
*** pdeore has quit IRC13:12
*** kwathore_ has joined #openstack-cinder13:15
*** khamtamtun has quit IRC13:16
*** kwathore__ has quit IRC13:18
*** edmondsw has joined #openstack-cinder13:18
openstackgerritMichal Dulko proposed openstack/cinder: Rolling upgrade procedure documentation  https://review.openstack.org/39339513:21
*** pgadiya has quit IRC13:27
*** markvoelker has joined #openstack-cinder13:28
*** gouthamr has joined #openstack-cinder13:31
*** ducttape_ has quit IRC13:35
*** sborkows has joined #openstack-cinder13:35
*** ducttape_ has joined #openstack-cinder13:35
*** kaisers__ has joined #openstack-cinder13:36
flip214How can I connect a volume to a hypervisor? "cinder" and "openstack" cli don't have any matching options, unless I misread them somehow13:38
*** kaisers_ has quit IRC13:38
*** ducttape_ has quit IRC13:40
*** zhangguoqing has quit IRC13:40
*** pbandark has quit IRC13:41
dulekflip214: http://docs.openstack.org/developer/python-brick-cinderclient-ext/readme.html13:42
*** jwcroppe has quit IRC13:47
flip214dulek: thanks, reading13:48
flip214"cinder local-attach", thank you very much13:49
flip214uh, what? "ERROR: This command requies root permissions." [sic!]13:50
flip214I thought that would just talk to cinder, too?!13:50
dulekflip214: I think e0ne can help you here. :)13:51
flip214and "requies" is a nice touch, too ;/13:51
e0nedulek, flip214: I'm on the meeting at the moment. I'll ping you a bit later13:51
*** kwathore_ has quit IRC13:56
*** kwathore has joined #openstack-cinder13:56
*** khamtamtun has joined #openstack-cinder13:58
smcginnisflip214: That command runs things directly instead of through the service.14:00
smcginnisflip214: As it's meant to be run on any given host.14:00
smcginnisflip214: So you need to run it with sudo so it can make the system changes needed.14:00
*** udesale has joined #openstack-cinder14:00
flip214smcginnis: oh, thanks.14:00
flip214  usage: cinder local-attach [--hostname <hostname>] [--mountpoint <mountpoint>]14:00
flip214I thought that --hostname would do what it implies ;/14:01
smcginnisflip214: Hmm, if you're doing a "local" attach, not sure what that is for.14:01
* smcginnis pulls up the source14:02
openstackgerritSean McGinnis proposed openstack/python-brick-cinderclient-ext: Fix typo in root permissions message  https://review.openstack.org/39775714:04
flip214^^ smcginnis: want a "please make a bug for that" review?? ;P14:07
smcginnisflip214: We don't require a bug filed for fixing obvious typos, but if you want to, go ahead. ;)14:07
flip214smcginnis: *I* don't want that. Only causes additional workload.14:07
dulek"requies"cat in pace if you're not root? ;)14:08
smcginnisflip214: I believe it's because the attach call into cinder required a hostname.14:10
smcginnisflip214: Well, optionally takes a host name.14:10
smcginnisflip214: I guess so we can keep track of that, even when it is a local attach.14:10
flip214well, is there some CLI to attach via cinder?14:13
flip214because that's what I'd like to test/debug14:13
*** ducttape_ has joined #openstack-cinder14:13
*** pcaruana has quit IRC14:13
*** dustins has joined #openstack-cinder14:14
*** gouthamr has quit IRC14:15
*** khamtamtun has quit IRC14:16
*** mriedem has joined #openstack-cinder14:16
smcginnisflip214: Only our normal path through Nova. Otherwise it's the local one with the brick extension.14:18
*** psachin has quit IRC14:18
*** sudipto has quit IRC14:18
smcginnisflip214: Someone needs to perform the compute side logging in, rescanning, etc. Can you explain a little more what you're trying to do?14:18
*** kwathore has quit IRC14:18
*** ducttape_ has quit IRC14:18
flip214smcginnis: I'd like to test/debug some attach/detach code in the DRBD driver.14:19
*** kwathore has joined #openstack-cinder14:19
flip214in my shell "cinder" and "openstack" CLIs work, but "nova" says "ERROR (AttributeError): 'Namespace' object has no attribute 'os_user_id'"14:19
smcginnisflip214: Hm, then I think those are you're only two options at the moment - do the normal path through Nova or perform it locally via sudo on the node that you want to mount to.14:20
flip214which is right, but why wouldn't nova be satisfied with the same credentials as everybody else?14:20
flip214smcginnis: I'd be happy with the nova path... if only it did work ;/14:20
smcginnisflip214: :/14:20
*** yangyapeng has joined #openstack-cinder14:23
e0neflip214: I'm back and ready to discuss local-attach if you have any questions14:23
flip214e0ne: do you know why the "nova" CLI would give me "ERROR (AttributeError): 'Namespace' object has no attribute 'os_user_id'"14:24
flip214although "cinder" and "openstack" CLI tools work?14:24
e0neflip214: I didn't see this issue yet14:24
flip214my devstack environment has that14:26
smcginnisflip214: Might have to bring it up with the nova folks. Maybe it's a known issue they're fixing.14:26
smcginnisscottda: ping14:27
flip214thanks, anyway!14:28
johnthetubaguyisn't OS_USERNAME the normal one?14:28
*** bkopilov_ has joined #openstack-cinder14:29
flip214johnthetubaguy: yeah, and this one is set14:30
johnthetubaguyhmm, thats odd14:30
*** easyasabc123 has joined #openstack-cinder14:34
*** easyasabc123 has quit IRC14:34
*** asselin has joined #openstack-cinder14:36
*** victorhe has joined #openstack-cinder14:37
*** yangyapeng has quit IRC14:39
*** ducttape_ has joined #openstack-cinder14:40
*** gouthamr has joined #openstack-cinder14:40
*** cknight has joined #openstack-cinder14:41
*** arecknag has quit IRC14:42
openstackgerritMerged openstack/python-brick-cinderclient-ext: Fix typo in root permissions message  https://review.openstack.org/39775714:43
*** amoralej is now known as amoralej|lunch14:43
*** eharney has joined #openstack-cinder14:45
*** eharney has quit IRC14:45
openstackgerritxing-yang proposed openstack/cinder: WIP: Migrate consistency groups to groups  https://review.openstack.org/35035014:45
*** eharney has joined #openstack-cinder14:45
*** khamtamtun has joined #openstack-cinder14:45
*** khushbu_ has joined #openstack-cinder14:48
*** khamtamtun has quit IRC14:50
dulekxyang1: Whoops, I think I've somehow missed some of your changes in api.py while reviewing. I will need to get back to it, but I think that my question about what happens to CGs calls still applies.14:51
*** ducttape_ has quit IRC14:52
xyang1dulek: hi, I just resolved merge conflicts.  looking at your comments now14:52
xyang1dulek: my later patches should address your question on CGs14:53
*** victorhe_ has joined #openstack-cinder14:53
*** udesale has quit IRC14:54
*** victorhe_ has quit IRC14:55
dulekxyang1: It may be tricky to split them safely for people deploying continuously from trunk, but I think we can address that in release notes by recommending waiting for all the patches to merge.14:55
*** victorhe_ has joined #openstack-cinder14:55
dulekBefore upgrading.14:55
xyang1dulek: sure14:55
*** jwcroppe has joined #openstack-cinder14:56
*** victorhe has quit IRC14:56
*** victorhe_ has quit IRC14:57
*** victorhe has joined #openstack-cinder14:57
*** victorhe has quit IRC14:57
*** victorhe has joined #openstack-cinder14:58
*** victorhe has quit IRC14:58
*** victorhe has joined #openstack-cinder14:58
*** kwathore has quit IRC14:59
*** kwathore has joined #openstack-cinder14:59
*** cknight has quit IRC14:59
smcginniseharney: ping15:00
eharneysmcginnis: hey15:00
smcginniseharney: Hey, question for you in regards to RHOSP, if you know.15:00
eharneysure15:01
smcginniseharney: For requirements handling, do they package the latest version?15:01
smcginniseharney: This is in regards to this: https://review.openstack.org/#/c/382573/215:01
*** ducttape_ has joined #openstack-cinder15:01
smcginniseharney: The latest oslo.concurrency library has what's needed, but requirements were never raised on that branch.15:01
eharneysmcginnis: well, in that case, we tracked the required changes to make prlimit work in oslo.concurrency etc., and handled that in the RPM packaging15:02
smcginniseharney: So it is possible someone could be running with an older version of osc.15:02
smcginniseharney: Cool, so it's not an issue for Red Hat deployments?15:02
eharneysmcginnis: correct15:02
smcginniseharney: Awesome, that's what I was hoping you'd say. ;)15:02
eharneysmcginnis: we're pretty good at managing this kind of fix :)15:03
*** Julien-zte has joined #openstack-cinder15:03
*** khushbu_ has quit IRC15:04
eharneysmcginnis: looking at the patchset though... kind of interesting15:04
*** enriquetaso has joined #openstack-cinder15:04
smcginniseharney: I would have thought so, but good to confirm.15:04
smcginniseharney: Oh?15:04
eharneysmcginnis: how Nova added some code to handle the dependency issue as a soft dependency15:04
*** alonma has quit IRC15:04
smcginniseharney: It was a good way to do it.15:05
*** jdillaman1 has quit IRC15:05
smcginniseharney: But this late in the game, I'd rather not try to make a change if it's not strictly needed.15:05
*** alonma has joined #openstack-cinder15:05
eharneysmcginnis: the update is available for oslo.concurrency, and the security problem is a DoS at worst, so i'm not super worried about it15:07
smcginnisxyang1: ping15:07
smcginniseharney: Yeah, kinda my thought too.15:08
xyang1smcginnis: hi15:08
smcginnisxyang1: Morning! Just noticed there is a question in the Cinder meeting agenda that we (you) might be able to easily answer beforehand.15:09
*** jgregor has joined #openstack-cinder15:09
smcginnisxyang1: If you have time.15:09
smcginnisxyang1: Otherwise fine to have it covered in the meeting. Just don't want to spend everyone's time if it's something simple.15:09
xyang1smcginnis: sure, what question15:09
smcginnis:)15:09
*** alonma has quit IRC15:09
smcginnisxyang1: From Steve: https://wiki.openstack.org/wiki/CinderMeetings#Next_Cinder_Team_meeting15:09
*** breitz has quit IRC15:10
*** jdillaman has joined #openstack-cinder15:10
*** xyang has joined #openstack-cinder15:10
*** nherciu has quit IRC15:15
*** kwathore_ has joined #openstack-cinder15:15
*** kwathore has quit IRC15:16
*** gouthamr has quit IRC15:16
xyang1smcginnis: this is very similar to volumes APIs15:17
xyang1smcginnis: what are they doing with create volume?15:18
*** wanghao_ has quit IRC15:21
*** nkrinner_afk has quit IRC15:24
smcginnisxyang1: Not sure15:24
xyang1smcginnis: is stevemar is here, we can ask him15:25
stevemaro/15:25
xyang1stevemar: you added an item for CG in meeting agenda?15:26
*** alonma has joined #openstack-cinder15:26
stevemarxyang1: yeah15:26
stevemarxyang1: those arguments (user id and project) in cinderclient are not used by anything15:26
xyang1stevemar: so we can handle it the same way as volume create15:26
xyang1stevemar: what are you doing with volume create?15:27
*** sandanar has quit IRC15:27
xyang1stevemar: this part is very similar to volume create15:28
stevemarxyang1: which also has no support for specifying user id and project id in cinder CLI15:28
stevemarxyang1: forget about OSC for now :)15:28
xyang1stevemar: :)15:29
stevemarxyang1: you provide the user id and project in the python bindings right?15:29
stevemarxyang1: why not the shell? https://github.com/openstack/python-cinderclient/blob/master/cinderclient/v3/shell.py#L348-L44015:29
xyang1stevemar: you are right, it is not doing anything.  I think I copied from volume create15:30
*** nkrinner_afk has joined #openstack-cinder15:30
*** slayer has joined #openstack-cinder15:30
*** porrua has quit IRC15:30
stevemarxyang1: so volume create also has them for the python binding call: https://github.com/openstack/python-cinderclient/blob/master/cinderclient/v3/volumes.py#L239-L24015:30
*** alonma has quit IRC15:30
stevemarand they are passed into the body15:31
stevemarmy question is, what does the cinder API actually do with those?15:31
stevemardoes the cinder API actually look up the user id on keystone? or does it get it from the token?15:32
xyangstevemar: in the API, it takes it from the context15:32
stevemarif i call cinderclient.volumes.create(foo, user_id=fake_user) then what happens15:32
stevemarso it only works for the user that is making the call?15:32
*** jgregor_ has joined #openstack-cinder15:33
stevemari'm an admin, i can't create a volume for someone else?15:33
xyang1stevemar: yes15:33
stevemari saw this and thought it looked funny, i don't know if it's actually correct or not. just wanted to bring it up15:33
xyangstevemar: are you handling it differently in OSC15:35
stevemarxyang1: we call the python bindings, so we're subject to whatever you guys are doing :)15:36
xyangstevemar: ok15:36
*** jgregor has quit IRC15:36
stevemarlet me see if i can find the cinder code..15:36
*** jgregor has joined #openstack-cinder15:39
*** porrua has joined #openstack-cinder15:40
*** jgregor_ has quit IRC15:42
*** mwhahaha has joined #openstack-cinder15:43
mwhahahahey cinder folks, so I wanted to bring https://bugs.launchpad.net/cinder/+bug/1641312 to your attention as we're hitting it quite frequently in the puppet openstack CI.15:43
openstackLaunchpad bug 1641312 in Cinder "CleanableInUse error in a deployment with just one cinder-volume" [Undecided,New]15:43
*** jungleboyj has joined #openstack-cinder15:43
stevemarxyang1: smcginnis so it looks like you always get the user id and project id from the auth context -- https://github.com/openstack/cinder/blob/7c95f7222e9d11e05f4d0c797e9c6c6fc04c6b44/cinder/volume/api.py#L107 and https://github.com/openstack/cinder/blob/a15459a27777e98599d70392191b0a6f0288ff34/cinder/context.py#L11815:44
xyangstevemar: yes15:44
stevemarxyang1: smcginnis providing a user and project argument in the client bindings is useless then :)15:45
stevemarhttps://github.com/openstack/python-cinderclient/blob/master/cinderclient/v3/volumes.py#L239-L24015:45
stevemaryou cannot override it, and you don't need to supply any, since they are in headers via the context15:45
xyangstevemar: I wonder if anyone else knows the history of this15:46
stevemari don't know if theres a usecase (where an admin can create a volume for someone) you want to support, or if you're fine with this then you'll have to deprecate and remove the args15:46
*** khushbu_ has joined #openstack-cinder15:46
stevemarmaybe it was planned and never done15:46
xyangjgriffith: around?15:46
smcginnisgeguileo: Mind taking a look at the comments on the bug from mwhahaha? ^^15:46
stevemarxyang1: smcginnis want me to file a bug?15:47
jgriffithxyang: I am15:47
smcginnisstevemar: That might be good for now while we sort things out.15:47
stevemarsmcginnis: okay15:47
smcginnisstevemar: Thank you15:47
xyangjgriffith: not sure if you are following the discussion above15:47
xyangjgriffith: we have user id and project id in CLI but we are not using them15:47
jgriffithxyang: wasn't... lemme read back15:48
xyangjgriffith: I don't know if there's a reason for that in the past15:48
*** openstackgerrit has quit IRC15:48
*** amoralej|lunch is now known as amoralej15:48
jgriffithxyang: no, I don't think there's a reason for that at all; not sure I'm caught up yet though either... hold on :)15:48
geguileosmcginnis: I'll have a look now15:48
xyangjgriffith: ok, thanks15:48
geguileosmcginnis: Because I think that's the same bug I've been fighting since yesterday  XD15:49
*** openstackgerrit has joined #openstack-cinder15:49
smcginnisgeguileo: :D15:49
geguileosmcginnis: Oh, it is because the reporter pinged me yesterday and we've been talking about it15:49
geguileosmcginnis: Just figured out the problem an hour ago or so15:50
smcginnisgeguileo: Nice!15:50
smcginnisgeguileo: Is it related to that patch?15:50
geguileosmcginnis: It's a bug in pymysql15:50
smcginnisgeguileo: Ruh roh15:50
geguileosmcginnis: That's fixed in 0.7.615:50
geguileosmcginnis: I'll take care of it15:50
smcginnisgeguileo: Is there a way to work around the bug? Or do we need to bump the minimum requirements for that lib?15:51
*** gouthamr has joined #openstack-cinder15:51
jgriffithxyang: I think that's an oversight; as in like it's cruft left over as we changed things15:52
xyangjgriffith: ok, so we can probably remove them?15:52
xyangjgriffith: unless if there's plan to actually use them in the future15:53
geguileosmcginnis: I need to bump global-requirements15:53
stevemarxyang1: jgriffith https://bugs.launchpad.net/cinder/+bug/164198215:53
openstackLaunchpad bug 1641982 in python-cinderclient "cinder only supports deriving a user and project id from auth context" [Undecided,New]15:53
smcginnisgeguileo: OK, easy enough I suppose. :)15:53
xyangstevemar: thanks15:53
jgriffithxyang: stevemar Better check with thingee it may have had something to do with some of the rework that he did in the past15:54
jgriffithas well as the disco pieces etc15:54
xyangjgriffith: ok15:54
geguileosmcginnis: Yup, though it was tought to find it, because it was a rounding issue that changed the microsecond when reading from the DB15:54
jgriffithxyang: stevemar but yeah, quick glance I don't think they're useful any longer15:54
stevemari'll leave it up to you guys now :)15:54
stevemarbug has been filed :D15:54
stevemaryou can remove me from the agenda15:54
jgriffithstevemar: thanks for catching it15:55
stevemarsmcginnis: ^15:55
stevemarjgriffith: np, the docstring looked a little funny15:55
stevemarthen i went digging...15:55
*** khushbu_ has quit IRC15:55
stevemarmistake #115:55
xyangstevemar: thanks for finding it15:55
jgriffithstevemar: that's how it always starts :)15:55
smcginnisstevemar: OK, will do. Thanks for bringing that up!15:55
smcginnisDisco pieces: http://gifs.alphacoders.com/gifs/view/334715:56
*** jdillaman has quit IRC15:56
xyang@smcginnis: :)15:56
*** asselin has quit IRC15:56
*** ManishD has joined #openstack-cinder15:57
jgriffithsmcginnis: LMAO15:57
jgriffithsmcginnis: I knew I shouldn't abbreviate that!!!15:58
jgriffithNow I'll have disco-cats in my brain all day15:58
smcginnisDoesn't matter, every time I see disco references something like that pops in my head. :)15:58
*** dave-mccowan has quit IRC15:59
jgriffithsmcginnis: it's actually kinda fitting15:59
smcginnis;)15:59
jgriffithsmcginnis: wonder if we could embed that gif in the docs15:59
smcginnisThat would be awesome.15:59
*** gouthamr has quit IRC16:00
*** nicolasbock has quit IRC16:01
*** rajinir has joined #openstack-cinder16:01
*** timcl has joined #openstack-cinder16:01
*** pcaruana has joined #openstack-cinder16:02
smcginnisscottda: ping16:04
*** sborkows has quit IRC16:04
*** jdillaman has joined #openstack-cinder16:05
smcginnise0ne: You still around?16:05
*** nicolasbock has joined #openstack-cinder16:05
*** gouthamr has joined #openstack-cinder16:07
*** alonma has joined #openstack-cinder16:08
smcginnisjgriffith: Can I get your thoughts on Tony's comments here: https://review.openstack.org/#/c/382573/216:09
smcginnisjgriffith: My inclination is: meh16:09
*** gouthamr_ has joined #openstack-cinder16:10
smcginniseharney confirmed RH is OK. Curious to find out about Helion and Mirantis, but guessing it's not an issue.16:10
*** vladani has quit IRC16:10
*** vladani has joined #openstack-cinder16:10
*** gouthamr_ has quit IRC16:11
e0nesmcginnis: yes16:11
smcginnise0ne: Hey, do you happen to know how Mirantis does packaging?16:11
smcginnise0ne: Trying to see if this is an issue: https://review.openstack.org/#/c/382573/216:11
*** gouthamr_ has joined #openstack-cinder16:11
*** gouthamr has quit IRC16:11
jgriffithsmcginnis: meh... so distros aside I'd be annoyed when this blew up.  Why not just add a follow up patch that adds the try/catch?16:12
*** asselin has joined #openstack-cinder16:12
smcginnise0ne: Minimum requirement wasn't raised to the needed release of oslo.concurrency, but not capped either.16:12
*** alonma has quit IRC16:12
e0nesmcginnis: we use versions from requirements.txt16:12
smcginnisjgriffith: Just thinking of the time merging to master and cherry picking back all the way to liberty. But it could certainly be done.16:13
smcginnisjgriffith: Had just hoped to wrap up liberty and be done with it. ;)16:13
jgriffithsmcginnis: I'll code it up and you and I can approve the chain :)16:13
smcginnisBut maybe I'm just being too lazy. Which is likely.16:13
jgriffithsmcginnis: or heck, just push straight to liberty AFAIC16:13
smcginnisjgriffith: Works for me.16:13
jgriffithsmcginnis: which :)16:13
smcginnisIt really is only an issue on LIberty.16:13
smcginnisThe other ones are above minimum required version for it.16:14
smcginnisSo I guess we could just patch Liberty directly.16:14
*** crose has joined #openstack-cinder16:14
jgriffithI'm game either way; I'll start with a bug and patch for liberty directly; if people object and we change our mind that's cool too16:15
*** ManishD has quit IRC16:15
SwansonReally wish I would have sorted out the extra volume thing last night as I've no idea what I was doing with it this morning...16:15
smcginnisjgriffith: Cool, I'll watch for it. Thanks!16:15
*** ManishD has joined #openstack-cinder16:15
smcginnisSwanson: I think you were about to do rm -fr /16:15
Swansonsmcginnis, can't fool me twice with that16:16
e0nesmcginnis, jgriffith: it sounds right for me if we need this patch only for liberty16:16
*** archies has joined #openstack-cinder16:16
smcginnise0ne: Yeah, I suppose that would be safer.16:16
*** dave-mccowan has joined #openstack-cinder16:17
*** ManishD has left #openstack-cinder16:18
jgriffithsmcginnis: https://bugs.launchpad.net/cinder/+bug/164199216:21
openstackLaunchpad bug 1641992 in Cinder "qemu img cpu limit backport not compatible with liberty oslo version" [Undecided,New]16:21
*** belmoreira has quit IRC16:22
*** dulek_ has joined #openstack-cinder16:23
*** sudipto has joined #openstack-cinder16:24
*** dulek_ has quit IRC16:25
archies@smcginnis: please can you help with suggestion on the bug: https://bugs.launchpad.net/nova/+bug/1636489.16:26
openstackLaunchpad bug 1636489 in OpenStack Compute (nova) "Volume attachment fails for all the available instances after running different volume operations for 1-2 hours or more." [Undecided,New]16:26
archies@smcginnis: for now, I have this bug raised on Nova but not sure which bracket it should fall.16:26
smcginnisarchies: Sure, I'll take a look.16:27
*** sudipto has quit IRC16:28
*** kfox1111_away is now known as kfox111116:28
*** jgregor_ has joined #openstack-cinder16:28
smcginnisarchies: Hmm, based on the analysis in there, it does look like something nova/libvirt related on first pass.16:29
smcginnisarchies: I see Matt just commented on there as well. That sounds like it might be a known issue?16:30
*** jgregor__ has joined #openstack-cinder16:30
*** jgregor has quit IRC16:31
*** nikeshm has joined #openstack-cinder16:31
*** alonmarx has joined #openstack-cinder16:32
*** e0ne has quit IRC16:33
*** e0ne has joined #openstack-cinder16:34
*** jgregor_ has quit IRC16:34
jgriffithsmcginnis: https://review.openstack.org/#/c/397820/16:37
jgriffithsmcginnis: I can't even get tox to setup a liberty env any more so I'm relying on the gate for testing :(16:38
smcginnisjgriffith: Yeah, not surprised there. :/16:41
smcginniseharney: Want to take a look too? ^^16:41
jgriffithsmcginnis: kind of annoying16:42
jgriffithOHHH16:42
jgriffithI know why :)16:42
smcginnisjgriffith: Part of the reason I just wanted it to go away. ;)16:42
nikeshmhi, what is report_discard_supported? In case of multiple backends, do we have to specify it in configuration group of each backend in cinder.conf16:42
jgriffithhmm... nope, I don't know why :)16:42
*** slayer has quit IRC16:43
*** slayer has joined #openstack-cinder16:44
*** jgregor__ has quit IRC16:44
smcginnisnikeshm: Yes, I think so. That tells nova if the backend supports discard. Mostly for thin provisioning to tell the storage to free up space when it's no longer being used.16:44
smcginnisjgriffith: Let's hope the gate has better luck.16:44
jgriffithsmcginnis: think I got it sorted, missing py3 stuff.  We'll see in a minute16:45
jgriffithsmcginnis: honestly we need to poop or get off the pot and move to py316:46
jgriffithIs that inappropriate for IRC ?  Hmmmm16:46
smcginnisIt's appropriate in this channel at least. ;)16:46
jgriffithoh well... I've said worse16:47
jgriffithsmcginnis: fair enough :)16:47
smcginnisFor sure! :D16:47
smcginnisI don't really like that py34 _and_ py35 have been added to the default list of things to run when you just call tox.16:47
smcginnisIf py35 is where we're going, I'd rather we just run that.16:47
smcginnisIt doesn't prevent the gate from running both.16:48
smcginnisBut one or the other locally.16:48
*** diablo_rojo_phon has quit IRC16:48
*** slayer has quit IRC16:49
jgriffithsmcginnis: huh; well got it to run, lots of errors but nothing I added :(16:51
*** archies has quit IRC16:51
*** slayer_ has joined #openstack-cinder16:51
jgriffithhttp://paste.openstack.org/show/589308/16:51
smcginnisjgriffith: Hmm. Newer pep8?16:52
*** bardia has joined #openstack-cinder16:52
jgriffithsmcginnis: yup16:52
jgriffithsmcginnis: interesting to see how much we "miss"16:53
jgriffithundefined name is typically a pretty bad thing I think16:53
jgriffithmost look bogus anyway16:55
*** slayer_ has quit IRC16:55
smcginnisThat's just odd: undefined name 'unicode'16:56
jgriffithI know right?16:56
jgriffithfriggin python :)16:56
smcginnis:)16:56
eharneyboth xrange and unicode are things that exist in python 2 and not 316:57
eharneyis that a py3 flake8 run?16:57
jgriffitheharney: yeah, that's what all of these appear to be16:57
jgriffitheharney: it apparantly is (unintentionally) which leads me back to my earlier statement about supporting both simultaneously kinda blows monkies16:58
*** cknight has joined #openstack-cinder16:58
jgriffithnot sure how that's getting invoked that way though; but I suppose it's irrelevant16:58
*** ckonstanski has joined #openstack-cinder16:59
*** kwathore__ has joined #openstack-cinder17:00
smcginnisI forget if that was accepted as an official "goal". Would be awesome if we could switch over to py3 exclusively in Pike or even Queens.17:02
smcginnisWe could probably get rid of a *ton* of code.17:02
eharneyhttps://wiki.openstack.org/wiki/Python3#Python_3_Status_of_OpenStack_projects17:02
*** haplo37_ has quit IRC17:03
smcginnisYeah, we're so close.17:03
*** kwathore_ has quit IRC17:03
*** mjura has quit IRC17:03
smcginnisInteresting there are Summit notes in there from back in Havana.17:04
nikeshmsmcginnis : thnks17:05
smcginnisnikeshm: no17:05
smcginnis.. problem :)17:05
hemnasmcginnis, is there an official timeline when we have to switch to py3?17:05
*** sudipto has joined #openstack-cinder17:05
nikeshm:)17:06
smcginnishemna: I know it was proposed as a cross-project goal, but I don't know if it was accepted.17:06
*** xyang has quit IRC17:07
smcginnishemna: Nope, not for Ocata: http://governance.openstack.org/goals/ocata/index.html17:07
smcginnishemna: So maybe soon.17:07
hemnaok17:08
*** porrua has quit IRC17:11
*** porrua has joined #openstack-cinder17:13
*** bardia has quit IRC17:18
*** e0ne has quit IRC17:21
*** laughterwym has quit IRC17:21
*** jwcroppe has quit IRC17:21
*** xyang has joined #openstack-cinder17:21
*** jwcroppe has joined #openstack-cinder17:22
*** timcl has quit IRC17:24
*** sudipto has quit IRC17:25
*** ChubYann has joined #openstack-cinder17:25
*** Julien-zte has quit IRC17:26
*** jwcroppe has quit IRC17:26
*** Julien-zte has joined #openstack-cinder17:26
*** jwcroppe has joined #openstack-cinder17:27
*** Julien-zte has quit IRC17:27
*** Julien-zte has joined #openstack-cinder17:27
*** Julien-zte has quit IRC17:27
*** Julien-zte has joined #openstack-cinder17:28
*** Julien-zte has quit IRC17:28
openstackgerritHelen Walsh proposed openstack/cinder: VMAX driver - Attach and detach snapshot  https://review.openstack.org/39384217:31
openstackgerritHelen Walsh proposed openstack/cinder: VMAX driver - MVs and SGs not reflecting correct protocol.  https://review.openstack.org/39565717:32
*** eharney has quit IRC17:35
*** jgregor has joined #openstack-cinder17:37
*** slayer has joined #openstack-cinder17:37
*** jgregor_ has joined #openstack-cinder17:37
*** sage_ has quit IRC17:38
*** Apoorva has joined #openstack-cinder17:38
*** mvk has quit IRC17:38
*** vladani_ has joined #openstack-cinder17:39
*** vladani has quit IRC17:39
*** bardia has joined #openstack-cinder17:40
*** xyang has quit IRC17:41
*** jgregor has quit IRC17:41
*** haplo37 has joined #openstack-cinder17:44
*** sage_ has joined #openstack-cinder17:46
*** jgregor_ has quit IRC17:47
*** timcl has joined #openstack-cinder17:49
*** sudipto has joined #openstack-cinder17:49
*** sudipto has quit IRC17:50
*** sudipto has joined #openstack-cinder17:50
*** sudipto has quit IRC17:51
openstackgerritHelen Walsh proposed openstack/cinder: VMAX driver - Storage assisted volume migration.  https://review.openstack.org/39786217:54
openstackgerritJon Bernard proposed openstack/cinder: RBD: allow a missing parent in delete operations  https://review.openstack.org/39786317:55
*** bardia has quit IRC18:00
*** jwcroppe has quit IRC18:02
*** jwcroppe has joined #openstack-cinder18:03
*** vladani_ has quit IRC18:03
*** eharney has joined #openstack-cinder18:03
*** vladani_ has joined #openstack-cinder18:03
*** jwcroppe has quit IRC18:07
*** e0ne has joined #openstack-cinder18:11
*** lpetrut has quit IRC18:11
*** alonma has joined #openstack-cinder18:14
openstackgerritJon Bernard proposed openstack/cinder: RBD: improve readability in over provisioning tests  https://review.openstack.org/39192718:17
*** nherciu has joined #openstack-cinder18:17
*** alonma has quit IRC18:18
*** e0ne has quit IRC18:23
*** jwcroppe has joined #openstack-cinder18:24
*** alonma has joined #openstack-cinder18:34
*** e0ne has joined #openstack-cinder18:37
*** harlowja has quit IRC18:38
*** alonma has quit IRC18:38
*** amoralej is now known as amoralej|off18:40
*** jgregor has joined #openstack-cinder18:44
*** knikolla has left #openstack-cinder18:49
*** bardia has joined #openstack-cinder18:54
*** jordanP has joined #openstack-cinder18:56
*** victorhe_ has joined #openstack-cinder18:58
*** nikeshm has quit IRC19:00
openstackgerritGoutham Pacha Ravi proposed openstack/cinder: [debug] do not merge  https://review.openstack.org/39788519:00
*** victorhe has quit IRC19:02
*** rcernin has joined #openstack-cinder19:02
*** sage_ has quit IRC19:03
*** sage_ has joined #openstack-cinder19:04
*** harlowja has joined #openstack-cinder19:06
*** sage_ has quit IRC19:06
*** victorhe_ has quit IRC19:06
*** victorhe has joined #openstack-cinder19:07
*** victorhe has quit IRC19:11
*** alonmarx has quit IRC19:14
*** victorhe has joined #openstack-cinder19:14
*** victorhe has quit IRC19:15
*** crose has quit IRC19:15
*** victorhe has joined #openstack-cinder19:15
*** alonma has joined #openstack-cinder19:16
raroraWould it be considered a bug worth fixing if code uses dict.pop('key') instead of using del dict['key'] if the value returned by pop() isn't used?19:18
*** crose has joined #openstack-cinder19:20
*** victorhe has quit IRC19:20
*** alonma has quit IRC19:21
xyang1rarora: I wouldn't consider it a bug19:27
*** crose has quit IRC19:27
raroraxyang1: okay, thank you!19:29
xyang1rarora: you are welcome!19:29
*** e0ne has quit IRC19:31
*** alonmarx has joined #openstack-cinder19:33
*** andymaier has joined #openstack-cinder19:36
*** crose has joined #openstack-cinder19:40
*** crose has quit IRC19:41
*** alonmarx has quit IRC19:42
*** asselin has quit IRC19:43
*** knikolla has joined #openstack-cinder19:45
*** knikolla has quit IRC19:45
*** knikolla has joined #openstack-cinder19:46
*** lpetrut has joined #openstack-cinder19:48
*** cknight has quit IRC19:49
*** alonmarx has joined #openstack-cinder19:49
*** ckonstanski has quit IRC20:00
*** ckonstanski has joined #openstack-cinder20:04
*** enriquetaso has quit IRC20:07
*** e0ne has joined #openstack-cinder20:13
*** gouthamr_ has quit IRC20:14
*** slayer has quit IRC20:20
openstackgerritJon Bernard proposed openstack/os-brick: RBD: ensure temporary config gets deleted  https://review.openstack.org/39670820:26
*** lpetrut has quit IRC20:31
*** slayer has joined #openstack-cinder20:31
openstackgerritLee Yarwood proposed openstack/os-brick: WIP encryptors: Mock only the executor  https://review.openstack.org/39793320:32
openstackgerritLee Yarwood proposed openstack/os-brick: WIP encryptors: Workaround mangled passphrases  https://review.openstack.org/39793420:32
*** asselin has joined #openstack-cinder20:33
openstackgerritJon Bernard proposed openstack/cinder: RBD: allow a missing parent in delete operations  https://review.openstack.org/39786320:35
*** lpetrut has joined #openstack-cinder20:35
*** kaisers__ has quit IRC20:43
*** kaisers_ has joined #openstack-cinder20:44
*** kaisers_ has quit IRC20:45
SwansonIs run_tests.sh supposed to run something? Is it broken or is my system broken?20:48
waj334Will Cinder v3 always maintain backwards compatibility with all prior versions since there won't be a v4?20:51
smcginniswaj334: Correct. v3 is set, and any new changes will be microversions to v3.20:53
smcginnisSwanson: Use "tox" instead. I think we should just get rid of run_tests.sh20:54
Swansonsmcginnis, No, I just needed to do an update.20:54
smcginnisSwanson: What do you mean?20:55
*** timcl has quit IRC20:56
Swansonsmcginnis, I rebased. All is well.20:56
waj334smcginnis: So, why does the client use 3.0 by default and not the highest available version? Or is there a possibility for a breaking change to appear after a certain microversion?20:56
*** takedakn has joined #openstack-cinder20:59
smcginniswaj334: Right, you wouldn't want it to just use the highest available by default because there could be a change in behavior you aren't expecting.20:59
*** takedakn has quit IRC20:59
*** e0ne has quit IRC21:04
*** victorhe has joined #openstack-cinder21:04
*** victorhe has quit IRC21:04
*** victorhe has joined #openstack-cinder21:05
waj334smcginnis: I find it quite annoying to have to specify the version each time. I figured existing functionality would remain consistent across versions, so using the newest version made sense to me.21:07
SwansonInteresting. Oodles of test failures in code I didn't touch.21:08
*** ccesario has joined #openstack-cinder21:09
*** Apoorva has quit IRC21:10
*** slayer has quit IRC21:11
*** porrua has quit IRC21:16
openstackgerritxing-yang proposed openstack/cinder: WIP: Migrate consistency groups to groups  https://review.openstack.org/35035021:24
*** alonmarx has quit IRC21:27
*** asselin has quit IRC21:29
*** pcaruana has quit IRC21:32
*** lpetrut has quit IRC21:34
*** Apoorva has joined #openstack-cinder21:34
*** lpetrut has joined #openstack-cinder21:34
*** alonma has joined #openstack-cinder21:42
*** Lee1092 has quit IRC21:46
*** alonma has quit IRC21:46
*** slayer has joined #openstack-cinder21:49
*** harlowja has quit IRC21:55
openstackgerritTom Swanson proposed openstack/cinder: WIP: Dell SC: Retype fixes  https://review.openstack.org/39797021:58
*** jwcroppe has quit IRC22:00
*** nherciu has quit IRC22:00
openstackgerritTom Swanson proposed openstack/cinder: WIP: Dell SC: Retype fixes  https://review.openstack.org/39797022:02
*** jwcroppe has joined #openstack-cinder22:02
SwansonBlame jungleboyj for this one cinder.tests.unit.volume.drivers.ibm.test_storwize_svc.StorwizeSVCFcDriverTestCase.test_storwize_get_host_from_connector_with_lshost_failure22:09
*** slayer has quit IRC22:09
jungleboyjSwanson: I didn't do it.22:11
*** harlowja has joined #openstack-cinder22:13
Swansonjungleboyj, woozle wuzzle?22:14
*** jwcroppe has quit IRC22:14
*** mvk has joined #openstack-cinder22:14
*** lpetrut has quit IRC22:15
openstackgerritxing-yang proposed openstack/cinder: WIP: Migrate consistency groups to groups  https://review.openstack.org/35035022:17
*** catinthe_ has joined #openstack-cinder22:18
*** gouthamr_ has joined #openstack-cinder22:19
*** cknight has joined #openstack-cinder22:21
*** catintheroof has quit IRC22:21
*** alonma has joined #openstack-cinder22:23
*** jgregor has quit IRC22:25
*** gouthamr_ has quit IRC22:26
*** jgregor has joined #openstack-cinder22:26
*** alonma has quit IRC22:28
*** khamtamtun has joined #openstack-cinder22:29
*** victorhe_ has joined #openstack-cinder22:30
*** jgregor has quit IRC22:30
*** victorhe has quit IRC22:33
*** victorhe_ has quit IRC22:34
*** catintheroof has joined #openstack-cinder22:35
*** catinthe_ has quit IRC22:36
*** edmondsw has quit IRC22:36
*** dustins has quit IRC22:36
*** mriedem has quit IRC22:41
*** rcernin has quit IRC22:42
*** asselin has joined #openstack-cinder22:44
SwansonOh, yeah, it could be hemna's fault now.22:51
*** xyang1 has quit IRC22:52
*** khamtamtun has quit IRC22:59
*** alonma has joined #openstack-cinder23:05
*** alonma has quit IRC23:09
*** jordanP has quit IRC23:10
*** dave-mccowan has quit IRC23:12
*** jwcroppe has joined #openstack-cinder23:15
*** ducttape_ has quit IRC23:27
*** cknight has quit IRC23:30
*** alonma has joined #openstack-cinder23:37
*** stakeda has joined #openstack-cinder23:38
openstackgerritKarthik Prabhu Vinod proposed openstack/cinder: Modeling ManageableSnaphots & ManageableVolumes as OVO  https://review.openstack.org/38878323:39
*** ckonstanski has quit IRC23:39
openstackgerritKarthik Prabhu Vinod proposed openstack/cinder: Change all occurences of List Manageable Volumes & snapshots to ovo  https://review.openstack.org/39073023:39
*** ckonstanski has joined #openstack-cinder23:40
*** alonma has quit IRC23:41
*** jungleboyj has quit IRC23:42
*** andymaier has quit IRC23:48
*** catintheroof has quit IRC23:52
*** catintheroof has joined #openstack-cinder23:54
*** catintheroof has quit IRC23:58

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