Monday, 2016-06-27

*** yangyapeng has quit IRC00:06
*** merooney has joined #openstack-cinder00:08
*** rdx_matt has quit IRC00:09
*** merooney has quit IRC00:14
*** ducttape_ has quit IRC00:14
*** merooney has joined #openstack-cinder00:14
*** merooney has quit IRC00:16
*** raunak has quit IRC00:31
*** laughterwym has joined #openstack-cinder00:35
*** laughterwym has quit IRC00:38
*** laughterwym has joined #openstack-cinder00:38
*** ducttape_ has joined #openstack-cinder00:40
*** akshai has joined #openstack-cinder00:45
*** markvoelker has joined #openstack-cinder00:51
*** raunak has joined #openstack-cinder00:52
*** markvoelker has quit IRC00:56
*** cheneydc has joined #openstack-cinder01:04
*** iceyao has joined #openstack-cinder01:06
*** cknight has joined #openstack-cinder01:06
*** rdx_matt has joined #openstack-cinder01:06
*** akshai has quit IRC01:06
*** yangyapeng has joined #openstack-cinder01:13
*** ducttape_ has quit IRC01:19
*** iceyao has quit IRC01:19
*** iceyao has joined #openstack-cinder01:19
*** akshai has joined #openstack-cinder01:22
*** akshai_ has joined #openstack-cinder01:25
*** salv-orl_ has quit IRC01:25
*** akshai has quit IRC01:28
*** raunak has quit IRC01:30
*** raunak has joined #openstack-cinder01:32
*** akshai has joined #openstack-cinder01:35
*** akshai_ has quit IRC01:38
*** rdx_matt has quit IRC01:40
*** Lee1092 has joined #openstack-cinder01:40
*** EinstCrazy has joined #openstack-cinder01:45
*** cknight1 has joined #openstack-cinder01:49
*** cknight has quit IRC01:49
*** wN has joined #openstack-cinder01:49
*** wN has quit IRC01:49
*** wN has joined #openstack-cinder01:49
*** iceyao_ has joined #openstack-cinder01:50
*** markvoelker has joined #openstack-cinder01:52
*** iceyao has quit IRC01:53
*** mriedem has quit IRC01:54
*** salv-orlando has joined #openstack-cinder01:56
*** cknight has joined #openstack-cinder01:56
*** cknight1 has quit IRC01:56
*** markvoelker has quit IRC01:57
openstackgerritQunyingRan proposed openstack/python-cinderclient: Use mock instead of mox in cinderclient  https://review.openstack.org/33267902:02
*** EinstCra_ has joined #openstack-cinder02:04
*** salv-orlando has quit IRC02:05
*** EinstCrazy has quit IRC02:06
*** houming has joined #openstack-cinder02:20
*** EinstCrazy has joined #openstack-cinder02:21
*** sdake_ has quit IRC02:23
*** EinstCra_ has quit IRC02:24
*** davechen has joined #openstack-cinder02:26
*** gouthamr_ has quit IRC02:27
*** raunak has quit IRC02:29
*** raunak has joined #openstack-cinder02:33
*** rdx_matt has joined #openstack-cinder02:37
*** amit213 has joined #openstack-cinder02:39
*** raunak has quit IRC02:46
*** ebalduf has quit IRC02:47
*** ducttape_ has joined #openstack-cinder02:50
*** markvoelker has joined #openstack-cinder02:53
*** ducttape_ has quit IRC02:54
*** markvoelker has quit IRC02:57
*** salv-orlando has joined #openstack-cinder03:02
*** ebalduf has joined #openstack-cinder03:03
*** akshai has quit IRC03:05
*** gcb has joined #openstack-cinder03:08
*** salv-orlando has quit IRC03:10
*** rdx_matt has quit IRC03:10
*** ducttape_ has joined #openstack-cinder03:12
openstackgerritwangxiyuan proposed openstack/cinder: Add backend driver for Huawei FusionStorage  https://review.openstack.org/32888703:12
*** karthikp__ has joined #openstack-cinder03:13
*** pleia2_ is now known as pleia203:15
*** karthikp__ has quit IRC03:15
*** akshai has joined #openstack-cinder03:20
*** SergeyLukjanov has quit IRC03:23
*** robcresswell has quit IRC03:24
*** Tridde has quit IRC03:24
*** Trident has joined #openstack-cinder03:25
*** robcresswell has joined #openstack-cinder03:27
*** SergeyLukjanov has joined #openstack-cinder03:30
*** jwcroppe has joined #openstack-cinder03:49
*** markvoelker has joined #openstack-cinder03:54
*** ducttape_ has quit IRC03:56
*** nkrinner_afk is now known as nkrinner03:56
*** markvoelker has quit IRC03:58
*** ducttape_ has joined #openstack-cinder04:06
*** salv-orlando has joined #openstack-cinder04:07
*** rdx_matt has joined #openstack-cinder04:08
*** akshai has quit IRC04:09
*** ducttape_ has quit IRC04:11
*** yangyape_ has joined #openstack-cinder04:15
*** yangyapeng has quit IRC04:15
*** links has joined #openstack-cinder04:15
*** karthikp__ has joined #openstack-cinder04:15
*** salv-orlando has quit IRC04:15
*** yangyapeng has joined #openstack-cinder04:20
*** yangyape_ has quit IRC04:20
*** yhayashi has joined #openstack-cinder04:20
*** karthikp__ has quit IRC04:24
*** yangyape_ has joined #openstack-cinder04:26
*** yangyapeng has quit IRC04:26
*** cknight has quit IRC04:27
*** cknight has joined #openstack-cinder04:27
*** karthikp__ has joined #openstack-cinder04:28
*** cknight has quit IRC04:32
*** karthikp__ has quit IRC04:41
*** rdx_matt has quit IRC04:41
*** pgadiya has joined #openstack-cinder04:41
*** salv-orlando has joined #openstack-cinder04:42
*** amit213 has quit IRC04:46
*** laughter_ has joined #openstack-cinder04:50
*** salv-orlando has quit IRC04:50
*** laughterwym has quit IRC04:53
*** markvoelker has joined #openstack-cinder04:54
*** markvoelker has quit IRC04:59
*** wN has quit IRC05:01
*** cknight has joined #openstack-cinder05:02
*** yangyape_ has quit IRC05:04
*** davechen has quit IRC05:05
*** wN has joined #openstack-cinder05:05
*** wN has quit IRC05:05
*** wN has joined #openstack-cinder05:05
*** yangyapeng has joined #openstack-cinder05:05
*** ducttape_ has joined #openstack-cinder05:07
*** ducttape_ has quit IRC05:12
*** ChubYann has quit IRC05:13
*** cknight has quit IRC05:15
*** gcb has quit IRC05:15
*** leeantho has joined #openstack-cinder05:16
*** savihou has joined #openstack-cinder05:20
*** laughter_ has quit IRC05:21
*** laughterwym has joined #openstack-cinder05:21
*** gcb has joined #openstack-cinder05:28
*** jwcroppe has quit IRC05:31
*** jwcroppe has joined #openstack-cinder05:32
*** lunarlamp is now known as mariusv05:36
*** jwcroppe has quit IRC05:36
*** salv-orlando has joined #openstack-cinder05:38
*** rdx_matt has joined #openstack-cinder05:38
*** davechen has joined #openstack-cinder05:38
*** mariusv has quit IRC05:43
*** gcb has quit IRC05:52
*** markvoelker has joined #openstack-cinder05:55
*** EinstCrazy has quit IRC05:59
*** markvoelker has quit IRC06:00
*** bhavik has joined #openstack-cinder06:02
*** Poornima has joined #openstack-cinder06:03
*** abhishekk has joined #openstack-cinder06:06
*** ducttape_ has joined #openstack-cinder06:08
*** gcb has joined #openstack-cinder06:11
*** rdx_matt has quit IRC06:12
*** ducttape_ has quit IRC06:12
*** rdx_matt has joined #openstack-cinder06:13
abhishekkmikal: hi you around?06:16
*** sgotliv__ has joined #openstack-cinder06:17
openstackgerritxiexs proposed openstack/python-cinderclient: Add strict Boolean checking  https://review.openstack.org/33429906:19
*** mjura has joined #openstack-cinder06:21
*** EinstCrazy has joined #openstack-cinder06:24
*** rcernin has joined #openstack-cinder06:28
*** mjura has quit IRC06:28
*** liverpooler has joined #openstack-cinder06:28
*** ManishD has joined #openstack-cinder06:32
*** M00nr41n has joined #openstack-cinder06:33
*** jwcroppe has joined #openstack-cinder06:33
*** pbandark has joined #openstack-cinder06:38
rdx_mattnikeshm: Regarding you code review https://review.openstack.org/#/c/332032/ ? I've read the guideline, and it seems like the code is aligned with it. Can you please be more specific?06:38
*** yangyape_ has joined #openstack-cinder06:38
*** yangyapeng has quit IRC06:39
*** jwcroppe has quit IRC06:40
*** mjura has joined #openstack-cinder06:41
*** EinstCrazy has quit IRC06:44
*** arecknag has joined #openstack-cinder06:49
*** gcb has quit IRC06:53
*** zhangjn has quit IRC06:54
*** zhangjn has joined #openstack-cinder06:54
*** markvoelker has joined #openstack-cinder06:56
*** EinstCrazy has joined #openstack-cinder06:56
*** lpetrut has joined #openstack-cinder06:57
*** tesseract- has joined #openstack-cinder06:59
*** markvoelker has quit IRC07:01
*** jamielennox is now known as jamielennox|away07:01
*** yangyape_ has quit IRC07:01
*** e0ne has joined #openstack-cinder07:01
*** yangyapeng has joined #openstack-cinder07:02
*** gcb has joined #openstack-cinder07:05
*** ducttape_ has joined #openstack-cinder07:09
*** ebalduf has quit IRC07:09
*** belmoreira has joined #openstack-cinder07:09
*** gcb has quit IRC07:11
*** sheel has joined #openstack-cinder07:12
*** ducttape_ has quit IRC07:14
*** fishcried has joined #openstack-cinder07:15
*** e0ne has quit IRC07:16
*** e0ne has joined #openstack-cinder07:16
*** fishcried has left #openstack-cinder07:19
*** gcb has joined #openstack-cinder07:29
*** lpetrut has quit IRC07:29
*** jwcroppe has joined #openstack-cinder07:37
*** jwcroppe has quit IRC07:42
*** lpetrut has joined #openstack-cinder07:48
*** bhavik has quit IRC07:54
*** yhayashi has quit IRC07:56
*** permalac has joined #openstack-cinder07:56
openstackgerritLisaLi proposed openstack/cinder: Add backup update function  https://review.openstack.org/26317907:57
*** davechen has left #openstack-cinder07:58
*** laughterwym has quit IRC07:58
*** zzzeek has quit IRC08:00
*** leeantho has quit IRC08:01
*** lpetrut has quit IRC08:01
*** zzzeek has joined #openstack-cinder08:02
*** salv-orlando has quit IRC08:04
*** laughterwym has joined #openstack-cinder08:04
*** laughte__ has joined #openstack-cinder08:05
*** bhavik has joined #openstack-cinder08:06
*** laughterwym has quit IRC08:08
openstackgerritDavanum Srinivas (dims) proposed openstack/cinder: [WIP] Testing latest u-c  https://review.openstack.org/30376608:10
*** ducttape_ has joined #openstack-cinder08:10
*** wilson1 has quit IRC08:13
*** zhongjun_ has quit IRC08:13
*** zhongjun_ has joined #openstack-cinder08:14
*** wilson_liu has joined #openstack-cinder08:14
*** bjornar_ has joined #openstack-cinder08:16
*** ducttape_ has quit IRC08:16
*** salv-orlando has joined #openstack-cinder08:25
*** vgridnev_ has joined #openstack-cinder08:31
openstackgerritSoffie Huang proposed openstack/cinder: FalconStor: New Cinder driver in Newton  https://review.openstack.org/33157908:33
openstackgerritParashuram Hallur proposed openstack/cinder: Implementation for CoprHD Cinder Drivers  https://review.openstack.org/31764308:35
*** laughte__ has quit IRC08:37
*** laughterwym has joined #openstack-cinder08:38
*** jwcroppe has joined #openstack-cinder08:38
*** laughterwym has quit IRC08:38
*** laughterwym has joined #openstack-cinder08:39
*** laughterwym has quit IRC08:39
*** bjornar_ has quit IRC08:39
*** laughterwym has joined #openstack-cinder08:39
*** jwcroppe has quit IRC08:43
*** laughterwym has quit IRC08:46
*** laughterwym has joined #openstack-cinder08:46
*** bhavik has quit IRC08:47
*** laughterwym has quit IRC08:51
*** vgridnev_ has quit IRC08:52
*** lpetrut has joined #openstack-cinder08:58
*** bhavik has joined #openstack-cinder09:00
*** zhongjun_ has quit IRC09:06
*** zhongjun_ has joined #openstack-cinder09:06
*** yangyape_ has joined #openstack-cinder09:09
*** yangyapeng has quit IRC09:10
*** ducttape_ has joined #openstack-cinder09:12
*** sheel has quit IRC09:15
*** ducttape_ has quit IRC09:17
*** jiaohaolin has joined #openstack-cinder09:18
jiaohaolin有人吗09:19
jiaohaolin问一下,cinder的driver,都包含哪些方法,除了基础的volume create/delete,其他还需要完成什么了?09:20
jiaohaolinis anybody here?09:20
jiaohaolinwhat should i、09:23
jiaohaolinWhat should I do if I want to commit a driver to cinder? Do i need to pay some money?09:25
*** jamielennox|away is now known as jamielennox09:26
dulekjiaohaolin: https://wiki.openstack.org/wiki/Cinder/how-to-contribute-a-driver09:27
jiaohaolinthanks,and i have already read this09:28
abhishekkdulek:  hi you around?09:29
jiaohaolinbut i still don't know how to write code,09:29
dulekjiaohaolin: So no money involved, you just send your code in Gerrit and setup a CI that will test every Cinder patch.09:30
dulekabhishekk: Hi.09:30
abhishekkneed input about your comment on https://review.openstack.org/#/c/287606/09:30
abhishekkdulek: hi, ^^09:30
dulekabhishekk: Well, my +2 means I'm okay with the patch and I'm not sure what DuncanT exactly wants.09:32
abhishekkdulek: yes me too09:32
dulekFrom my perspective this was answered. You can copy the answer to the inline comment reply. :)09:32
DuncanTThe question is why doesn't the list in the cinder code match the one in glance?09:33
*** lixiaoy1_ has quit IRC09:33
abhishekkdulek: I will try to pind duncanT on this09:33
DuncanTIf it is answered somewhere and I missed it, please point me at it and I'll happily change my review09:33
abhishekkDuncanT: I have addressed that in Patch709:34
jiaohaolinis there any Chinese ?09:34
abhishekkIn the link [1] 6 image-formats are specified out of which 'qed' and 'vpc' are not supported by glance and vhd format raises 'Stderr: u"qemu-img: Unknown file format 'vhd'\n"' in cinder-volume, so I have limited it to raw, vmdk, vdi and qcow2.09:34
abhishekk[1] http://docs.openstack.org/image-guide/convert-images.html#qemu-img-convert-raw-qcow2-qed-vdi-vmdk-vhd09:34
abhishekkDuncanT: just before +2 comment given by dulek, I have answered it09:34
*** lixiaoy1 has joined #openstack-cinder09:35
DuncanTabhishekk: Got it. Sorry, I somehow missed that reply entirely. Review updated.09:37
abhishekkDuncanT: thank you09:37
abhishekkdulek: thank you09:37
dulek:)09:37
*** jwcroppe has joined #openstack-cinder09:39
openstackgerritxiexs proposed openstack/cinder: Trivial Fix  https://review.openstack.org/33437309:40
*** lixiaoy1 has quit IRC09:42
*** pgadiya has quit IRC09:43
*** lixiaoy1 has joined #openstack-cinder09:44
*** mdavidson has joined #openstack-cinder09:44
*** jwcroppe has quit IRC09:44
*** tyrola has joined #openstack-cinder09:52
openstackgerritMerged openstack/python-cinderclient: Fix Service.__repr__ to remove the undefined attribute  https://review.openstack.org/32023009:53
*** Lallana has joined #openstack-cinder10:04
*** cheneydc has quit IRC10:06
*** Poornima has quit IRC10:07
rdx_mattDuncanT: https://review.openstack.org/#/c/332032/. The latest code review specified the guidelines for oslo logging. However, I don't understand where in my patch I've not used the logging according to the guideline. Can you please have a look? Thanks.10:08
*** ducttape_ has joined #openstack-cinder10:13
*** laughterwym has joined #openstack-cinder10:13
*** dims has quit IRC10:14
DuncanTrdx_matt: I can't see anything egregious. There's a translation of a debug message in _run_cmd (rdx_cli_api.py) that should probably be fixed, but in general the logging looks right to me10:14
rdx_mattDuncanT: Thanks! I will fix the logging in _run_cmd and submit another patch.10:15
*** Poornima has joined #openstack-cinder10:16
DuncanTrdx_matt: You might want to follow up with the reviewer directly to see if they've spotted something else - it's not a very helpful review I'm afraid - if they don't get back to you then just fix the bit I spotted. I'll try to do a fill review later, but I've not got a lot of time at the moment, sadly10:17
*** ducttape_ has quit IRC10:17
*** laughterwym has quit IRC10:18
*** iceyao_ has quit IRC10:19
rdx_mattDuncanT: Yeah, thanks for helping me. I will contact nikesh.10:19
*** lprice1 has quit IRC10:19
*** lprice has joined #openstack-cinder10:20
*** dims has joined #openstack-cinder10:20
*** sdague has joined #openstack-cinder10:32
*** Poornima has quit IRC10:33
*** Poornima has joined #openstack-cinder10:39
*** Poornima has quit IRC10:39
*** jwcroppe has joined #openstack-cinder10:41
*** links has quit IRC10:43
*** GB21 has joined #openstack-cinder10:45
*** EinstCrazy has quit IRC10:45
*** yangyape_ has quit IRC10:46
openstackgerritSongming Yan proposed openstack/python-cinderclient: Delete mox in cinderclient  https://review.openstack.org/33267910:46
openstackgerritSongming Yan proposed openstack/python-cinderclient: Delete mox in cinderclient  https://review.openstack.org/33267910:46
*** jwcroppe has quit IRC10:46
openstackgerritxiexs proposed openstack/python-cinderclient: Add strict Boolean checking  https://review.openstack.org/33429910:48
*** houming has quit IRC10:50
*** houming has joined #openstack-cinder10:51
*** links has joined #openstack-cinder10:56
*** erlon has joined #openstack-cinder10:57
*** salv-orl_ has joined #openstack-cinder10:59
*** markvoelker has joined #openstack-cinder10:59
*** bhavik has quit IRC10:59
*** arecknag has quit IRC11:01
*** salv-orl_ has quit IRC11:01
dulekxinli: You're right with thinking described in that paste (http://paste.openstack.org/show/521977/).11:02
*** salv-orlando has quit IRC11:02
*** markvoelker has quit IRC11:03
*** arecknag has joined #openstack-cinder11:04
*** tyrola has quit IRC11:08
*** tyrola has joined #openstack-cinder11:09
*** tpsilva has joined #openstack-cinder11:12
*** ducttape_ has joined #openstack-cinder11:13
*** bhavik has joined #openstack-cinder11:14
*** wanghao_ has joined #openstack-cinder11:16
*** bjornar has quit IRC11:17
*** wilson has joined #openstack-cinder11:17
*** ducttape_ has quit IRC11:18
*** wilson is now known as Guest2392011:18
dulekHm, is gate-cinder-tox-compliance broken?11:20
dulekhttp://logs.openstack.org/73/334373/1/check/gate-cinder-tox-compliance/da5d12d/console.html#_2016-06-27_10_01_30_92891811:20
*** wilson_liu has quit IRC11:20
*** bjornar has joined #openstack-cinder11:21
*** savihou has quit IRC11:29
*** savihou has joined #openstack-cinder11:29
*** Poornima has joined #openstack-cinder11:30
*** GB21 has quit IRC11:35
*** adrianofr has joined #openstack-cinder11:41
*** houming has quit IRC11:42
*** jwcroppe has joined #openstack-cinder11:42
*** akshai has joined #openstack-cinder11:46
*** ManishD has quit IRC11:47
*** jwcroppe has quit IRC11:47
*** akshai has quit IRC11:48
*** yangyapeng has joined #openstack-cinder11:50
openstackgerritMichal Dulko proposed openstack/cinder: Don't reuse pep8 env in compliance tox runs  https://review.openstack.org/33443011:53
duleke0ne, geguileo, DuncanT: ^ I think this will fix the gate.11:54
*** raildo-a` is now known as raildo11:56
openstackgerritvenkata krishna proposed openstack/cinder: Refactoring Kaminario iSCSI driver  https://review.openstack.org/33347811:57
geguileodulek: I don't think that will fix anything11:57
geguileodulek: I don't know what the problem is though11:57
geguileodulek: envdir is just a config option that defaults to {toxworkdir}/{envname}11:58
geguileodulek: What we do there is just change the envname so it reuses it like you say, but it would still create it if it doesn't exist11:58
*** GB21 has joined #openstack-cinder11:58
dulekgeguileo: Yup, and run-tox.sh is expecting pip to be in .tox/compliance.11:58
geguileodulek: Ok...11:59
geguileodulek: Let me have a quick look in case there's another solution that doesn't require us creating yet another env...11:59
dulekBut it's not there, as compliance check creates only pep8 with envname.11:59
dulekgeguileo: Cool, I would totally prefer such solution.12:00
*** ducttape_ has joined #openstack-cinder12:00
geguileodulek: Can you give me a like to one of those error on the gate?12:02
geguileodulek: Because to me looks like it's a problem with tox, that is enabling the wrong env12:02
*** laughterwym has joined #openstack-cinder12:05
geguileodulek: And I can't reproduce it locally12:08
*** laughterwym has quit IRC12:08
*** laughterwym has joined #openstack-cinder12:08
*** arecknag has quit IRC12:13
*** arecknag has joined #openstack-cinder12:13
geguileodulek: Ok, the problem seems to be that jenkins is expecting our env to have the same name12:13
abhishekkhi can anybody let me know why12:13
abhishekkgate-cinder-tox-compliance job is failing on jenkins?12:13
geguileoabhishekk: Yes, it's failing12:14
abhishekkdo I need to add recheck?12:14
geguileodulek: And it only checks if pbr exists in the expected path and if it doesn't it assumes pip will exist in that path12:14
geguileoabhishekk: It's not fixed yet12:14
geguileoabhishekk: https://bugs.launchpad.net/cinder/+bug/159650812:14
openstackLaunchpad bug 1596508 in Cinder "gate-cinder-tox-compliance is broken" [Undecided,In progress] - Assigned to Michal Dulko (michal-dulko-f)12:14
*** markvoelker has joined #openstack-cinder12:14
abhishekkgeguileo: thank you for update12:15
*** krtaylor has quit IRC12:18
geguileoabhishekk: np12:18
geguileodulek: I +2 that patch12:18
geguileoe0ne: DuncanT When you have a second, please look at dulek patch to fix the gate: https://review.openstack.org/33443012:19
*** iceyao has joined #openstack-cinder12:19
*** ducttape_ has quit IRC12:24
*** akerr has joined #openstack-cinder12:26
*** eharney has joined #openstack-cinder12:29
*** GB21 has quit IRC12:35
*** lpetrut has quit IRC12:40
*** akerr is now known as akerr_away12:42
openstackgerritlakshman proposed openstack/cinder: Provide Kaminario K2 all-flash array FC driver  https://review.openstack.org/33416612:43
*** jwcroppe has joined #openstack-cinder12:44
*** salv-orlando has joined #openstack-cinder12:44
*** ccesario has joined #openstack-cinder12:45
geguileoeharney: Can you have a look at this patch to fix the gate, please? https://review.openstack.org/33443012:48
*** jwcroppe has quit IRC12:48
eharneygeguileo: ok12:49
geguileoeharney: Thanks!12:49
geguileoeharney: It's a oneliner  ;-)12:49
geguileoeharney: Oh, e0ne already reviewed it12:50
geguileoSorry for pinging you for nothing  :-(12:50
eharneygeguileo: so i guess there's some unwritten expectation that we use a directory that matches the env name12:50
*** rlrossit has joined #openstack-cinder12:51
*** faiz89 has joined #openstack-cinder12:51
geguileoeharney: Yes, whoever is calling run-tox.sh expects that12:51
geguileoAnd that's what run-tox expects as well12:51
geguileo:-(12:51
*** salv-orlando has quit IRC12:51
*** akshai has joined #openstack-cinder12:51
*** sayalilunkad has joined #openstack-cinder12:52
*** gcb has quit IRC12:55
*** porrua has joined #openstack-cinder12:57
*** timcl has joined #openstack-cinder12:59
*** edmondsw has joined #openstack-cinder13:01
*** gouthamr has joined #openstack-cinder13:02
dulekgeguileo: Sorry, I've had a meeting. So you're right, it's an expectation in project-config scripts that venv dir == venv name.13:03
*** akerr_away has quit IRC13:05
*** akshai has quit IRC13:05
nikeshmDuncanT: is this allowed LOG.debug("Entering " + method_verbose)13:07
*** akshai has joined #openstack-cinder13:09
*** sdake has joined #openstack-cinder13:10
nikeshmrdx_matt: hi13:10
nikeshmfor string concatenation do not use +13:11
rdx_mattnikeshm: Hi, What should I use instead?13:11
nikeshmalso for line continuation do not use '\'13:11
*** krtaylor has joined #openstack-cinder13:11
*** bhavik has quit IRC13:12
*** ducttape_ has joined #openstack-cinder13:12
*** nkrinner is now known as nkrinner_afk13:13
nikeshm LOG.debug("Entering  %s", method_verbose), where method_verbose is string right13:13
rdx_mattnikeshm: How can I continue a line without '\'? I'm limited to 79 chars.13:14
nikeshmuse ()13:15
rdx_mattnikeshm: Thanks, will update a new patch soon.13:15
*** EinstCrazy has joined #openstack-cinder13:16
nikeshmrdx_matt: run flake8 on each file13:16
*** ducttape_ has quit IRC13:16
*** dustins has joined #openstack-cinder13:17
*** e0ne has quit IRC13:18
scottdardx_matt: run 'tox -efast8' to check only the files you've changed.13:18
*** e0ne has joined #openstack-cinder13:18
rdx_mattscottda: Thanks guys for the help.13:19
nikeshmrdx_matt: please confirm with cores on '\', we should not use it at other places, but i am not sure () will work in import modules or not13:19
*** cknight has joined #openstack-cinder13:19
eharneythe rule is basically "make it as readable and clean as possible"13:20
*** sdake_ has joined #openstack-cinder13:20
rdx_mattnikeshm: They don't work with imports. Do I have any other option besides using '\' for imports?13:20
*** pbandark has quit IRC13:20
*** sdake has quit IRC13:21
nikeshmrdx_matt: not sure, please confirm with others13:21
rdx_mattscottda: Any idea how to handle '\' for imports?13:22
*** mriedem has joined #openstack-cinder13:23
*** pbandark has joined #openstack-cinder13:23
guitarzanrdx_matt: is your package name too long to fit on one line?13:24
rdx_mattguitarzan: yes13:25
guitarzanyikes13:25
rdx_mattguitarzan: :(13:25
*** bhavik has joined #openstack-cinder13:25
guitarzanthen you have to use line continuation13:25
* guitarzan shrugs13:25
eharneythe problem is that you are importing individual exceptions from a module13:26
guitarzanah, I haven't seen the actual example13:26
eharneyjust import the module itself instead13:26
eharneyfirst item under http://docs.openstack.org/developer/hacking/#imports13:27
*** timcl has quit IRC13:29
rdx_matteharney: I've missed it. Thanks.13:29
*** e0ne has quit IRC13:29
*** diogogmt has quit IRC13:29
*** e0ne has joined #openstack-cinder13:30
*** belmoreira has quit IRC13:31
*** baojg has joined #openstack-cinder13:36
openstackgerritMattan Shalev proposed openstack/cinder: Added initial backend ISCSI driver for Reduxio, implementing the basic ISCSI driver functionality and supports volumes backdating by snapshots or clones  https://review.openstack.org/33203213:37
*** ebalduf has joined #openstack-cinder13:39
*** timcl has joined #openstack-cinder13:40
*** xyang1 has joined #openstack-cinder13:41
rdx_mattDoes someone know what's with Cisco CI? Constantly failing, and they seem to be unreachable.13:45
*** jwcroppe has joined #openstack-cinder13:45
*** eharney has quit IRC13:46
*** tongli has joined #openstack-cinder13:47
*** kfarr has joined #openstack-cinder13:48
*** salv-orlando has joined #openstack-cinder13:48
swamireddyhi13:48
*** merooney has joined #openstack-cinder13:49
swamireddyin cinder/releasenotes/notes/<feature_name+NUMBER>.yaml.   -- Here what is NUMBER indicates? is it BP/bug ID?13:49
*** jwcroppe has quit IRC13:50
*** eharney has joined #openstack-cinder13:51
rdx_mattswamireddy: http://docs.openstack.org/developer/reno/usage.html#creating-new-release-notes13:53
rdx_mattinstall reno and use it to generate a release note. I guess the number is a unique id.13:54
*** salv-orlando has quit IRC13:55
*** ducttape_ has joined #openstack-cinder13:56
*** M00nr41n has quit IRC13:56
*** akerr has joined #openstack-cinder13:57
rdx_mattYour brunch name and commit should contain the information regarding the BP/bug.13:58
*** akshai has quit IRC13:59
*** ametts has joined #openstack-cinder14:00
*** EinstCrazy has quit IRC14:00
*** reduxio has joined #openstack-cinder14:03
*** akshai has joined #openstack-cinder14:03
*** attish has joined #openstack-cinder14:03
*** liverpooler has quit IRC14:05
attishHi! I cannot find the policy.json line which controls snapshot creation. Is there any?14:06
*** mfisch has joined #openstack-cinder14:06
*** mfisch has quit IRC14:06
*** mfisch has joined #openstack-cinder14:06
*** bardia has joined #openstack-cinder14:07
*** stefan_amann has joined #openstack-cinder14:08
*** diogogmt has joined #openstack-cinder14:12
*** jsheeren has joined #openstack-cinder14:12
dulekattish: https://github.com/openstack/cinder/blob/1574ccf2d22cc86b83f828eadb5778a631fa9789/etc/cinder/policy.json#L19-L1914:12
*** cdelatte has joined #openstack-cinder14:13
jsheerenhi all, is it normal that i cannot boot an instance and create a volume when using raw disk-format?  the instance complains with: no bootable device found14:13
*** leeantho has joined #openstack-cinder14:13
*** leeantho has quit IRC14:13
jsheerenqcow2 is no issue14:13
*** leeantho has joined #openstack-cinder14:13
*** mjura has quit IRC14:15
*** GB21 has joined #openstack-cinder14:17
*** lpetrut has joined #openstack-cinder14:17
*** rajinir has joined #openstack-cinder14:20
*** mkoderer__ has quit IRC14:21
*** attish has quit IRC14:21
*** attish has joined #openstack-cinder14:22
*** Guest23920 has quit IRC14:22
*** Guest23920 has joined #openstack-cinder14:23
*** jseiler__ has quit IRC14:26
*** bardia has quit IRC14:26
*** bardia has joined #openstack-cinder14:27
*** Poornima has quit IRC14:27
*** attish has quit IRC14:29
*** salv-orlando has joined #openstack-cinder14:30
*** attish has joined #openstack-cinder14:30
nikeshmi got -1 from jenkins on two patches in gate-cinder-tox-compliance  for this .tox/compliance/bin/pip: No such file or directory14:31
attishdulek: thx, this line seems to be missing from the rdo/liberty install14:31
nikeshmi did recheck14:31
openstackgerritMerged openstack/cinder: Don't reuse pep8 env in compliance tox runs  https://review.openstack.org/33443014:32
nikeshmhope nothing wrong in my patch14:32
nikeshmah see Merged openstack/cinder: Don't reuse pep8 env in compliance tox runs  https://review.openstack.org/33443014:33
*** cdelatte has quit IRC14:34
smcginnisnikeshm: Yep, should be OK now.14:34
*** karthikp__ has joined #openstack-cinder14:38
*** amit213 has joined #openstack-cinder14:43
*** bhavik has quit IRC14:44
*** lprice1 has joined #openstack-cinder14:44
*** karthikp__ has quit IRC14:45
*** attish has quit IRC14:45
*** karthikp__ has joined #openstack-cinder14:45
*** ociuhandu has joined #openstack-cinder14:46
*** lprice has quit IRC14:46
*** akshai has quit IRC14:46
*** attish has joined #openstack-cinder14:47
*** jwcroppe has joined #openstack-cinder14:47
*** jsheeren has quit IRC14:47
*** xinli has joined #openstack-cinder14:48
*** arch-nemesis has joined #openstack-cinder14:48
savihousmcginnis: Hey Sean, Hope your doing well ! may I bother you (again .. :/) with https://review.openstack.org/#/c/263026/ ?14:49
smcginnissavihou: Got it open in a tab now. I should be able to get to it later today.14:50
*** belmoreira has joined #openstack-cinder14:50
savihousmcginnis: Thanks ! God bless multi-tabs :)14:50
*** akshai has joined #openstack-cinder14:51
*** jwcroppe has quit IRC14:55
attishDoes policy.json's "volume_extension:services:index" has any use? ``cinder service-list`` seem to require admin role in a hard-wired way14:55
*** Suyash has joined #openstack-cinder14:55
guitarzanattish: the db requires isAdmin to be true14:57
guitarzanmaybe?14:58
*** arecknag has quit IRC14:58
guitarzanya, looks like the api doesn't use an admin context to make the db call14:59
*** karthikp__ has quit IRC15:01
*** sdake has joined #openstack-cinder15:01
*** jistr is now known as jistr|mtg15:01
*** mvk_ has quit IRC15:02
*** sdake_ has quit IRC15:03
*** baojg has quit IRC15:04
*** krotscheck is now known as krotscheck_dcm15:05
*** iceyao has quit IRC15:08
*** doffm has quit IRC15:09
*** chris_morrell has joined #openstack-cinder15:09
openstackgerritxing-yang proposed openstack/cinder-specs: Replication with More Granularity (Tiramisu)  https://review.openstack.org/22972215:09
*** leeantho has quit IRC15:09
*** e0ne has quit IRC15:09
*** mdenny has quit IRC15:10
*** belmoreira has quit IRC15:10
*** tyrola has quit IRC15:11
*** tyrola has joined #openstack-cinder15:11
openstackgerritSimon Dodsley proposed openstack/cinder: Increment Pure Storage Driver Versions  https://review.openstack.org/33452615:13
*** attish has quit IRC15:13
*** attish has joined #openstack-cinder15:14
*** enriquetaso has joined #openstack-cinder15:15
openstackgerritSimon Dodsley proposed openstack/cinder: Increment Pure Storage Driver Versions  https://review.openstack.org/33452615:16
*** jistr|mtg is now known as jistr15:16
openstackgerritMerged openstack/cinder: Add strict Boolean checking for qos delete  https://review.openstack.org/33400315:18
*** xyang has joined #openstack-cinder15:19
*** karthikp__ has joined #openstack-cinder15:24
*** pbandark has quit IRC15:24
openstackgerritTom Swanson proposed openstack/cinder: Dell SC: Use Live Volume for replication  https://review.openstack.org/32953115:26
openstackgerritXinli Guan proposed openstack/cinder: Move QualityOfServiceSpecs to use VersionedObject  https://review.openstack.org/25198915:29
nikeshmsmcginnis: is six a standard library or third-party library?   http://docs.openstack.org/developer/hacking/#import-order-template what should be position of six in https://review.openstack.org/#/c/332032/4/cinder/volume/drivers/reduxio/rdx_iscsi_driver.py15:30
*** harlowja_at_home has joined #openstack-cinder15:31
*** daneyon has joined #openstack-cinder15:31
smcginnisnikeshm: six is a third party library.15:32
nikeshmoops, then do i have to change it in my patches, i put it in standard library15:33
openstackgerritxing-yang proposed openstack/cinder-specs: Replication with More Granularity (Tiramisu)  https://review.openstack.org/22972215:33
*** daneyon_ has quit IRC15:34
smcginnisnikeshm: It's not too critical, so I wouldn't worry about it. If you make other changes you can move it, but I wouldn't bother doing a patch specifically for that.15:34
*** cdelatte has joined #openstack-cinder15:34
*** leeantho has joined #openstack-cinder15:35
nikeshmsmcginnis: i do not have other changes to do in code, so i will keep as it is thanks15:36
rdx_mattsmcginnis: I didn't get this: https://review.openstack.org/#/c/332032/5/releasenotes/notes/reduxio-iscsci-driver-5827c32a0c498949.yaml15:36
*** xyang has quit IRC15:38
*** tesseract- has quit IRC15:40
smcginnisrdx_matt: Sure. Take a look at how these end up getting published: http://docs.openstack.org/releasenotes/cinder/mitaka.html15:40
smcginnisrdx_matt: So all we really want in the release notes is a note stating that the driver was added.15:41
*** xyang has joined #openstack-cinder15:41
smcginnisrdx_matt: Other details will be documented in the configuration reference for the driver.15:41
smcginnisrdx_matt: The release notes isn't the right place for all of that detail.15:41
*** M00nr41n has joined #openstack-cinder15:41
rdx_mattsmcginnis: Thanks. So I only should keep the first feature? ("- Added backend ISCSI driver for Reduxio.")15:42
smcginnisrdx_matt: Yeah, I think that's really all you need. But if you want to append the text from the second line of what you have in there now, that would be OK to include as well.15:43
rdx_mattsmcginnis: The first feature line is ok by me. Thanks Sean!15:43
smcginnisrdx_matt: Thanks!15:44
*** asselin has joined #openstack-cinder15:44
*** laughterwym has quit IRC15:44
xinlidulek: I resolve the issue in your comment (following your instruction): https://review.openstack.org/#/c/251989/22/cinder/db/api.py@607, but I still don't understand it very well, could you please help to explain in the paste when you have time? thanks ahead. http://paste.openstack.org/show/523597/15:44
openstackgerritMichal Dulko proposed openstack/cinder: WIP, DNM: Try running multinode grenade job  https://review.openstack.org/33453915:44
dulekxinli: I've seen the paste. I think that what you're explaining there is correct.15:45
*** cdelatte has quit IRC15:45
*** attish has quit IRC15:46
*** darosale has joined #openstack-cinder15:46
*** GB21 has quit IRC15:46
dulekxinli: So you're asking what is this line doing? It's all related to https://github.com/openstack/cinder/blob/a85d95084bf00a35efa40c9c3c2d38c9e73f45b8/cinder/objects/base.py#L265-L28015:46
dulekxinli: Basically you're specifying an exception from consistency of naming db api methods.15:51
*** cdelatte has joined #openstack-cinder15:51
*** cdelatte has quit IRC15:52
dulekscottda, karthikp__: https://review.openstack.org/#/c/33453915:52
openstackgerritMattan Shalev proposed openstack/cinder: Added initial backend ISCSI driver for Reduxio  https://review.openstack.org/33203215:52
xinlidulek: thanks a lot15:52
*** links has quit IRC15:52
dulekscottda, karthikp__: This and two patches as nested Depends-On. I'm not sure if it will work so I welcome any feedback and pointers. :)15:52
scottdadulek: The Dependency chain looks correct. Let's see if it works...15:56
dulekscottda: Zuul tells me it isn't: http://status.openstack.org/zuul/15:58
dulek:(15:58
karthikp__I see that you invoke the zuul to read the gerrit event stream writing check experimental..16:00
dulekkarthikp__: Yup, I've added the job to experimental pipeline. Anyway it isn't catching up this job.16:01
karthikp__dulek: do we satisfy the trigger though..let me check16:01
*** jungleboyj has joined #openstack-cinder16:03
dulekkarthikp__: Well, job is running in zuul, it just haven't catched gate-grenade-dsvm-cinder-multinode job.16:03
*** timcl has quit IRC16:05
*** faiz89 has quit IRC16:06
scottdadulek: Yeah, it looks like the cross-repo dependency isn't getting picked up...16:08
dulekscottda: Actually it looks quite like it is picked up: http://imgur.com/L0Vwih816:10
scottdadulek: Yeah, but why did you get a "This change depends on a change that failed to merge." From Jenkins?16:11
scottdadulek: I guess I shouldn't say "dependency isn't getting picked up", zuul certainly sees it...16:12
dulekscottda: I think it's because I've changed the second patch in the chain, while it was tested.16:12
scottdaahh...ok, that makes sense. It's just the message we get when the job is killed.16:13
karthikp__dulek : i think your pipeline should be experimental and not gate here.. maybe I am wrong16:15
*** slunkad_ has quit IRC16:15
karthikp__https://review.openstack.org/#/c/334537/2/jenkins/jobs/projects.yaml16:15
karthikp__line 80016:15
dulekkarthikp__: Okay, I'll try moving it to check.16:16
*** pauloewerton has joined #openstack-cinder16:17
*** stefan_amann has quit IRC16:17
openstackgerritMerged openstack/python-cinderclient: Delete mox in cinderclient  https://review.openstack.org/33267916:18
*** Suyash has quit IRC16:19
openstackgerritxiexs proposed openstack/cinder: Add validation for container name  https://review.openstack.org/33455416:20
*** sdague has quit IRC16:21
rdx_mattnikeshm: I've seen your comment. Strange. When I run flake I don't get any errors. Also when I run "./run_tests.sh test_reduxio -p" I also don't get any errors.16:26
rdx_mattSorry, I got only one error in rdx_iscsi_driver, but the rest of the errors you've listed are not generated.16:28
nikeshmhm,   flake8 --version 2.2.4 (pep8: 1.5.7, pep257: 0.2.1.post1, hacking.core: 0.0.1, ProxyChecker: 0.0.1, pyflakes: 0.8.1, mccabe: 0.2.1) CPython 2.7.6 on Linux16:29
nikeshmalso one command is pylint16:29
nikeshmrelated16:29
nikeshmalso i deleted some text16:30
nikeshmwhich was not useful16:30
nikeshmfor you while pasting16:30
rdx_mattflake8 --version16:30
rdx_matt2.5.5 (pep8: 1.5.7, pyflakes: 0.8.1, hacking.core: 0.0.1, ProxyChecker: 0.0.1, mccabe: 0.2.1) CPython 3.5.1 on Darwin16:30
nikeshmoh mine is old, no harm in fixing those if go for other patch, but check pylint errror,16:33
*** jseiler has joined #openstack-cinder16:34
*** Apoorva has joined #openstack-cinder16:35
rdx_mattThis is the one I'm more curious about. tox passes and so does ./run_tests.sh, so assertNotEqual does exist for this class.16:35
rdx_mattmore=most16:35
*** GB21 has joined #openstack-cinder16:35
*** timcl has joined #openstack-cinder16:36
*** harlowja_at_home has quit IRC16:37
*** xinli has quit IRC16:37
*** tyrola has quit IRC16:37
dulekkarthikp__: It doesn't look like moving job to check helped.16:38
karthikp__dulek: Ohh .. How about experimental pipeline16:40
dulekkarthikp__: I'm trying to do it similarly to Neutron job, and it doesn't define experimental pipeline.16:41
karthikp__dulek: hmm okay16:41
scottdadulek: Don't you need something in jobs/jenkins/devstack-gate?16:44
scottdai.e. https://review.openstack.org/#/c/330678/5/jenkins/jobs/devstack-gate.yaml16:44
scottda(Note that I'm not sure that ^^ is correct)16:44
openstackgerritMattan Shalev proposed openstack/cinder: Added initial backend ISCSI driver for Reduxio  https://review.openstack.org/33203216:44
*** e0ne has joined #openstack-cinder16:46
*** amit213 has quit IRC16:46
*** rdx_matt has quit IRC16:47
dulekscottda: I have changes in devstack-gate.yaml.16:48
dulekscottda: http://paste.openstack.org/show/523619/16:48
dulekkarthikp__: ^16:48
dulekOkay, I cannot add a new job, to run it I need to modify any of the existing ones.16:49
dulekI mean to test it I need to reuse an existing one. :)16:49
karthikp__dulek: are you gonna wait for 334537 to merge?16:54
*** _ducttape_ has joined #openstack-cinder16:55
dulekkarthikp__: Nah, I'll try to modify grenade-dsvm-neutron-multinode job just to prove that I'm able to run multinode on Cinder. And we can work from there.16:56
*** ntpttr_ has quit IRC16:56
*** ducttape_ has quit IRC16:58
*** salv-orl_ has joined #openstack-cinder16:59
openstackgerritMichal Dulko proposed openstack/cinder: WIP, DNM: Try running multinode grenade job  https://review.openstack.org/33453916:59
*** belmoreira has joined #openstack-cinder17:00
dulekscottda, karthikp__: Okay, gotta finish for today. If my last changes won't work feel free to play with these patches.17:01
scottdadulek: Alright. Have a good evening.17:02
karthikp__dulek: cool..Have a good night :)17:02
*** salv-orlando has quit IRC17:02
*** belmoreira has quit IRC17:03
openstackgerritMerged openstack/python-cinderclient: Add strict Boolean checking  https://review.openstack.org/33429917:05
karthikp__scottda: I don't think that worked..17:07
scottda:(17:08
*** diablo_rojo has joined #openstack-cinder17:08
scottdakarthikp__: I won't have time to make changes until later today, so feel free to mess with it if you've time.17:13
karthikp__scottda: oh ya ... I will work on this.17:15
erlonscottda: hey17:17
scottdaerlon: Hi17:17
swamireddyrdx_matt: Thank you for the pointer.17:17
erlonscottda: can you give a look on this when you have a chance?17:17
erlonscottda: https://review.openstack.org/#/c/333037/2/cinder/volume/flows/manager/create_volume.py17:17
erlonscottda: im trying to use a common way to fix the clone from snap/vol problem17:18
scottdaerlon: OK, I'll have a look17:18
*** faiz89 has joined #openstack-cinder17:18
*** cdelatte has joined #openstack-cinder17:19
erlonscottda: thanks, the PS is not fixing tests because there are some points I need to get feedback on the best approach to do17:19
*** satya_ has joined #openstack-cinder17:20
scottdaerlon: yeah, OK. I can give some quick feedback, but I've no backend to actually test.17:20
erlonscottda: no problem, as soon as the tempest test get merged, we can know if it will work/break in specific BEs17:22
*** Suyash has joined #openstack-cinder17:23
*** krtaylor has quit IRC17:24
*** harlowja has quit IRC17:25
*** cbader has quit IRC17:28
*** harlowja has joined #openstack-cinder17:33
*** cbader has joined #openstack-cinder17:34
*** aleksey1 has joined #openstack-cinder17:39
*** aleksey1 is now known as alekseyruban17:39
*** salv-orl_ has quit IRC17:47
*** rdx_matt has joined #openstack-cinder17:48
openstackgerritMerged openstack/cinder: Fix wrong usage of exception's attribute message  https://review.openstack.org/28571217:48
*** rdx_matt has quit IRC17:53
*** xinli has joined #openstack-cinder17:53
xinlixyang1: I have submitted PatchSet 23 regarding 251989 which addressed all your comments:https://review.openstack.org/#/c/251989/18:00
openstackgerritKendall Nelson proposed openstack/os-brick: WIP: Splitting Out Connectors from connector.py :WIP  https://review.openstack.org/30797418:02
*** Apoorva has quit IRC18:03
*** salv-orlando has joined #openstack-cinder18:03
ildikovjgriffith: how much is the Cinder code ready for testing?18:03
*** Apoorva has joined #openstack-cinder18:03
*** kfarr_ has joined #openstack-cinder18:03
*** kfarr has quit IRC18:04
ildikovjgriffith: I also remember some discussions about not well structured unit tests in Cinder18:04
scottdajgriffith: Here's an example of devstack WIP:DNM changes to get the config running in the gate: https://review.openstack.org/#/c/331840/18:05
*** salv-orlando has quit IRC18:06
*** salv-orlando has joined #openstack-cinder18:06
*** porrua has quit IRC18:09
*** diogogmt has quit IRC18:09
*** karthikp__ has quit IRC18:09
*** diogogmt has joined #openstack-cinder18:10
*** _ducttape_ has quit IRC18:10
smcginnisildikov: Still around?18:18
ildikovsmcginnis: yeap18:18
*** porrua has joined #openstack-cinder18:18
smcginnisildikov: Cool. Thinking about the midcycle sync up.18:19
smcginnisildikov: Should we just pick a time slot? Or is there a schedule in place yet for the Nova topics?18:19
openstackgerritMerged openstack/cinder: Added coverage report to tox.ini  https://review.openstack.org/33411218:19
ildikovsmcginnis: I haven't followed those discussions, but I will check18:20
smcginnisildikov: OK. We're pretty flexible on this end. I was thinking we could just set aside soemthing like 2-3 Tuesday afternoon, but whatever works for the Nova side we can probably accomodate.18:22
*** lpetrut has quit IRC18:22
ildikovsmcginnis: do you have an agenda for Cinder already or plans?18:22
smcginnisildikov: We just have some topics started near the bottom here: https://etherpad.openstack.org/p/newton-cinder-midcycle18:23
smcginnisildikov: Last time I just broke up the topics into each day and we worked our way through them.18:23
scottdasmcginnis: Probably best to schedule cinder-nova in the AM, so that overseas/Europe devs can attend.18:23
smcginnisSo easy enough to have a set time for Nova, then we can just cover topics around that block.18:23
smcginnisscottda: Oh, good point.18:23
*** asselin_ has joined #openstack-cinder18:23
xyang1xinli: sure, I'll take a look18:24
smcginnisProbably no earlier than 9am MDT though.18:24
_alastor_Do we have an alternative to the tintri ci watch for 3rd party driver reporting status?  The tintri site has been down for several days now18:24
smcginnis_alastor_: A spec was accepted for an official one, but I haven't seen that actually come about.18:26
smcginnis_alastor_: https://specs.openstack.org/openstack-infra/infra-specs/specs/deploy-ci-dashboard.html18:26
*** asselin has quit IRC18:27
_alastor_smcginnis: I stumbled upon that on Friday and hopefully some headway is made soon.  I'm more looking for what the PTLs use to determine if a 3rd party is reporting consistently enough :)18:27
*** lpetrut has joined #openstack-cinder18:28
smcginnis_alastor_: Unfortunately we don't really have good tools there. I've written a few scripts, as have others, but nothing good right now.18:28
_alastor_smcginnis: ok.  Good to know. I'll probably hack out my own :)18:29
smcginnis_alastor_: If you get something useful, I'd love to take a look. :)18:29
ildikovsmcginnis: 9am sounds fine, I don't think we would need earlier18:34
*** rcernin has quit IRC18:34
smcginnisildikov: Will that be too early for Oregon? Whatever works best. We can push it back an hour too if that doesn't work for the onsite folks.18:38
*** rdx_matt has joined #openstack-cinder18:38
*** rdx_matt has quit IRC18:38
*** rdx_matt has joined #openstack-cinder18:39
*** lpetrut has quit IRC18:39
*** salv-orlando has quit IRC18:40
*** cbader has quit IRC18:42
ildikovsmcginnis: we can pick a later one as well18:43
*** bjornar_ has joined #openstack-cinder18:43
*** savihou has quit IRC18:43
*** rdx_matt has quit IRC18:43
ildikovsmcginnis: the weekly meeting is 11am MDT I think18:44
*** karthikp__ has joined #openstack-cinder18:44
openstackgerritIvan Kolodyazhny proposed openstack/cinder: TEST COMMIT! DO NOT MERGE! Testing BDD gate job  https://review.openstack.org/33462018:44
*** jungleboyj has quit IRC18:48
*** kfarr_ has quit IRC18:50
*** ChubYann has joined #openstack-cinder18:51
*** fernnest_ has quit IRC18:53
jgriffithildikov: it's pretty much ready now18:56
*** Delair has joined #openstack-cinder18:56
ildikovjgriffith: ok, cool18:57
ildikovjgriffith: any hints for the unit tests?18:57
jgriffithildikov: hehe :)18:58
jgriffithildikov: I'm going to start writing a test per method18:58
jgriffithildikov: in other words 'real' unit tests that just check the method18:58
jgriffithildikov: not the "load everything and test the whole chain"18:58
smcginnisjgriffith: +100018:58
jgriffithildikov: if you want to work together on it we can?18:58
ildikovjgriffith: yeah, that one pretty much sounds like functional test to me18:59
jgriffithI can get some things started and work with you later this week?18:59
Delair@all .. i am getting error message in cinder/api.log file "ERROR cinder.volume.volume_types [req-9bfcf388-b059-4a89-b400-67377e1463da - - - - -] Default volume type is not found. Please check default_volume_type config:"18:59
Delaircan cant create any volumes18:59
jgriffithildikov: well... that's how most of our unit tests work :(18:59
Delairthe default_volume_type is set to <none> in my confg file19:00
*** rlrossit has quit IRC19:00
Delairany help is appreciaed19:00
ildikovjgriffith: :( maybe you should move them to the functional test folder then19:00
ildikovjgriffith: :)19:00
ildikovjgriffith: ok, bad joke, please blame Monday for it :)19:00
smcginnisDelair: Is it "default_volume_type=" or "default_volume_type=<None>" ?19:01
smcginnisDelair: Or have you tried removing the line completely?19:01
jgriffithildikov: lol19:02
ildikovjgriffith: I wanted to sync up with you so that we don't step on each other's while writing tests19:03
ildikovjgriffith: so I can try to make myself a bit familiar with the Cinder code and then ping you during the second half of the week19:03
jgriffithildikov: sure, that sounds great19:04
*** rcernin has joined #openstack-cinder19:04
*** amit213 has joined #openstack-cinder19:04
ildikovjgriffith: ok, cool, tnx!19:05
SuyashDelair, if you have configured a backend, you can set default_volume_type=<backend> and try restarting cinder services19:05
*** faiz89 has quit IRC19:08
*** rdx_matt has joined #openstack-cinder19:09
jay-mehtaDelair, normally its set to lvmdriver-119:09
*** rdx_matt_ has joined #openstack-cinder19:10
*** rdx_matt has quit IRC19:10
*** e0ne has quit IRC19:10
*** arch-nemesis has quit IRC19:10
*** arch-nemesis has joined #openstack-cinder19:11
*** e0ne has joined #openstack-cinder19:11
*** akshai has quit IRC19:12
*** akshai has joined #openstack-cinder19:12
*** salv-orlando has joined #openstack-cinder19:12
*** e0ne_ has joined #openstack-cinder19:13
*** ducttape_ has joined #openstack-cinder19:14
*** rdx_matt_ has quit IRC19:14
*** akshai has quit IRC19:16
jgriffithSuyash: you can do "cinder type-default" to find the def type.. then 'cinder extra-specs-list' to see how it's set up.  It would be cool if the default-type was dynamic and not tied to conf but it is so... oh well.19:17
*** salv-orlando has quit IRC19:17
*** e0ne has quit IRC19:17
Delair@jay-mehta @ Suyash Sorry guys was looking into logs and completly forgot to respond19:20
Delair@smcginnis it is set to default_volume_type=<None>19:20
Delairwhat is this suppose to be and why do we need it ?19:21
jgriffithDelair: you only need it for multi-backend so that you can have... well a default19:21
jgriffithDelair: otherwise you're leaving everything up to the scheduler19:21
jgriffithDelair: or specifying the type every time19:22
jay-mehtaDelair, its used while creating a volume, if default type is configured, you don't need to specify it while creating19:22
Delairok so may be this error has nothing to do with the whty i am NOT able to create volume19:22
Delairi dont see any other error msg19:22
jgriffithDelair: it's because there's a call being made to "volume-type show" with no id specified19:24
jgriffithDelair: which say's "get default" but if you don't have one then that fails19:24
jgriffithDelair: it would appear something/someone is calling volume-type show19:24
jgriffithDelair: https://github.com/openstack/cinder/blob/master/cinder/api/v2/types.py#L4619:27
openstackgerritTom Swanson proposed openstack/cinder: Dell SC: Specify High Availability for create replication  https://review.openstack.org/33462819:27
*** salv-orlando has joined #openstack-cinder19:27
Suyashjgriffith, right, so Delair has not given the default type and scheduler can't find anything specified?19:30
Delairthanks guys it is working..19:30
Delairi had to restart cinder-scheduler seervice :)19:30
Delairthanks for ur help19:30
SuyashDelair, nice19:30
Delairthanks suyash for pointing it out19:30
*** Delair has quit IRC19:30
*** ociuhandu has quit IRC19:36
*** akshai has joined #openstack-cinder19:36
*** harlowja has quit IRC19:36
*** faiz89 has joined #openstack-cinder19:37
*** sdake has quit IRC19:38
*** sdake has joined #openstack-cinder19:40
*** e0ne_ has quit IRC19:43
openstackgerritMerged openstack/cinder: Trivial Fix  https://review.openstack.org/33437319:44
*** rajinir has quit IRC19:44
*** julim has joined #openstack-cinder19:51
*** gouthamr has quit IRC19:51
openstackgerritTom Swanson proposed openstack/cinder: Dell SC: Specify High Availability for create replication  https://review.openstack.org/33462819:51
*** harlowja has joined #openstack-cinder19:52
*** jungleboyj has joined #openstack-cinder19:52
*** akshai_ has joined #openstack-cinder19:54
*** akshai has quit IRC19:58
*** kfarr has joined #openstack-cinder20:01
*** tongli has quit IRC20:04
*** gouthamr has joined #openstack-cinder20:08
*** rdx_matt has joined #openstack-cinder20:09
*** timcl has quit IRC20:13
*** rdx_matt has quit IRC20:14
*** merooney has quit IRC20:14
*** GB21 has quit IRC20:16
*** akerr has quit IRC20:18
*** sdake_ has joined #openstack-cinder20:19
*** sdake has quit IRC20:22
*** ociuhandu has joined #openstack-cinder20:22
*** Lee1092 has quit IRC20:23
*** rlrossit has joined #openstack-cinder20:24
*** GB21 has joined #openstack-cinder20:34
*** Apoorva has quit IRC20:40
*** Apoorva has joined #openstack-cinder20:40
openstackgerritMerged openstack/cinder: Increment Pure Storage Driver Versions  https://review.openstack.org/33452620:41
openstackgerritMerged openstack/cinder: Nexenta: Add NBD driver for NexentaEdge.  https://review.openstack.org/32006020:43
*** cknight has quit IRC20:47
*** rdx_matt has joined #openstack-cinder20:47
*** sdake has joined #openstack-cinder20:49
*** karthikp__ has quit IRC20:49
*** karthikp__ has joined #openstack-cinder20:50
*** reduxio has quit IRC20:50
*** edmondsw has quit IRC20:52
*** rdx_matt has quit IRC20:52
*** sdake_ has quit IRC20:53
*** catintheroof has joined #openstack-cinder20:55
*** rdx_matt has joined #openstack-cinder20:57
*** rdx_matt has quit IRC20:58
*** mdenny has joined #openstack-cinder20:58
openstackgerritMerged openstack/cinder: volume upload-to-image fails for iso disk-format  https://review.openstack.org/28760621:00
*** porrua has quit IRC21:01
*** sgotliv__ has quit IRC21:01
*** kfarr has quit IRC21:01
*** mvk_ has joined #openstack-cinder21:01
*** cdelatte has quit IRC21:03
*** cdelatte has joined #openstack-cinder21:03
openstackgerritCarl Pecinovsky proposed openstack/cinder: Fix exception type check ProcessExecutionError  https://review.openstack.org/33466421:04
*** alekseyruban has quit IRC21:05
*** mkrcmari__ has joined #openstack-cinder21:05
*** raildo is now known as raildo-afk21:07
*** mvk has joined #openstack-cinder21:07
*** mvk_ has quit IRC21:09
*** gouthamr has quit IRC21:09
*** mkrcmari__ has quit IRC21:10
*** sdake_ has joined #openstack-cinder21:12
*** sdake has quit IRC21:13
*** sdake has joined #openstack-cinder21:16
*** sdake_ has quit IRC21:17
*** eharney has quit IRC21:17
*** rlrossit has quit IRC21:19
*** cknight has joined #openstack-cinder21:23
*** rdx_matt has joined #openstack-cinder21:25
*** rdx_matt has quit IRC21:25
*** _ducttape_ has joined #openstack-cinder21:34
*** mvk_ has joined #openstack-cinder21:35
*** permalac_ has joined #openstack-cinder21:35
*** pauloewerton has quit IRC21:35
*** permalac has quit IRC21:36
*** ducttape_ has quit IRC21:38
*** mvk has quit IRC21:38
openstackgerritDeepti Ramakrishna proposed openstack/cinder-specs: New public core APIs to expose system capabilities  https://review.openstack.org/30693021:38
*** mkrcmari__ has joined #openstack-cinder21:41
*** mkrcmari__ has quit IRC21:43
*** mvk has joined #openstack-cinder21:44
*** mvk_ has quit IRC21:44
openstackgerritRyan Selden proposed openstack/cinder: Added unit test coverage to test_limits.py for api v2  https://review.openstack.org/33468021:45
*** faiz89 has quit IRC21:48
*** _ducttape_ has quit IRC21:50
*** xinli has quit IRC21:51
*** bjornar_ has quit IRC21:52
*** rdx_matt has joined #openstack-cinder21:53
*** rdx_matt has quit IRC21:53
*** mriedem has quit IRC21:55
*** catintheroof has quit IRC21:55
*** dustins has quit IRC21:57
*** enriquetaso has quit IRC22:04
*** wanghao_ has quit IRC22:08
*** zul_ has joined #openstack-cinder22:09
*** zul has quit IRC22:12
*** karthikp__ has quit IRC22:14
*** akshai_ has quit IRC22:19
*** rdx_matt has joined #openstack-cinder22:20
*** rdx_matt has quit IRC22:21
*** jamielennox is now known as jamielennox|away22:21
*** ametts has quit IRC22:28
*** ducttape_ has joined #openstack-cinder22:30
*** tyrola has joined #openstack-cinder22:34
*** Apoorva has quit IRC22:38
*** Apoorva has joined #openstack-cinder22:38
*** ducttape_ has quit IRC22:39
*** abhi has joined #openstack-cinder22:39
*** ducttape_ has joined #openstack-cinder22:39
*** ametts has joined #openstack-cinder22:41
*** darosale has quit IRC22:43
*** julim has quit IRC22:46
*** ametts has quit IRC22:48
*** rdx_matt has joined #openstack-cinder22:48
*** rdx_matt has quit IRC22:49
*** ducttape_ has quit IRC22:52
*** gouthamr has joined #openstack-cinder22:54
*** mvk_ has joined #openstack-cinder22:58
*** salv-orl_ has joined #openstack-cinder22:59
*** jwcroppe has joined #openstack-cinder23:00
*** adrianofr has quit IRC23:01
*** salv-orlando has quit IRC23:01
*** mvk has quit IRC23:02
*** xyang has quit IRC23:06
*** xyang1 has quit IRC23:08
*** bardia has quit IRC23:11
*** rajinir has joined #openstack-cinder23:15
*** cknight1 has joined #openstack-cinder23:16
*** cknight has quit IRC23:16
*** rdx_matt has joined #openstack-cinder23:16
*** rdx_matt has quit IRC23:16
*** karthikp__ has joined #openstack-cinder23:21
*** bjornar has quit IRC23:29
*** fernnest has joined #openstack-cinder23:33
*** ducttape_ has joined #openstack-cinder23:34
*** karthikp__ has quit IRC23:37
*** karthikp__ has joined #openstack-cinder23:37
*** merooney has joined #openstack-cinder23:38
*** bjornar has joined #openstack-cinder23:39
*** cknight1 has quit IRC23:40
*** tyrola has quit IRC23:41
*** tyrola has joined #openstack-cinder23:42
*** jamielennox|away is now known as jamielennox23:43
openstackgerritRyan Selden proposed openstack/cinder: Added unit test coverage to test_limits.py for api v2  https://review.openstack.org/33468023:43
*** rdx_matt has joined #openstack-cinder23:44
*** rdx_matt has quit IRC23:44
*** ducttape_ has quit IRC23:47
*** gouthamr has quit IRC23:47
*** karthikp__ has quit IRC23:51
*** karthikp__ has joined #openstack-cinder23:51
*** ducttape_ has joined #openstack-cinder23:52
*** satya_ has quit IRC23:54
*** tyrola has quit IRC23:55

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