Monday, 2016-02-29

*** haomaiwang has quit IRC00:01
*** haomaiwang has joined #openstack-cinder00:01
*** ircuser-1 has joined #openstack-cinder00:02
*** Fdaisuke has joined #openstack-cinder00:05
*** laughterwym has quit IRC00:07
*** mylu has joined #openstack-cinder00:07
*** wmolicki has joined #openstack-cinder00:17
*** ntpttr has quit IRC00:19
*** laughterwym has joined #openstack-cinder00:22
*** bardia has joined #openstack-cinder00:23
*** DiaoSi has joined #openstack-cinder00:26
*** laughterwym has quit IRC00:26
*** wmolicki has quit IRC00:28
*** wmolicki has joined #openstack-cinder00:28
*** Gao has quit IRC00:29
*** avishay has quit IRC00:36
*** markvoelker has joined #openstack-cinder00:43
*** wmolicki has quit IRC00:44
*** mylu has quit IRC00:46
*** markvoelker has quit IRC00:47
*** avishay has joined #openstack-cinder00:48
*** mylu has joined #openstack-cinder00:50
*** haomaiwang has quit IRC00:58
*** laughterwym has joined #openstack-cinder01:01
*** cheneydc has joined #openstack-cinder01:03
*** yangyapeng has joined #openstack-cinder01:05
*** yangyapeng has quit IRC01:05
*** laughterwym has quit IRC01:06
*** EinstCrazy has joined #openstack-cinder01:06
*** mylu has quit IRC01:07
*** mylu has joined #openstack-cinder01:09
*** salv-orlando has joined #openstack-cinder01:12
*** bjornar has quit IRC01:14
*** yangyapeng has joined #openstack-cinder01:19
*** salv-orlando has quit IRC01:20
openstackgerritMerged openstack/cinder: VMware: manage_existing_get_size for VMDK driver  https://review.openstack.org/27606301:27
openstackgerritMerged openstack/cinder: Fix race condition in RemoteFS create_snapshot_online  https://review.openstack.org/28175301:29
*** haomaiwa_ has joined #openstack-cinder01:42
*** mylu has quit IRC01:44
*** markvoelker has joined #openstack-cinder01:44
*** mylu has joined #openstack-cinder01:46
*** markvoelker has quit IRC01:48
openstackgerritMerged openstack/cinder: Huawei: Judgement failure when creating hypermetro  https://review.openstack.org/28064101:56
*** haomaiwa_ has quit IRC02:01
*** haomaiwang has joined #openstack-cinder02:01
*** mylu has quit IRC02:18
openstackgerritMerged openstack/os-brick: Mock time.sleep() in 3 unit tests  https://review.openstack.org/28530202:18
*** mylu has joined #openstack-cinder02:20
*** avishay has quit IRC02:25
*** mylu has quit IRC02:28
*** mylu has joined #openstack-cinder02:29
*** mylu has quit IRC02:35
*** salv-orlando has joined #openstack-cinder02:37
*** mylu has joined #openstack-cinder02:37
*** avishay has joined #openstack-cinder02:38
*** lixiaoy1 has joined #openstack-cinder02:39
*** mylu has quit IRC02:40
openstackgerritMerged openstack/cinder: Disable multi-attach for RBD  https://review.openstack.org/28369502:42
*** yangyapeng has quit IRC02:44
*** yangyapeng has joined #openstack-cinder02:44
*** markvoelker has joined #openstack-cinder02:44
*** cbader_ has quit IRC02:45
*** mdavidson has quit IRC02:46
*** mylu has joined #openstack-cinder02:46
*** precurse has quit IRC02:46
*** lennyb__ has quit IRC02:46
*** gouthamr has quit IRC02:47
*** lennyb__ has joined #openstack-cinder02:47
*** gouthamr has joined #openstack-cinder02:48
*** markvoelker has quit IRC02:49
*** precurse has joined #openstack-cinder02:50
*** boris-42 has joined #openstack-cinder02:50
*** mdavidson has joined #openstack-cinder02:50
openstackgerritMerged openstack/cinder: Return all target_wwpns for FC storwize  https://review.openstack.org/28254102:51
*** salv-orlando has quit IRC02:53
*** winston-d_ has joined #openstack-cinder02:57
*** haomaiwang has quit IRC03:01
*** haomaiwang has joined #openstack-cinder03:01
*** mylu has quit IRC03:07
*** mylu has joined #openstack-cinder03:07
*** julim has quit IRC03:07
*** baojg has joined #openstack-cinder03:07
openstackgerritWilson Liu proposed openstack/cinder: Huawei: Mask chap password in log  https://review.openstack.org/27159503:09
openstackgerritMerged openstack/cinder: Handle OverQuota exception during volume transfer  https://review.openstack.org/28105303:14
openstackgerritMerged openstack/cinder: Improve logging to debug invalid "extra_specs" entries  https://review.openstack.org/27590903:14
*** mylu has quit IRC03:23
openstackgerrithuanan proposed openstack/cinder: Huawei: Check when attach hypermetro volume  https://review.openstack.org/28396403:31
*** huanan has joined #openstack-cinder03:34
*** bardia has quit IRC03:34
*** dave-mccowan has quit IRC03:42
*** links has joined #openstack-cinder03:44
*** stevemar has joined #openstack-cinder03:53
*** gouthamr has quit IRC03:53
*** salv-orlando has joined #openstack-cinder03:57
*** haomaiwang has quit IRC04:01
*** haomaiwang has joined #openstack-cinder04:01
*** links has quit IRC04:01
*** baojg has quit IRC04:02
*** salv-orlando has quit IRC04:03
*** shausy has joined #openstack-cinder04:11
*** david-lyle has joined #openstack-cinder04:13
*** links has joined #openstack-cinder04:18
*** avishay has quit IRC04:29
*** baojg has joined #openstack-cinder04:30
*** haomaiwang has quit IRC04:31
*** salv-orlando has joined #openstack-cinder04:32
*** david-lyle has quit IRC04:36
*** salv-orlando has quit IRC04:38
*** laughterwym has joined #openstack-cinder04:44
*** cknight has quit IRC04:44
*** laughter_ has joined #openstack-cinder04:45
*** markvoelker has joined #openstack-cinder04:45
*** laughter_ has joined #openstack-cinder04:45
openstackgerritLisaLi proposed openstack/cinder: Add backup update function  https://review.openstack.org/26317904:45
*** laughterwym has quit IRC04:48
openstackgerritMatt Smith proposed openstack/cinder: Updating Datera DataFabric Driver to v2 of Datera DataFabric API  https://review.openstack.org/28496704:49
*** markvoelker has quit IRC04:50
*** david-lyle has joined #openstack-cinder04:50
*** david-lyle has quit IRC04:56
*** laughter_ has quit IRC05:03
*** chhavi has joined #openstack-cinder05:06
*** mylu has joined #openstack-cinder05:07
*** Poornima has joined #openstack-cinder05:15
*** cvstealth has quit IRC05:15
*** laughterwym has joined #openstack-cinder05:16
*** laughterwym has quit IRC05:17
*** laughterwym has joined #openstack-cinder05:18
*** ishant has joined #openstack-cinder05:19
*** laughter_ has joined #openstack-cinder05:20
*** laughte__ has joined #openstack-cinder05:21
*** baojg has quit IRC05:22
*** laughterwym has quit IRC05:22
*** laughter_ has quit IRC05:24
*** baojg has joined #openstack-cinder05:24
*** laughte__ has quit IRC05:25
*** asselin__ has quit IRC05:25
*** laughterwym has joined #openstack-cinder05:36
*** [1]Thelo has joined #openstack-cinder05:36
*** salv-orlando has joined #openstack-cinder05:36
*** zhangjn has quit IRC05:38
*** Thelo has quit IRC05:39
*** [1]Thelo is now known as Thelo05:39
openstackgerritMerged openstack/cinder: Delete volumes with snapshots  https://review.openstack.org/26498205:40
*** avishay has joined #openstack-cinder05:42
*** laughterwym has quit IRC05:42
*** zhangjn has joined #openstack-cinder05:42
*** chhavi has quit IRC05:45
*** laughterwym has joined #openstack-cinder05:45
*** salv-orlando has quit IRC05:47
*** laughterwym has quit IRC05:49
*** ankit_ag has joined #openstack-cinder05:50
*** laughterwym has joined #openstack-cinder05:52
*** mylu has quit IRC05:53
*** laughter_ has joined #openstack-cinder05:54
*** laughterwym has quit IRC05:57
*** laughter_ has quit IRC05:59
*** laughterwym has joined #openstack-cinder06:21
openstackgerrithuanan proposed openstack/cinder: Huawei: Support huawei consistency group  https://review.openstack.org/26093006:24
*** gcb has joined #openstack-cinder06:26
*** jeanfabrice has quit IRC06:26
openstackgerritLisaLi proposed openstack/cinder: Make nullable of fields in db model and object match  https://review.openstack.org/28508606:28
openstackgerritLisaLi proposed openstack/cinder: Make nullable of fields in db model and object match  https://review.openstack.org/28508606:29
*** lpetrut has joined #openstack-cinder06:37
*** salv-orlando has joined #openstack-cinder06:42
*** liverpooler has joined #openstack-cinder06:44
*** markvoelker has joined #openstack-cinder06:46
*** yangyapeng has quit IRC06:46
*** zhangjn has quit IRC06:47
openstackgerritxiaoqin proposed openstack/cinder: IBM Storwize with pool-aware-cinder-scheduler  https://review.openstack.org/28461406:49
*** zhangjn has joined #openstack-cinder06:49
*** Fdaisuke has quit IRC06:49
*** markvoelker has quit IRC06:50
*** laughterwym has quit IRC06:52
*** laughterwym has joined #openstack-cinder06:55
*** nkrinner has joined #openstack-cinder06:57
*** cvstealth has joined #openstack-cinder06:59
*** laughterwym has quit IRC07:01
*** sheel has joined #openstack-cinder07:02
*** jeanfabrice has joined #openstack-cinder07:03
*** laughterwym has joined #openstack-cinder07:05
openstackgerritLisaLi proposed openstack/cinder: Fix exception during service update  https://review.openstack.org/28590807:10
*** jeanfabrice has quit IRC07:14
*** baojg has quit IRC07:17
*** baojg has joined #openstack-cinder07:18
*** anshul has joined #openstack-cinder07:20
*** rcernin has joined #openstack-cinder07:24
*** arecknag has joined #openstack-cinder07:24
*** belmoreira has joined #openstack-cinder07:25
*** laughterwym has quit IRC07:27
*** laughterwym has joined #openstack-cinder07:27
*** jeanfabrice has joined #openstack-cinder07:28
*** laughterwym has quit IRC07:31
*** laughterwym has joined #openstack-cinder07:31
*** laughterwym has quit IRC07:32
*** laughterwym has joined #openstack-cinder07:32
*** jeanfabrice has quit IRC07:32
*** laughter_ has joined #openstack-cinder07:33
*** zhangjn has quit IRC07:33
*** laughte__ has joined #openstack-cinder07:35
*** laughte__ has quit IRC07:36
*** laughte__ has joined #openstack-cinder07:36
*** laughterwym has quit IRC07:37
*** laughter_ has quit IRC07:38
*** lpetrut has quit IRC07:38
*** zhangjn has joined #openstack-cinder07:39
*** haomaiwang has joined #openstack-cinder07:41
*** lpetrut has joined #openstack-cinder07:48
*** avishay has quit IRC07:50
*** gus has quit IRC07:50
*** yangyapeng has joined #openstack-cinder07:51
*** gus has joined #openstack-cinder07:52
*** zhangjn has quit IRC07:54
*** zhangjn has joined #openstack-cinder08:00
*** permalac_ has quit IRC08:00
*** permalac_ has joined #openstack-cinder08:00
*** haomaiwang has quit IRC08:01
*** permalac_ has quit IRC08:01
*** haomaiwang has joined #openstack-cinder08:01
*** mattymo has quit IRC08:03
*** mattymo has joined #openstack-cinder08:05
*** arnewiebalck_ has joined #openstack-cinder08:07
*** lpetrut has quit IRC08:08
*** zhangjn has quit IRC08:18
*** lpetrut has joined #openstack-cinder08:23
*** baojg has quit IRC08:24
*** baojg has joined #openstack-cinder08:26
*** zhangjn has joined #openstack-cinder08:29
openstackgerritMatan Sabag proposed openstack/cinder: Changes in ScaleIO configurations options.  https://review.openstack.org/28154108:29
*** rcernin has quit IRC08:29
*** jyoti-ranjan has joined #openstack-cinder08:32
*** jyoti-ranjan has quit IRC08:32
*** rcernin has joined #openstack-cinder08:37
openstackgerritwanghao proposed openstack/cinder-specs: Improvement to query consistency group detail  https://review.openstack.org/24299808:39
openstackgerritAlon Marx proposed openstack/cinder: Remove a vol in error state from a CG  https://review.openstack.org/27046008:45
*** lpetrut has quit IRC08:45
openstackgerritMatan Sabag proposed openstack/cinder: Changes in ScaleIO configurations options.  https://review.openstack.org/28154108:45
*** baojg has quit IRC08:46
*** markvoelker has joined #openstack-cinder08:47
*** anshul has quit IRC08:48
*** anshul has joined #openstack-cinder08:48
*** laughte__ has quit IRC08:49
*** laughterwym has joined #openstack-cinder08:49
*** laughter_ has joined #openstack-cinder08:51
*** markvoelker has quit IRC08:51
openstackgerritwanghao proposed openstack/cinder-specs: Report backend state in service list  https://review.openstack.org/25292108:53
*** laughterwym has quit IRC08:54
*** Julien-zte has joined #openstack-cinder08:54
*** laughter_ has quit IRC08:55
*** baojg has joined #openstack-cinder08:56
openstackgerritwanghao proposed openstack/cinder-specs: Get the replica volume ref for DR  https://review.openstack.org/18215008:56
*** ndipanov has joined #openstack-cinder08:57
*** haomaiwang has quit IRC09:01
*** haomaiwang has joined #openstack-cinder09:01
*** jordanP has joined #openstack-cinder09:02
*** jistr has joined #openstack-cinder09:20
*** permalac has joined #openstack-cinder09:23
*** permalac has quit IRC09:23
*** permalac has joined #openstack-cinder09:23
*** patrickeast has quit IRC09:24
*** patrickeast has joined #openstack-cinder09:25
*** shausy has quit IRC09:28
*** shausy has joined #openstack-cinder09:30
*** Julien-zte has quit IRC09:32
*** Lee1092 has joined #openstack-cinder09:34
*** e0ne has joined #openstack-cinder09:43
*** baojg has quit IRC09:44
*** lpetrut has joined #openstack-cinder09:52
*** boris-42 has quit IRC09:54
*** e0ne has quit IRC09:55
*** salv-orl_ has joined #openstack-cinder09:59
*** haomaiwang has quit IRC10:01
*** haomaiwang has joined #openstack-cinder10:01
*** cheneydc has quit IRC10:01
*** salv-orlando has quit IRC10:02
*** baojg has joined #openstack-cinder10:04
openstackgerritMerged openstack/cinder: replace deprecated oslo_messaging _impl_messaging  https://review.openstack.org/25942310:05
*** ishant has quit IRC10:05
*** avishay has joined #openstack-cinder10:06
*** EinstCrazy has quit IRC10:14
*** yangyapeng has quit IRC10:18
openstackgerritMerged openstack/cinder: Remove unused kill method from Service  https://review.openstack.org/28087410:29
openstackgerritMerged openstack/cinder: Propoerly call oslo's Service stop and wait  https://review.openstack.org/28087510:29
openstackgerritAvishay Traeger proposed openstack/cinder: List manageable volumes and snapshots  https://review.openstack.org/28529610:33
avishayDuncanT: Added PoC code to list manageable. Would like to get it merged early in Newton to get good driver support.10:34
*** baojg has quit IRC10:42
*** salv-orlando has joined #openstack-cinder10:42
*** salv-orlando has quit IRC10:43
*** salv-orl_ has quit IRC10:43
*** salv-orlando has joined #openstack-cinder10:43
openstackgerritSrushti Gadadare proposed openstack/cinder: Use is_int_like method from oslo_utils  https://review.openstack.org/28597310:45
*** zerda has joined #openstack-cinder10:46
*** markvoelker has joined #openstack-cinder10:47
*** markvoelker has quit IRC10:52
*** sdague has joined #openstack-cinder10:55
*** haomaiwang has quit IRC11:01
*** haomaiwang has joined #openstack-cinder11:01
*** wilson has joined #openstack-cinder11:03
*** wilson is now known as Guest96511:03
openstackgerritMerged openstack/python-cinderclient: Fix return type in consistencygroups docstring  https://review.openstack.org/28513711:06
openstackgerritMerged openstack/python-cinderclient: Word Misspelling  https://review.openstack.org/26570811:06
openstackgerritMerged openstack/python-cinderclient: Use instanceof instead of type  https://review.openstack.org/28437711:06
*** Guest965 has quit IRC11:08
*** tpsilva has joined #openstack-cinder11:09
*** wilson_liu has joined #openstack-cinder11:09
*** xek has joined #openstack-cinder11:12
*** yhayashi has quit IRC11:13
*** laughterwym has joined #openstack-cinder11:16
*** asselin__ has joined #openstack-cinder11:22
*** yuriy_n17 has joined #openstack-cinder11:24
*** asselin__ has quit IRC11:26
DuncanTavishay: Sure, I'll take a look at the PoC before Newton opens, and it makes sense to get it in as early as possible11:26
avishayDuncanT: thanks a lot - i put the links in a comment in the spec11:27
openstackgerritMichal Dulko proposed openstack/cinder: Remove unused columns from Service ORM model  https://review.openstack.org/28073911:27
*** dims has joined #openstack-cinder11:29
*** dims has quit IRC11:32
*** dims has joined #openstack-cinder11:32
openstackgerritAlon Marx proposed openstack/cinder: Remove a vol in error state from a CG  https://review.openstack.org/27046011:33
*** liverpooler has quit IRC11:36
*** liverpooler has joined #openstack-cinder11:37
*** Julien-zte has joined #openstack-cinder11:38
yuriy_n17dulek: Hi! Very sorry for disturbing. Please review https://review.openstack.org/#/c/271696/ whenever you have free time.11:38
*** Poornima has quit IRC11:40
*** bjornar__ has joined #openstack-cinder11:45
*** raildo-afk is now known as raildo11:49
*** dave-mccowan has joined #openstack-cinder11:55
*** marcusvrn_ has joined #openstack-cinder11:56
openstackgerritSheel Rana proposed openstack/cinder: Use dictionary literal for dictionary creation  https://review.openstack.org/28599511:58
openstackgerritAndreas Jaeger proposed openstack/cinder: always use pip constraints  https://review.openstack.org/28549412:00
*** haomaiwang has quit IRC12:01
*** avishay has quit IRC12:01
*** haomaiwa_ has joined #openstack-cinder12:02
*** baojg has joined #openstack-cinder12:02
dulekyuriy_n17: Done, patch looks fine. :)12:04
-openstackstatus- NOTICE: Infra currently has a long backlog. Please be patient and where possible avoid rechecks while it catches up.12:05
yuriy_n17dulek: Thanks a lot.12:06
*** ndipanov has quit IRC12:10
openstackgerritSheel Rana proposed openstack/cinder: Use dictionary literal for dictionary creation  https://review.openstack.org/28599512:11
*** zerda has quit IRC12:11
*** EinstCrazy has joined #openstack-cinder12:17
openstackgerritwanghao proposed openstack/cinder: Unmanaging vol/snap reduces quota incorrectly  https://review.openstack.org/23243612:27
*** cheneydc has joined #openstack-cinder12:28
*** Gao has joined #openstack-cinder12:29
openstackgerritwanghao proposed openstack/cinder: Unmanaging vol/snap reduces quota incorrectly  https://review.openstack.org/23243612:29
*** DiaoSi has quit IRC12:33
*** cheneydc has quit IRC12:39
*** wanghao has quit IRC12:41
*** haomaiwa_ has quit IRC12:42
*** wanghao has joined #openstack-cinder12:42
*** haomaiwang has joined #openstack-cinder12:43
*** haomaiwang has quit IRC12:43
*** haomaiwa_ has joined #openstack-cinder12:43
*** haomaiwa_ has quit IRC12:43
*** chhavi has joined #openstack-cinder12:44
*** shausy has quit IRC12:45
*** markvoelker has joined #openstack-cinder12:48
*** markvoelker has quit IRC12:53
*** anshul has quit IRC12:54
*** yangyapeng has joined #openstack-cinder12:55
*** julim has joined #openstack-cinder12:56
openstackgerritjaveme proposed openstack/cinder: Use sizelimit from oslo.middleware  https://review.openstack.org/28217612:57
*** smoriya_ has quit IRC12:57
*** anshul has joined #openstack-cinder12:59
openstackgerritMerged openstack/cinder: Add pip-missing-reqs tox env  https://review.openstack.org/26825512:59
*** gcb has quit IRC13:01
*** Oku_OS has joined #openstack-cinder13:05
*** wanghao has quit IRC13:07
*** wanghao has joined #openstack-cinder13:08
*** ociuhandu has quit IRC13:11
*** haomaiwang has joined #openstack-cinder13:12
*** takedakn has joined #openstack-cinder13:16
*** links has quit IRC13:18
*** edmondsw has joined #openstack-cinder13:25
*** ociuhandu has joined #openstack-cinder13:31
*** gouthamr has joined #openstack-cinder13:32
*** ankit_ag has quit IRC13:35
openstackgerritMerged openstack/cinder: Move anyjson to test-requirements  https://review.openstack.org/28269013:35
*** takedakn has quit IRC13:37
*** akerr has joined #openstack-cinder13:39
*** tongli has joined #openstack-cinder13:40
*** scottda_is_outta is now known as scottda13:41
*** liverpooler has quit IRC13:42
scottdabjornar__: Hi13:43
bjornar__scottda, hi13:43
openstackgerritMichal Dulko proposed openstack/cinder: Clean up replication v2.1 (Cheesecake) RPC API  https://review.openstack.org/28604713:44
scottdabjornar__: I tried to fix the python3 breaking problem you pointed out above with the exception in L#115613:44
scottdabjornar__: What issue are you seeing?13:44
bjornar__scottda, currently or yesterday?13:44
scottdano, this was a while back before the patch merged.13:45
scottdaThere were issues in the py34 tests, so I did the try:....except: to deal with it.13:45
scottdaWhat issues are there right now?13:46
bjornar__it is written in the issue I think.13:46
bjornar__https://bugs.launchpad.net/cinder/+bug/155095113:47
openstackLaunchpad bug 1550951 in Cinder "26ac08ca79bb9e69d63a4a3cead567556bb31c97 breaks python3" [Undecided,New] - Assigned to Scott DAngelo (scott-dangelo)13:47
*** markvoelker has joined #openstack-cinder13:47
bjornar__scottda, You see some headers are utf813:47
scottdabjornar__: OK, I did not see that bug. Can you put some info on how to reproduce this?13:48
bjornar__scottda, I guess it can be reproduced by rinning cinder service-list for example under python313:49
scottdaThanks.13:49
bjornar__scottda, The problem disappears if I replace val.encode('utf-8') with six.text_type(val)13:49
bjornar__scottda, So guessing that there is no exception for python313:50
scottdaOK, I'll have a look. Thanks.13:50
*** wilson has joined #openstack-cinder13:51
*** wilson is now known as Guest4657913:51
bjornar__scottda, thanks13:51
*** Guest46579 has quit IRC13:52
openstackgerritMichal Dulko proposed openstack/cinder: Add volume RPC API v2.0  https://review.openstack.org/28131713:54
openstackgerritSheel Rana proposed openstack/cinder: Use dictionary literal for dictionary creation  https://review.openstack.org/28599513:54
openstackgerritMichal Dulko proposed openstack/cinder: Add scheduler RPC API v2.0  https://review.openstack.org/28130713:55
openstackgerritVivek Dhayaal proposed openstack/cinder: Support ZeroMQ messaging in cinder multibackend  https://review.openstack.org/27711313:55
sheelscottda: hi there13:56
scottdasheel: HI13:56
sheelscottda: could you have a look at https://review.openstack.org/#/c/278605/13:57
sheelscottda: and let me know if you need any change in it?13:57
scottdaYes. I'll look this morning.13:57
sheelk thanks13:58
*** wilson-liu has joined #openstack-cinder13:58
scottdasheel: It looks like it is failing Jenkins. Have you had a look at that?13:58
*** links has joined #openstack-cinder13:59
sheelscottda: yes I had a quick glance, i think this was due to microversion dependency...13:59
scottdasheel: OK. You may need a rebase, since microversion has landed.13:59
bjornar__sheel, dotn say "glance"13:59
sheelbjornar__: haha14:00
sheelbjornar__: ok, i had a quick view :)14:00
bjornar__better14:00
scottdasheel: And you will need a dependency on the patch to fix the string for api_version.matches()14:00
*** rlrossit has joined #openstack-cinder14:00
sheelscottda: sure, i will add14:00
sheelscottda: could you provide link?14:00
scottdahttps://review.openstack.org/#/c/28529314:00
*** haomaiwang has quit IRC14:01
*** adrianofr_ has joined #openstack-cinder14:01
sheelscottda: ok, thanks will add in a while14:01
*** bswartz has joined #openstack-cinder14:01
sheelscottda: may be you can have a quick look if you see any other requirement in it14:01
*** haomaiwang has joined #openstack-cinder14:01
openstackgerritSheel Rana proposed openstack/cinder: Added Bootable filter for cinder list command  https://review.openstack.org/27860514:01
scottdasheel: I will14:01
sheelscottda: ok, thank you14:01
*** cknight has joined #openstack-cinder14:03
openstackgerritSheel Rana proposed openstack/cinder: Added Bootable filter for cinder list command  https://review.openstack.org/27860514:06
*** baojg has quit IRC14:06
*** mriedem has joined #openstack-cinder14:11
yuriy_n17geguileo: Hi! Very sorry for disturbing. Please review https://review.openstack.org/#/c/271696/ whenever you have free time.14:14
*** martyturner has joined #openstack-cinder14:14
*** baojg has joined #openstack-cinder14:15
*** links has quit IRC14:16
*** boris-42 has joined #openstack-cinder14:16
geguileoyuriy_n17: Done14:17
openstackgerritMerged openstack/cinder: Modify the number of params of hypermetro in HuaweiDriver  https://review.openstack.org/28099714:18
*** kvidvans has quit IRC14:19
*** cdelatte has joined #openstack-cinder14:21
yuriy_n17geguileo: Thank you ever so much.14:21
geguileoyuriy_n17: Thank you for fixing it14:21
yuriy_n17geguileo: Thanks. Sorry for disturbing. ;)14:22
geguileoyuriy_n17: Not a problem14:22
openstackgerritMichal Dulko proposed openstack/cinder: Add necessary fields to volume creation  https://review.openstack.org/28460014:22
*** merooney has joined #openstack-cinder14:22
yuriy_n17jgriffith: Hi! Very sorry for disturbing. Please review https://review.openstack.org/#/c/271696/ whenever you have free time.14:26
*** andymaier has joined #openstack-cinder14:26
*** porrua has joined #openstack-cinder14:26
yuriy_n17jgriffith: Thank you ever so much.14:30
jgriffithyuriy_n17: sure, I was actually already reviewing that14:30
yuriy_n17jgriffith: thanks. sorry for disturbing.14:31
jgriffithyuriy_n17: not at all14:32
*** anshul has quit IRC14:32
*** baojg has quit IRC14:32
*** baojg has joined #openstack-cinder14:34
*** mylu has joined #openstack-cinder14:34
*** eharney has joined #openstack-cinder14:34
*** asselin__ has joined #openstack-cinder14:35
*** cknight has quit IRC14:35
*** cknight has joined #openstack-cinder14:36
*** mylu has quit IRC14:38
*** asselin__ has quit IRC14:40
*** mylu has joined #openstack-cinder14:42
*** knikolla has joined #openstack-cinder14:42
*** diablo_rojo has joined #openstack-cinder14:42
*** diablo_rojo has left #openstack-cinder14:42
*** jgregor has joined #openstack-cinder14:44
openstackgerritMichal Dulko proposed openstack/cinder: Add ConsistencyGroupVolumeTypeMapping table  https://review.openstack.org/24585414:44
*** dustins has joined #openstack-cinder14:45
*** diablo_rojo has joined #openstack-cinder14:48
*** timcl has joined #openstack-cinder14:50
*** baojg has quit IRC14:50
*** wanghao has quit IRC14:51
*** gcb has joined #openstack-cinder14:51
*** wanghao has joined #openstack-cinder14:51
*** ndipanov has joined #openstack-cinder14:52
*** edtubill has joined #openstack-cinder14:53
*** asselin has joined #openstack-cinder14:54
*** baojg has joined #openstack-cinder14:57
*** dustins has quit IRC14:59
*** baumann has joined #openstack-cinder15:00
*** haomaiwang has quit IRC15:01
*** haomaiwang has joined #openstack-cinder15:01
*** EinstCra_ has joined #openstack-cinder15:01
*** EinstCra_ has quit IRC15:02
*** EinstCrazy has quit IRC15:02
*** lennyb__ is now known as lennyb15:03
*** EinstCrazy has joined #openstack-cinder15:03
*** wilson-liu has quit IRC15:08
*** asselin__ has joined #openstack-cinder15:12
*** dustins has joined #openstack-cinder15:12
*** asselin has quit IRC15:15
*** jungleboyj has joined #openstack-cinder15:16
*** mtanino has joined #openstack-cinder15:16
*** tjcocozz has joined #openstack-cinder15:17
*** asselin has joined #openstack-cinder15:18
*** asselin__ has quit IRC15:19
*** permalac has quit IRC15:19
*** permalac has joined #openstack-cinder15:19
*** permalac has quit IRC15:19
*** permalac has joined #openstack-cinder15:20
*** yangyapeng has quit IRC15:20
*** asselin has quit IRC15:22
*** asselin has joined #openstack-cinder15:23
*** kfarr has joined #openstack-cinder15:24
*** mylu has quit IRC15:28
*** laughterwym has quit IRC15:28
*** laughterwym has joined #openstack-cinder15:29
*** mriedem is now known as mriedem_meeting15:30
*** kvidvans has joined #openstack-cinder15:31
*** knomura has joined #openstack-cinder15:34
*** laughterwym has quit IRC15:34
*** mdenny has joined #openstack-cinder15:34
*** mylu has joined #openstack-cinder15:36
openstackgerritShay Halsband proposed openstack/cinder: XtremIO handle errors in terminate_connection:  https://review.openstack.org/28611615:37
*** [1]Thelo has joined #openstack-cinder15:37
openstackgerritTom Barron proposed openstack/cinder: Fix invalid uuid warnings in backup unit tests  https://review.openstack.org/28611815:39
*** Thelo has quit IRC15:40
*** [1]Thelo is now known as Thelo15:40
scottdaIn order to use python3 with devstack, is it sufficient to put "export USE_PYTHON3=TRUE" in stackrc?15:43
scottdaDoes anyone know a good way to verify which python we are using with Devstack/Cinder?15:43
openstackgerritAlex O'Rourke proposed openstack/cinder: 3PAR: Update replication to v2.1  https://review.openstack.org/27930715:43
*** knomura has quit IRC15:44
scottdahmmm.. I can see from cinder-api logs That USE_PYTHON3 does not mean we are using PYTHON315:44
bswartzscottda: if everything passed normally, you know it was python2. If it catches fire and explodes horribly, you can bet it was probably python3.15:44
smcginnis:D15:45
scottdabswartz: Thanks. So does that mean we are not ready for python3?15:45
*** knomura has joined #openstack-cinder15:45
sheel:)15:45
bswartz¬_¬15:45
*** arecknag has quit IRC15:45
bswartzno I'm just kidding15:45
smcginnisKind of.15:45
scottdabjornar__: Filed this bug that 'cinder-manage service list" fails when using python3: https://bugs.launchpad.net/cinder/+bug/155095115:45
openstackLaunchpad bug 1550951 in Cinder "26ac08ca79bb9e69d63a4a3cead567556bb31c97 breaks python3" [Undecided,New] - Assigned to Scott DAngelo (scott-dangelo)15:45
bswartzI think python3 support has come a long way, but I'm not sure it works 100%15:45
smcginnisscottda: We just got all unit tests passing, but that's just unit tests.15:46
smcginnisAFAIK, no one has actually fully run with py3.15:46
smcginnisIt wasn't even possible until a weel or so ago.15:46
smcginnisAnd I would be very surprised if everything were to just work.15:46
scottdabjornar__: Did you repro that bug on Devstack? Can you please put info in that bug as to how to reproduce?15:46
scottdasmcginnis: That sounds about right. I'm just trying to fix a Monday morning microversion bug :)15:47
*** jgregor has left #openstack-cinder15:47
*** jgregor has joined #openstack-cinder15:47
*** baojg has quit IRC15:48
smcginnisscottda: I'm sure that one is just one of many, but good to start working through some of them for sure.15:49
*** mriedem_meeting is now known as mriedem15:51
*** laughterwym has joined #openstack-cinder15:51
*** julim has quit IRC15:51
*** julim has joined #openstack-cinder15:53
*** wolsen has joined #openstack-cinder15:53
*** haplo37 has joined #openstack-cinder15:54
*** EinstCrazy has quit IRC15:55
*** dustins has quit IRC15:57
*** dustins has joined #openstack-cinder15:58
*** huyang has quit IRC15:58
*** huyang has joined #openstack-cinder15:59
*** salv-orl_ has joined #openstack-cinder16:00
scottdasheel: Can you verify that your bootable filter patch meets the requirements for "When do I need a new Microversion?" http://docs.openstack.org/developer/cinder/devref/api_microversion_dev.html16:00
*** jungleboyj has quit IRC16:00
*** haomaiwang has quit IRC16:01
sheelscottda: hey sure16:01
*** haomaiwang has joined #openstack-cinder16:01
sheelscottda: by the way this was suggestion from DuncanT16:01
scottdasheel: Thanks. Just want to make sure a microversion is really needed before we go further.16:01
*** arch-nemesis has joined #openstack-cinder16:01
*** arch-nemesis has quit IRC16:01
scottdasheel: Yeah, lotsa dodgy advice from that chap :)16:02
*** arch-nemesis has joined #openstack-cinder16:02
mriedemyikes, anyone ever seen this? http://logs.openstack.org/49/281149/5/gate/gate-tempest-dsvm-full-ceph/82e9e00/logs/screen-c-api.txt.gz#_2016-02-29_12_16_36_11616:02
*** crose has joined #openstack-cinder16:02
*** jungleboyj has joined #openstack-cinder16:02
*** salv-orlando has quit IRC16:03
DuncanTDefinitely don't take advice from me. Terrible idea, never ends up well16:03
*** nkrinner has quit IRC16:03
smcginnismriedem: Ceph has been making me sad.16:03
*** laughterwym has quit IRC16:04
mriedemhttp://logs.openstack.org/49/281149/5/gate/gate-tempest-dsvm-full-ceph/82e9e00/logs/screen-c-bak.txt.gz16:04
smcginnismriedem: Oh, but that's a new one!16:04
mriedemhttp://logs.openstack.org/49/281149/5/gate/gate-tempest-dsvm-full-ceph/82e9e00/logs/screen-c-bak.txt.gz#_2016-02-29_12_13_20_50116:04
mriedemlooks like the backup service puked16:04
*** laughterwym has joined #openstack-cinder16:04
sheelDuncanT: hahaha16:05
*** ntpttr has joined #openstack-cinder16:05
mriedemsmcginnis: reported a bug https://bugs.launchpad.net/cinder/+bug/155130516:06
openstackLaunchpad bug 1551305 in Cinder "backup service crashes in ceph job" [Medium,Confirmed]16:06
mriedemso far only 1 hit in logstash16:06
smcginnismriedem: Thanks for the heads up.16:06
*** lprice has joined #openstack-cinder16:06
DuncanTmriedem: That's ugly. I can't see a code change likely to have caused it in the cinder tree either :-( I wonder if the rbd libs have released recently16:06
sheelscottda I think DuncanT advice is ok for sure16:06
*** asselin__ has joined #openstack-cinder16:07
sheelscottda: the list of query parameter values for non free form fields16:07
sheelExample: parameter filter_by takes a small set of constants/enums “A”, “B”, “C”. Adding support for new enum “D”.16:07
sheelscottda: this suffice bootable filter patch for microversion16:07
sheelscottda: what you say?16:07
mriedemii  python-ceph                          0.80.11-0ubuntu1.14.04.1              amd64        Python libraries for the Ceph distributed filesystem16:07
*** laughterwym has quit IRC16:08
*** laughterwym has joined #openstack-cinder16:08
scottdasheel: I'm not so sure. Looking at that flow chart, I don't see how you get to "Yes" with that. But I could be wrong.16:08
*** crose has quit IRC16:08
mriedemreleased on Mon, 18 Jan 2016 14:25:14 +000016:09
scottdaI think DuncanT had a good idea, that it would be nice to get an example of code that uses microversions into Mitaka, but we shouldn't be doing it just for the sake of an example.16:09
DuncanTmriedem That's a metapackage though, python-rbd is the one we actually use I think16:10
mriedemi believe librados provides it16:10
*** asselin has quit IRC16:10
DuncanTscottda: It turns a previously valid request into an error, so I think that needs a microversion according to the chart?16:10
scottdaDuncanT: Yes, it does. I just wanted to confirm that was the case. Thanks.16:11
sheelscottda: ah, happy to be in microversion16:12
sheelDuncanT: thank you...16:12
DuncanTmriedem: I think "pure virtual method called" is coming from C++ not from python, which strongly suggests something changed / broken in the ceph libraries somewhere16:12
mriedemhttp://tracker.ceph.com/issues/1363616:13
mriedemyah16:13
mriedemprobably need jdurgin to review16:13
mriedemyeah, and that fix isn't in the list of changes in the recent release from ubuntu https://bugs.launchpad.net/ubuntu/trusty/+source/ceph/+bug/153527816:15
openstackLaunchpad bug 1535278 in ceph (Ubuntu Trusty) "0.80.11 stable point release" [Medium,Fix released] - Assigned to James Page (james-page)16:15
*** laughterwym has quit IRC16:15
*** laughterwym has joined #openstack-cinder16:15
*** rcernin has quit IRC16:16
DuncanTWe're using EOL ceph code too? I guess that is due to the Ubuntu LTS cycle16:16
mriedemyeah16:16
mriedemi commented on the ubuntu bug asking if they can get that patch backported16:16
DuncanTMind if I ask how you found that specific ceph PR? I'm struggling to make the connection16:17
sheelsmcginnis: hi there16:17
mriedemDuncanT: i just googled the error16:17
sheelsmcginnis: could you please find some spare time to review https://review.openstack.org/#/c/274589/16:17
mriedemhttp://lmgtfy.com/?q=librados+%22pure+virtual+method+called%2216:17
smcginnissheel: Sure, no problem.16:18
sheelsmcginnis: thank you16:18
DuncanTmriedem: Ah, fair enough then :-) I didn't have libradios in my search. Thanks16:18
mriedemso yeah, we seem to have a few ceph job regressions since the recent update16:19
mriedemi opened this 2 days ago https://bugs.launchpad.net/cinder/+bug/155040916:20
openstackLaunchpad bug 1550409 in Cinder ""Detected volume stuck in downloading status, setting to ERROR." in ceph job" [Undecided,New]16:20
*** laughterwym has quit IRC16:20
*** yangyapeng has joined #openstack-cinder16:21
*** wilson_liu has quit IRC16:21
*** wilson_liu has joined #openstack-cinder16:22
*** xyang has joined #openstack-cinder16:22
*** belmoreira has quit IRC16:24
*** mylu has quit IRC16:25
*** chris_morrell has joined #openstack-cinder16:26
lpricesmcginnis: Would you mind giving me a review on a consistency group patch if you have time? https://review.openstack.org/#/c/28238816:26
*** ndipanov has quit IRC16:26
smcginnislprice: Tab opened.16:26
*** ndipanov has joined #openstack-cinder16:26
lpricesmcginnis: Thanks16:27
*** vincent_hou has joined #openstack-cinder16:27
*** chris_morrell has quit IRC16:28
*** mylu has joined #openstack-cinder16:28
openstackgerritMerged openstack/cinder: Add description when rename LUN in Huawei driver  https://review.openstack.org/27160316:28
openstackgerritMerged openstack/cinder: Huawei: delete_snapshot need not return any value  https://review.openstack.org/28451416:28
openstackgerritMerged openstack/cinder: Remove unused pngmath sphinx extension  https://review.openstack.org/28583316:29
*** yangyapeng has quit IRC16:29
*** hemna has joined #openstack-cinder16:30
sheelsmcginnis: thanks16:32
*** dustins_ has joined #openstack-cinder16:34
*** boichev has joined #openstack-cinder16:39
*** fthiagogv has joined #openstack-cinder16:39
*** mylu has quit IRC16:42
*** boichev2 has quit IRC16:46
mriedemdoes cinder set default logging levels for other libraries somewhere?16:46
jgriffithmriedem: no... but there was a patch by eharney to at least route them into cinder logs instead of stdout16:47
mriedemhmm, ok16:48
jgriffithmriedem: any log routing/leve setting currently is in the drivers that import them IIRC16:48
eharneywell that was for the "warnings" module16:48
jgriffiths/leve/level/16:48
eharneythere is the default_log_levels setting that sets levels for different libraries16:48
mriedemproblem is unit test runs are logging stevedore debug logging which emits a log every time an extension is loaded16:48
jgriffitheharney: yeah... sorry, I should've been more clear16:48
mriedemin a py27 run on liberty, that's over 65K times16:48
mriedemhttp://logs.openstack.org/periodic-stable/periodic-cinder-python27-liberty/c36b836/console.html#_2016-02-28_06_09_35_75916:48
jgriffitheharney: I didn't think that worked?  Ok... ignore me then :)16:49
mriedemwhich explodes the console log, which is indexed for logstash16:49
mriedemso it could also be killing logstash16:49
smcginnisI only see three stevedore references in the code.16:49
smcginnisNone of them setting a different logging level.16:49
mriedemyeah16:49
eharneyi think you can add ",stevedore=WARN" to default_log_levels for that16:49
jgriffithmriedem: ohhh!  Sorry, I thought you meant the driver libs16:49
smcginnismriedem: Are you saying we should set it's logging level to something differnt in unit tests?16:49
mriedemwell, i was looking for something like this https://github.com/openstack/nova/blob/master/nova/config.py#L4316:50
jgriffithmriedem: I see what you're saying, that's rather annoying16:50
*** markstur has joined #openstack-cinder16:50
mriedemanyway, i'm checking into it16:50
*** xyang has quit IRC16:51
jgriffithmriedem: smcginnis let me know if you want to open a bug and assign to me or if somebody else wants it.  We should certainly be doing that IMO16:51
mriedemhttps://bugs.launchpad.net/cinder/+bug/155132516:51
openstackLaunchpad bug 1551325 in Cinder ""found extension EntryPoint.parse" blows up the size of the console log in py27 runs" [Undecided,New]16:51
guitarzanmriedem: yeah, you have to dig into oslo.log to find that since cinder doesn't have one set :)16:51
mriedemi already did16:51
jgriffithmriedem: :)16:51
smcginnisAt least should suppress most of that for test runs.16:51
*** knomura has quit IRC16:51
jgriffithanybody else got dibs?  I'll assign it to myself and if somebody else wants it hollar16:51
jgriffithI can probably fix it this afternoon16:52
smcginnisjgriffith: Go for it.16:52
jgriffithsmcginnis: I can bear to spend another day on the attach unit test disaster16:52
jgriffiths/can/CAN'T/16:52
smcginnis:)16:52
*** dustins has quit IRC16:52
*** dustins_ is now known as dustins16:52
smcginnisjgriffith: How about more replication tweaks?16:52
*** thingee has quit IRC16:53
mriedemjgriffith: hmm, https://github.com/openstack/oslo.log/blob/master/oslo_log/_options.py#L27 stevedore=WARN is already set as the default in oslo.log16:53
*** thingee has joined #openstack-cinder16:53
smcginnisMaybe another lib overwriting it?16:54
smcginnisProbably still not a bad idea to set it like nova is doing.16:54
openstackgerritMerged openstack/python-cinderclient: Trival: Remove 'MANIFEST.in'  https://review.openstack.org/26556516:55
*** gcb has quit IRC16:56
*** apoorvad has joined #openstack-cinder16:57
*** vincent_hou has quit IRC16:58
mriedemjgriffith: i think i might know what's up17:00
*** haomaiwang has quit IRC17:01
mriedemthe base test class is setting up a log fixture17:01
mriedemthat might be getting set to debug17:01
*** haomaiwa_ has joined #openstack-cinder17:01
*** garthb has joined #openstack-cinder17:04
mriedemjgriffith: fyi https://github.com/openstack/nova/blob/master/nova/tests/fixtures.py#L8217:06
mriedemmight want to re-use that in cinder17:06
*** boris-42 has quit IRC17:14
*** rhagarty has joined #openstack-cinder17:14
*** openstackgerrit has quit IRC17:18
*** openstackgerrit has joined #openstack-cinder17:19
*** bjornar has joined #openstack-cinder17:23
openstackgerritTom Barron proposed openstack/cinder: Fix invalid uuid warnings in scheduler unit tests  https://review.openstack.org/28616717:24
duleksmcginnis: Hi, can we switch https://blueprints.launchpad.net/cinder/+spec/rpc-object-compatibility to completed? I would do it myself, but I'm not assigned to it.17:25
duleksmcginnis: Main functionality is done I believe, the follow ups are release notes, docs, maybe bugfixes.17:25
dulek(hopefully no bugfixes ;))17:26
*** jistr has quit IRC17:26
lpricesmcginnis: Just saw your review, thanks.17:28
*** timcl has quit IRC17:28
*** david-lyle has joined #openstack-cinder17:30
lpriceDuncanT: Would you mind giving me a second review on https://review.openstack.org/#/c/282388?17:30
*** bardia has joined #openstack-cinder17:30
openstackgerritMatt Riedemann proposed openstack/cinder: WIP: Copy StandardLogging fixture from Nova  https://review.openstack.org/28617417:32
mriedemjgriffith: we'll see if this does anything ^17:32
*** wanghao has quit IRC17:32
*** timcl has joined #openstack-cinder17:33
*** wanghao has joined #openstack-cinder17:34
bjornarscottda, So, you cant reproduce?17:39
*** porrua has quit IRC17:39
scottdabjornar: Not on devstack. But I don't think py34 is working for cinder on devstack yet.17:39
*** lpetrut has quit IRC17:40
bjornarOk, Im not using devstack..17:40
bjornarscottda, but I do believe I provided enough information. Problem is that python34 does not fail in the try/catch17:41
*** laughterwym has joined #openstack-cinder17:43
scottdabjornar: OK. I'll keep on this. But py34 is only passing unit tests for Cinder. It is not *officially* supported.17:43
*** _alastor_ has quit IRC17:44
*** _alastor_ has joined #openstack-cinder17:45
scottdabjornar: So, you are saying that in the Cinder-api logs, there is no exception ?17:45
*** wanghao has quit IRC17:45
*** wanghao has joined #openstack-cinder17:46
bjornarscottda, yeah there is exception17:48
*** laughterwym has quit IRC17:48
_alastor_I have a system that's currently in this state, when I run the test manually I get the same exception17:49
_alastor_The c-vol service is definitely running17:50
mriedemcinder people might like this https://review.openstack.org/#/c/286180/17:52
mriedemget with the times cinder!17:52
scottda_alastor_: Are you talking about the same issue as bjornar ?17:54
*** david-lyle has quit IRC17:55
*** david-lyle_ has joined #openstack-cinder17:55
_alastor_I'm not sure?  I lost my chat history.  Is his issue dealing with 'test_get_service_by_hostname'?17:55
scottda_alastor_: No, not that I know of.17:56
*** tongli has quit IRC17:56
_alastor_scottda: In that test when services are queried by hostname I'm only getting the cinder-sceduler and not the cinder-volume service.17:57
_alastor_But cinder-volume is definitely running17:58
scottda_alastor_: OK. Sorry, but I don't have any context as to what your initial conversation was about. I was talking with bjornar about a different subject.17:58
*** krtaylor has quit IRC17:59
_alastor_scottda: np, mostly grasping at straws here to see if anyone else has seen this.17:59
scottda_alastor_: Well, I think most of us have seen c-vol fail to start at some time or another :)18:00
scottda_alastor_: What does it say in the c-vol logs?18:00
*** haomaiwa_ has quit IRC18:01
*** haomaiwang has joined #openstack-cinder18:01
*** Guest67703 is now known as me_18:01
_alastor_scottda: No Exceptions or Errors.  The service is running, I can make queries to cinder, create volumes.  Every tempest test but this one passes18:01
*** me_ is now known as med_18:01
*** med_ has quit IRC18:01
*** med_ has joined #openstack-cinder18:01
*** thangp has joined #openstack-cinder18:02
smcginnis_alastor_: There was a patch out there to fix handling when filtering by host name.18:03
smcginnis_alastor_: I thought it went in, but maybe not.18:03
scottda_alastor_: ahhh.ok. I mis-read "c-vol service is definitely running" as "definitely  NOT running". Sorry18:03
_alastor_smcginnis: You have a commit id?  I could check my cinder repo.18:04
smcginnis_alastor_: Looking now18:04
smcginnis_alastor_: That one was merged: https://review.openstack.org/#/c/263257/18:05
smcginnis_alastor_: See if your case is different.18:05
_alastor_smcginnis: Thanks18:05
_alastor_smcginnis: I have that commit18:07
smcginnisHmm18:07
_alastor_smcginnis: Maybe it's something to do with restarting the cinder service?18:07
*** vincent_hou has joined #openstack-cinder18:10
_alastor_smcginnis: when I run `cinder service-list --host=<my_host>` I get both services18:10
*** krtaylor has joined #openstack-cinder18:11
smcginnis_alastor_: But without --host you don't?18:11
_alastor_It's the same output18:11
_alastor_smcginnis: It's only the test that doesn't get that18:12
smcginnis_alastor_: I'm confused. I thought the issue was the service was not showing up.18:12
smcginnis_alastor_: Oh, just the test query is failing?18:12
_alastor_Yeah, the test query for the services by hostname gets just the scheduler, which causes the test to fail18:12
_alastor_But everything else points to the service being up and running18:13
smcginnis_alastor_: Which test is this? Can you give me the full path? I can try running it locally here and see what I get.18:13
_alastor_smcginnis: tempest.api.volume.admin.test_volume_services.VolumesServicesV2TestJSON.test_get_service_by_host_name18:13
_alastor_smcginnis: the V1 test fails the same way18:14
smcginnis_alastor_: Oh, those are skipped in tempest runs right now. Hmmm.18:15
*** bjornar has quit IRC18:15
*** vincent_hou_ has joined #openstack-cinder18:15
smcginnis_alastor_: https://github.com/openstack/tempest/blob/master/tempest/api/volume/admin/test_volume_services.py#L5418:15
*** bardia has quit IRC18:16
*** vincent_hou has quit IRC18:16
smcginnis_alastor_: But the bug referred to is marked Fix Released.18:16
_alastor_smcginnis: The fix's commit-id is the same one I verified was in my cinder branch earlier18:17
*** bjornar has joined #openstack-cinder18:18
*** vincent_hou_ is now known as vincent_hou18:19
scottdaIt looks like kmartin_ pointed out CI failure on the tempest test for review of PS #1618:20
*** chris_morrell has joined #openstack-cinder18:22
openstackgerritWalter A. Boring IV (hemna) proposed openstack/cinder: CONF add suppress_requests_ssl_warnings  https://review.openstack.org/26787018:23
openstackgerritJacob Gregor proposed openstack/cinder: Storwize SVC multiple management IPs  https://review.openstack.org/28426718:23
*** jgregor has quit IRC18:23
bjornarIs anyone considering creaking up cinder.conf into multiple parts?18:23
*** Guest63721 is now known as mfisch18:24
*** mfisch is now known as Guest200418:25
*** timcl has quit IRC18:25
*** yangyapeng has joined #openstack-cinder18:26
DuncanTbjornar: You can do that now if you like... just change the service startup script to list them all on the command line18:26
*** timcl has joined #openstack-cinder18:26
*** raildo is now known as raildo-afk18:27
*** Suyash has joined #openstack-cinder18:27
*** krtaylor has quit IRC18:27
_alastor_smcginnis: Ok, so I just ran through the test in pdb and this is what seems to be happening18:27
bjornarDuncanT, problem is noone knows what options belongs where ;)18:28
*** ntpttr has quit IRC18:28
bjornarDuncanT, talking about -api -volume -backup -scheduler18:28
_alastor_smcginnis: the `self.services` object has two services in it, but their 'host' properties are 'ubuntu' and 'ubuntu@datera' respectively18:28
*** Guest2004 is now known as mfisch18:28
*** mfisch has quit IRC18:28
*** mfisch has joined #openstack-cinder18:28
_alastor_smcginnis: 'self.host_name` (which is used as the filter) is just `ubuntu`18:28
bjornarDuncanT, Splitting config would force documentation in the right direction18:28
*** ntpttr has joined #openstack-cinder18:29
_alastor_smcginnis: so the list comprehension on this line is filtering out cinder-volume even though it's there: https://github.com/openstack/tempest/blob/master/tempest/api/volume/admin/test_volume_services.py#L6218:29
smcginnis_alastor_: So the list comprehension needs to be updated it appears.18:29
*** raildo-afk is now known as raildo18:29
smcginnisOr more accurately, the get_host call.18:29
*** boris-42 has joined #openstack-cinder18:30
_alastor_smcginnis: I think it's directly related to restarting the cinder service, but that's just a hunch18:30
smcginnis_alastor_: I'm not sure I follow.18:30
_alastor_smcginnis: *cinder-vol service18:30
_alastor_smcginnis: I restart the c-vol service after stacking in my CI setup (because the patch for our backend hasn't merged yet), and that's the service that isn't getting matched correctly18:31
_alastor_smcginnis: I don't remember adding anything that would change the hostname though18:33
_alastor_smcginnis: But I bet this wouldn't happen if the c-vol service isn't restarted.  I'll test it out18:33
*** ntpttr has quit IRC18:33
openstackgerritKaitlin Farr proposed openstack/cinder: Replace key manager with Castellan  https://review.openstack.org/28049218:33
*** yangyapeng has quit IRC18:34
smcginnis_alastor_: But if you're saying both services are in self.services, then it appears it's just incorrectly being filtered out in the tempest test.18:34
*** ntpttr has joined #openstack-cinder18:34
*** porrua has joined #openstack-cinder18:34
_alastor_smcginnis: Yeah, it's definitely a bug, but i think it's a bug that only happens on a volume service restart18:34
smcginnis_alastor_: Still not following why, but let me know if you track that down.18:35
*** jgregor has joined #openstack-cinder18:35
lpriceDuncanT: Thanks18:35
*** angela-s has joined #openstack-cinder18:37
*** chhavi has quit IRC18:39
openstackgerritMatt Riedemann proposed openstack/cinder: Copy StandardLogging fixture from Nova  https://review.openstack.org/28617418:39
*** krtaylor has joined #openstack-cinder18:39
openstackgerritTom Barron proposed openstack/cinder: Fix invalid uuid warnings in block device unit tests  https://review.openstack.org/28619718:39
openstackgerritMatt Riedemann proposed openstack/cinder: Copy StandardLogging fixture from Nova  https://review.openstack.org/28617418:39
smcginnisjgriffith: I can't change the Status of a blueprint. Are you able to do that?18:42
_alastor_smcginnis: I'm a dumb-dumb, I'm missing this commit in tempest https://github.com/openstack/tempest/commit/86d90f2d4c5d037e69b63dde04e0d5c86a361ad118:42
_alastor_Not sure why considering I cloned today18:43
smcginnis_alastor_: But the bug that that is skipping is supposedly fixed by that commit.18:43
smcginnis_alastor_: It should be able to be unskipped now from what I can tell.18:43
smcginnis_alastor_: I'm spinning up now to test that on my environment.18:43
_alastor_smcginnis: Part of the commit is adding the get_host function, which is missing in my repository18:44
smcginnis_alastor_: But yeah, getting the current master will let you get by that for now.18:44
smcginnis_alastor_: Ah!18:44
smcginnisMakes sense now.18:44
openstackgerritMerged openstack/cinder: always use pip constraints  https://review.openstack.org/28549418:45
smcginnis_alastor_: And here is the tempest patch unskipping it:18:47
smcginnishttps://review.openstack.org/#/c/267765/18:47
_alastor_smcginnis: I've added an explicit tempest pull call to the CI so it ensures we're running the latest tempest.  That should fix this on my side.18:49
smcginnis_alastor_: Awesome, glad you got it figured out.18:49
*** bardia has joined #openstack-cinder18:49
_alastor_smcginnis: thanks for your help18:49
smcginnis_alastor_: Any time.18:49
*** e0ne has joined #openstack-cinder18:51
jgriffithsmcginnis: weird... which one?18:55
smcginnisjgriffith: https://blueprints.launchpad.net/cinder/+spec/rpc-object-compatibility18:55
smcginnisTried change approver, assignee, etc, but I can change any field but Status.18:55
jgriffithsmcginnis: I think that's an "auto" thing... once it's "started"18:56
smcginnisjgriffith: Weird that we can't manually set that.18:57
smcginnisSo probably since there are some out there that have the tag "Implements: blueprint rpc-object-compatibility" we need to wait for all of those to be completed?18:57
jgriffithI'm guessing... or we just don't have permissions anymore18:58
jgriffithI don't remember to be honest18:58
smcginnisjgriffith: Heh, fair enough. :)18:58
smcginnisOh well, I tried dulek.18:58
jgriffithLemme check one other thing18:58
*** andymaier has quit IRC18:59
*** delattec has joined #openstack-cinder19:00
*** delattec has quit IRC19:00
*** haomaiwang has quit IRC19:01
jgriffithyeah, I can't change it.  :(19:01
bjornarHas there been any discussion on splitting cinder.conf in multiple parts by default?19:01
smcginnisjgriffith: Oh well.19:01
*** haomaiwang has joined #openstack-cinder19:01
bjornarOr eventually, how would a suggestion like that be received?19:01
jgriffithsmcginnis: should look into that... doesn't seem right.19:01
jgriffithbjornar: well, it already kinda is :)19:02
smcginnisjgriffith: Yeah, it does seem odd to me.19:02
jgriffithbjornar: sections at any rate19:02
smcginnisjgriffith: I can mark it obsolete or superceded to get rid of it, but that wouldn't be right.19:02
bjornarjgriffith, Yes, but problem now (I think) is that it is very hard or impossible to understand what cinder-part relies on what options19:03
*** cdelatte has quit IRC19:03
bjornarjgriffith, splitting it in cinder-common.conf cinder-backup.conf makes sense to me..19:03
*** asselin__ has quit IRC19:05
smcginnisjgriffith: Hmm, and here is one that is complete but still says started: https://blueprints.launchpad.net/cinder/+spec/rebrand-hp-xp-driver19:05
eharneysmcginnis: maybe "implementation" needs to be set to "implemented"?19:06
smcginniseharney: derp19:07
smcginnisjgriffith: eharney is way smarter than us. ^^19:07
*** julim has quit IRC19:07
hemnasmcginnis, that one has been implemented19:07
jgriffithsmcginnis: I've always said he was :)19:08
eharneybugzilla has trained me well in obscure-tracker-field-manipulation19:08
smcginnishemna: Yeah, couldn't figure out how to change the bp.19:08
smcginniseharney: ;)19:08
*** julim has joined #openstack-cinder19:10
openstackgerritAngela Smith proposed openstack/cinder: Fix HTTP sessions left open in Brocade zone driver  https://review.openstack.org/28436219:10
*** dims_ has joined #openstack-cinder19:11
*** dims has quit IRC19:11
openstackgerritKendall Nelson proposed openstack/cinder-specs: Dynamic Reconfiguration  https://review.openstack.org/28623419:11
*** mtanino has quit IRC19:14
jordanPDear os-brick core. Could you review https://review.openstack.org/#/c/285382/ (Fix Scality SOFS support) or at least tell me whether there's a change a new minor release of os-brick could happen before Mitake is cut off ?19:14
jordanP*a chance19:14
smcginnisjordanP: We're past the Mitaka cutoff for non-client libraries.19:14
smcginnisjordanP: But I can take a look.19:14
jordanPmy CI is working now. smcginnis I'd appreciate it.19:15
jordanPto my defense I'd say that the patch that "broke" my driver was merged 10 days ago which left my little time to come up with a fix19:15
jordanP*left me19:15
*** ivase has joined #openstack-cinder19:16
smcginnishemna: Got some time to take a look at that one too? ^^19:19
jordanPthanks guys :)19:20
*** e0ne has quit IRC19:20
bjornarDoes cinder-volume and cinder-backup really need database access? Is any work beeing done to hand this off to for example scheduler?19:22
*** cdelatte has joined #openstack-cinder19:22
smcginnisbjornar: No, they need to access the database.19:22
bjornarsmcginnis, they do now, I wondered if anything is done to change this?19:23
bjornarcinder is the only project in the openstack family I believe with 4 separate parts all doing database19:24
jungleboyjsmcginnis: When you have a chance do you mind looking at this:  https://review.openstack.org/#/c/282068/19:24
bjornarAlso, I wonder, why is scheduler registered to a spesific host? isnt scheduler service "global"?19:26
*** jdillaman has quit IRC19:27
openstackgerritMichelle Mandel proposed openstack/cinder: Fix volume filtering for quoted display name  https://review.openstack.org/23872219:29
*** laughterwym has joined #openstack-cinder19:30
*** martyturner has quit IRC19:31
*** IlyaG has joined #openstack-cinder19:34
*** IlyaG has quit IRC19:34
*** knikolla has quit IRC19:34
*** mtanino has joined #openstack-cinder19:35
*** laughterwym has quit IRC19:36
*** martyturner has joined #openstack-cinder19:36
*** larsks has left #openstack-cinder19:39
*** jordanP has quit IRC19:41
*** tongli has joined #openstack-cinder19:42
*** jordanP has joined #openstack-cinder19:43
*** jordanP has quit IRC19:43
*** lmtaylor has joined #openstack-cinder19:44
*** fthiagogv has quit IRC19:45
*** mmandel has joined #openstack-cinder19:46
jgriffithbjornar: :)  You're making my case (kinda)19:46
jgriffithbjornar: so there's two things to this19:46
jgriffithbjornar: cinder-manager and API is where we typically should have access19:47
jgriffithbjornar: API to check and finalize things19:47
jgriffithbjornar: manager to "do stuff"19:47
jgriffithbjornar: certainly doesn't *have* to be that way19:47
jgriffithbjornar: FWIW, the object work dulek and others are doing will solve some of this19:47
jgriffithbjornar: because at least in my mind, that means DB access can all be isolated to the objects19:48
jgriffithbjornar: you might still have indirect actions from other places, but the only actual DB calls would be in the object19:48
jgriffithbjornar: make sense?19:48
*** fthiagogv has joined #openstack-cinder19:49
jgriffithbjornar: historically, there was a time when the ONLY place you had db updates was in the manager code, other places could do queries but no updates19:49
jgriffithbjornar: then we wondered why we had races :)19:49
*** dalgaaf has joined #openstack-cinder19:51
*** knikolla has joined #openstack-cinder19:51
mmandelsmcginnis: I resolved the merge conflict for change 238722.19:51
*** lmtaylor has left #openstack-cinder19:52
*** lmtaylor has joined #openstack-cinder19:53
bjornarjgriffith, makes sense19:53
bjornarjgriffith, I have just recently tried to look at the details on this, and I must say I dont really like what I see here.19:53
mtaninodulek: hi,19:54
*** jdillaman has joined #openstack-cinder19:56
bjornarjgriffith, What about my "scheduler that registers on a spesific host" question?19:56
jgriffithbjornar: I'm not sure I follow that?19:56
bjornarjgriffith, currently cinder-scheduler registers on host spesific queue, and also registers host-spesific scheduler-service19:57
jgriffithbjornar: ummm... you sure about that?19:57
bjornarjgriffith, but afaik cinder-scheduler is a global service..19:57
jgriffithbjornar: c-scheduler is a single/global service19:58
*** dims_ has quit IRC19:58
*** geguileo has quit IRC19:58
jgriffithbjornar: takes input from hosts (c-vol services... which interact via manager.py instantiation for each backend)19:58
bjornarjgriffith, yeah, look at rabbitmq queue binding for example19:58
bjornarjgriffith, also, cinder service-list ..19:58
jgriffithbjornar: sure, those are the recepients19:58
jgriffithbjornar: equivalent of a nova compute-node19:59
openstackgerritMatt Smith proposed openstack/cinder: Updating Datera DataFabric Driver to v2 of Datera DataFabric API  https://review.openstack.org/28496719:59
bjornarjgriffith, ok, so not related to that of volume, then19:59
*** martyturner has quit IRC19:59
bjornarjgriffith, what do you think about this: https://blueprints.launchpad.net/cinder/+spec/split-cinder-config19:59
jgriffithbjornar: well, that's a tough one for me20:00
jgriffithbjornar: so we added "sections" to alleviate part of what you're talking about20:00
jgriffithbjornar: the challenge is that if you break them out into files, as you can imagine you could have a bit of a mess20:00
*** haomaiwang has quit IRC20:01
bjornarjgriffith, yeah, but problem is it is not documented properly/at all, so I believe splitting would force that20:01
jgriffithbjornar: especially if you do it for *all* projects.  You'd have literally 100's of config files20:01
bjornarjgriffith, I mean, if I want to deploy a -volume only host somewhere, I want to know what options I need to specify20:01
*** haomaiwang has joined #openstack-cinder20:01
jgriffithbjornar: perhaps... but knowing how things go I think it would lead to "hidden" files all over the place20:01
jgriffithbjornar: sure, I get ya20:01
openstackgerritPatrick East proposed openstack/cinder: Allow for Pure drivers to verify HTTPS requests  https://review.openstack.org/28162520:01
jgriffithbjornar: but my preference there would be "better docs"20:01
bjornarjgriffith, yeah, that sure would work. or more sections20:02
*** fthiagogv_ has joined #openstack-cinder20:02
*** dramakri has joined #openstack-cinder20:02
jgriffithbjornar: in other words, IMO it's not a problem with the config file as much as it is with docs and with "too many options"20:02
bjornarjgriffith, with [DEFAULT] actually beeing the shared stuff20:02
jgriffithbjornar: yeah.. the "DEFAULT" == "shared-stuff" is sort of icky right now too20:02
bjornarjgriffith, ..and -service spesiic options in [serrvice_ ..20:03
jgriffithbjornar: particularly with the way we do multi-backend, the DEFAULT doesn't really work the way it should20:03
jgriffithbjornar: yeah, I think that's a fine approach to take20:03
bjornaryeah, its messed up20:03
jgriffithbjornar: it gets sort of ugly though I'll warn you :)20:03
bjornarjgriffith, its not that I want lots of config files, but I think that may be the only way to make this rigjt20:03
jgriffithbjornar: you very well may be right20:04
jgriffithbjornar: by all means it would be good to get input from others20:04
bjornarit cant be worse!20:04
openstackgerritPatrick East proposed openstack/cinder: Allow for Pure drivers to verify HTTPS requests  https://review.openstack.org/28162520:04
bjornarIts a "devs use devstack" problem20:04
jgriffithbjornar: my experience and first response would be that it would actually be worse.  BUT I most certainly could be wrong20:04
jgriffithbjornar: yeah.. perhaps20:05
jgriffithbjornar: TBF, it's been hard enough to get devs to even do that, let alone try and deploy via packages20:05
jgriffithbjornar: somebody needs to tackle the *real* problem20:05
*** fthiagogv has quit IRC20:05
bjornarjgriffith, say for example cinder-volume.conf would be a separate file, it would be alot easier for everyone to see and understand what the volume service actually does, and how one could tweak and abuse20:06
jgriffithbjornar: and figure out how to make OpenStack something that a normal person can actually deploy without having to hire a bunch of consultants or pay vendors for their dep tools20:06
jgriffithbjornar: sure... but that service also interacts with things like keystone, db, rabbit....20:06
bjornarjgriffith, yeah, and then you see that it does..20:07
jgriffithbjornar: then throw in drivers and all the baggage they bring along20:07
jgriffithbjornar: yeah... maybe it works out better than I think.  You should draft something up and get input from others20:07
jgriffithbjornar: I certainly don't have all the answers here :)20:08
openstackgerritPatrick East proposed openstack/cinder: Update Pure replication to cheesecake  https://review.openstack.org/27698120:08
bjornarrootwrap for example, its volume and backup, right20:08
jgriffithbjornar: hehe20:08
jgriffithbjornar: so that's currently being reworked to priv-sep20:08
jgriffithbjornar: but yes20:08
jgriffithbjornar: and I see where you're heading...20:08
jgriffithbjornar: backup-rootwrap, volume-rootwrap, scheduler-rootwrap maybe etc etc20:09
jgriffithbjornar: how do you categorize and where do you draw the lines?  How many config files and rootwrap files do you end up with?20:09
bjornarjgriffith, thats up for discussion ofcorse20:10
bjornarjgriffith, but having separate configs will add the ability atleast20:11
*** dims has joined #openstack-cinder20:12
*** david-lyle_ is now known as david-lyle20:12
jgriffithbjornar: fair enough20:13
bjornarThe reason I bring this up is I have spent a fair amount of time trying to figure out what options are used/required where, and it ends up not beeing documented anywhere.20:14
*** eharney_ has joined #openstack-cinder20:15
*** angela-s has quit IRC20:16
bjornarjgriffith, also, in a production deployment scenario, the different cinder-services are not nessesarly running on the same hosts or datacenters20:16
jgriffithbjornar: very true20:17
*** eharney has quit IRC20:19
openstackgerritJay Bryant proposed openstack/cinder: IBM Storwize with pool-aware-cinder-scheduler  https://review.openstack.org/28461420:19
openstackgerritTom Swanson proposed openstack/cinder: Dell SC: Support Replication V2.1  https://review.openstack.org/28625620:24
*** laughterwym has joined #openstack-cinder20:25
dramakriI have 2 fairly simple patches (couple of months old) which already have one +2. Appreciate if other cores could take a look at them. https://review.openstack.org/#/c/238722/ and https://review.openstack.org/#/c/242391/20:29
dramakriThanks! 20:29
*** dims has quit IRC20:29
dramakrihttps://review.openstack.org/#/c/238722/ and https://review.openstack.org/#/c/242391/20:30
openstackgerritSurya Ghatty proposed openstack/cinder: Fix sshpool.remove code  https://review.openstack.org/28568720:30
*** yangyapeng has joined #openstack-cinder20:31
*** laughterwym has quit IRC20:31
openstackgerritAdriano Freires Rosso proposed openstack/cinder: WIP: HNAS drivers refactoring  https://review.openstack.org/28546920:32
*** tongli has quit IRC20:34
*** pots72 has joined #openstack-cinder20:36
*** pots72 has quit IRC20:36
*** salv-orlando has joined #openstack-cinder20:36
*** mylu has joined #openstack-cinder20:37
*** yangyapeng has quit IRC20:37
*** xyang1 has joined #openstack-cinder20:38
*** lmtaylor has left #openstack-cinder20:39
*** salv-orl_ has quit IRC20:39
*** xyang has joined #openstack-cinder20:39
openstackgerritAdriano Freires Rosso proposed openstack/cinder: WIP: HNAS drivers refactoring  https://review.openstack.org/28546920:41
*** eharney_ is now known as eharney20:41
*** pots is now known as Guest2423820:42
*** lpetrut has joined #openstack-cinder20:43
*** pots has joined #openstack-cinder20:43
*** pots2 has joined #openstack-cinder20:43
*** pots2 has quit IRC20:44
openstackgerritEric Harney proposed openstack/cinder: Return BadRequest for invalid unicode names (continued)  https://review.openstack.org/28627120:45
*** knikolla has quit IRC20:45
*** geguileo has joined #openstack-cinder20:45
scottdaIs anyone seeing issues with running "tox -epy34 -- -n <regex_for_test>" or "tox -epy34 -- -regex <regex_for_test>"  ?20:48
scottdaOn fresh devstack, it keeps on running all tests.20:48
scottdaIs that known to be broken for py34 unit tests?20:49
smcginnisscottda: Do you have the switch back to ostestr?20:49
*** ivase_ has joined #openstack-cinder20:49
*** ociuhandu has quit IRC20:49
scottdasmcginnis: Switch?20:49
scottdaWasn't that patch merged?20:50
scottdasmcginnis: tox.ini has this, if that is what you mean:20:51
smcginnisscottda: In tox.ini, we inadvertently switched back from ostestr to whatever that other one is, then found it and switched back.20:51
scottdahttps://www.irccloud.com/pastebin/p8p0BBiY/20:51
eharneyscottda: i was looking at that earlier... i'm not sure how to fix it but i think it's because the py34 env doesn't look at {posargs}20:51
smcginnisscottda: What do you have under [textenv] for "commands = *"20:51
eharneyi.e. compare "commands" in testenv vs testenv:py3420:52
scottdaeharney: OK, so you are seeing this too? Thanks.20:52
eharneyyes20:52
*** Lee1092 has quit IRC20:52
eharneyit's annoying when you have py34 failing on your patch :)20:52
scottdacommands = ostestr {posargs}20:52
smcginnisscottda: Oh, wait. The whitelist file is gone. You do have an older setup.20:52
scottdaIt's a fresh VM and fresh devstack20:53
scottdaeharney: Yes, which is my case20:53
*** ivase has quit IRC20:53
smcginnisscottda: Doesn't look like your on master: https://github.com/openstack/cinder/blob/master/tox.ini20:54
*** IlyaG has joined #openstack-cinder20:54
scottdasmcginnis: you are right. I need to rebase. When did this go in? My patch is from Friday.20:54
smcginnisscottda: Patch probably just needs to be rebased.20:54
openstackgerritMerged openstack/cinder: Clean up replication v2.1 (Cheesecake) RPC API  https://review.openstack.org/28604720:54
*** ivase_ is now known as ivase20:55
smcginnisscottda: Probably immediately after you branched. :)20:55
smcginnishttps://github.com/openstack/cinder/commit/a09143cb31697ede2f2fc8becd23c9b1f4da7f5920:55
eharneyahh, good to know20:55
openstackgerritEric Harney proposed openstack/cinder: Capture warnings into logs  https://review.openstack.org/28482820:56
scottdasmcginnis: This is why you make the Big Bucks20:57
*** vincent_hou has quit IRC20:57
*** IlyaG has quit IRC20:57
smcginnisHah!20:57
eharneyso i rebased.... http://paste.openstack.org/raw/488635/20:58
eharneyarg20:58
smcginniseharney: Ooo, I've seen that before.20:58
smcginniseharney: What version of pip?20:58
*** dramakri has quit IRC20:58
eharneysmcginnis: is it the version on the system or in the tox env?20:59
eharneylooks like tox env20:59
smcginniseharney: Hm, looks like venv: /home/eharney/src/cinder-trunk8/.tox/py34/bin/pip20:59
eharneywhich is 6.0.820:59
*** vincent_hou has joined #openstack-cinder20:59
smcginnisWhich would be odd I guess.20:59
eharneyso we probably require a newer pip but aren't saying that somewhere21:00
scottdaeharney: After rebase, it worked for me with "tox -epy34 -- -n <blah>"21:00
*** haomaiwang has quit IRC21:01
*** chlong_ has joined #openstack-cinder21:01
*** haomaiwang has joined #openstack-cinder21:01
eharneyi wonder where we should be requiring that21:02
smcginniseharney: It's a little chicken and egg.21:03
eharneyi'm trying to find... either of those :)21:03
smcginniseharney: Were you able to check which version was being used in the venv?21:03
eharneysmcginnis: yeah, it's 6.0.821:03
hemnasmcginnis, ok checking it now21:04
smcginniseharney: Oh, that is old/21:04
eharneysmcginnis: yeah, but i don't know how it's choosing that version21:04
smcginnishemna: No rush. I asked and was told we need to wait until after feature freeze.21:04
hemnaok21:04
smcginnishemna: But I suppose the sooner we get it through the better.21:04
*** martyturner has joined #openstack-cinder21:04
smcginnishemna: We just can't cut a new release until later.21:04
hemnaok21:05
hemnaso they got a scality CI running on that patch I see21:05
eharneysmcginnis: it doesn't correlate to my system pip (7.1.0), so it's something in cinder probably21:05
smcginniseharney: Yeah, I don't know how that works either. We can't put it in test-requirements because that needs pip to install.21:05
smcginniseharney: Interesting we're not seeing this with py27 though.21:05
*** ndipanov has quit IRC21:06
eharneysmcginnis: oh, uh, i am seeing it there now too21:06
smcginnisgrr21:06
eharneyso basically i can't run unit tests on my box now21:06
smcginniseharney: Should be able to activate the venv manually and do a pip install -U pip.21:07
*** knikolla has joined #openstack-cinder21:07
*** thangp has left #openstack-cinder21:07
smcginniseharney: Crappy workaround, but it should get things working.21:07
smcginnishemna: BTW, apt-mirror and bandersnatch == 455G. Dang!21:08
scottdaeharney: maybe related to this commit 379f272924f1e3946f8eb08eb9690e42d09b0a2921:08
*** salv-orlando has quit IRC21:08
eharneyscottda: yeah, that adds a requirement on a particular version that supports constraints, but nothing seems to ensure that you actually install that version21:08
*** salv-orlando has joined #openstack-cinder21:09
eharneyi think it's using 6.0.8 because that's what my python-virtualenv defaults to21:09
hemnasmcginnis, yah it's no joke21:09
hemnabetter to do that once, and have CI pull that data locally, then trying to do that every run!21:10
smcginnishemna: No kidding!21:11
smcginnishemna: Ran over the weekend with apt-mirror. No problems.21:11
hemnagreat21:11
hemna:)21:11
smcginnishemna: Now to add pip mirror.21:11
hemnaI had a rough time w/ getting pip to work with my bandersnatch mirror21:11
hemnaI gave up21:12
smcginnishemna: Ruh roh. Well, I'll let you know if I have any better luck.21:12
hemnaok :)21:12
hemnaI just haven't messed with it since21:12
*** raildo is now known as raildo-afk21:12
smcginniseharney: Might be interesting to roll back to https://github.com/openstack/cinder/commit/077b8593ae40e759a04517c00767891e6e65228a21:12
smcginniseharney: Clear the venv and see if that has some unintended side effects.21:13
eharneysmcginnis: well that's what adds the "-c" parameter which is now missing21:13
openstackgerritMatt Smith proposed openstack/cinder: Updating Datera DataFabric Driver to v2 of Datera DataFabric API  https://review.openstack.org/28496721:13
eharneysmcginnis: the problem seems to be that that code assumes a version of pip that isn't guaranteed to actually be there21:13
*** merooney has quit IRC21:14
smcginniseharney: Ah! Maybe AJaeger has seen this before?21:14
smcginnisI don't see him around anywhere though.21:14
*** sheel has quit IRC21:17
*** laughterwym has joined #openstack-cinder21:19
smcginnishemna: You ever get ReadTimeout errors when running bandersnatch?21:20
eharneysmcginnis: installing python-virtualenv 14 from Fedora 24 seems to fix it.  But that's kinda cheesy, because this means we have an untracked dependency in our test rig :/21:21
hemnasmcginnis, I haven't ran it in a while21:23
hemnaI don't remember seeing that21:23
*** dims has joined #openstack-cinder21:23
jungleboyjsmcginnis: When are we cutting off features merging?  Today, tomorrow?21:25
smcginnisjungleboyj: Tomorrow is the published Cinder M-3.21:25
smcginnisSo that would be feature freeze and soft string freeze.21:25
smcginnisReplication being an exception to the "feature" freeze.21:25
jungleboyjSo anything that doesn't go in today isn't going in.21:25
smcginnisjungleboyj: I did give us the out with: "Any changes past that date will be at the discretion of the core team."21:26
smcginnisjungleboyj: But basically, yeah.21:26
*** laughterwym has quit IRC21:26
jungleboyjsmcginnis: Ok ...21:28
*** akerr has quit IRC21:28
eharneysmcginnis: could you +W https://review.openstack.org/#/c/284828/ again?21:28
smcginniseharney: Done21:29
eharneysmcginnis: thanks21:29
*** esker has joined #openstack-cinder21:32
openstackgerritJacob Gregor proposed openstack/cinder: Storwize SVC multiple management IPs  https://review.openstack.org/28426721:33
*** cdelatte has quit IRC21:33
*** ociuhandu has joined #openstack-cinder21:35
*** tpsilva has quit IRC21:37
*** dims has quit IRC21:40
*** cdelatte has joined #openstack-cinder21:42
*** dims has joined #openstack-cinder21:44
*** timcl has quit IRC21:48
*** garthb_ has joined #openstack-cinder21:49
*** garthb has quit IRC21:49
*** cknight has quit IRC21:51
*** geguileo has quit IRC21:51
*** hambuergaer has joined #openstack-cinder21:52
*** cdelatte has quit IRC21:52
*** cdelatte has joined #openstack-cinder21:53
*** edmondsw has quit IRC21:53
*** cdelatte has quit IRC21:54
*** haplo37 has quit IRC21:54
*** darosale has joined #openstack-cinder21:54
*** hambuergaer has quit IRC21:55
*** haplo37 has joined #openstack-cinder21:58
*** salv-orl_ has joined #openstack-cinder21:59
*** haomaiwang has quit IRC22:01
openstackgerritJohn Griffith proposed openstack/cinder: Add ability to failback for replication V2.1  https://review.openstack.org/28500722:01
*** haomaiwa_ has joined #openstack-cinder22:01
*** salv-orlando has quit IRC22:02
openstackgerritTom Swanson proposed openstack/cinder: Dell SC: Support Replication V2.1  https://review.openstack.org/28625622:05
openstackgerritMerged openstack/python-cinderclient: Avoid logging sensitive info in http requests.  https://review.openstack.org/27575522:06
*** geguileo has joined #openstack-cinder22:08
*** bardia has quit IRC22:08
*** bardia has joined #openstack-cinder22:08
*** knikolla has quit IRC22:09
*** dims has quit IRC22:09
*** tpatil has joined #openstack-cinder22:10
jungleboyjAt least patch set 8 did for the first one.22:13
*** mylu_ has joined #openstack-cinder22:13
*** mylu has quit IRC22:13
openstackgerritMerged openstack/cinder: Re-enable -1 child limits for nested quotas  https://review.openstack.org/28206822:17
*** sdague has quit IRC22:18
*** dustins has quit IRC22:22
*** eharney has quit IRC22:26
*** vincent_hou has quit IRC22:26
*** diablo_rojo has quit IRC22:32
*** diablo_rojo has joined #openstack-cinder22:32
*** yangyapeng has joined #openstack-cinder22:35
*** garthb_ has quit IRC22:36
*** garthb has joined #openstack-cinder22:36
*** diablo_rojo has quit IRC22:36
*** jgregor has quit IRC22:36
*** jungleboyj has quit IRC22:39
*** garthb has quit IRC22:39
*** julim has quit IRC22:39
*** garthb has joined #openstack-cinder22:39
*** yangyapeng has quit IRC22:40
*** xyang has quit IRC22:43
openstackgerritMerged openstack/cinder: VMware: manage_existing for VMDK driver  https://review.openstack.org/28051322:44
openstackgerritSurya Ghatty proposed openstack/cinder: Fix sshpool.remove code  https://review.openstack.org/28568722:44
*** baumann has quit IRC22:45
openstackgerritScott DAngelo proposed openstack/cinder: Allow api_version_request.matches to accept a string  https://review.openstack.org/28529322:46
openstackgerritMatt Riedemann proposed openstack/cinder: Copy StandardLogging fixture from Nova  https://review.openstack.org/28617422:47
mriedemjgriffith: smcginnis: ^ also fixes the stevedore debug logging bug22:47
openstackgerritMerged openstack/python-cinderclient: snapshot-list now supports filtering by tenant  https://review.openstack.org/24239122:47
jgriffithmriedem: I saw that22:47
jgriffithmy work here is done22:48
jgriffith:)22:48
mriedem\o/22:48
*** rlrossit has quit IRC22:48
*** haplo37 has quit IRC22:48
mriedemand as a nice side effect, catches bad log formatting22:48
jgriffithmriedem: :)  Yeah, noticed a few strays you gathered up there22:49
jgriffithmriedem: good stuff22:49
*** edtubill has quit IRC22:51
*** kfarr has quit IRC22:53
*** akerr has joined #openstack-cinder22:57
*** ivase has quit IRC22:58
*** haomaiwa_ has quit IRC23:01
*** haomaiwang has joined #openstack-cinder23:01
*** darosale has quit IRC23:03
*** huanan has quit IRC23:03
*** huanan has joined #openstack-cinder23:04
openstackgerritPatrick East proposed openstack/cinder: Update Pure replication to cheesecake  https://review.openstack.org/27698123:05
*** pots has quit IRC23:07
*** laughterwym has joined #openstack-cinder23:07
*** mmandel has quit IRC23:08
*** lpetrut has quit IRC23:12
*** Julien-zte has quit IRC23:13
*** laughterwym has quit IRC23:13
openstackgerritMerged openstack/python-cinderclient: Add replication v2.1 (cheesecake) calls  https://review.openstack.org/28159723:13
*** dims has joined #openstack-cinder23:14
*** salv-orl_ has quit IRC23:15
*** salv-orlando has joined #openstack-cinder23:19
*** akerr has quit IRC23:21
*** akerr has joined #openstack-cinder23:22
*** akerr has quit IRC23:22
openstackgerritMichael Rowden proposed openstack/cinder: Correcting thin provisioning behavior  https://review.openstack.org/27540823:27
*** EinstCrazy has joined #openstack-cinder23:34
*** EinstCrazy has quit IRC23:35
*** bardia_ has joined #openstack-cinder23:36
*** bardia has quit IRC23:36
*** andymaier has joined #openstack-cinder23:37
openstackgerritTom Swanson proposed openstack/cinder: Dell SC: Support Replication V2.1  https://review.openstack.org/28625623:41
*** xyang1 has quit IRC23:46
*** gouthamr has quit IRC23:48
*** jungleboyj has joined #openstack-cinder23:49
openstackgerritScott DAngelo proposed openstack/cinder: Allow api_version_request.matches to accept a string  https://review.openstack.org/28529323:50
*** yhayashi has joined #openstack-cinder23:51
*** mylu_ has quit IRC23:52
*** smoriya_ has joined #openstack-cinder23:54
*** mylu has joined #openstack-cinder23:55

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