Thursday, 2016-03-03

*** haomaiwang has quit IRC00:01
*** haomaiwang has joined #openstack-cinder00:01
*** dims has joined #openstack-cinder00:06
*** markvoelker has quit IRC00:06
*** mriedem has joined #openstack-cinder00:12
openstackgerritMerged openstack/python-cinderclient: Fix Resource.__eq__ mismatch semantics of object equal  https://review.openstack.org/25102100:13
*** IlyaG has quit IRC00:14
*** sdague has quit IRC00:18
*** ByPasS has quit IRC00:20
*** mylu has joined #openstack-cinder00:21
*** mmandel has joined #openstack-cinder00:23
*** hemna is now known as hemnafk00:24
*** bjornar has quit IRC00:43
*** resker has joined #openstack-cinder00:45
*** esker has quit IRC00:46
*** laughterwym has joined #openstack-cinder00:56
*** bardia has quit IRC00:56
*** bardia has joined #openstack-cinder00:57
*** laughter_ has joined #openstack-cinder00:59
*** laughterwym has quit IRC01:00
*** haomaiwang has quit IRC01:01
*** haomaiwang has joined #openstack-cinder01:01
*** laughterwym has joined #openstack-cinder01:02
*** mmandel has quit IRC01:02
*** laughter_ has quit IRC01:04
*** laughterwym has quit IRC01:05
*** laughterwym has joined #openstack-cinder01:05
*** Suyash has quit IRC01:06
*** cheneydc has joined #openstack-cinder01:06
*** dims has quit IRC01:08
*** ntpttr has quit IRC01:08
*** sasukeh has joined #openstack-cinder01:09
*** markvoelker has joined #openstack-cinder01:10
openstackgerritWilson Liu proposed openstack/cinder: Huawei: Check the real size before extend volume  https://review.openstack.org/28344001:13
*** garthb has quit IRC01:15
*** yangyapeng has joined #openstack-cinder01:15
*** takedakn has joined #openstack-cinder01:15
*** cknight1 has quit IRC01:17
openstackgerritMerged openstack/cinder: Fix test isolation issues related to versions  https://review.openstack.org/28721801:17
openstackgerritMerged openstack/cinder: Fix test_create_volume_flow test issue  https://review.openstack.org/28721901:17
*** EinstCrazy has joined #openstack-cinder01:17
openstackgerritPatrick East proposed openstack/cinder: Revert "Remove Cisco FC Zone Manager Driver"  https://review.openstack.org/28754101:18
* patrickeast hides01:18
openstackgerritPatrick East proposed openstack/cinder: Allow for Pure drivers to verify HTTPS requests  https://review.openstack.org/28162501:19
*** yangyapeng has quit IRC01:19
openstackgerritMerged openstack/os-brick: Updated from global requirements  https://review.openstack.org/28399601:22
*** yangyapeng has joined #openstack-cinder01:23
*** bardia has quit IRC01:23
*** bardia has joined #openstack-cinder01:24
*** haomaiwang has quit IRC01:25
*** davechen has joined #openstack-cinder01:26
openstackgerritMerged openstack/cinder: Huawei: Manage volume fails due to lower array version  https://review.openstack.org/28105001:35
*** laughterwym has quit IRC01:39
*** laughterwym has joined #openstack-cinder01:39
*** laughterwym has quit IRC01:39
*** laughterwym has joined #openstack-cinder01:40
*** amit213 has joined #openstack-cinder01:41
*** openstackgerrit has quit IRC01:47
*** openstackgerrit has joined #openstack-cinder01:48
*** mmandel has joined #openstack-cinder01:50
*** baojg has joined #openstack-cinder01:58
*** dims has joined #openstack-cinder01:59
*** cknight has joined #openstack-cinder01:59
*** mtanino has quit IRC02:01
*** haomaiwang has joined #openstack-cinder02:02
openstackgerritWilson Liu proposed openstack/cinder: Huawei: Record and check LUN wwn  https://review.openstack.org/27159202:02
*** takedakn has quit IRC02:02
*** crose has quit IRC02:04
*** ntpttr has joined #openstack-cinder02:11
*** Lee1092 has joined #openstack-cinder02:14
*** markstur_ has joined #openstack-cinder02:16
*** xek_ has joined #openstack-cinder02:17
*** liewegas_ has joined #openstack-cinder02:18
*** IlyaG has joined #openstack-cinder02:18
*** dosaboy_ has joined #openstack-cinder02:18
*** guitarza1 has joined #openstack-cinder02:19
*** tobasco_ has joined #openstack-cinder02:19
*** cknight has quit IRC02:20
*** mkoderer___ has joined #openstack-cinder02:22
*** dosaboy_ has quit IRC02:23
*** mkoderer__ has quit IRC02:23
*** guitarzan has quit IRC02:23
*** dosaboy has quit IRC02:23
*** stevemar has quit IRC02:23
*** bjornar__ has quit IRC02:23
*** markstur has quit IRC02:23
*** ircuser-1 has quit IRC02:23
*** ccesario has quit IRC02:23
*** xek has quit IRC02:23
*** tobasco has quit IRC02:23
*** liewegas has quit IRC02:23
*** bjornar__ has joined #openstack-cinder02:24
*** stevemar has joined #openstack-cinder02:24
*** dosaboy has joined #openstack-cinder02:24
openstackgerritMerged openstack/cinder: Remove duplicated code in volume manager and base driver  https://review.openstack.org/27133102:24
smcginnispatrickeast: CI coverage looks good. Thanks for doing that.02:24
*** ircuser-1 has joined #openstack-cinder02:25
*** ccesario has joined #openstack-cinder02:26
*** sasukeh has quit IRC02:29
*** sasukeh has joined #openstack-cinder02:31
*** sasukeh has quit IRC02:31
*** sasukeh has joined #openstack-cinder02:32
lixiaoy1smcginnis: hey! About the patch 'add sheepdog support in os-brick', it passed sheepdog tempest test.02:33
*** ntpttr has quit IRC02:36
lixiaoy1smcginnis: as current sheepdog job template gets os-brick, but doesn't use it. There is a job template in experimental to use new os-brick patch to do tempest test02:36
openstackgerritMerged openstack/cinder: Use OSprofiler options consolidated in lib itself  https://review.openstack.org/27815502:38
*** ntpttr has joined #openstack-cinder02:38
openstackgerritMerged openstack/cinder: Fix exception during service update  https://review.openstack.org/28590802:38
*** ntpttr has quit IRC02:42
smcginnislixiaoy1: Thanks for the reminder. I saw your comment on their with the link to the results, but didn't get a chance to get back in to review.02:43
smcginnislixiaoy1: I'll take a look at that now.02:43
lixiaoy1smcginnis: thanks!02:43
smcginnislixiaoy1: All those TODOs make me a little nervous.02:46
smcginnislixiaoy1: Is there someone involved with Sheepdog that can help resolve some of them?02:46
*** sasukeh has quit IRC02:47
*** sasukeh has joined #openstack-cinder02:47
*** resker has quit IRC02:48
lixiaoy1smcginnis: this is interface inherited from parent class, and current no body uses it. Meanwhile, I keep it as RBD also do same thing02:48
smcginnislixiaoy1: Fair enough. :)02:49
lixiaoy1smcginnis: and for such interface, raise exception or return nothing is current implementations02:50
*** esker has joined #openstack-cinder02:52
*** gcb has joined #openstack-cinder02:54
*** vilobhmm11 has joined #openstack-cinder02:54
*** bardia has quit IRC03:00
*** haomaiwang has quit IRC03:01
*** haomaiwang has joined #openstack-cinder03:01
*** houming has joined #openstack-cinder03:02
*** laughterwym has quit IRC03:06
*** laughterwym has joined #openstack-cinder03:06
*** apoorvad has quit IRC03:07
*** laughter_ has joined #openstack-cinder03:10
*** laughterwym has quit IRC03:10
*** laughter_ has quit IRC03:14
*** jwang_ has joined #openstack-cinder03:21
*** jwcroppe_ has joined #openstack-cinder03:22
*** kmartin__ has joined #openstack-cinder03:23
*** jdurgin1 has quit IRC03:25
*** wNz has joined #openstack-cinder03:25
*** sborkows has joined #openstack-cinder03:25
*** dansmith_ has joined #openstack-cinder03:25
*** tjcocozz_ has joined #openstack-cinder03:25
*** spring_ has joined #openstack-cinder03:25
*** strictlyb has joined #openstack-cinder03:25
*** netapp_ has joined #openstack-cinder03:26
*** thurloat_ has joined #openstack-cinder03:26
*** divab has joined #openstack-cinder03:26
*** frakt_ has joined #openstack-cinder03:26
*** xiaohui has joined #openstack-cinder03:26
*** dansmith has quit IRC03:26
*** netapp has quit IRC03:26
*** sborkows_ has quit IRC03:26
*** wN has quit IRC03:26
*** abhi has quit IRC03:26
*** jwang__ has quit IRC03:26
*** sb- has quit IRC03:26
*** tjcocozz has quit IRC03:26
*** kmartin_ has quit IRC03:26
*** spring__ has quit IRC03:26
*** xiaohui_ has quit IRC03:26
*** IlyaG has quit IRC03:26
*** jwcroppe has quit IRC03:26
*** ildikov has quit IRC03:26
*** lennyb has quit IRC03:26
*** HenryG has quit IRC03:26
*** thurloat has quit IRC03:26
*** Roamer` has quit IRC03:26
*** IlyaG has joined #openstack-cinder03:26
*** Roamer` has joined #openstack-cinder03:26
*** frakt has quit IRC03:26
*** wNz is now known as wN03:26
*** wN has quit IRC03:26
*** wN has joined #openstack-cinder03:26
*** thurloat_ is now known as thurloat03:26
*** HenryG has joined #openstack-cinder03:26
*** dansmith_ is now known as dansmith03:27
*** ildikov has joined #openstack-cinder03:27
*** netapp_ is now known as netapp03:27
*** lennyb has joined #openstack-cinder03:27
*** dansmith is now known as Guest6049703:27
*** links has joined #openstack-cinder03:29
*** spring_ is now known as grassy03:31
*** cknight has joined #openstack-cinder03:33
*** baojg has quit IRC03:35
*** lennyb__ has joined #openstack-cinder03:35
*** baojg has joined #openstack-cinder03:38
*** [1]Thelo has joined #openstack-cinder03:39
*** lennyb has quit IRC03:40
*** Thelo has quit IRC03:41
*** [1]Thelo is now known as Thelo03:41
openstackgerritwangxiyuan proposed openstack/cinder: Make query to quota usage table order preserved  https://review.openstack.org/28757803:41
*** dims has quit IRC03:48
*** hemna_ has joined #openstack-cinder03:50
*** hemna has joined #openstack-cinder03:58
*** hemna_ has quit IRC04:00
*** shausy has joined #openstack-cinder04:00
*** haomaiwang has quit IRC04:01
openstackgerritLisaLi proposed openstack/cinder: Handle exceptions about snapshot in backup create  https://review.openstack.org/28457704:01
openstackgerritVincent Hou proposed openstack/cinder: Storwize: Update replication to v2.1  https://review.openstack.org/28748304:02
*** divab is now known as abhi04:03
openstackgerritMerged openstack/cinder: Fix spelling mistake in docstring  https://review.openstack.org/28688804:04
openstackgerritMerged openstack/cinder: Fix invalid uuid warnings in block device unit tests  https://review.openstack.org/28619704:04
openstackgerritMerged openstack/cinder: NexentaStor5 iSCSI driver unit tests  https://review.openstack.org/27153704:04
openstackgerritMerged openstack/cinder: Use openstack.org URLs in README  https://review.openstack.org/28740404:05
*** mriedem has quit IRC04:16
*** IlyaG has quit IRC04:17
openstackgerritgundarapu kalyan reddy proposed openstack/python-cinderclient: Search_opts for cinder types  https://review.openstack.org/28758404:24
*** haomaiwang has joined #openstack-cinder04:32
*** sasukeh has quit IRC04:32
*** sasukeh has joined #openstack-cinder04:33
*** mylu has quit IRC04:36
*** lprice has joined #openstack-cinder04:37
*** mylu has joined #openstack-cinder04:37
*** sasukeh has quit IRC04:39
*** resker has joined #openstack-cinder04:45
*** esker has quit IRC04:47
*** mylu has quit IRC04:48
*** fthiagogv_ has joined #openstack-cinder04:51
*** fthiagogv has quit IRC04:54
*** gcb has quit IRC04:54
*** mylu has joined #openstack-cinder04:56
openstackgerritChangBo Guo(gcb) proposed openstack/cinder: Let method CONF.set_override with enfore_type=true by default  https://review.openstack.org/25332804:57
*** haomaiwang has quit IRC05:01
*** haomaiwang has joined #openstack-cinder05:01
gushemna: the 3rd-party CI bots are failing on https://review.openstack.org/#/c/277224 (os-brick/privsep) because they aren't configured correctly.  The devstack patches I depend-on either aren't executed in the 3rd-party environment or aren't being applied.05:02
gushemna: (fyi)05:02
*** gouthamr has quit IRC05:03
gushemna: I don't know what I can do to influence the 3rd-party CI bots - if there was something else I was meant to do in devstack, I'd be happy to do so.05:05
*** takedakn has joined #openstack-cinder05:06
*** cknight has quit IRC05:08
*** gcb has joined #openstack-cinder05:09
*** ivase_ has quit IRC05:09
hemnagus nothing we can do about it?05:09
*** martyturner has joined #openstack-cinder05:09
gushemna: meh, then there's no point rechecking them - they'll fail 100% of the time.05:10
hemnaok05:10
hemna:(05:10
hemnaboo05:10
gusPerhaps when the depends-on changes are finally approved the bots might pick them up (assuming the issue is that they don't support depends-on)05:10
hemnadidn't we merge the cinder dep patch?05:12
gusyes.05:12
gusbut it needs the nova equivalent, and the two matching cinder/nova devstack changes too.05:12
hemnanova folks not want to merge it ?05:13
gusI am unable to speculate on what leads to nova changes actually getting approved.05:13
*** martyturner has quit IRC05:14
* bswartz imagines gus could be a politician05:14
hemnaI +1'd the nova patch fwiw05:14
hemnathat's all I can do there05:14
gusthanks.  I gather the nova cores just aren't looking.05:15
hemnamriedem isn't on05:16
gus(sdague/mriedem asked the same question early on for 277670, I replied with another question, and then crickets.  I've prompted them again over irc explicitly)05:16
gus(a while back)05:17
*** salv-orl_ has joined #openstack-cinder05:19
*** salv-orlando has quit IRC05:22
*** mmandel has quit IRC05:23
*** takedakn has quit IRC05:27
*** smoriya_ has joined #openstack-cinder05:28
*** takedakn has joined #openstack-cinder05:31
*** takedakn has quit IRC05:36
*** takedakn has joined #openstack-cinder05:37
*** sasukeh has joined #openstack-cinder05:39
*** sasukeh has quit IRC05:40
*** fthiagogv__ has joined #openstack-cinder05:40
*** sasukeh has joined #openstack-cinder05:40
*** bharathk has joined #openstack-cinder05:41
openstackgerritPatrick East proposed openstack/cinder: Revert "Remove Cisco FC Zone Manager Driver"  https://review.openstack.org/28754105:41
openstackgerritMerged openstack/cinder: Add necessary fields to volume creation  https://review.openstack.org/28460005:42
lixiaoy1smcginnis: and for such interface, raise exception or return nothing is current implementations05:42
lixiaoy1smcginnis: sorry please ignore above comments.05:43
*** fthiagogv_ has quit IRC05:43
*** ankit_ag has joined #openstack-cinder05:44
*** gcb has quit IRC05:50
*** hemna has quit IRC05:52
*** edtubill has joined #openstack-cinder05:55
*** bardia has joined #openstack-cinder06:01
*** haomaiwang has quit IRC06:01
*** resker has quit IRC06:01
*** dave-mccowan has quit IRC06:01
*** haomaiwang has joined #openstack-cinder06:01
openstackgerritBhagyashri Shewale proposed openstack/python-cinderclient: Fix volume type 'is_public' flag updation  https://review.openstack.org/28638206:01
*** edtubill has quit IRC06:03
*** esker has joined #openstack-cinder06:05
*** laughterwym has joined #openstack-cinder06:06
*** sasukeh has quit IRC06:06
*** sasukeh_ has joined #openstack-cinder06:06
*** edtubill has joined #openstack-cinder06:07
*** gcb has joined #openstack-cinder06:10
*** laughterwym has quit IRC06:10
*** edtubill has quit IRC06:11
openstackgerritAbhishek Kekane proposed openstack/cinder: volume upload-to-image fails for iso disk-format  https://review.openstack.org/28760606:11
*** takedakn has quit IRC06:14
*** takedakn has joined #openstack-cinder06:19
*** sheel has joined #openstack-cinder06:20
*** edtubill has joined #openstack-cinder06:20
*** Thelo has quit IRC06:23
*** laughterwym has joined #openstack-cinder06:27
*** laughterwym has quit IRC06:31
*** amit213 has quit IRC06:33
*** edtubill has quit IRC06:35
*** edtubill has joined #openstack-cinder06:36
*** bardia has quit IRC06:36
openstackgerritWilson Liu proposed openstack/cinder: Huawei: Check the real size before extend volume  https://review.openstack.org/28344006:37
*** mylu has quit IRC06:38
*** mylu has joined #openstack-cinder06:39
*** mylu_ has joined #openstack-cinder06:41
*** vilobhmm11 has quit IRC06:43
*** anshul has joined #openstack-cinder06:43
*** mylu has quit IRC06:44
*** edtubill has quit IRC06:54
*** mylu_ has quit IRC06:54
openstackgerritWilson Liu proposed openstack/cinder: Huawei: Record and check LUN wwn  https://review.openstack.org/27159206:56
lixiaoy1DuncanT: please help to review https://review.openstack.org/#/c/280588, add sheepdog to os-brick06:57
openstackgerritWilson Liu proposed openstack/cinder: Huawei: Check before delete host  https://review.openstack.org/28295006:58
lixiaoy1DuncanT: currently got one +2 already06:58
*** lpetrut has joined #openstack-cinder06:58
*** takedakn has quit IRC07:01
*** haomaiwang has quit IRC07:01
*** takedakn has joined #openstack-cinder07:01
*** haomaiwang has joined #openstack-cinder07:01
*** amit213 has joined #openstack-cinder07:04
openstackgerritWilson Liu proposed openstack/cinder: Huawei: Mask chap password in log  https://review.openstack.org/27159507:04
*** amit213 has quit IRC07:06
*** amit213 has joined #openstack-cinder07:08
*** amit213 has quit IRC07:10
*** amit213 has joined #openstack-cinder07:15
*** amit213 has quit IRC07:18
*** haomaiw__ has joined #openstack-cinder07:20
*** haomaiwang has quit IRC07:20
*** arecknag has joined #openstack-cinder07:25
*** takedakn has quit IRC07:29
*** takedakn has joined #openstack-cinder07:30
*** boris-42 has quit IRC07:34
*** takedakn has quit IRC07:34
*** boris-42 has joined #openstack-cinder07:35
*** laughterwym has joined #openstack-cinder07:38
*** lpetrut has quit IRC07:39
*** amit213 has joined #openstack-cinder07:40
*** amit213 has quit IRC07:42
*** andymaier has joined #openstack-cinder07:44
*** Poornima has joined #openstack-cinder07:45
*** EinstCra_ has joined #openstack-cinder07:52
*** yangyape_ has joined #openstack-cinder07:53
*** e0ne has joined #openstack-cinder07:55
*** EinstCrazy has quit IRC07:56
*** yangyapeng has quit IRC07:56
*** e0ne has quit IRC07:56
*** lprice has quit IRC07:56
*** lprice has joined #openstack-cinder07:57
*** laughterwym has quit IRC07:57
*** laughterwym has joined #openstack-cinder07:57
*** yangyapeng has joined #openstack-cinder07:59
*** haomaiw__ has quit IRC08:01
*** resker has joined #openstack-cinder08:01
*** esker has quit IRC08:01
*** haomaiwang has joined #openstack-cinder08:01
openstackgerritIvan Kolodyazhny proposed openstack/python-brick-cinderclient-ext: Fix extension name to brick_local_volume_management  https://review.openstack.org/26371008:02
*** laughterwym has quit IRC08:02
*** yangyape_ has quit IRC08:02
*** markvoelker has quit IRC08:03
*** takedakn has joined #openstack-cinder08:04
*** daneyon has joined #openstack-cinder08:05
*** daneyon has quit IRC08:06
*** EinstCra_ has quit IRC08:06
*** laughterwym has joined #openstack-cinder08:06
*** EinstCrazy has joined #openstack-cinder08:06
*** abhishekk has joined #openstack-cinder08:08
abhishekkeharney: hi you around?08:09
*** daneyon_ has quit IRC08:09
*** amit213 has joined #openstack-cinder08:09
*** andymaier has quit IRC08:09
*** amit213 has quit IRC08:13
*** Poornima has quit IRC08:13
*** neophy has joined #openstack-cinder08:14
*** abhishekk has quit IRC08:15
*** amit213 has joined #openstack-cinder08:16
*** laughterwym has quit IRC08:16
*** vilobhmm11 has joined #openstack-cinder08:17
*** amit213 has quit IRC08:18
*** amit213 has joined #openstack-cinder08:24
*** andymaier has joined #openstack-cinder08:25
*** laughterwym has joined #openstack-cinder08:33
openstackgerritMerged openstack/cinder: Add attach/detach doc to index  https://review.openstack.org/28698108:34
*** e0ne has joined #openstack-cinder08:36
*** e0ne has quit IRC08:38
*** Poornima has joined #openstack-cinder08:39
*** lprice has quit IRC08:42
*** haomaiwang has quit IRC08:45
*** laughter_ has joined #openstack-cinder08:47
*** laughterwym has quit IRC08:48
*** laughterwym has joined #openstack-cinder08:48
*** haomaiwang has joined #openstack-cinder08:48
*** ndipanov has joined #openstack-cinder08:49
*** laughterwym has quit IRC08:50
*** laughter_ has quit IRC08:51
*** daneyon has joined #openstack-cinder08:54
*** laughterwym has joined #openstack-cinder08:57
*** haomaiwang has quit IRC09:01
*** laughterwym has quit IRC09:01
*** 18VAACMSD has joined #openstack-cinder09:01
*** huanan has quit IRC09:01
*** resker has quit IRC09:01
*** huanan has joined #openstack-cinder09:02
*** sasukeh_ has quit IRC09:02
*** andymaier has quit IRC09:03
*** markvoelker has joined #openstack-cinder09:03
*** esker has joined #openstack-cinder09:05
*** laughterwym has joined #openstack-cinder09:07
*** ishant has joined #openstack-cinder09:07
*** liverpooler has joined #openstack-cinder09:11
*** Fdaisuke has joined #openstack-cinder09:12
*** jordanP has joined #openstack-cinder09:14
*** andymaier has joined #openstack-cinder09:15
*** sasukeh has joined #openstack-cinder09:16
*** jistr has joined #openstack-cinder09:17
*** EinstCrazy has quit IRC09:18
*** EinstCrazy has joined #openstack-cinder09:19
*** takedakn has quit IRC09:20
*** sasukeh has quit IRC09:22
*** e0ne has joined #openstack-cinder09:24
*** sasukeh has joined #openstack-cinder09:28
*** houming has quit IRC09:39
*** belmoreira has joined #openstack-cinder09:51
*** grassy is now known as davechen_afk09:52
*** davechen has left #openstack-cinder09:52
*** openstackgerrit_ has joined #openstack-cinder09:54
*** vilobhmm11 has quit IRC09:56
*** haomaiwang has joined #openstack-cinder09:56
*** 18VAACMSD has quit IRC09:57
*** sasukeh has quit IRC10:00
*** e0ne has quit IRC10:00
*** haomaiwang has quit IRC10:01
*** haomaiwang has joined #openstack-cinder10:01
*** openstackgerrit_ has quit IRC10:02
*** cheneydc has quit IRC10:02
*** boris-42 has quit IRC10:04
openstackgerritMerged openstack/os-brick: Add sheepdog support  https://review.openstack.org/28058810:05
*** e0ne has joined #openstack-cinder10:10
*** EinstCrazy has quit IRC10:13
*** EinstCrazy has joined #openstack-cinder10:13
*** yangyapeng has quit IRC10:16
*** jordan_ has joined #openstack-cinder10:16
*** EinstCrazy has quit IRC10:18
openstackgerritHelen Walsh proposed openstack/cinder: EMC VMAX - Operations and timeout issues  https://review.openstack.org/27276010:20
*** jordanP has quit IRC10:20
*** laughterwym has quit IRC10:28
*** laughterwym has joined #openstack-cinder10:29
*** deepakcs has joined #openstack-cinder10:31
*** laughterwym has quit IRC10:33
*** laughterwym has joined #openstack-cinder10:33
*** zerda has joined #openstack-cinder10:38
*** laughterwym has quit IRC10:39
*** Poornima has quit IRC10:39
*** Poornima has joined #openstack-cinder10:40
*** Poornima has quit IRC10:42
*** raildo-afk is now known as raildo10:42
*** ishant has quit IRC10:43
*** Poornima has joined #openstack-cinder10:43
*** mvk has joined #openstack-cinder10:43
*** yhayashi has quit IRC10:50
*** Poornima has quit IRC10:51
*** laughterwym has joined #openstack-cinder10:52
*** tpsilva has joined #openstack-cinder10:55
openstackgerritMerged openstack/cinder: Fix volume filtering for quoted display name  https://review.openstack.org/23872210:55
*** lpetrut has joined #openstack-cinder10:56
*** Poornima has joined #openstack-cinder10:56
*** baojg has quit IRC10:58
*** haomaiwang has quit IRC11:01
*** haomaiwang has joined #openstack-cinder11:01
*** salv-orl_ has quit IRC11:01
*** laughterwym has quit IRC11:01
*** salv-orlando has joined #openstack-cinder11:02
*** Poornima has quit IRC11:03
*** laughterwym has joined #openstack-cinder11:04
*** Poornima has joined #openstack-cinder11:04
dulekgeguileo: I've read the thread you've provided in https://review.openstack.org/28143411:04
geguileodulek: Hi11:05
dulekgeguileo: I've noticed that we don't really have wsgi_default_pool_size11:05
dulekgeguileo: Should we add it? Or we inherit it somewhere?11:05
geguileodulek: It's inherited by oslo11:05
geguileos/by/from11:05
dulekgeguileo: It doesn't look like it's documented in http://docs.openstack.org/liberty/config-reference/content/section_cinder.conf.html11:06
*** ankit_ag has quit IRC11:06
*** ociuhandu has joined #openstack-cinder11:06
geguileodulek: I don't know, I just looked at the code when creating and testing the patch11:06
*** laughter_ has joined #openstack-cinder11:06
*** ankit_ag has joined #openstack-cinder11:07
dulekgeguileo: Okay!11:07
*** deepakcs has quit IRC11:08
*** links has quit IRC11:08
*** laughterwym has quit IRC11:09
dulekgeguileo: I also wonder about the names and descriptions of the options.11:09
*** deepakcs has joined #openstack-cinder11:09
geguileodulek: Yeah, it's always painful for me11:09
geguileodulek: Suggestions would be appreciated11:09
dulekgeguileo: My concern is worker/thread/greenthread relation.11:09
*** laughterwym has joined #openstack-cinder11:10
dulekYou're saying thread, when meaning greenthread. That should be fine if we don't use threads.11:10
*** links has joined #openstack-cinder11:10
dulekBut I really never got that deep into oslo.service.11:10
*** laughter_ has quit IRC11:11
geguileoIn cinder greenthreads use is hardcoded11:12
geguileoiirc11:12
*** raildo is now known as raildo-afk11:12
geguileoBut oslo.service is not limited only to them11:13
*** dims has joined #openstack-cinder11:14
geguileodulek: Maybe I should add wsgi_default_pool_size to the patch11:14
geguileoAnd make it explicit11:14
dulekgeguileo: Neutron and Nova does that.11:15
dulekgeguileo: http://git.openstack.org/cgit/openstack/nova/tree/nova/conf/wsgi.py#n5111:15
geguileoI know11:15
geguileoBut I'm not a big fan of repeating stuff that is already somewhere else11:15
geguileoAlthough in this case it is best to do it11:15
dulekgeguileo: At least adding "greenthread" instead of "thread" in option description would be cool.11:15
geguileoOk, I'll make both changes11:16
dulekgeguileo: To keep consistency with wsgi_default_pool_size description in Nova.11:16
*** bharathk has quit IRC11:16
dulekAnd about adding option - I would be fine without it if it was documented in the reference.11:16
dulekI really don't know how this doc generation stuff work.11:16
openstackgerritGorka Eguileor proposed openstack/cinder: Refactor sqlalchemy service methods  https://review.openstack.org/28659811:16
openstackgerritGorka Eguileor proposed openstack/cinder: Add service_nodes table and related methods  https://review.openstack.org/28659911:16
openstackgerritGorka Eguileor proposed openstack/cinder: Add ServiceNode Versioned Object  https://review.openstack.org/28660011:16
openstackgerritGorka Eguileor proposed openstack/cinder: Change Job Distribution for HA A-A  https://review.openstack.org/28660111:16
geguileodulek: It's best I add it in the patch as well11:16
dulekMaybe we just need to import the option from oslo.wsgi?11:16
*** neophy has quit IRC11:17
*** salv-orl_ has joined #openstack-cinder11:19
*** laughterwym has quit IRC11:20
*** salv-orlando has quit IRC11:22
*** laughterwym has joined #openstack-cinder11:22
openstackgerritMerged openstack/cinder: LeftHand: Update replication to v2.1  https://review.openstack.org/28633611:24
openstackgerritMerged openstack/cinder: 3PAR: Update replication to v2.1  https://review.openstack.org/27930711:24
*** laughterwym has quit IRC11:26
*** dims has quit IRC11:26
*** dims has joined #openstack-cinder11:27
*** yangyapeng has joined #openstack-cinder11:32
*** mvk has quit IRC11:32
openstackgerritxiaoqin proposed openstack/cinder: Update unittest for Storwize pool-aware-cinder-scheduler  https://review.openstack.org/28712011:33
*** yangyapeng has quit IRC11:33
*** yangyapeng has joined #openstack-cinder11:33
*** sasukeh has joined #openstack-cinder11:36
*** laughterwym has joined #openstack-cinder11:41
*** sdague has joined #openstack-cinder11:50
*** laughterwym has quit IRC11:51
*** zerda has quit IRC11:52
*** laughterwym has joined #openstack-cinder11:54
*** nikeshm has joined #openstack-cinder11:54
*** EinstCrazy has joined #openstack-cinder11:55
*** dave-mccowan has joined #openstack-cinder11:56
*** mvk has joined #openstack-cinder11:59
*** haomaiwang has quit IRC12:01
*** haomaiwang has joined #openstack-cinder12:01
*** openstackgerrit has quit IRC12:03
*** openstackgerrit has joined #openstack-cinder12:04
openstackgerritMerged openstack/cinder: Fixed help message for AllocatedCapacityWeigher  https://review.openstack.org/28326312:06
*** Poornima has quit IRC12:09
*** deepakcs has quit IRC12:09
*** Julien-zte has joined #openstack-cinder12:11
*** Poornima has joined #openstack-cinder12:18
*** huanan has quit IRC12:20
*** huanan has joined #openstack-cinder12:21
*** edtubill has joined #openstack-cinder12:23
*** haomaiwang has quit IRC12:24
*** haomaiwang has joined #openstack-cinder12:25
*** Poornima has quit IRC12:26
*** haomaiwang has quit IRC12:29
*** belmoreira has quit IRC12:31
*** belmoreira has joined #openstack-cinder12:31
*** sasukeh has quit IRC12:38
*** haomaiwang has joined #openstack-cinder12:40
*** Poornima has joined #openstack-cinder12:43
openstackgerritGorka Eguileor proposed openstack/cinder: Add non API thread pool size configuration options  https://review.openstack.org/28143412:44
*** gouthamr has joined #openstack-cinder12:44
*** haomaiwang has quit IRC12:44
dulekgeguileo: https://review.openstack.org/#/c/268320/1/cinder/db/sqlalchemy/api.py12:46
dulekgeguileo: It's not like I want to blame anyone, but you know how rolling upgrades work, so maybe you can help me a little on how (or "if") we can fix this?12:47
geguileodulek: looking...12:48
geguileodulek: I think the best way would be reverting that patch12:49
dulekgeguileo: At least migration part of it, rest is fine.12:50
geguileodulek: Yes, revert the migration and add it in next release12:50
dulekgeguileo: But I wonder what guidelines we have on monotonicity of DB migrations?12:50
geguileodulek: You mean deleting 063 directly in a patch?12:50
geguileodulek: I'll post a patch to fix this12:51
geguileodulek: Just a sec that I figured out why the wsgi cfg was not appearing12:51
dulekgeguileo: Some people are are deploying continously.12:52
*** baojg has joined #openstack-cinder12:52
dulekSo removing a migration that was in the past would break them.12:52
dulekIn this case they would need to readd the table automatically.12:52
dulekHm…12:52
geguileodulek: It won't break them12:52
geguileodulek: Because migration keeps track of the last migration applied12:53
dulekgeguileo: Yeah, they've already deleted it. Newton's migration can be made idempotent.12:53
dulekAnd delete only in case table exists.12:53
dulekOkay, I can agree with your point of view.12:53
geguileodulek: No, rolling upgrades would still fail12:54
dulekgeguileo: In which case?12:54
geguileodulek: Mmmmmm, nop, you are right12:54
dulekOnly if someone deploys continously - he has the table deleted.12:54
dulekBut well - he need to readd it if he want to do rolling upgrade.12:55
dulekThat can be said in release notes I think.12:55
geguileodulek: We don't need to read it12:55
geguileodulek: We need to remove that migration from there and add it as a post rolling upgrade migration12:55
*** haomaiwang has joined #openstack-cinder12:56
dulekgeguileo: That would work, but we don't have the post-rolling-upgrade infrastructure in cinder-manage, and it's a little late in the cycle to add it IMO.12:57
*** porrua has joined #openstack-cinder12:57
geguileodulek: Then, we do it the bad way12:57
geguileodulek: Set a placeholder for 063 migration and set a new migration that will add the table if it doesn't exist12:57
geguileodulek: Then on next release we drop the table12:58
*** smoriya_ has quit IRC12:58
dulekgeguileo: This works in every case.12:58
geguileoYep12:58
*** haomaiwang has quit IRC12:59
dulekYou're going to work on that or should I grab this?12:59
*** haomaiwa_ has joined #openstack-cinder12:59
*** haomaiwa_ has quit IRC13:01
*** haomaiwa_ has joined #openstack-cinder13:01
geguileodulek: I break it, I fix it13:02
geguileodulek: :-)13:02
dulekgeguileo: Thanks! :)13:03
geguileodulek: Thank you for noticing in time!!!13:03
dulek:) Back to tests!13:04
*** martyturner has joined #openstack-cinder13:04
geguileodulek: This is the reason why wsgi config option did not appear in our config sample: https://review.openstack.org/28778713:06
*** haomaiwa_ has quit IRC13:07
dulekgeguileo: Oh! Nice!13:08
*** timcl has joined #openstack-cinder13:10
*** haomaiwa_ has joined #openstack-cinder13:10
*** raildo-afk is now known as raildo13:11
dulekgeguileo: We don't really use any code from oslo.service.sslutils. Should it still be here? Is it used internally by oslo.service.sth?13:11
*** markvoelker has quit IRC13:11
*** markvoelker has joined #openstack-cinder13:11
geguileodulek: Good question, with the euphoria of finding the issue I just added everything13:11
geguileoXD13:11
dulekgeguileo: WSGI does use it.13:12
dulekI mean oslo_service.wsgi13:12
geguileodulek: Then it's settled  ;-)13:12
dulekI think so.13:12
*** ndipanov has quit IRC13:14
*** laughterwym has quit IRC13:16
*** laughterwym has joined #openstack-cinder13:18
*** Poornima has quit IRC13:20
*** laughter_ has joined #openstack-cinder13:21
*** Poornima has joined #openstack-cinder13:22
*** haomaiwa_ has quit IRC13:23
*** huanan has quit IRC13:24
*** laughterwym has quit IRC13:25
*** huanan has joined #openstack-cinder13:25
*** JoseMello has joined #openstack-cinder13:26
*** laughter_ has quit IRC13:26
*** Julien-zte has quit IRC13:27
*** akerr has joined #openstack-cinder13:28
*** crose has joined #openstack-cinder13:31
*** Poornima has quit IRC13:33
*** shausy has quit IRC13:33
*** akerr_ has joined #openstack-cinder13:33
*** links has quit IRC13:34
EmilienMhello cinder experts!13:34
EmilienMI'm trying to deploy Cinder API with SSL but even with the options in cinder.conf, it keeps starting in http13:34
EmilienMthe cinder.conf is here: http://logs.openstack.org/99/286899/15/check/gate-puppet-openstack-integration-scenario002-tempest-dsvm-centos7/1ca5928/logs/etc/cinder/cinder.conf.txt.gz13:35
EmilienMor you can easily read it like this: http://paste.openstack.org/show/489142/13:36
EmilienMbut as you can see here: http://logs.openstack.org/99/286899/15/check/gate-puppet-openstack-integration-scenario002-tempest-dsvm-centos7/1ca5928/logs/cinder/api.txt.gz#_2016-03-03_03_10_55_57713:36
EmilienMit's starting will http :(13:36
*** akerr has quit IRC13:36
*** akerr_ is now known as akerr13:36
*** xyang1 has joined #openstack-cinder13:37
openstackgerritDuncan Thomas proposed openstack/cinder: Continue volume delete on encryption key delete errors  https://review.openstack.org/28712513:39
*** adrianofr_ has quit IRC13:40
openstackgerritGorka Eguileor proposed openstack/cinder: Readd iscsi_target table  https://review.openstack.org/28780013:43
geguileodulek: ^13:43
dulekgeguileo: I'll take a look, thanks.13:44
*** edmondsw has joined #openstack-cinder13:48
EmilienMcan we deploy cinder api with ssl in eventlet?13:49
*** Poornima has joined #openstack-cinder13:50
*** yangyapeng has quit IRC13:50
*** yangyapeng has joined #openstack-cinder13:50
*** adrianofr_ has joined #openstack-cinder13:50
dulekEmilienM: I think so. I think it should be possible with the same options as Nova has.13:51
dulekEmilienM: But I don't have a recipe ready unfortunately.13:51
*** baojg has quit IRC13:51
openstackgerritPhilipp Marek proposed openstack/cinder: DRBD: Policy-based waiting for completion.  https://review.openstack.org/28780713:51
EmilienMdulek: have you looked my config?13:52
EmilienMdulek: http://paste.openstack.org/show/489142/13:52
EmilienMcinder api keeps starting in http13:52
*** krtaylor has quit IRC13:54
*** ankit_ag has quit IRC13:55
flip214Hi smcginnis, sorry about being so late with https://review.openstack.org/#/c/287807/ - but that would make sense to get the Cinder driver compatible with the soon-to-be released versions of DRBDmanage. After all, when Mitaka is released, nobody will (wants to) install an old version on purpose, right?13:55
*** EinstCrazy has quit IRC13:55
*** EinstCrazy has joined #openstack-cinder13:56
scottdaHas anyone seen cinder-python34 failing Jenkins with:13:57
scottdacinder.tests.unit.volume.flows.test_create_volume_flow.CreateVolumeFlowTestCase.test_cast_create_volume13:57
scottda?13:57
scottdaIt passes on my workstation.13:57
dulekEmilienM: enabled_ssl_apis option is missing?13:57
dulekEmilienM: I've just looked how to do that for Nova and noticed that you don't have that option set.13:58
EmilienMdulek: I haven't seen this option in cinder code13:58
dulekEmilienM: I believe the way to configure SSL should be the same.13:58
dulekEmilienM: We may not import it, but oslo.wsgi may use it.13:58
*** rlrossit has joined #openstack-cinder13:58
EmilienMoslo.wsgi?13:58
dulekEmilienM: Look on my discussion with geguileo an hour ago about Cinder not having all the oslo.service options exposed.13:59
dulekoslo.service.wsgi13:59
dulekEmilienM: https://review.openstack.org/#/c/287787/13:59
dulekEmilienM: Anyway I'm just waving my hands, never deployed it in SSL.13:59
EmilienMdulek: can I deploy Cinder API with apache?14:00
dulekEmilienM: Sure.14:00
dulekEmilienM: http://docs.openstack.org/developer/cinder/devref/api.apache.html14:00
EmilienMI'll do that, much simplier.14:01
EmilienMdulek: so you think I can use enabled_ssl_apis now, without 287787 ?14:02
dulekEmilienM: Yeah, 287787 is only fixing configuration reference generator.14:03
EmilienMok14:03
EmilienMyeah, all works fine in nova14:03
dulekEmilienM: But I'm not 100% sure that is the problem, I've just noticed that.14:03
geguileoEmilienM: Yep you can use it14:03
EmilienMgeguileo: enabled_ssl_apis = ?14:03
geguileoEmilienM: I don't think that's an Oslo config option, so probably no...14:04
dulekEmilienM: Ah, I agree with geguileo, I've just dug deeper…14:04
EmilienMso why cinder is starting in http: http://logs.openstack.org/99/286899/15/check/gate-puppet-openstack-integration-scenario002-tempest-dsvm-centos7/1ca5928/logs/cinder/api.txt.gz#_2016-03-03_03_10_55_57714:05
dulekEmilienM: Hah, because it looks like we're never passing use_ssl=True to oslo_service.wsgi.Server14:06
dulekEmilienM: Looks like its simply impossible in current state of cinder-api.14:06
dulekEmilienM: But again I may be mistaken.14:07
EmilienMI would be surprised to see Cinder API not supporting SSL, SSL is like everywhere in production14:07
dulekEmilienM: Oh, you can overwrite it…14:07
dulekEmilienM: Waiiit…14:07
dulekEmilienM: Nope, looks like oslo_service.wsgi will turn on SSL only if you pass use_ssl=True, and it doesn't look like we do it in any case.14:08
EmilienMdulek: so I can't use use_ssl = True, cinder does not support it right?14:09
scottdahmm..looks like oslo_versionedobjects.fields.py is throwing an exception when the fake_consistencygroup vol id is '1'....14:10
scottdahttps://github.com/openstack/oslo.versionedobjects/blob/master/oslo_versionedobjects/fields.py#L31414:10
scottdahttp://logs.openstack.org/01/286901/2/check/gate-cinder-python34/e5cb6cc/console.html#_2016-03-02_20_00_11_87114:10
dulekEmilienM: I think so, you may want a second person to confirm.14:10
dulekscottda: Oh, oslo_versionedobject already changed that?14:10
scottdadulek: I'm trying to figure that out...14:11
dulekscottda: No, they only warn us - https://github.com/openstack/oslo.versionedobjects/blob/master/oslo_versionedobjects/fields.py#L318-L32514:11
scottdadulek: Do you know if it changed?14:11
tbarronscottda: on the python3 issue, make sure you've got https://review.openstack.org/#/c/287219/14:11
tbarronscottda: which just recently merged14:11
*** jaypipes has joined #openstack-cinder14:12
*** cdelatte has joined #openstack-cinder14:13
scottdatbarron: Thanks. That could be my issue. Or it could be the cause of my issue :)14:13
*** jordan__ has joined #openstack-cinder14:13
*** mylu has joined #openstack-cinder14:14
tbarronscottda: I was having intermittent problems with that test yesterday before the patch14:14
tbarronscottda: it *looks* like it will fix it.14:14
*** links has joined #openstack-cinder14:15
openstackgerritoliver-leahy-l proposed openstack/cinder: Fix call to Barbican Secrets create()  https://review.openstack.org/28781614:15
*** mylu has quit IRC14:15
openstackgerritScott DAngelo proposed openstack/cinder: support new HTTP microversion header  https://review.openstack.org/28690114:15
*** tobasco_ is now known as tobasco14:16
*** Julien-zte has joined #openstack-cinder14:16
scottdatbarron: Thanks. I rebased . We'll see if that does the trick. Of course, we could also change fakes_consitencygroup to use a valid UUID at some point....I think I'll file a bug!14:17
openstackgerritTom Barron proposed openstack/cinder: Don't run test_volume.VolumeTestCase twice  https://review.openstack.org/28694414:17
*** delattec has joined #openstack-cinder14:17
*** jordan_ has quit IRC14:17
*** cdelatte has quit IRC14:20
*** delattec has quit IRC14:21
*** mriedem has joined #openstack-cinder14:21
*** haomaiwang has joined #openstack-cinder14:21
*** ndipanov has joined #openstack-cinder14:21
*** edtubill has quit IRC14:24
smcginnisflip214: Will the current state be incompatible with the new DRDB?14:24
smcginnisflip214: Reading that commit message, that makes it sound like it will just work better.14:25
smcginnisflip214: But it doesn't sound like it won't work at all.14:25
*** arch-nemesis has joined #openstack-cinder14:25
flip214smcginnis: the new DRBDmanage release will be async; while the currently upstream code is compatible enough,14:26
flip214it will wait for *all* nodes to be ready before proceeding.14:26
*** edtubill has joined #openstack-cinder14:26
smcginnisflip214: So it could cause unnecessarily long delays without this patch?14:26
flip214ie. if a volume is replicated across 3 nodes, *any* of them failing will cause snapshots/resizing or even new volumes (that should be deployed on the failed node) to freeze14:26
flip214yeah, right14:27
*** alkhodos has joined #openstack-cinder14:27
smcginnisflip214: OK, got it.14:27
flip214with the patch the admin can configure a policy - one node is enough / I want all of them / more than half is good to go / etc14:27
flip214thanks!14:27
*** cknight has joined #openstack-cinder14:28
*** links has quit IRC14:30
*** eharney has quit IRC14:31
*** edtubill has quit IRC14:31
*** erlon has joined #openstack-cinder14:33
*** guitarza1 is now known as guitarzan14:34
*** Guest60497 is now known as dansmith14:36
*** daneyon has quit IRC14:38
*** mmandel has joined #openstack-cinder14:39
*** crose has quit IRC14:39
*** jgregor has joined #openstack-cinder14:40
openstackgerritAlon Marx proposed openstack/cinder: Remove a vol in error state from a CG  https://review.openstack.org/27046014:41
*** knikolla has joined #openstack-cinder14:41
*** akshai has joined #openstack-cinder14:42
*** mriedem has left #openstack-cinder14:42
*** mriedem has joined #openstack-cinder14:43
*** mylu has joined #openstack-cinder14:43
smcginnisDuncanT: This looks good to you? https://review.openstack.org/#/c/263710/14:44
*** cbader has joined #openstack-cinder14:44
smcginnisI think I need to release that, so want to make sure that one is in first.14:44
DuncanTOne sec....14:45
smcginnisDuncanT: Thanks! No rush.14:45
DuncanTDone14:45
smcginnisDuncanT: Awesome, thanks.14:46
e0nesmcginnis, DuncanT: thanks!14:46
smcginnise0ne: I think that's the only one ready. Do you think we're good to release after that goes in?14:46
smcginnise0ne: There's this one: https://review.openstack.org/#/c/268405/14:46
smcginnise0ne: But looks like hemnafk had some problems with it.14:46
e0nesmcginnis: gimme few minutes. I'll try it on my env14:47
DuncanTAnybody know what style mains plug they use in china?14:48
smcginnisDuncanT: http://www.interpower.com/ic/guide.html14:48
smcginnisGB 2099.1 and GB 1002-114:49
DuncanTThanks14:49
smcginnisDuncanT: When are you going?14:49
openstackgerritMerged openstack/python-brick-cinderclient-ext: Fix extension name to brick_local_volume_management  https://review.openstack.org/26371014:49
DuncanTSunday14:50
*** haomaiwang has quit IRC14:50
smcginnisDuncanT: Nice. How long of a trip is that from Isreal? Seems long, but have no perception.14:50
*** haomaiwang has joined #openstack-cinder14:51
smcginnisDuncanT: Here's a better reference: http://www.worldstandards.eu/electricity/plugs-and-sockets/i/14:51
DuncanTActual flight time, it's not so bad, but the times are aweful so it's a minimum of 27 hours :-(14:52
openstackgerritScott DAngelo proposed openstack/cinder: fake_consistencygroup uses invalid UUID  https://review.openstack.org/28785414:52
smcginnisDuncanT: Holy crap. Worse than I thought. :)14:52
DuncanTIt's about 7 hours in the air14:52
smcginnis20 hours of layovers?14:52
DuncanTYup14:54
*** markstur has joined #openstack-cinder14:55
e0neDuncanT: I don't remember, I've used Europe or US power adopters in China w/o any issues14:55
*** dustins has joined #openstack-cinder14:55
smcginnisHah, I like this: Type D plugs are among the most dangerous ones in the world14:55
smcginnisI've gotten some sparks from that.14:55
*** eharney has joined #openstack-cinder14:55
DuncanTI've two universal adaptors that look like they should work14:55
DuncanTAt least they aren't on the 120v nonsense....14:56
flip214smcginnis: thanks a lot for the quick review!14:58
jgriffithdulek: OMG!!!  https://review.openstack.org/#/c/285908/3/cinder/db/sqlalchemy/api.py15:00
jgriffithdulek: I mean... "tank you" for noticing it15:00
*** haomaiwang has quit IRC15:01
*** belmoreira has quit IRC15:01
jgriffithdulek: but I'm a bit annoyed that we just "change" exceptions15:01
e0nejgriffith: what's wrong with that patch?15:01
*** haomaiwang has joined #openstack-cinder15:01
*** esker has quit IRC15:02
jgriffithdulek: IMO the correct fix to the call catching the wrong exception would be to catch the right exception no?  Not change the exception15:02
jgriffithe0ne: ^^15:02
*** markstur has left #openstack-cinder15:02
*** liverpooler has quit IRC15:02
*** haomaiwang has quit IRC15:02
dulekjgriffith: Well, wasn't it inconsistent before that?15:02
jgriffithe0ne: and it's not so much "what's wrong with it", I"m just venting because I now have yet another rebase error on my patch, that actually would have gone undetected if it wasn't for dulek15:02
jgriffithThe exception ?  Not really15:03
*** haomaiwang has joined #openstack-cinder15:03
dulekjgriffith: HostBinaryNotFound? What kind of resource is HostBinary?15:04
*** haomaiwang has quit IRC15:04
dulekIMO ServiceNotFound is fine - we're looking for a service when making the DB call.15:04
*** haomaiwang has joined #openstack-cinder15:04
e0nedulek: +115:04
*** rcernin has joined #openstack-cinder15:04
*** alkhodos has quit IRC15:07
*** esker has joined #openstack-cinder15:07
*** EinstCrazy has quit IRC15:08
*** yangyapeng has quit IRC15:08
*** EinstCrazy has joined #openstack-cinder15:08
*** mylu has quit IRC15:08
*** yangyapeng has joined #openstack-cinder15:09
*** ntpttr has joined #openstack-cinder15:11
*** ntpttr has quit IRC15:12
smcginnisjgriffith: You've been living the rebase dream.15:13
e0nesmcginnis: I've added comment to https://review.openstack.org/#/c/268405/15:14
*** kaisers_ has joined #openstack-cinder15:14
smcginnise0ne: Oh! Yeah, now looking at his paste, that totally makes sense.15:15
e0nesmcginnis: let's wait for hemnafk15:15
*** alyson_ has joined #openstack-cinder15:16
smcginnise0ne: Sounds good.15:16
e0nesmcginnis: it can't work w/o root permissions and we must use "sudo" for it15:16
smcginnise0ne: I wonder if we could build in some checking and give a good error if someone does that.15:17
e0nesmcginnis: I'll try to make a patch15:17
smcginnisMaybe something like "if os.getuid() != 0: raise RunAsRootRequiredError()" or something like that.15:18
*** kaisers_92 has joined #openstack-cinder15:19
*** kaisers_ has quit IRC15:21
*** krtaylor has joined #openstack-cinder15:23
*** kaisers_92 has quit IRC15:25
e0nesmcginnis: the question is: how should check it: os-brick or python-brick-cinderclient-ext?15:26
openstackgerritAdriano Freires Rosso proposed openstack/cinder: WIP: HNAS drivers refactoring  https://review.openstack.org/28546915:26
smcginnise0ne: I would think in the cinderclient.15:26
*** dave-mccowan has quit IRC15:28
*** dave-mccowan has joined #openstack-cinder15:28
*** mtanino has joined #openstack-cinder15:29
openstackgerritMerged openstack/cinder: Fix oslo.service config generation  https://review.openstack.org/28778715:30
*** lprice has joined #openstack-cinder15:31
*** john5223 has joined #openstack-cinder15:32
*** baumann has joined #openstack-cinder15:34
e0nesmcginnis: ok. I have to head off in ~20 minutes. I'll make a patch later tonight15:35
*** mmandel has quit IRC15:37
*** Poornima has quit IRC15:42
smcginnise0ne: Sounds good. No rush.15:43
*** crose has joined #openstack-cinder15:43
*** esker has quit IRC15:45
*** Julien-zte has quit IRC15:46
*** Poornima has joined #openstack-cinder15:46
*** Poornima has quit IRC15:50
*** EinstCrazy has quit IRC15:50
*** EinstCrazy has joined #openstack-cinder15:50
*** mmandel has joined #openstack-cinder15:51
*** EinstCra_ has joined #openstack-cinder15:55
*** esker has joined #openstack-cinder15:55
*** jungleboyj has joined #openstack-cinder15:56
*** EinstCrazy has quit IRC15:57
*** esker has quit IRC15:58
*** arecknag has quit IRC15:58
openstackgerritIvan Kolodyazhny proposed openstack/python-brick-cinderclient-ext: WIP. Require root permissions for get-connector operation  https://review.openstack.org/28793815:58
e0nesmcginnis: I'll add tests and remove WIP tonight ^^15:59
*** anshul has quit IRC16:00
*** haomaiwang has quit IRC16:01
*** EinstCra_ has quit IRC16:01
*** jseiler has quit IRC16:01
scottdasmcginnis: dulek DuncanT and everyone else, Here's a high-priority fix to the way we do microversion headers:16:01
scottdahttps://review.openstack.org/#/c/286901/16:01
*** EinstCrazy has joined #openstack-cinder16:01
scottdaAs discussed in meeting yesterday16:01
*** haomaiwa_ has joined #openstack-cinder16:01
*** e0ne has quit IRC16:02
*** ntpttr has joined #openstack-cinder16:04
*** EinstCrazy has quit IRC16:06
*** rcernin has quit IRC16:06
*** haplo37 has joined #openstack-cinder16:07
*** jistr is now known as jistr|call16:08
*** jdurgin1 has joined #openstack-cinder16:11
*** permalac has joined #openstack-cinder16:12
openstackgerritJohn Griffith proposed openstack/cinder: Use get_by_args instead of host_and_topic  https://review.openstack.org/28697616:13
*** martyturner has quit IRC16:14
scottdajgriffith: You might want to review that as well ^^^16:14
*** nsasherr has quit IRC16:14
Swansonjgriffith, Have the tests passing on this one now ...  https://review.openstack.org/#/c/286256/16:14
*** nkrinner has quit IRC16:16
*** esker has joined #openstack-cinder16:16
*** laughterwym has joined #openstack-cinder16:17
*** hemnafk is now known as hemna16:18
hemnamornin16:18
smcginnishemna: Morning16:18
hemnawhat's up16:19
smcginnishemna: Mind taking a look at the response here when you get a chance: https://review.openstack.org/#/c/268405/16:19
hemnaok so the official answer is to sudo cinder ?16:19
smcginnishemna: Yeah. e0ne has a WIP up to check that it's running as root and give a better error message.16:20
hemnaok, that was going to be my next request :)16:20
hemnaso that will be a follow up?16:20
smcginnisBut I think I need to release that today, so probably better to get what we have ready.16:20
smcginnishemna: Yeah.16:20
hemnaok I'll +A it then16:20
*** diablo_rojo has joined #openstack-cinder16:21
hemnadone16:21
smcginnishemna: Here's that WIP: https://review.openstack.org/#/c/268405/16:21
smcginnishemna: Thanks!16:21
hemnawrong url16:21
hemna:P16:22
smcginnisOh, weird. I pasted in to IRC and it was wrong and pasted into browser and got the right patch. :S16:22
smcginnishttps://review.openstack.org/#/c/287938/16:22
* smcginnis is confused16:22
hemnaok coolio16:22
smcginnishemna: BTW, running apt-mirror and bandersnatch for a few days now. Some pip sync issues at first, but appears to be running well now.16:23
* smcginnis knocks on wood16:23
hemnaoh nice16:23
hemnacan I snag your bandersnatch setup?  pip.conf ?16:23
hemnaI couldn't get pip to work with mine.16:24
hemna:(16:24
smcginnisDoesn't seem to have speeded things up much, but it at least is a little nicer on our internet pipe.16:24
hemnawe noticed a 2x speedup on our end just for apt16:24
smcginnishemna: How'd you try the pip client side config?16:24
hemnaI'll have to check it out again16:24
*** bardia has joined #openstack-cinder16:24
hemnait's been a few weeks16:24
smcginnishemna: Took me a while to figure out what was going on with cloud-init overwriting apt source.list16:25
bswartzsmcginnis: did you try devpi?16:25
smcginnisbswartz: No, using bandersnatch.16:26
bswartzI thought devpi offered equivalent speedup to bandersnatch with much less effort16:26
smcginnisbswartz: I think I briefly looked at devpi in the paste.16:26
* bswartz uses devpi and is happy with it16:26
*** jistr|call is now known as jistr16:26
smcginnisbswartz: I forget which one it was, but the one I tried initially had some weird authentication restriction.16:26
smcginnishemna: In case this is of any use: http://paste.openstack.org/show/489178/16:26
smcginnisFile name is wrong there, but that's what my /etc/pip.conf is.16:27
smcginnisbswartz: Yeah, I did try devpi.16:27
smcginnisbswartz: I might have just screwed myself up with the client side config.16:27
bswartzpip.conf for devpi is basically identical to your paste16:28
smcginnisI remember doing something hokey with setting HTTP_PROXY.16:28
smcginnisSo I'm sure it was my fail.16:28
bswartzthe "trusted-host" thing is essential or it will barf on your SSL ceritificate16:28
bswartz(or lack of certificate)16:29
smcginnisPfft, who needs encryption?16:29
smcginnis:)16:29
bswartzikr16:29
*** garthb has joined #openstack-cinder16:29
hemnasmcginnis, thanks16:31
openstackgerritWalter A. Boring IV (hemna) proposed openstack/os-brick: remove unneeded in_use flag checking for multipath  https://review.openstack.org/26467816:35
*** laughter_ has joined #openstack-cinder16:36
*** jaypipes has quit IRC16:36
smcginnishemna: Oh yeah, one point of warning. Make sure your cron job or whatever is running frequently.16:38
hemnahow frequently ?16:39
hemnaonce an hour ?16:39
smcginnishemna: I was surprised by how quickly I started getting stacking errors when I got behind.16:39
hemnadoh16:39
smcginnishemna: Could just be the time of the cycle too.16:39
smcginnishemna: I actually have it run every two minutes.16:39
hemnawhoa16:39
smcginnishemna: Maybe a little overkill.16:40
smcginnisBut it just checks the version serial and if it's the same it's done.16:40
smcginnisBut looking back through the log, I'm getting 1 to 7 new packages nearly every time.16:40
*** laughterwym has quit IRC16:40
*** timcl has quit IRC16:40
smcginnisA few "0 packages to sync" messages, but probably < 50%16:41
smcginnisMost likely safe to up that to 5-15 minute intervals, but I'm being cautious. ;)16:41
hemnadamn16:43
smcginnisChanged it. Here's my cron: http://paste.openstack.org/show/489180/16:43
*** diogogmt has joined #openstack-cinder16:43
*** timcl has joined #openstack-cinder16:44
diogogmtis there any docs explaning the volume type settings configuration? like iops, qos, availability, encyrption, etc… ?16:46
smcginnisdiogogmt: Some extra spec settings might be here depending on the backend driver you're using: http://docs.openstack.org/liberty/config-reference/content/section_volume-drivers.html16:48
diogogmtsmcginnis: thanks16:50
diogogmtsmcginnis: currently, can i select a volume type for the root volume when creating a instance?  it is possible during the volume create but i haven’t seem that option on the instance create16:51
*** sdague has quit IRC16:52
diogogmtbtw, in the ceph config reference i don’t see any mention to volume type settings: http://docs.openstack.org/liberty/config-reference/content/ceph-rados.html16:52
smcginnisdiogogmt: I don't believe it's exposed through the instance creation workflow.16:53
smcginnisdiogogmt: You would need to create the volume first, then make it a bootable volume16:53
smcginnisdiogogmt: Not all drivers have all extra spec documentation there.16:53
smcginnisdiogogmt: It depends if someone went out and added it.16:54
tbarronwinston-d: smcginnis: when you get a chance, https://review.openstack.org/#/c/286944/ lost your +2s and +A due to commit msg update16:55
smcginnistbarron: Will do.16:55
diogogmtsmcginnis: ok, well, if anybody ever configured some volume types with ceph please let me know, i would be interested in hearing more about the details16:56
*** esker has quit IRC16:58
tbarronsmcginnis: ty16:58
*** haomaiwa_ has quit IRC17:01
* hemna is bandersnatching17:01
smcginnis:)17:01
*** haomaiwa_ has joined #openstack-cinder17:01
openstackgerritVincent Hou proposed openstack/cinder: Storwize: Update replication to v2.1  https://review.openstack.org/28748317:03
*** chris_morrell has joined #openstack-cinder17:08
openstackgerritGorka Eguileor proposed openstack/cinder: Add non API thread pool size configuration options  https://review.openstack.org/28143417:09
krotscheckHey everyone- I'm working on cinder's version of #1551836 - are there any cores around to take a look at https://review.openstack.org/#/c/286795 ?17:09
krotscheckALso, is there a process by which bugs are triaged?17:10
* krotscheck would like to get a priority and a milestone on ^^17:10
smcginniskrotscheck: How important is it to get this in Mitaka?17:12
smcginniskrotscheck: Or can it wait until Newton?17:12
*** IlyaG has joined #openstack-cinder17:13
krotschecksmcginnis: Well, the nova team specifically requested it because keeping the config in paste imposes a maintenance burden moving forwards.17:13
krotschecksmcginnis: The list discussion seemed to agree that it's a mitaka release blocker for them, but cinder may not agree.17:13
hemnakrotscheck, so I dunno if Cinder responds to any HEAD requests at all, just noticed it was missing17:14
*** porrua has quit IRC17:14
krotscheckhemna: Yeah, HEAD is one of those things frequently overlooked during implementation.17:14
krotscheckhemna: I can do a quick grep, see if ther's any handling....17:15
hemnaI was about to do the same :)17:15
*** apoorvad has joined #openstack-cinder17:16
krotscheckhemna: I see some references to it in the REST client....17:17
*** lcurtis has joined #openstack-cinder17:17
krotscheckhemna: Only one tested instance -> path = '/api/storage/v1/pools'17:17
krotscheckSo, either way, should be added.17:17
hemnaok yah easy enough to do :)17:17
hemnathanks17:17
smcginniskrotscheck: I agree with the nova decision. If we can get this fixed up, looks like it should probably go in.17:18
krotschecksmcginnis: Alright17:18
krotschecksmcginnis: Adding the header you mentioned now.17:18
*** salv-orlando has joined #openstack-cinder17:20
krotschecksmcginnis: Is the volume-microversion a thing that I need to keep in there for backwards compatibility, or is that a brand new mitaka thing?17:20
*** salv-orl_ has quit IRC17:22
krotscheckAh, 2/24/16. Mitaka only17:23
hemnakrotscheck, this is new17:23
openstackgerritMichael Krotscheck proposed openstack/cinder: Moved CORS middleware configuration into oslo-config-generator  https://review.openstack.org/28679517:24
openstackgerritApoorva Deshpande proposed openstack/cinder: Tintri image direct clone  https://review.openstack.org/26112617:24
krotscheckhemna, smcginnis ^^ - includes depens-on for microversion patch, HEAD.17:25
*** andymaier has quit IRC17:25
krotscheckIf one of you two could flag https://bugs.launchpad.net/oslo.config/+bug/1551836/ as a mitaka thing and high priority, that'll keep the release team informed (I think)17:26
openstackLaunchpad bug 1551836 in neutron "CORS middleware's latent configuration options need to change" [High,In progress] - Assigned to Michael (krotscheck)17:26
* krotscheck doesn't seem to have the access rights to do that.17:26
*** martyturner has joined #openstack-cinder17:28
*** laughterwym has joined #openstack-cinder17:28
*** ndipanov has quit IRC17:28
scottdasmcginnis: or anyone, I've a question about this doc bug for microversions: https://bugs.launchpad.net/cinder/+bug/154954817:28
openstackLaunchpad bug 1549548 in Cinder " cinder-api-microversions code" [High,New] - Assigned to Scott DAngelo (scott-dangelo)17:28
scottdaWhat format do I use to provide the doc info to the manuals team?17:29
scottdasome .rst and attach to bug? or ??17:29
*** JoseMello has quit IRC17:29
scottdadulek: You seemed to know what to do ^^^17:30
*** adrianofr_ has quit IRC17:30
*** laughter_ has quit IRC17:31
*** jistr has quit IRC17:33
*** jaypipes has joined #openstack-cinder17:36
hemnasmcginnis, hey, looks like pip is working w/ my BS mirror17:38
*** laughterwym has quit IRC17:42
*** laughterwym has joined #openstack-cinder17:43
*** vilobhmm11 has joined #openstack-cinder17:45
*** bardia has quit IRC17:45
*** vilobhmm111 has joined #openstack-cinder17:46
*** laughterwym has quit IRC17:48
*** vilobhmm11 has quit IRC17:49
*** salv-orl_ has joined #openstack-cinder17:49
*** salv-orlando has quit IRC17:52
*** jdurgin1 has quit IRC17:53
*** e0ne has joined #openstack-cinder17:55
openstackgerritMerged openstack/cinder: Fix invalid uuid warnings in scheduler unit tests  https://review.openstack.org/28616717:55
openstackgerritMerged openstack/cinder: Don't run test_volume.VolumeTestCase twice  https://review.openstack.org/28694417:58
*** haomaiwa_ has quit IRC18:01
*** haomaiwa_ has joined #openstack-cinder18:01
*** vilobhmm111 has quit IRC18:02
*** salv-orl_ has quit IRC18:02
*** salv-orl_ has joined #openstack-cinder18:03
*** Poornima has joined #openstack-cinder18:03
*** mriedem has quit IRC18:10
*** salv-orl_ has quit IRC18:10
*** salv-orlando has joined #openstack-cinder18:11
thingeeeharney: can you explain to me why upper-constraints has oauth2 but not the global requirements re: https://review.openstack.org/#/c/284929/3/requirements.txt18:13
ntpttrIf any cores have a moment would you mind taking a quick look at this https://review.openstack.org/#/c/252476/ ? Got a +2 a couple months ago and needs workflow18:15
eharneythingee: i don't really know, but i think the constraints info was automatically generated (probably because google-api-python-client was pulling it as a dep)18:15
thingeeeharney: I'd like to understand that before approving this. I guess I can reread discussions with upper-constraint again18:16
eharneythingee: ok, i don't really have a good answer18:16
*** permalac has quit IRC18:17
eharneythingee: did you see https://review.openstack.org/#/c/284927/ ?18:18
thingeeeharney: well then I'm happy as a clam18:18
eharneygood :)18:18
eharneyi assume we're ticking toward some deadline for openstack/requirements changes, hopefully that lands soon..18:20
*** mriedem has joined #openstack-cinder18:20
*** mtanino__ has quit IRC18:22
openstackgerritTom Barron proposed openstack/cinder: Fix invalid uuid warnings in test_volume.py  https://review.openstack.org/28804718:22
patrickeastthingee: smcginnis: thanks for pushing that cisco thing back in so quick18:24
smcginnispatrickeast: Thanks for bailing them out. :)18:24
thingeeI did it as quick as I removed once I knew it was up for question18:24
patrickeasthaha18:24
thingeeI'm really a fair person18:24
smcginnisscottda: Just add some notes to the bug report and reassign it.18:24
smcginnisscottda: They will take care of writing something up and formatting it.18:25
thingeepatrickeast: I was sort of wondering about that. Since I knew initially you were already helping them earlier last year18:25
*** apoorvad has quit IRC18:25
thingeepatrickeast: so what's the story there? you started using brocade or something and now switching back just for now?18:26
patrickeastthingee: yea... I was kind of hoping they would get their CI ready in time18:26
patrickeastthingee: well, using the fczm is kind of hard with the ci system18:26
patrickeastthingee: you can't reliably test multiple 'clouds' sharing the same switch18:26
thingeepatrickeast: ah I see, so you wanted no zone manager on it18:26
patrickeastthingee: so parallel tempest runs were stepping on eachother18:26
patrickeastthingee: exactly, switching to static zones and no fczm was the last key to stabilizing my ci for lots of green18:27
thingeepatrickeast: interesting... the stable issues though are related to the zone manager solution itself? So ideally something that would be worked out with the zone manager company itself.18:27
patrickeastthingee: ehh sort of, it uses locking but the locks are shared between clouds18:28
patrickeastthingee: which isn't really a big deal, in a real deployment the same switch wouldn't be sharing18:28
patrickeastthingee: s/are shared/are *not* shared/18:29
smcginnisthingee: What update is needed here? https://review.openstack.org/#/c/258295/18:29
*** lpetrut has quit IRC18:29
thingeesmcginnis: ddt is not in our requirements18:29
thingeeit's in global18:30
smcginnisthingee: https://github.com/openstack/cinder/blob/master/test-requirements.txt#L1018:30
*** takedakn has joined #openstack-cinder18:30
*** jgregor has quit IRC18:30
smcginnisthingee: Is there a newer version needed?18:30
smcginnisWe've been using ddt for some time.18:30
thingeeah right it's in unit tests, my bad18:30
smcginnisthingee: OK, good. You had me worried. ;)18:31
*** sdague has joined #openstack-cinder18:32
*** bardia has joined #openstack-cinder18:33
openstackgerritIvan Kolodyazhny proposed openstack/python-brick-cinderclient-ext: Query volume paths implementation  https://review.openstack.org/26840518:37
aorourkehemna, did anything ever merge for disabling those insecure platform warnings?18:39
openstackgerritIvan Kolodyazhny proposed openstack/python-brick-cinderclient-ext: Query volume paths implementation  https://review.openstack.org/26840518:41
*** takedakn has quit IRC18:42
*** Lee1092 has quit IRC18:42
openstackgerritIvan Kolodyazhny proposed openstack/python-brick-cinderclient-ext: Query volume paths implementation  https://review.openstack.org/26840518:43
hemnaaorourke, https://review.openstack.org/#/c/267870/18:43
*** takedakn has joined #openstack-cinder18:44
*** rcernin has joined #openstack-cinder18:44
*** takedakn has quit IRC18:45
e0nesmcginnis, hemna: I've rebased patch ^^ on master18:45
e0neunfortunately, I don't knoe Anthony Lee's irc nick:(18:45
openstackgerritVincent Hou proposed openstack/cinder: Storwize: Update replication to v2.1  https://review.openstack.org/28748318:48
smcginnise0ne: leeantho I believe.18:49
*** IlyaG_ has joined #openstack-cinder18:52
openstackgerritIvan Kolodyazhny proposed openstack/python-brick-cinderclient-ext: Require root permissions for CLI operations  https://review.openstack.org/28793818:53
e0nesmcginnis, hemna: patch ^^ to check for root permissions18:53
hemnae0ne, k18:53
e0nealso, I've found issue with mocking some sudo calls in the tests :(, will fix it later tonigh or tomorrow morning18:54
*** vilobhmm11 has joined #openstack-cinder18:55
*** IlyaG has quit IRC18:55
*** daneyon has joined #openstack-cinder18:56
*** mvk has quit IRC18:57
* e0ne bb in ~2 hours18:57
*** e0ne has quit IRC18:57
*** neophy has joined #openstack-cinder18:58
openstackgerritMerged openstack/cinder: Fixes running error for storwize _run_ssh  https://review.openstack.org/28702518:58
*** daneyon_ has joined #openstack-cinder18:59
*** haomaiwa_ has quit IRC19:01
*** 20WAAG6RV has joined #openstack-cinder19:01
openstackgerritMerged openstack/cinder: Add 'conf' param for TextGuruMeditation autorun setup  https://review.openstack.org/28081519:01
*** daneyon has quit IRC19:01
*** jgregor has joined #openstack-cinder19:02
*** andymaier has joined #openstack-cinder19:02
*** apoorvad has joined #openstack-cinder19:04
*** ociuhandu has quit IRC19:09
*** kprabhu has joined #openstack-cinder19:13
*** sheel has quit IRC19:17
*** laughterwym has joined #openstack-cinder19:17
*** alkhodos has joined #openstack-cinder19:19
alkhodosHi guys, if I am planning on adding a new feature, say migrate, to the driver - do I need to create a blueprint for that?19:19
smcginnisalkhodos: I don't require one for driver features.19:20
smcginnisalkhodos: You can submit one if it helps with tracking, but otherwise it's not needed.19:20
*** Suyash has joined #openstack-cinder19:21
alkhodossmcginnis: okay, thank you19:21
*** laughterwym has quit IRC19:22
*** Poornima has quit IRC19:23
*** angela-s has joined #openstack-cinder19:31
openstackgerritVasanthi Thirumalai proposed openstack/cinder: Violin Memory ISCSI storage for 7000 series AFA  https://review.openstack.org/26979619:31
openstackgerritMerged openstack/cinder: Revert "Remove Cisco FC Zone Manager Driver"  https://review.openstack.org/28754119:34
patrickeastanyone know if there are open issues with snapshot-list not respecting the limit parameter?19:37
patrickeastor maybe i'm just doing it wrong...19:38
patrickeasti can't get it to give me more than 1000 at a time from the cli19:38
eharneypatrickeast: i think you have to raise osapi_max_limit on the server19:38
openstackgerritMerged openstack/cinder: Fix call to Barbican Secrets create()  https://review.openstack.org/28781619:39
patrickeasteharney: ahh perfect, thanks!19:39
openstackgerritRyan McNair proposed openstack/cinder: Update quota_utils with import for keystone_auth  https://review.openstack.org/28692219:40
*** daneyon_ has quit IRC19:40
eharneypatrickeast: though i thought pagination was supposed to handle that anyway... so... that seems odd19:40
patrickeasteharney: yea it looks like pagination will limit the number back up until you hit the osapi_max_limit, which then takes over.... guess I could just go look in the code and see what its doing19:43
*** lpetrut has joined #openstack-cinder19:43
*** bjornar has joined #openstack-cinder19:45
*** diablo_rojo has quit IRC19:48
*** jordan__ has quit IRC19:52
openstackgerritMichael Rowden proposed openstack/cinder: Correcting thin provisioning behavior  https://review.openstack.org/27540819:53
*** boris-42 has joined #openstack-cinder19:54
*** rlrossit has quit IRC19:57
*** rlrossit_ has joined #openstack-cinder19:59
*** lcurtis has quit IRC20:00
*** 20WAAG6RV has quit IRC20:01
*** haomaiwang has joined #openstack-cinder20:01
*** porrua has joined #openstack-cinder20:05
*** cebruns has quit IRC20:07
*** cebruns has joined #openstack-cinder20:08
*** breton has quit IRC20:16
*** rlrossit_ has quit IRC20:19
*** eharney has quit IRC20:19
*** leeantho has joined #openstack-cinder20:22
*** chris_morrell has quit IRC20:24
*** IlyaG_ has quit IRC20:24
*** dave-mccowan has quit IRC20:26
*** cdelatte has joined #openstack-cinder20:27
*** e0ne has joined #openstack-cinder20:32
*** Suyash has quit IRC20:32
*** sdague has quit IRC20:45
*** lprice has quit IRC20:47
*** laughterwym has joined #openstack-cinder20:47
*** dave-mccowan has joined #openstack-cinder20:51
*** laughterwym has quit IRC20:52
*** dave-mcc_ has joined #openstack-cinder20:54
*** e0ne has quit IRC20:54
*** esker has joined #openstack-cinder20:54
*** salv-orlando has quit IRC20:55
openstackgerritMerged openstack/cinder: VMware: Fix volume copy across vCenter datacenters  https://review.openstack.org/25829520:55
*** salv-orlando has joined #openstack-cinder20:55
openstackgerritMerged openstack/cinder: Remove an useless and wrong call  https://review.openstack.org/27700920:55
Swanson'an'?20:57
*** dave-mccowan has quit IRC20:57
*** rlrossit_ has joined #openstack-cinder20:59
*** e0ne has joined #openstack-cinder21:00
scottdaSwanson: File a bug: improper use of 'an' in commit message21:00
scottdapeople love it when you do that :)21:00
*** haomaiwang has quit IRC21:01
Swansonscottda, Oh, I know.21:01
*** haomaiwa_ has joined #openstack-cinder21:01
e0neleeantho: hi. just fyi, I've just rebased your patch to master because my one won't be accepted in Mitaka21:02
bswartzcommit messages are unmodifiable for all time though21:02
*** nikeshm has quit IRC21:02
bswartzeternal misspelling 4tw21:03
leeanthoe0ne, ok, I saw the messages a little earlier.  I left a comment because I need to make a small update to the patch.21:03
*** timcl has quit IRC21:04
*** raildo is now known as raildo-afk21:08
openstackgerritIvan Kolodyazhny proposed openstack/python-brick-cinderclient-ext: Require root permissions for CLI operations  https://review.openstack.org/28793821:09
*** esker has quit IRC21:17
*** e0ne has quit IRC21:17
*** e0ne has joined #openstack-cinder21:17
openstackgerritWalter A. Boring IV (hemna) proposed openstack/cinder: 3PAR use same LUN id for each export path  https://review.openstack.org/28688921:18
openstackgerritWalter A. Boring IV (hemna) proposed openstack/os-brick: WIP os-brick refactor get_connector_properties  https://review.openstack.org/27594321:19
*** daneyon has joined #openstack-cinder21:21
*** leeantho has quit IRC21:24
*** leeantho has joined #openstack-cinder21:27
*** e0ne has quit IRC21:28
*** crose has quit IRC21:30
*** dave-mcc_ has quit IRC21:31
*** cknight has quit IRC21:34
*** e0ne has joined #openstack-cinder21:37
*** jungleboyj has quit IRC21:39
*** dave-mccowan has joined #openstack-cinder21:40
*** laughterwym has joined #openstack-cinder21:42
openstackgerritVincent Hou proposed openstack/cinder: Storwize: Update replication to v2.1  https://review.openstack.org/28748321:44
*** rcernin has quit IRC21:45
*** laughterwym has quit IRC21:46
*** jgregor has quit IRC21:52
*** edtubill has joined #openstack-cinder21:56
*** jgregor has joined #openstack-cinder21:57
*** kprabhu has quit IRC21:57
*** jgregor has quit IRC21:58
*** haomaiwa_ has quit IRC22:01
*** haomaiwang has joined #openstack-cinder22:01
*** alyson_ has quit IRC22:02
openstackgerritKurt Martin proposed openstack/cinder: Don't fail on clearing 3PAR object volume key  https://review.openstack.org/28750222:02
*** akerr has quit IRC22:07
*** arch-nemesis has quit IRC22:08
*** leeantho has quit IRC22:09
*** lpetrut has quit IRC22:11
*** vilobhmm11 has quit IRC22:11
*** vilobhmm11 has joined #openstack-cinder22:11
*** ociuhandu has joined #openstack-cinder22:14
*** jgregor has joined #openstack-cinder22:17
mtaninoe0ne: thank you & good morning :)22:20
*** knikolla has quit IRC22:20
e0nemtanino: hm... 12am... not really a morning:)22:20
mtaninooops,,,22:21
e0neneed to take a break. I'll be back in ~12 hours22:23
smcginnise0ne: That all depends if you stay up late or get up early. ;)22:23
e0nesmcginnis: I like to call 'morning' that take, when I wake up22:24
e0nesmcginnis: the same thing for "night" word22:24
e0nesmcginnis: are you going to cut python-brick-cinderclient-ext for Mitaka today or tomorrow?22:27
*** edmondsw has quit IRC22:28
*** baumann has quit IRC22:29
smcginnise0ne: Yep, it actually just went throug. p-b-c-e 0.1.0 is official.22:31
openstackgerritMerged openstack/cinder: VMAX-Replacing deprecated API EMCGetTargetEndpoints  https://review.openstack.org/24432822:31
e0nesmcginnis: thanks!22:32
smcginnise0ne: Thank you for getting it there.22:32
e0nesmcginnis: I'll do my best, to implement more needed and useful features in N22:33
openstackgerritMerged openstack/cinder: Remove a vol in error state from a CG  https://review.openstack.org/27046022:33
e0neseems that we know what we want and it's great22:33
smcginnise0ne: I do think it will be very useful. Lot's of good things to come, I'm sure.22:34
*** e0ne has quit IRC22:35
*** andymaier_ has joined #openstack-cinder22:35
*** tpsilva has quit IRC22:37
hemnaI think we should rename Newton to Ni!22:37
hemnahttps://youtu.be/crX4E-dul4Y?t=31s22:37
*** jgregor has quit IRC22:38
*** andymaier has quit IRC22:38
*** rlrossit_ has quit IRC22:38
*** daneyon_ has joined #openstack-cinder22:40
*** mriedem has quit IRC22:41
*** daneyon has quit IRC22:43
*** lprice has joined #openstack-cinder22:44
*** martyturner has quit IRC22:45
*** mmandel has quit IRC22:47
*** porrua has quit IRC22:48
*** lprice1 has joined #openstack-cinder22:49
mtaninoCould any core push this bug fix? This always prevents my migration & retype testing...  https://review.openstack.org/#/c/276747/22:50
*** lprice has quit IRC22:51
*** edtubill has quit IRC22:58
*** haomaiwang has quit IRC23:01
*** haomaiwang has joined #openstack-cinder23:01
*** akerr has joined #openstack-cinder23:02
*** vilobhmm11 has quit IRC23:04
*** vilobhmm11 has joined #openstack-cinder23:04
*** vilobhmm11 has quit IRC23:04
*** vilobhmm11 has joined #openstack-cinder23:04
*** dustins has quit IRC23:05
*** akerr_ has joined #openstack-cinder23:05
*** xyang1 has quit IRC23:06
openstackgerritMerged openstack/cinder: Fix invalid uuid warnings in test_volume.py  https://review.openstack.org/28804723:06
*** xyang1 has joined #openstack-cinder23:07
*** marcusvrn_ has quit IRC23:07
*** akerr has quit IRC23:09
*** xyang1 has quit IRC23:09
*** cdelatte has quit IRC23:14
*** cdelatte has joined #openstack-cinder23:14
*** akerr_ has quit IRC23:16
*** haplo37 has quit IRC23:18
mtaninopatrickeast: thank you for pushing that bug fix :)23:19
*** Julien-zte has joined #openstack-cinder23:29
*** Julien-zte has quit IRC23:29
*** laughterwym has joined #openstack-cinder23:30
*** laughterwym has quit IRC23:31
*** laughterwym has joined #openstack-cinder23:31
*** neophy has quit IRC23:34
*** haomaiwang has quit IRC23:40
*** knikolla has joined #openstack-cinder23:44
*** Suyash has joined #openstack-cinder23:47
*** laughterwym has quit IRC23:50
*** angela-s has quit IRC23:50
*** knikolla has quit IRC23:51
*** laughterwym has joined #openstack-cinder23:51
*** salv-orl_ has joined #openstack-cinder23:52
*** cdelatte has quit IRC23:52
*** laughterwym has quit IRC23:55
*** salv-orlando has quit IRC23:55

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