Wednesday, 2014-06-18

marunanybody with ml2 chops here?00:00
*** dvorkinista has quit IRC00:00
*** Sukhdev has quit IRC00:01
*** dvorkini_ has quit IRC00:01
*** banix has joined #openstack-neutron00:02
openstackgerritSalvatore Orlando proposed a change to openstack/neutron: ML2: Prevent lock wait timeout in update_port  https://review.openstack.org/9135800:02
openstackgerritSalvatore Orlando proposed a change to openstack/neutron: Prevent lock wait timeout on floating ip ops  https://review.openstack.org/10072400:02
*** y_fujioka has joined #openstack-neutron00:03
*** carlp has quit IRC00:05
*** trad511 has quit IRC00:09
*** y_fujioka has quit IRC00:10
*** marun has quit IRC00:11
*** marun has joined #openstack-neutron00:11
*** marun has quit IRC00:12
*** Sukhdev has joined #openstack-neutron00:13
*** nati_ueno has quit IRC00:14
*** mwagner_lap has joined #openstack-neutron00:14
*** nlahouti has quit IRC00:21
*** manishg has joined #openstack-neutron00:24
*** tomoe_ has quit IRC00:26
*** karimb has quit IRC00:27
*** Jay__ has quit IRC00:28
kevinbentonsalv-orlando: ping00:29
*** manishg has quit IRC00:30
*** a_le_ has quit IRC00:31
*** a_le has joined #openstack-neutron00:32
openstackgerritKevin Benton proposed a change to openstack/neutron: Big Switch: Remove unnecessary initialization code  https://review.openstack.org/10073100:34
*** spandhe has quit IRC00:39
*** kopparam has joined #openstack-neutron00:40
*** Jay__ has joined #openstack-neutron00:40
*** spandhe has joined #openstack-neutron00:41
*** yamahata has quit IRC00:41
*** trinaths has joined #openstack-neutron00:42
openstackgerritOpenStack Proposal Bot proposed a change to openstack/neutron: Updated from global requirements  https://review.openstack.org/9976200:43
*** [1]trinaths has joined #openstack-neutron00:44
*** kopparam has quit IRC00:45
*** a_le has quit IRC00:45
*** trinaths has quit IRC00:47
*** [1]trinaths is now known as trinaths00:47
openstackgerritOpenStack Proposal Bot proposed a change to openstack/python-neutronclient: Updated from global requirements  https://review.openstack.org/9626700:47
*** bandarji has quit IRC00:49
*** rwsu has quit IRC00:58
openstackgerritPaul Michali proposed a change to openstack/neutron: VPNaaS: Allow vendor extend/override valiadation  https://review.openstack.org/9694601:01
*** pcm_ has joined #openstack-neutron01:01
*** a_le has joined #openstack-neutron01:01
*** lykinsbd has quit IRC01:03
*** trinaths has quit IRC01:03
*** lykinsbd has joined #openstack-neutron01:03
*** ranger81_ has joined #openstack-neutron01:05
*** sweston has quit IRC01:06
*** ranger81 has quit IRC01:06
*** lykinsbd has quit IRC01:07
*** prad_ has quit IRC01:09
*** flwang has quit IRC01:10
*** gildub has quit IRC01:11
*** lykinsbd has joined #openstack-neutron01:12
*** gildub has joined #openstack-neutron01:14
*** leenheer has joined #openstack-neutron01:15
*** sbfox has joined #openstack-neutron01:17
*** pcm_ has quit IRC01:17
*** chuckC has quit IRC01:17
*** matsuhashi has joined #openstack-neutron01:17
*** leenheer has quit IRC01:21
*** iwamoto has joined #openstack-neutron01:22
*** salv-orlando has quit IRC01:25
*** yamahata has joined #openstack-neutron01:26
*** _cjones_ has quit IRC01:31
*** _cjones_ has joined #openstack-neutron01:31
*** xuhanp has joined #openstack-neutron01:32
*** zhipeng has quit IRC01:34
*** _cjones_ has quit IRC01:35
*** Sukhdev has quit IRC01:37
*** chuckC has joined #openstack-neutron01:38
openstackgerrityangxurong proposed a change to openstack/neutron: Invalid tunnel network shouldn't be created  https://review.openstack.org/8998201:38
*** sweston has joined #openstack-neutron01:42
*** tomoe_ has joined #openstack-neutron01:42
*** a_le has quit IRC01:43
*** a_le has joined #openstack-neutron01:43
*** popow1 has joined #openstack-neutron01:47
*** popow has quit IRC01:48
*** a_le has quit IRC01:49
*** ramishra has joined #openstack-neutron01:50
*** WackoRobie has joined #openstack-neutron01:50
*** dims has quit IRC01:58
openstackgerrityangxurong proposed a change to openstack/neutron-specs: Ml2 mechanism driver for Huawei SDN controller.  https://review.openstack.org/9749001:58
*** sbalukoff has joined #openstack-neutron02:00
*** _cjones_ has joined #openstack-neutron02:03
*** sbfox has quit IRC02:03
*** 77CAADSW3 has quit IRC02:06
*** xuhanp has quit IRC02:08
*** ranger81_ has quit IRC02:09
*** ranger81 has joined #openstack-neutron02:10
openstackgerrityangxurong proposed a change to openstack/neutron-specs: Ml2 mechanism driver for Huawei SDN controller.  https://review.openstack.org/9749002:11
*** otherwiseguy has quit IRC02:11
*** sweston has quit IRC02:12
*** popow has joined #openstack-neutron02:13
*** _cjones_ has quit IRC02:13
*** xuhanp has joined #openstack-neutron02:14
*** ranger81 has quit IRC02:15
*** xuhanp has quit IRC02:15
*** popow1 has quit IRC02:16
*** mestery has joined #openstack-neutron02:19
*** dims has joined #openstack-neutron02:24
*** coolsvap|afk is now known as coolsvap02:25
*** networkstatic has joined #openstack-neutron02:27
*** SumitNaiksatam has joined #openstack-neutron02:28
*** dims has quit IRC02:28
*** matsuhashi has quit IRC02:29
*** nplanel has quit IRC02:30
*** german_ has joined #openstack-neutron02:30
*** matsuhashi has joined #openstack-neutron02:32
*** WackoRobie has quit IRC02:33
*** WackoRobie has joined #openstack-neutron02:34
*** zhipeng has joined #openstack-neutron02:36
*** Sukhdev has joined #openstack-neutron02:38
*** WackoRobie has quit IRC02:38
*** suresh12 has quit IRC02:39
*** Jay__ has quit IRC02:39
*** gildub has quit IRC02:42
*** popow1 has joined #openstack-neutron02:42
*** popow has quit IRC02:44
*** zhiyan_ is now known as zhiyan02:44
*** harlowja is now known as harlowja_away02:48
*** emagana has joined #openstack-neutron02:49
*** seizadi has joined #openstack-neutron02:52
*** yamamoto_ has joined #openstack-neutron02:54
*** sweston has joined #openstack-neutron02:57
*** vivekjain has joined #openstack-neutron02:58
*** sbfox has joined #openstack-neutron02:59
*** matsuhashi has quit IRC03:00
*** vivekjain has quit IRC03:02
*** spandhe has quit IRC03:03
*** essessv has joined #openstack-neutron03:07
*** gildub has joined #openstack-neutron03:09
*** trinaths has joined #openstack-neutron03:11
*** Jay__ has joined #openstack-neutron03:11
*** banix has quit IRC03:11
*** Jay__ has quit IRC03:15
*** changbl has joined #openstack-neutron03:19
*** ranger81 has joined #openstack-neutron03:25
*** sweston has quit IRC03:31
*** sunrenjie6 has joined #openstack-neutron03:33
*** seizadi has quit IRC03:34
*** lykinsbd has quit IRC03:34
*** lykinsbd has joined #openstack-neutron03:35
*** SridharG has joined #openstack-neutron03:37
*** sweston has joined #openstack-neutron03:37
*** lykinsbd has quit IRC03:39
*** sunrenjie6 has quit IRC03:39
*** sunrenjie6 has joined #openstack-neutron03:40
*** ranger81 has quit IRC03:42
*** sunrenjie6 has quit IRC03:44
*** nlahouti has joined #openstack-neutron03:47
*** nlahouti has quit IRC03:47
*** nlahouti has joined #openstack-neutron03:48
*** suresh12 has joined #openstack-neutron03:50
*** leenheer has joined #openstack-neutron03:52
openstackgerritshihanzhang proposed a change to openstack/neutron-specs: Spec for adding ipset to security group  https://review.openstack.org/10076103:52
*** matsuhashi has joined #openstack-neutron03:53
*** suresh12 has quit IRC03:54
*** lykinsbd has joined #openstack-neutron03:55
*** lykinsbd has quit IRC03:56
openstackgerritshihanzhang proposed a change to openstack/neutron-specs: Spec for adding ipset to security group  https://review.openstack.org/10076103:58
*** mestery has quit IRC04:00
openstackgerritshihanzhang proposed a change to openstack/neutron-specs: Spec for adding ipset to security group  https://review.openstack.org/10076104:01
*** yfried_ has quit IRC04:02
*** mestery has joined #openstack-neutron04:02
*** mestery has quit IRC04:04
*** _cjones_ has joined #openstack-neutron04:04
*** seizadi has joined #openstack-neutron04:05
*** _cjones_ has quit IRC04:08
*** sbfox has quit IRC04:09
*** ranger81 has joined #openstack-neutron04:11
*** Jay__ has joined #openstack-neutron04:12
*** matsuhashi has quit IRC04:12
*** amotoki has joined #openstack-neutron04:12
*** matsuhashi has joined #openstack-neutron04:12
*** german_ has quit IRC04:13
*** Jay__ has quit IRC04:16
*** matsuhashi has quit IRC04:17
*** dguitarbite has joined #openstack-neutron04:19
*** jgrimm has joined #openstack-neutron04:19
*** matsuhashi has joined #openstack-neutron04:21
*** Longgeek has joined #openstack-neutron04:26
*** dims has joined #openstack-neutron04:27
*** essessv has quit IRC04:31
*** dims has quit IRC04:32
*** garyk has quit IRC04:35
*** matsuhashi has quit IRC04:36
*** matsuhashi has joined #openstack-neutron04:36
*** Sukhdev has quit IRC04:38
*** lori|away is now known as lori04:39
*** matsuhashi has quit IRC04:39
*** matsuhashi has joined #openstack-neutron04:39
*** yfried_ has joined #openstack-neutron04:39
*** seizadi has quit IRC04:40
*** sbfox has joined #openstack-neutron04:43
*** dims has joined #openstack-neutron04:46
*** seizadi has joined #openstack-neutron04:47
*** dims has quit IRC04:51
*** ranger81 has quit IRC04:57
*** ranger81 has joined #openstack-neutron04:57
openstackgerritshihanzhang proposed a change to openstack/neutron-specs: Spec for adding ipset to security group  https://review.openstack.org/10076104:59
*** ramishra has quit IRC05:00
*** rdo has quit IRC05:00
*** ramishra has joined #openstack-neutron05:00
*** rdo has joined #openstack-neutron05:01
*** ramishra has quit IRC05:03
*** ramishra has joined #openstack-neutron05:03
*** matsuhashi has quit IRC05:11
*** matsuhashi has joined #openstack-neutron05:11
*** emagana has quit IRC05:12
*** Jay__ has joined #openstack-neutron05:12
*** dguitarbite has quit IRC05:13
*** ranger81 has quit IRC05:16
*** matsuhashi has quit IRC05:16
*** matsuhashi has joined #openstack-neutron05:17
*** Jay__ has quit IRC05:17
*** ranger81 has joined #openstack-neutron05:17
*** ranger81 has quit IRC05:17
*** ranger81 has joined #openstack-neutron05:18
*** dguitarbite has joined #openstack-neutron05:18
*** spandhe has joined #openstack-neutron05:21
*** spandhe_ has joined #openstack-neutron05:22
*** ranger81 has quit IRC05:22
*** ihrachyshka has joined #openstack-neutron05:23
*** matsuhashi has quit IRC05:24
*** ihrachyshka has quit IRC05:24
*** matsuhashi has joined #openstack-neutron05:24
*** kopparam has joined #openstack-neutron05:24
*** spandhe has quit IRC05:25
*** spandhe_ is now known as spandhe05:25
openstackgerritA change was merged to openstack/neutron: Big Switch: Import DB module in unit test  https://review.openstack.org/10072005:26
*** seizadi has quit IRC05:34
*** a_le has joined #openstack-neutron05:45
*** seizadi has joined #openstack-neutron05:46
*** dims has joined #openstack-neutron05:47
*** lukego has joined #openstack-neutron05:51
*** rcarrillocruz has joined #openstack-neutron05:51
*** dims has quit IRC05:51
*** irenab has joined #openstack-neutron05:53
*** rcarrill` has quit IRC05:53
*** afazekas_ has joined #openstack-neutron05:54
*** alex_klimov has joined #openstack-neutron05:56
*** a_le has quit IRC06:01
*** a_le has joined #openstack-neutron06:01
*** SridharG has quit IRC06:02
*** kopparam has quit IRC06:02
*** zhipeng has quit IRC06:03
*** _cerberus_ has quit IRC06:04
*** wendar has quit IRC06:04
*** wendar_ has joined #openstack-neutron06:04
*** _cjones_ has joined #openstack-neutron06:05
*** a_le has quit IRC06:05
*** kopparam has joined #openstack-neutron06:05
*** _cerberus_ has joined #openstack-neutron06:05
openstackgerritLi Ma proposed a change to openstack/neutron: Race condition of L3-agent to add/remove routers  https://review.openstack.org/7323406:06
*** spandhe has quit IRC06:07
openstackgerritOpenStack Proposal Bot proposed a change to openstack/neutron: Imported Translations from Transifex  https://review.openstack.org/10042906:09
*** alex_klimov has quit IRC06:09
*** devvesa_ has joined #openstack-neutron06:09
*** _cjones_ has quit IRC06:10
openstackgerritpritesh proposed a change to openstack/neutron: Remove vim editor configuration from comments  https://review.openstack.org/9950806:10
*** zhipeng has joined #openstack-neutron06:12
*** Jay__ has joined #openstack-neutron06:13
*** doude has joined #openstack-neutron06:18
*** Jay__ has quit IRC06:18
*** garyk has joined #openstack-neutron06:19
*** lukego has quit IRC06:19
*** dguitarbite has quit IRC06:19
*** nlahouti has quit IRC06:19
*** gildub has quit IRC06:19
*** jckasper_ has quit IRC06:19
*** ajo has quit IRC06:19
*** jamespd has quit IRC06:19
*** openstackgerrit has quit IRC06:19
*** dguitarbite has joined #openstack-neutron06:21
*** amuller has joined #openstack-neutron06:22
*** alex_klimov has joined #openstack-neutron06:22
*** lukego has joined #openstack-neutron06:23
*** nlahouti has joined #openstack-neutron06:23
*** gildub has joined #openstack-neutron06:23
*** jckasper_ has joined #openstack-neutron06:23
*** ajo has joined #openstack-neutron06:23
*** jamespd has joined #openstack-neutron06:23
*** openstackgerrit has joined #openstack-neutron06:23
*** kopparam is now known as vm06:24
*** iwamoto has quit IRC06:25
*** vm is now known as kopparam06:25
*** seizadi has quit IRC06:29
*** zhipeng has quit IRC06:39
*** kong has joined #openstack-neutron06:42
*** zhipeng has joined #openstack-neutron06:43
kongspec review request: https://review.openstack.org/#/c/98966/, thanks very much!06:44
openstackgerritJaume Devesa proposed a change to openstack/neutron: Autocreated ports count against port quota  https://review.openstack.org/9967906:44
*** dims_ has joined #openstack-neutron06:47
*** moha_hunt has joined #openstack-neutron06:48
*** ramishra has quit IRC06:48
*** ramishra has joined #openstack-neutron06:51
*** kopparam has quit IRC06:51
*** kopparam has joined #openstack-neutron06:52
openstackgerrityangxurong proposed a change to openstack/neutron: Invalid tunnel network shouldn't be created  https://review.openstack.org/8998206:52
*** dims_ has quit IRC06:53
openstackgerritZhang Hua proposed a change to openstack/neutron: IPsec strongswan driver implemention  https://review.openstack.org/10079106:55
*** samuelbercovici has joined #openstack-neutron06:56
*** [1]evgenyf has joined #openstack-neutron06:58
*** nlahouti has quit IRC07:00
*** sunrenjie6 has joined #openstack-neutron07:00
*** sunrenjie6 has quit IRC07:01
*** zhipeng has quit IRC07:01
*** sunrenjie6 has joined #openstack-neutron07:01
*** kopparam has quit IRC07:02
*** kopparam has joined #openstack-neutron07:03
*** moha_hunt has quit IRC07:03
*** ygbo has quit IRC07:05
*** samuelbercovici1 has joined #openstack-neutron07:05
*** zhipeng has joined #openstack-neutron07:06
*** garyk1 has joined #openstack-neutron07:06
*** ygbo has joined #openstack-neutron07:06
*** kopparam has quit IRC07:07
*** garyk has quit IRC07:08
*** matsuhashi has quit IRC07:08
*** matsuhas_ has joined #openstack-neutron07:08
*** samuelbercovici has quit IRC07:09
*** samuelbercovici1 is now known as samuelbercovici07:09
*** Asrivastava has joined #openstack-neutron07:09
AsrivastavaHi all, I want my neutron ml2_plyugin to work for both KVM and Hyperv. But both are not working suimultaneously. mechanism_driver either work on "hyperv" or "openvswitch". The comma separated value which is mentioned as an ordered list doesn't work or it could be a configuration issue. ML2 claims that it supports multiple mechanism drivers simultaneously07:09
openstackgerrityangxurong proposed a change to openstack/neutron: Invalid tunnel network shouldn't be created  https://review.openstack.org/8998207:10
*** leenheer has quit IRC07:10
openstackgerritpritesh proposed a change to openstack/neutron: Remove old quantum scripts  https://review.openstack.org/10079507:12
*** xianghui^ has joined #openstack-neutron07:12
*** Jay__ has joined #openstack-neutron07:14
*** Jay__ has quit IRC07:15
*** Jay__ has joined #openstack-neutron07:16
openstackgerritSami J. Mäkinen proposed a change to openstack/neutron-specs: Implements: blueprint neutron-ml2-mech-vmware-dvswitch Closes-Bug: NA  https://review.openstack.org/10079607:16
*** xianghui has quit IRC07:16
*** chandan_kumar has quit IRC07:17
*** sunrenjie6 has quit IRC07:17
*** ramishra has quit IRC07:18
*** sunrenjie6 has joined #openstack-neutron07:18
*** Jay__ has quit IRC07:20
*** jroovers has joined #openstack-neutron07:20
*** salv-orlando has joined #openstack-neutron07:21
*** sunrenjie6 has quit IRC07:22
*** jroovers2 has joined #openstack-neutron07:23
*** paraa has joined #openstack-neutron07:24
*** jgallard has joined #openstack-neutron07:24
*** jroovers has quit IRC07:24
*** doude has quit IRC07:25
*** kickinz1|afk has quit IRC07:28
*** kickinz1|afk has joined #openstack-neutron07:28
openstackgerritKevin Benton proposed a change to openstack/neutron: Big Switch ML2: sync detection in port-update  https://review.openstack.org/10079807:29
*** nlahouti has joined #openstack-neutron07:29
*** samuelbercovici1 has joined #openstack-neutron07:31
*** xuhanp has joined #openstack-neutron07:32
*** zhipeng has quit IRC07:33
*** samuelbercovici has quit IRC07:34
*** samuelbercovici1 is now known as samuelbercovici07:35
*** gildub has quit IRC07:35
*** luqas has joined #openstack-neutron07:35
*** zzelle has joined #openstack-neutron07:37
*** nlahouti has quit IRC07:37
*** kickinz1|afk is now known as kickinz107:38
*** jlibosva has joined #openstack-neutron07:38
*** jgallard has quit IRC07:40
*** jgallard has joined #openstack-neutron07:41
*** popow has joined #openstack-neutron07:41
*** trinaths has quit IRC07:41
*** kopparam has joined #openstack-neutron07:44
*** popow1 has quit IRC07:45
openstackgerritKevin Benton proposed a change to openstack/neutron-specs: Allow router external IPs to be set and read  https://review.openstack.org/9501607:46
openstackgerritSami J. Mäkinen proposed a change to openstack/neutron-specs: VMware dvSwitch/vSphere API support for Neutron ML2  https://review.openstack.org/10080307:48
openstackgerritSalvatore Orlando proposed a change to openstack/neutron: ML2: Prevent lock wait timeout in update_port  https://review.openstack.org/9135807:48
openstackgerrityangxurong proposed a change to openstack/neutron: Implement bp/huawei-ml2-mechanism-driver  https://review.openstack.org/6814807:50
openstackgerrityangxurong proposed a change to openstack/neutron-specs: Ml2 mechanism driver for Huawei SDN controller.  https://review.openstack.org/9749007:53
openstackgerritSalvatore Orlando proposed a change to openstack/neutron: Prevent lock wait timeout on floating ip ops  https://review.openstack.org/10072407:54
*** networkstatic has quit IRC07:54
*** ihrachyshka has joined #openstack-neutron07:54
*** Yalei has joined #openstack-neutron07:55
*** coolsvap is now known as coolsvap|afk07:55
*** moha_hunt has joined #openstack-neutron07:55
openstackgerritSami J. Mäkinen proposed a change to openstack/neutron-specs: VMware dvSwitch/vSphere API support for Neutron ML2  https://review.openstack.org/10080807:57
*** jpich has joined #openstack-neutron07:57
ihrachyshkasalv-orlando: hey. I want to briefly discuss db locks again, I think I've found an instance of unsafe code under transaction that could result in yield when deleting a port07:58
*** zzelle has quit IRC07:58
*** zzelle has joined #openstack-neutron07:59
openstackgerritSami J. Mäkinen proposed a change to openstack/neutron-specs: VMware dvSwitch/vSphere API support for Neutron ML2  https://review.openstack.org/10081007:59
*** tomoe__ has joined #openstack-neutron08:00
*** yamamoto_ has quit IRC08:01
*** amaretskiy has joined #openstack-neutron08:01
*** leseb has joined #openstack-neutron08:02
*** safchain has joined #openstack-neutron08:02
*** jistr has joined #openstack-neutron08:03
salv-orlandoihrachyshka: go ahead08:03
*** tomoe_ has quit IRC08:04
*** nlahouti has joined #openstack-neutron08:04
*** tomoe_ has joined #openstack-neutron08:04
*** kopparam has quit IRC08:05
*** roeyc has joined #openstack-neutron08:05
ihrachyshkasalv-orlando: so, this is for floating IPs. when we delete_port(), we disassociate_floatingips(), and we call it under subtransaction. And in disassociate_floatingips(), we call to self.l3_rpc_notifier.routers_updated(...), which is RPC network interaction, which is probably a way to yield.08:05
*** yamahata has quit IRC08:05
ihrachyshkathis is for ML208:05
*** tomoe__ has quit IRC08:05
*** kopparam has joined #openstack-neutron08:05
*** yamahata has joined #openstack-neutron08:05
*** leseb has quit IRC08:06
ihrachyshkasalv-orlando: does reasoning look sane to you?08:07
*** leseb has joined #openstack-neutron08:07
*** kopparam has quit IRC08:07
*** kopparam has joined #openstack-neutron08:08
salv-orlandoihrachyshka: makes sense to me.08:08
*** nlahouti has quit IRC08:09
ihrachyshkasalv-orlando: and since floating_ip  status update is not locked with db-access, it proceeds with its own subtransaction and lock-timeouts08:09
ihrachyshkasalv-orlando: good, thanks for checking. now the question is how to fix it correct ;)08:09
salv-orlandoI will add a lock on the floating ip resource in disassociate_floating_ips for https://review.openstack.org/#/c/10072408:10
salv-orlandothe question is that we’re potentially exposing now to deadlocks (see comments on related bug)08:11
salv-orlandoand we need to do a bit of reasonign to check we won’t have deadlocks in this case.08:11
*** doude has joined #openstack-neutron08:12
*** kopparam has quit IRC08:12
ihrachyshkasalv-orlando: honestly, I don't think this is the right way to handle it (or better - should not be the final fix)08:12
ihrachyshkasalv-orlando: since we now see a cause, we're better fixing it instead of wrapping into yet another lock08:12
salv-orlandowe’ve discussed with a few people the removal of code which might trigger yields08:13
salv-orlandoturned out to be too many things08:13
salv-orlandoeven a LOG will yield.08:13
ihrachyshkayes, I know, but only if logging remotely I guess08:13
salv-orlandonope.08:13
ihrachyshkafile access does not yield I guess08:13
salv-orlandolog yields because eventlet monkeypatches os to add a yield.08:14
*** amotoki has quit IRC08:14
salv-orlandoanyway I’m ok with abandoning that patch. I do not feel strongly.08:14
ihrachyshkadoes it patch file os.write?08:14
salv-orlandoI don’t remember the details, but it should not be hard to check.08:14
*** moha_hunt has quit IRC08:16
*** Jay__ has joined #openstack-neutron08:16
*** samuelbercovici1 has joined #openstack-neutron08:18
*** zzelle has quit IRC08:18
ihrachyshkasalv-orlando: as per https://github.com/eventlet/eventlet/blob/master/eventlet/green/os.py, it yields only if getting EAGAIN or EPIPE from read/write syscall08:18
*** ramishra has joined #openstack-neutron08:18
*** zzelle has joined #openstack-neutron08:19
ihrachyshkaand this is probably impossible if fd is not a pipe or a network socket08:19
ihrachyshka[not that I want to say that logging to network is smth we don't support...]08:19
*** moha_hunt has joined #openstack-neutron08:20
*** Jay__ has quit IRC08:21
*** sbfox has quit IRC08:21
salv-orlandoihrachyshka: cool, so for you issue all you need to do is move the rpc notifcation outside the transaction?08:21
*** samuelbercovici has quit IRC08:21
*** samuelbercovici1 is now known as samuelbercovici08:22
ihrachyshkasalv-orlando: yes, should be enough. the problem is that notification occurs in a function that we call from another transaction, so probably we need to move call to disassociate_floatingips() from the root transaction.08:22
ihrachyshkadisassociate_floatingips() is fine itself, what's wrong is how we call to it (from transaction)08:23
salv-orlandook I.m abandoning my patch then.08:23
salv-orlandoI think we do that for disassociatE_floating_ips because it does DB changes too.08:23
ihrachyshkasalv-orlando: I will try to test before sending the patch, our QA said it's easily reproducible08:23
ihrachyshkasalv-orlando: disassociate_floatingips() already opens its own transaction for db access08:24
salv-orlandoAnd originally the developers did not want to have multiple transactions to avoid dealing with partially changed state upon failures08:24
salv-orlandohave you noticed it uses subtransactions?08:24
ihrachyshkasalv-orlando: we may postpone notification (move it to a separate function and call to it from both delete_port() and disassociate_floatingips() passing things to notify08:24
salv-orlandoit is *not* a distinct transaction08:25
salv-orlandoyeah you can do that I guess08:25
ihrachyshkaok, I need to come out saying that I don't know how db transactions work in our case08:25
salv-orlandoit’s just that we use generally subtransactions so as long as the ctx managers are nested all is done in a signel sql transaction08:26
salv-orlandoNow it;s not mandatory to have a single transaction for each API call08:26
salv-orlandothere are plenty of cases where this is not the right thing to do08:27
*** boumbastic has joined #openstack-neutron08:27
*** boumbastic is now known as karim08:28
*** Demitar has joined #openstack-neutron08:28
salv-orlandohowever, in this specific case, if we split the two transactions - if we disassociate floating ips, and then delete_port fails for some reason - we should reassociate the floating IP for keeping the status consistent (a failed API operation ideally should not change the state)08:28
ihrachyshkawhat does this 'subtransactions=True' argument really mean? Is it 'open transaction if it's not done already'? or 'remember the state at this point to be able to roll back to it in case of failure while leaving nesting transaction running'?08:28
*** karim is now known as Guest4669408:28
salv-orlandosubtransactions basicaly either uses a parent transaction if open or opens a new one.08:29
*** sunrenjie6 has joined #openstack-neutron08:29
ihrachyshkaok, then it's 'open transaction if it's not done already', ok08:29
*** kopparam has joined #openstack-neutron08:30
*** Guest46694 is now known as karimb08:30
salv-orlandoanyway, when it comes to API operations and atomicity it’s really not an exact science. I’m pretty sure there are other groups which will make an argument that it is ok for failing operations to partially update the state of a resource.08:32
*** amotoki has joined #openstack-neutron08:32
*** luqas has quit IRC08:33
*** samuelbercovici1 has joined #openstack-neutron08:35
kevinbentonsalv-orlando: I have issues :-)08:37
kevinbentonsalv-orlando: related to this conversation somewhat08:37
kevinbentonsalv-orlando: when I call network.db.api’s get_session() and then run a select statement using that session, it will mess up other transactions in progress08:38
*** samuelbercovici has quit IRC08:39
*** samuelbercovici1 is now known as samuelbercovici08:39
kevinbentonsalv-orlando: seems to cause them to commit in some cases prematurely or not commit at all08:39
salv-orlandoI have no idea.08:39
salv-orlandobut from what you say it seems sqlalchemy is broken?08:39
kevinbentonsalv-orlando: i don’t think so. i think it’s related to using get_session() which translates to the global facade in network.db.api08:41
salv-orlandokevinbenton: I mean, it seems like you’re seeing a violation of the isolation proerty08:41
kevinbentonsalv-orlando: so i end up sharing the same engine as the current transaction08:41
kevinbentonsalv-orlando: the work around i had that was ugly was a completely new engine08:42
kevinbentonsalv-orlando: but that seems like overkill08:42
*** eezhova has joined #openstack-neutron08:42
salv-orlandoI’m afraid I need more pointers to understand exactly what would going on. Notmally it is ok to use the same engine for multiple sessions - and trasnsactions within them08:44
kevinbentonsalv-orlando: https://review.openstack.org/#/c/98949/2/neutron/plugins/bigswitch/db/consistency_db.py08:44
salv-orlandoinstead what you’re witnessing appears to be that committing a transaction on a session triggers commit also on all the other transactions running on other sessions?08:44
kevinbentonsalv-orlando: it’s even worse than that. simply running a query on the result of db.get_session() would break things up in the plugin layer that were in the context.session.begin() contexts08:46
kevinbentonsalv-orlando: no changes, just a query on the model with a filter by id and a call to first()08:46
*** Longgeek has quit IRC08:46
salv-orlandothis might also mean that for every pkugin not just yours, if two api are executed concurrently, one will trigger a commit in the other?08:46
*** Longgeek has joined #openstack-neutron08:46
kevinbentonsalv-orlando: i think it may be isolated better when it comes to multiple threads08:48
*** jp_at_hp has joined #openstack-neutron08:48
kevinbentonsalv-orlando: maybe it would help if i can distill this down to a simple example so you don’t have to get context of everything else in that patch?08:48
*** Longgeek has quit IRC08:48
openstackgerritenikanorov proposed a change to openstack/neutron: Improve docstring for OVSNeutronAgent constructor  https://review.openstack.org/10082208:49
*** Longgeek has joined #openstack-neutron08:49
salv-orlandoI kind of get it… you had to do the consistency hash thing in a distinct session with a distinct engine08:49
*** dims_ has joined #openstack-neutron08:51
salv-orlandoI just think that this problem cannot be specific to your plugin - in the terms you’ve put if affects *all* db operations08:51
openstackgerritElena Ezhova proposed a change to openstack/neutron: Check port value when creating firewall rule with icmp protocol  https://review.openstack.org/9937208:51
*** Longgeek has quit IRC08:53
*** Longgeek_ has joined #openstack-neutron08:53
kevinbentonsalv-orlando: in that case yes, but I was actually willing to sacrifice independent fates of the transactions (a backend failure means no new hash anyway)08:53
kevinbentonsalv-orlando: but I couldn’t get it to work right. let me get back to you with something less convoluted08:53
salv-orlandoI mean - what you’ve found appears pretty serious ;)08:53
kevinbentonsalv-orlando: that’s usually how i know i’m just doing something incorrectly08:54
salv-orlandoas this happens to you in your consistency hashing patch, this could normally happen betwen say.. a get port and an update port operation08:54
*** leseb has quit IRC08:54
salv-orlandokevinbenton: in patch #98949 the old one suffers from this problem and the new not. Is that right?08:55
*** dims_ has quit IRC08:56
kevinbentonsalv-orlando: by old do you mean the version in master?08:56
salv-orlandothe version in master sorry - get_consistency_hash causes other transactions to commit as well08:56
kevinbentonno, because i only call get_consistency_hash during plugin startup08:57
salv-orlandoyup, saw that08:57
*** ghyoc has joined #openstack-neutron08:58
kevinbentonIIRC I can actually duplicate the problem by calling get_consistency_hash08:58
kevinbentonone sec08:58
salv-orlandook08:59
openstackgerritKevin Benton proposed a change to openstack/neutron: Code for demonstation purposes. DO NOT REVIEW  https://review.openstack.org/10082609:01
kevinbentonsalv-orlando: ^^09:01
kevinbentonsalv-orlando: in that particular case it will break some of the rollback tests09:01
*** matsuhas_ has quit IRC09:01
kevinbentonsalv-orlando: tox -epy27 neutron.tests.unit.bigswitch.test_restproxy_plugin.TestBigSwitchProxyPortsV2.test_rollback_for_port_update09:02
*** karimb has left #openstack-neutron09:02
kevinbentonsalv-orlando: there are several across port rollbacks and router rollbacks09:03
kevinbentonsalv-orlando: perhaps the fundamental issue is how transactions are being handled in the plugin or something09:03
*** luqas has joined #openstack-neutron09:03
*** nlahouti has joined #openstack-neutron09:05
*** boumbastic has joined #openstack-neutron09:05
openstackgerritA change was merged to openstack/neutron-specs: Allow router external IPs to be set and read  https://review.openstack.org/9501609:06
*** boumbastic has quit IRC09:07
*** boumbastic has joined #openstack-neutron09:07
*** boumbastic has quit IRC09:09
*** matsuhashi has joined #openstack-neutron09:09
*** boumbastic has joined #openstack-neutron09:09
*** nlahouti has quit IRC09:09
*** samuelbercovici1 has joined #openstack-neutron09:11
openstackgerritshihanzhang proposed a change to openstack/neutron-specs: Spec for adding ipset to security group  https://review.openstack.org/10076109:11
salv-orlandokevinbenton: does it rollback only the current transaction or does it rollback pretty much any open transaction?09:11
openstackgerritshihanzhang proposed a change to openstack/neutron-specs: Spec for adding ipset to security group  https://review.openstack.org/10076109:12
*** isviridov|away is now known as isviridov09:13
openstackgerritshihanzhang proposed a change to openstack/neutron-specs: Spec for adding ipset to security group  https://review.openstack.org/10076109:13
*** tomoe_ has quit IRC09:13
*** samuelbercovici has quit IRC09:14
*** samuelbercovici1 is now known as samuelbercovici09:14
*** atmark has joined #openstack-neutron09:14
kevinbentonsalv-orlando: so in the particular port_update test referenced above, the update that receives an error is incorrectly being committed09:14
atmarkhi, i'm getting this error when i try to add a pool in the Load Balancer09:15
atmarkhttp://pastebin.com/raw.php?i=QD8k1shX09:15
kevinbentonsalv-orlando: when it should be rolled back instead09:15
*** ramishra has quit IRC09:16
*** Jay__ has joined #openstack-neutron09:17
*** samuelbercovici1 has joined #openstack-neutron09:19
*** Jay__ has quit IRC09:21
*** samuelbercovici has quit IRC09:22
*** samuelbercovici1 is now known as samuelbercovici09:22
openstackgerritAssaf Muller proposed a change to openstack/neutron: Split API and RPC workers in to one separate process each (WIP)  https://review.openstack.org/7602109:24
openstackgerritAssaf Muller proposed a change to openstack/neutron: Fix ML2, db base plugin concurrent subnet delete (WIP)  https://review.openstack.org/9971109:24
*** ilyashakhat has joined #openstack-neutron09:27
*** xianghui^ has quit IRC09:28
openstackgerritIsaku Yamahata proposed a change to openstack/neutron-specs: API Extension for l2-gateway  https://review.openstack.org/10027809:28
*** geekinut1h has joined #openstack-neutron09:30
*** boumbastic is now known as karim09:31
enikanorovatmark: is your neutron server running?09:31
*** karim is now known as Guest9161709:31
*** Guest91617 is now known as karimb09:32
atmarkyes09:33
*** geekinutah has quit IRC09:33
atmarki have 3 nodes: 1 controller, 1 compute and 1 network node09:33
salv-orlandokevinbenton: are you seeing this in production as well, with mysql/postgre? I’m tempted to blame sqllite for that.09:34
atmarkcontroller - neutron server09:34
atmarknetwork - lbaas-agent, haproxy09:34
*** sunrenjie6 has quit IRC09:35
kevinbentonsalv-orlando: no, i’ve never tried it in production because i couldn’t get the unit tests to work09:35
atmarki have this service_plugins = neutron.services.loadbalancer.plugin.LoadBalancerPlugin on neutron.conf in controller and network09:35
*** sunrenjie6 has joined #openstack-neutron09:35
kevinbentonsalv-orlando: is it possible to back the unit tests with mysql to verify?09:36
*** nlahouti has joined #openstack-neutron09:36
atmarkin the network node, lbaas-agent.ini has these config09:36
*** matsuhashi has quit IRC09:38
*** samuelbercovici1 has joined #openstack-neutron09:39
*** sunrenjie6 has quit IRC09:39
*** nlahouti has quit IRC09:40
openstackgerritSumit Naiksatam proposed a change to openstack/neutron: Group Policy API-1: EP, EPG, L2 Policy, L3 Policy  https://review.openstack.org/9590009:41
*** igordcard has joined #openstack-neutron09:41
openstackgerritshihanzhang proposed a change to openstack/neutron-specs: Spec for adding ipset to security group  https://review.openstack.org/10076109:41
salv-orlandokevinbenton: I’m looking at a few trivial things. I’m following the principle that all weird problems have trivial causes.09:42
*** samuelbercovici has quit IRC09:42
*** samuelbercovici1 is now known as samuelbercovici09:42
*** SridharG has joined #openstack-neutron09:43
atmarkenikanorov:09:46
enikanorovatmark: it's hard to say. some issue with rpc connectivity between lbaas agent and neutron server09:47
kevinbentonsalv-orlando: makes sense.09:49
atmarkenikanorov: yeah, issue is similar to this https://answers.launchpad.net/neutron/+question/23873809:51
*** zhiyan is now known as zhiyan_09:51
atmarki did tried the recommendation but it didn't help09:51
*** dims_ has joined #openstack-neutron09:51
enikanorovatmark: have you tried the suggested workaround?09:52
kevinbentonsalv-orlando: thanks for taking a look. i’m going to bed now. I will look at this again tomorrow09:52
salv-orlandobie good night09:53
*** xianghui^ has joined #openstack-neutron09:53
*** zhipeng has joined #openstack-neutron09:53
atmarkyep but it didn't work09:54
atmarki can see this in lbaas-agent.log09:55
atmarkneutron.openstack.common.rpc.impl_qpid [-] Connected to AMQP server on 10.103.0.2:567209:55
*** dims_ has quit IRC09:56
*** ihrachyshka has quit IRC09:56
*** ramishra has joined #openstack-neutron09:57
*** leseb has joined #openstack-neutron09:58
*** hoangdo has joined #openstack-neutron10:00
*** hoangdo has left #openstack-neutron10:00
*** matsuhashi has joined #openstack-neutron10:01
*** tomoe_ has joined #openstack-neutron10:04
*** matsuhashi has quit IRC10:06
*** nlahouti has joined #openstack-neutron10:07
*** zzelle has quit IRC10:08
*** matsuhashi has joined #openstack-neutron10:08
openstackgerrityangxurong proposed a change to openstack/neutron: Invalid tunnel network shouldn't be created  https://review.openstack.org/8998210:09
*** ramishra has quit IRC10:09
*** nlahouti has quit IRC10:11
*** beagles has quit IRC10:11
*** pcm_ has joined #openstack-neutron10:12
*** leseb has quit IRC10:13
*** tomoe_ has quit IRC10:14
*** pcm_ has quit IRC10:14
*** pcm_ has joined #openstack-neutron10:14
*** b3nt_pin has joined #openstack-neutron10:15
*** matsuhashi has quit IRC10:17
*** matsuhashi has joined #openstack-neutron10:18
*** xuhanp has quit IRC10:18
openstackgerrityangxurong proposed a change to openstack/neutron: Implement bp/huawei-ml2-mechanism-driver  https://review.openstack.org/6814810:18
*** Jay__ has joined #openstack-neutron10:18
openstackgerritFeng Ju proposed a change to openstack/python-neutronclient: Hash X-Auth-Token in neutron client log  https://review.openstack.org/9386610:18
*** kopparam has quit IRC10:20
*** kopparam has joined #openstack-neutron10:21
openstackgerritLi Ma proposed a change to openstack/neutron: Race condition of L3-agent to add/remove routers  https://review.openstack.org/7323410:21
*** Jay__ has quit IRC10:22
*** kopparam has quit IRC10:25
*** jroovers has joined #openstack-neutron10:29
*** xianghui^ has quit IRC10:30
openstackgerritenikanorov proposed a change to openstack/neutron-specs: neutron-flavor-framework  https://review.openstack.org/9007010:30
*** yamahata_ has joined #openstack-neutron10:31
*** jroovers2 has quit IRC10:32
*** WackoRobie has joined #openstack-neutron10:35
*** SridharG has quit IRC10:37
*** kopparam has joined #openstack-neutron10:37
*** nlahouti has joined #openstack-neutron10:37
kevinbentonsalv-orlando: ok, going to bed for real this time :-)10:39
kevinbentonsalv-orlando: but i think i’ve distilled it into something easy to understand now10:39
salv-orlandoI’ve commented on the test patch you put on gerrit10:39
openstackgerritKevin Benton proposed a change to openstack/neutron: Code for demonstation purposes. DO NOT REVIEW  https://review.openstack.org/10082610:39
salv-orlandoah I see10:40
kevinbentonyes, i was afraid that would be the answer10:40
*** jgallard has quit IRC10:42
*** nlahouti has quit IRC10:42
openstackgerritenikanorov proposed a change to openstack/neutron-specs: neutron-flavor-framework  https://review.openstack.org/9007010:43
salv-orlandokevinbenton: maybe there are ways to use two sessions but I think it’s not worth10:43
enikanorovhow can one not review the patch which has DO NOT REVIEW in the title?10:43
*** dims_ has joined #openstack-neutron10:43
kevinbentonenikanorov: with a lot of willpower :-)10:44
enikanorovi've never had it enough for that :)10:44
*** Asrivastava has quit IRC10:45
kevinbentonsalv-orlando: the tentacles of tight coupling are strangling my servermanager module!10:45
kevinbentonsalv-orlando: maybe time for a refactor then10:45
kevinbentonsalv-orlando: well thanks for the pointer. I’ll read more about the session stuff tomorrow and see if there is a work_correctly flag i can pass to one of the sessions that i never noticed before10:46
salv-orlandoor a magic wand that you can use to make everything work as you expect?10:47
salv-orlandook, goodnight then.10:47
kevinbentonprecisely10:47
kevinbentongoodnight10:47
*** samuelbercovici1 has joined #openstack-neutron10:49
*** leseb has joined #openstack-neutron10:51
*** jpich has quit IRC10:51
*** samuelbercovici has quit IRC10:52
*** samuelbercovici1 is now known as samuelbercovici10:52
*** samuelbercovici1 has joined #openstack-neutron10:54
*** xianghui^ has joined #openstack-neutron10:56
*** leseb has quit IRC10:56
*** samuelbercovici has quit IRC10:56
*** samuelbercovici1 is now known as samuelbercovici10:57
*** Longgeek_ has quit IRC10:58
*** jroovers2 has joined #openstack-neutron10:58
*** jroovers has quit IRC10:58
*** Longgeek has joined #openstack-neutron10:59
*** yamahata has quit IRC10:59
*** ihrachyshka has joined #openstack-neutron11:00
*** flaviof_zzz is now known as flaviof11:02
*** jpich has joined #openstack-neutron11:04
*** nlahouti has joined #openstack-neutron11:08
*** ramishra has joined #openstack-neutron11:10
*** zzelle has joined #openstack-neutron11:10
*** samuelbercovici1 has joined #openstack-neutron11:12
*** nlahouti has quit IRC11:13
*** Longgeek has quit IRC11:13
*** paraa has quit IRC11:14
*** Longgeek has joined #openstack-neutron11:14
*** ramishra has quit IRC11:14
*** samuelbercovici has quit IRC11:15
*** samuelbercovici1 is now known as samuelbercovici11:15
*** dims_ has quit IRC11:16
*** amaretskiy has quit IRC11:16
*** rotbart has joined #openstack-neutron11:17
*** Jay__ has joined #openstack-neutron11:19
*** Longgeek has quit IRC11:19
*** Longgeek has joined #openstack-neutron11:19
*** ihrachyshka has quit IRC11:21
*** Jay__ has quit IRC11:23
*** samuelbercovici1 has joined #openstack-neutron11:24
*** ihrachyshka has joined #openstack-neutron11:25
*** amaretskiy has joined #openstack-neutron11:26
openstackgerritA change was merged to openstack/neutron: Improve docstring for OVSNeutronAgent constructor  https://review.openstack.org/10082211:26
*** samuelbercovici has quit IRC11:27
*** samuelbercovici1 is now known as samuelbercovici11:27
*** samuelbercovici1 has joined #openstack-neutron11:29
*** jpich has quit IRC11:29
*** samuelbercovici has quit IRC11:32
*** samuelbercovici1 is now known as samuelbercovici11:32
*** paraa has joined #openstack-neutron11:33
*** prad_ has joined #openstack-neutron11:36
*** Demitar has quit IRC11:37
*** nlahouti has joined #openstack-neutron11:39
*** jpich has joined #openstack-neutron11:42
*** kopparam has quit IRC11:43
*** kopparam has joined #openstack-neutron11:43
*** nlahouti has quit IRC11:44
*** samuelbercovici1 has joined #openstack-neutron11:47
*** kopparam has quit IRC11:47
*** matsuhashi has quit IRC11:48
*** kopparam has joined #openstack-neutron11:49
*** jpich has quit IRC11:49
*** samuelbercovici has quit IRC11:50
*** samuelbercovici1 is now known as samuelbercovici11:50
*** b3nt_pin is now known as beagles11:50
*** karimb has left #openstack-neutron11:50
*** matsuhashi has joined #openstack-neutron11:51
*** leseb has joined #openstack-neutron11:52
*** luqas has quit IRC11:52
*** matsuhashi has quit IRC11:56
*** lukego has quit IRC11:56
*** matsuhashi has joined #openstack-neutron11:56
*** leseb has quit IRC11:57
*** dguitarbite has quit IRC11:57
*** Longgeek has quit IRC11:57
*** Longgeek has joined #openstack-neutron11:58
*** matsuhashi has quit IRC12:01
*** jistr is now known as jistr|english12:02
*** armax has joined #openstack-neutron12:02
*** jpich has joined #openstack-neutron12:02
*** amaretskiy has quit IRC12:02
*** jpich_ has joined #openstack-neutron12:05
*** kong has quit IRC12:05
*** Demitar has joined #openstack-neutron12:06
*** jpich has quit IRC12:06
*** mwagner_lap has quit IRC12:10
*** nlahouti has joined #openstack-neutron12:10
*** ramishra has joined #openstack-neutron12:11
*** yamahata has joined #openstack-neutron12:11
*** amaretskiy has joined #openstack-neutron12:11
*** german__ has joined #openstack-neutron12:13
*** nlahouti has quit IRC12:14
*** ramishra has quit IRC12:15
*** jpich_ is now known as jpich12:16
*** samuelbercovici1 has joined #openstack-neutron12:19
*** zzelle has quit IRC12:20
*** tomoe_ has joined #openstack-neutron12:21
*** trad511 has joined #openstack-neutron12:21
*** samuelbercovici has quit IRC12:21
*** samuelbercovici1 is now known as samuelbercovici12:21
*** kopparam has quit IRC12:25
*** kopparam has joined #openstack-neutron12:25
*** tomoe_ has quit IRC12:27
*** german__ has quit IRC12:27
*** trinaths has joined #openstack-neutron12:28
*** kopparam has quit IRC12:30
*** trinaths has quit IRC12:34
*** aburaschi has joined #openstack-neutron12:34
openstackgerritLi Ma proposed a change to openstack/neutron: Race condition of L3-agent to add/remove routers  https://review.openstack.org/7323412:36
*** leseb has joined #openstack-neutron12:38
*** lukego has joined #openstack-neutron12:39
*** ijw has joined #openstack-neutron12:40
*** zzelle has joined #openstack-neutron12:42
*** samuelbercovici1 has joined #openstack-neutron12:42
*** luqas has joined #openstack-neutron12:44
*** samuelbercovici has quit IRC12:46
*** samuelbercovici1 is now known as samuelbercovici12:46
*** Longgeek has quit IRC12:48
*** xuhanp has joined #openstack-neutron12:49
*** julim has joined #openstack-neutron12:51
*** Jay__ has joined #openstack-neutron12:54
*** afaranha has quit IRC12:55
*** sbalukoff has quit IRC12:56
*** samuelbercovici has quit IRC12:57
*** Longgeek has joined #openstack-neutron12:57
*** SridharG has joined #openstack-neutron12:58
*** yfried_ has quit IRC13:01
*** samuelbercovici has joined #openstack-neutron13:01
*** afaranha has joined #openstack-neutron13:03
*** erecio has joined #openstack-neutron13:04
*** amuller_ has joined #openstack-neutron13:04
*** rotbart has quit IRC13:04
*** dims_ has joined #openstack-neutron13:04
*** amuller has quit IRC13:05
*** yamahata has quit IRC13:07
*** yamahata has joined #openstack-neutron13:07
*** flaviof is now known as flaviof_zzz13:08
*** samuelbercovici1 has joined #openstack-neutron13:08
*** kbringard has joined #openstack-neutron13:11
*** samuelbercovici has quit IRC13:11
*** samuelbercovici1 is now known as samuelbercovici13:11
*** nlahouti has joined #openstack-neutron13:12
*** Jay__ has quit IRC13:13
*** kashyap` has joined #openstack-neutron13:13
*** dims_ has quit IRC13:13
*** leseb has quit IRC13:15
*** leseb has joined #openstack-neutron13:16
*** zzelle has quit IRC13:16
*** nlahouti has quit IRC13:16
*** dims_ has joined #openstack-neutron13:17
*** kashyap has quit IRC13:17
*** irenab has quit IRC13:18
*** tomoe_ has joined #openstack-neutron13:19
*** kashyap`` has joined #openstack-neutron13:20
*** regXboi has joined #openstack-neutron13:22
*** kashyap` has quit IRC13:22
*** samuelbercovici1 has joined #openstack-neutron13:26
*** samuelbercovici1 has quit IRC13:27
*** otherwiseguy has joined #openstack-neutron13:28
*** dims_ has quit IRC13:28
*** samuelbercovici has quit IRC13:29
openstackgerritenikanorov proposed a change to openstack/neutron: Fix auto_schedule_networks to resist DBDuplicateEntry  https://review.openstack.org/10090313:32
*** mestery has joined #openstack-neutron13:32
*** kashyap`` is now known as kashyap13:36
*** jgallard has joined #openstack-neutron13:36
openstackgerritA change was merged to openstack/neutron: Fix pool statistics for LBaaS Haproxy driver  https://review.openstack.org/10029913:37
*** leseb has quit IRC13:38
*** changbl has quit IRC13:38
*** ghyoc has quit IRC13:38
*** ghyoc has joined #openstack-neutron13:39
*** jckasper_ has quit IRC13:41
*** jistr|english is now known as jistr13:42
*** mestery has quit IRC13:44
openstackgerritAvishay Balderman proposed a change to openstack/neutron: Enable Python 3.4 enum in Neutron. Define enum validation as part of the REST validarion framework  https://review.openstack.org/10090813:45
*** aveiga has joined #openstack-neutron13:47
*** ygbo has quit IRC13:47
*** mwagner_lap has joined #openstack-neutron13:47
*** sbalukoff has joined #openstack-neutron13:48
*** devvesa_ has quit IRC13:48
*** salv-orlando has quit IRC13:49
*** salv-orlando_ has joined #openstack-neutron13:49
ihrachyshkasalv-orlando_: now that I look into that db lock issue, I think it would be great if our db context would support attaching postponed actions to it while in transaction to run them on __exit__ from session context13:51
*** ramishra has joined #openstack-neutron13:51
ihrachyshkasalv-orlando_: that would give us a nice way to postpone action items to the time 'after db session is closed'13:52
*** jmsoares has joined #openstack-neutron13:52
*** beagles is now known as beagles_brb13:52
*** [1]eezhova has joined #openstack-neutron13:53
*** bklei has joined #openstack-neutron13:53
openstackgerritijw-ubuntu proposed a change to openstack/neutron-specs: VLAN trunking networks for NFV  https://review.openstack.org/9771413:53
*** bklei has quit IRC13:53
*** sballe has joined #openstack-neutron13:54
openstackgerritCedric Brandily proposed a change to openstack/neutron: Allow to delegate some tunnel provider attributes choice  https://review.openstack.org/7405513:54
openstackgerritCedric Brandily proposed a change to openstack/neutron: Allow to delegate some vlan provider attributes choice to neutron  https://review.openstack.org/7190413:54
*** s3wong has joined #openstack-neutron13:55
*** amotoki has quit IRC13:55
*** ramishra has quit IRC13:56
*** Longgeek_ has joined #openstack-neutron13:57
openstackgerritAvishay Balderman proposed a change to openstack/neutron: Enable Python 3.4 enum in Neutron. Define enum validation as part of the REST validation framework  https://review.openstack.org/10090813:58
*** jistr has quit IRC13:59
*** yfried_ has joined #openstack-neutron13:59
francois_eleouetroot13:59
*** Longgeek has quit IRC13:59
francois_eleouetsorry wrong window...13:59
*** jistr has joined #openstack-neutron14:01
*** ygbo has joined #openstack-neutron14:02
*** sdague has quit IRC14:02
openstackgerritOpenStack Proposal Bot proposed a change to openstack/neutron: Updated from global requirements  https://review.openstack.org/9976214:06
*** trinaths has joined #openstack-neutron14:07
*** VijayB has joined #openstack-neutron14:07
openstackgerritijw-ubuntu proposed a change to openstack/neutron-specs: NFV unaddressed interfaces  https://review.openstack.org/9771514:07
[1]eezhovaSumitNaiksatam, Hello! May I ask you a question?14:08
*** german__ has joined #openstack-neutron14:08
*** sdague has joined #openstack-neutron14:08
openstackgerritijw-ubuntu proposed a change to openstack/neutron-specs: VLAN trunking networks for NFV  https://review.openstack.org/9771414:09
*** [1]trinaths has joined #openstack-neutron14:09
openstackgerritDoug Wiegley proposed a change to openstack/neutron-specs: LBaaS Driver Interface changes for new object model, new interface  https://review.openstack.org/10069014:10
*** TrevorV has joined #openstack-neutron14:12
*** yamamoto has joined #openstack-neutron14:13
*** trinaths has quit IRC14:13
*** [1]trinaths is now known as trinaths14:13
*** TrevorV has quit IRC14:13
*** TrevorV has joined #openstack-neutron14:13
*** nlahouti has joined #openstack-neutron14:14
*** [1]evgenyf has quit IRC14:14
*** avishayb has quit IRC14:14
*** rcarrillocruz has quit IRC14:14
*** evgenyf has joined #openstack-neutron14:15
*** amuller__ has joined #openstack-neutron14:16
*** jckasper has joined #openstack-neutron14:16
*** mestery has joined #openstack-neutron14:16
*** markmcclain has joined #openstack-neutron14:18
*** nlahouti has quit IRC14:18
*** rcarrillocruz has joined #openstack-neutron14:20
*** amuller_ has quit IRC14:20
*** mlavalle has joined #openstack-neutron14:21
*** jckasper has quit IRC14:22
*** jckasper has joined #openstack-neutron14:22
*** amuller__ has quit IRC14:25
*** VijayB has quit IRC14:26
*** manishg has joined #openstack-neutron14:26
*** jobewan has joined #openstack-neutron14:26
openstackgerritAvishay Balderman proposed a change to openstack/neutron: Enable Python 3.4 enum in Neutron.  https://review.openstack.org/10090814:26
*** dims_ has joined #openstack-neutron14:27
*** leseb has joined #openstack-neutron14:27
*** nplanel has joined #openstack-neutron14:27
*** vivek-ebay has joined #openstack-neutron14:28
openstackgerritA change was merged to openstack/neutron-specs: lbaas-api-and-objmodel-improvement  https://review.openstack.org/8990314:29
*** rwsu has joined #openstack-neutron14:29
*** trinaths has quit IRC14:30
*** kopparam has joined #openstack-neutron14:31
*** trinaths has joined #openstack-neutron14:31
*** ramishra has joined #openstack-neutron14:32
*** rolledback has joined #openstack-neutron14:36
*** dlundquist has joined #openstack-neutron14:37
*** dlundquist has left #openstack-neutron14:37
*** aneesh_ has joined #openstack-neutron14:38
*** rm_work|away is now known as rm_work14:38
*** VijayB_ has joined #openstack-neutron14:39
aneesh_hi!, i have one doubt in neutron networking configuration can any help me14:40
aneesh_i have only one private network in my collage with 10.0.0.0/814:40
aneesh_it was subnetted to and provided to all with class c 10.4.16.0/2414:41
aneesh_other subnets are not reachable14:41
aneesh_like 10.4.15.0/2414:41
openstackgerritenikanorov proposed a change to openstack/neutron: Move _filter_non_model_columns method to CommonDbMixin  https://review.openstack.org/10092614:42
*** Longgeek_ has quit IRC14:43
*** jistr has quit IRC14:44
*** Longgeek has joined #openstack-neutron14:44
boris-42aneesh_ hi there14:45
*** Longgeek has quit IRC14:45
*** ranger81 has joined #openstack-neutron14:46
aneesh_it was subnetted to and provided to all with class c 10.4.16.0/2414:46
*** garyk1 has quit IRC14:47
boris-42aneesh_ enikanorov ^14:47
*** Longgeek has joined #openstack-neutron14:47
*** alex_klimov has quit IRC14:48
*** jistr has joined #openstack-neutron14:48
sc68calaneesh_: are you private messaging people in this channel for help?14:48
sc68calif you are, that is rude.14:48
aneesh_i don't know whom to ask my question14:49
sc68calthen ask it here so everyone can see, instead of picking people out of the list of users on this channel and sending private messages14:50
*** flaviof_zzz is now known as flaviof14:50
aneesh_thanks <sc68cal>14:50
aneesh_hi!, i have one doubt in neutron networking configuration14:50
enikanorovaneesh_: you should ask on ask.openstack.org14:50
enikanorovthis is a channel for development questions14:51
aneesh_thanks enikanorov ...14:51
*** BradLsys has joined #openstack-neutron14:51
*** ramishra has quit IRC14:54
openstackgerritPhillip Toohill proposed a change to openstack/neutron-specs: Improving LBaaS reference implementation  https://review.openstack.org/10093114:56
*** dfarrell07 has joined #openstack-neutron14:56
*** ranger81 has quit IRC14:57
openstackgerritIhar Hrachyshka proposed a change to openstack/neutron: Avoid notifying while inside transaction opened in delete_port()  https://review.openstack.org/10093415:01
*** ijw has quit IRC15:02
*** ijw has joined #openstack-neutron15:02
*** roeyc has quit IRC15:04
openstackgerritBradley Klein proposed a change to openstack/python-neutronclient: Add keystone v3 auth support  https://review.openstack.org/9239015:04
*** kopparam has quit IRC15:05
openstackgerritIhar Hrachyshka proposed a change to openstack/neutron: Avoid notifying while inside transaction opened in delete_port()  https://review.openstack.org/10093415:06
*** kopparam has joined #openstack-neutron15:06
ihrachyshkasalv-orlando_: you may be interested in this ^^15:06
openstackgerritZhang Hua proposed a change to openstack/neutron: IPsec strongswan driver implemention  https://review.openstack.org/10079115:06
*** dims_ has quit IRC15:07
*** aneesh_ has quit IRC15:08
*** afazekas_ has quit IRC15:10
*** kopparam has quit IRC15:11
*** ihrachyshka has quit IRC15:11
*** yamahata has quit IRC15:13
*** alagalah has quit IRC15:14
*** dguitarbite has joined #openstack-neutron15:15
*** WackoRobie has quit IRC15:18
*** WackoRobie has joined #openstack-neutron15:18
openstackgerritsean mooney proposed a change to openstack/neutron: enables patch ports when availble for connecting openvswitch bridges  https://review.openstack.org/10093915:18
*** VijayB has joined #openstack-neutron15:19
*** Longgeek has quit IRC15:20
*** otherwiseguy has quit IRC15:21
*** sweston has quit IRC15:21
*** beagles_brb is now known as beagles15:21
*** nlahouti has joined #openstack-neutron15:22
*** jpich has quit IRC15:22
*** VijayB has quit IRC15:22
*** WackoRobie has quit IRC15:23
*** VijayB_ has quit IRC15:23
*** VijayB has joined #openstack-neutron15:23
openstackgerritPhillip Toohill proposed a change to openstack/neutron-specs: Improving LBaaS reference implementation  https://review.openstack.org/10093115:24
*** thomas_morin has joined #openstack-neutron15:25
openstackgerritRalf Haferkamp proposed a change to openstack/neutron: Don't convert numeric protocol values to int  https://review.openstack.org/10094215:26
mesteryrossella_s: Ping regarding your bulk API patch for get_device_details15:27
*** xuhanp has quit IRC15:27
rossella_smestery: Hi Kyle, I have read your comment15:29
mesteryrossella_s: Thanks! And sorry for putting -2. :(15:29
mesteryrossella_s: Can you put a quick spec together for this? It can be very lightweight.15:29
mesteryrossella_s: I will work to approve it ASAP. :)15:29
rossella_smestery: thanks!! I will try to do that today15:30
mesteryrossella_s: Thank you very much! Ping me in-channel once it lands, I will work to get it approved ASAP. Sorry for the trouble. :)15:30
rossella_smestery: thanks Kyle! No worries, we have a process, we should follow it. Will ping you later :)15:31
mesteryrossella_s: Thanks much! :)15:31
*** dims_ has joined #openstack-neutron15:32
*** leenheer has joined #openstack-neutron15:32
*** seizadi has joined #openstack-neutron15:32
*** otherwiseguy has joined #openstack-neutron15:33
openstackgerritZang MingJie proposed a change to openstack/neutron-specs: Option to disable dns on some subnets  https://review.openstack.org/10024215:34
*** jecarey has quit IRC15:35
*** ijw has quit IRC15:35
markmcclainihrachyshka: https://review.openstack.org/#/c/97291/15:35
*** packet has joined #openstack-neutron15:38
*** jpich has joined #openstack-neutron15:39
*** garyk has joined #openstack-neutron15:39
*** SumitNaiksatam has quit IRC15:42
*** dims_ has quit IRC15:42
*** SumitNaiksatam has joined #openstack-neutron15:43
*** dims_ has joined #openstack-neutron15:44
*** shivharis has joined #openstack-neutron15:44
*** changbl has joined #openstack-neutron15:44
*** Sukhdev has joined #openstack-neutron15:45
SumitNaiksatameezhova: hi15:46
openstackgerritA change was merged to openstack/neutron: Add vlan type driver unittests  https://review.openstack.org/10023315:48
*** leenheer has quit IRC15:48
rkukuraSukhdev: Are you planning to chair the ML2 meeting today?15:51
Sukhdevrkukura: Sure, I can do it..15:51
Sukhdevrkukura: can you check the agenda?15:52
rkukuraSukhdev: Looked OK a while ago, will look again15:52
Sukhdevrkukura: cool - thanks15:53
*** sballe_ has joined #openstack-neutron15:53
*** aveiga has quit IRC15:53
*** thomas_morin has quit IRC15:54
*** tomoe_ has quit IRC15:54
*** bandarji has joined #openstack-neutron15:55
*** SridharG has quit IRC15:55
*** banix has joined #openstack-neutron15:55
*** jpich has quit IRC15:55
*** sballe has quit IRC15:56
*** dims_ has quit IRC15:57
*** vthapar has joined #openstack-neutron15:58
*** slogan has joined #openstack-neutron15:58
*** salv-orlando_ has quit IRC16:00
openstackgerritDoug Wiegley proposed a change to openstack/neutron-specs: LBaaS Driver Interface changes for new object model  https://review.openstack.org/10069016:01
*** leseb has quit IRC16:01
*** sbfox has joined #openstack-neutron16:01
*** pcm_ has quit IRC16:01
openstackgerritEvgeny Fedoruk proposed a change to openstack/neutron-specs: lbaas-tls  https://review.openstack.org/9864016:01
*** leseb has joined #openstack-neutron16:01
*** amotoki has joined #openstack-neutron16:02
*** pcm_ has joined #openstack-neutron16:02
*** _cjones_ has joined #openstack-neutron16:03
*** jlibosva has quit IRC16:03
*** terryw has joined #openstack-neutron16:03
openstackgerritA change was merged to openstack/neutron: Big Switch ML2: sync detection in port-update  https://review.openstack.org/10079816:04
openstackgerritA change was merged to openstack/neutron: Big Switch: Remove unnecessary initialization code  https://review.openstack.org/10073116:04
openstackgerritA change was merged to openstack/neutron: Configure agents using neutron.common.config.init (formerly .parse)  https://review.openstack.org/9750516:04
openstackgerritA change was merged to openstack/neutron: Don't instantiate RPC clients on import  https://review.openstack.org/9750616:04
openstackgerritA change was merged to openstack/neutron: Don't explicitly call .stop() on mock.patch objects  https://review.openstack.org/9880716:04
*** WackoRobie has joined #openstack-neutron16:05
*** WackoRobie has quit IRC16:05
*** ijw has joined #openstack-neutron16:05
*** WackoRobie has joined #openstack-neutron16:05
*** leseb has quit IRC16:06
*** rcurran has joined #openstack-neutron16:06
openstackgerritPhillip Toohill proposed a change to openstack/neutron-specs: Improving LBaaS reference implementation  https://review.openstack.org/10093116:06
*** otherwiseguy has quit IRC16:07
openstackgerritOpenStack Proposal Bot proposed a change to openstack/neutron: Updated from global requirements  https://review.openstack.org/9976216:07
openstackgerritA change was merged to openstack/neutron: Make sure we call BaseTestCase.setUp() first  https://review.openstack.org/9762316:08
*** pcm_ has quit IRC16:08
*** jgallard has quit IRC16:09
*** spandhe has joined #openstack-neutron16:09
*** evgenyf has quit IRC16:11
*** terryw has quit IRC16:13
*** otherwiseguy has joined #openstack-neutron16:14
*** sweston has joined #openstack-neutron16:15
*** ygbo has quit IRC16:16
enikanorovSumitNaiksatam: hi16:18
*** chandan_kumar has joined #openstack-neutron16:18
enikanorovwhat do you mean by 'schedme for developers'? (on your commen in flavors review)16:18
*** pcm_ has joined #openstack-neutron16:19
SumitNaiksatamenikanorov: so the suggestion in the proposal is that the developers distinguish the tags16:20
SumitNaiksatamenikanorov: when the add them16:20
SumitNaiksatamenikanorov: and you point out three categories: generic, …16:20
SumitNaiksatamenikanorov: how will the developer express the association of the new tag with any of these categories?16:21
enikanorovSumitNaiksatam: so they add something like tag_names = { ... , "TAG_FW_INSERTION_TYPE":"InsertionType"}16:21
enikanorovthere's no association16:21
enikanorovit's just by name and is logical16:21
enikanorovfor instance16:22
enikanorovsupported_cypher capability16:22
enikanorovthe tag is then just a proper name for it16:22
SumitNaiksatamenikanorov: so when you say “Developers need to distinguish tags they going to add by types”16:23
*** sbfox has quit IRC16:23
SumitNaiksatamenikanorov: you are referring to "InsertionType" as one of those types?16:23
*** sbfox has joined #openstack-neutron16:23
enikanorovInsertionType is an example of generic tag, that may apply to different services - vpn, lbaas, fwaas16:24
SumitNaiksatamenikanorov: ok so how did "InsertionType" come into existence, and where is the information captured that this is a “generic” tag?16:25
SumitNaiksatamenikanorov: as opposed to being service or driver specific16:25
enikanorovmy suggestion is that some common place will have the name "InsertionType"16:26
SumitNaiksatamenikanorov: ok16:26
openstackgerritA change was merged to openstack/neutron-specs: LBaaS Driver Interface changes for new object model  https://review.openstack.org/10069016:26
SumitNaiksatamenikanorov: as long as we identify that16:26
enikanorovand nothing really says that it is a generic tag, other than location of it's name16:27
*** sbfox has quit IRC16:27
SumitNaiksatamenikanorov: location?16:27
SumitNaiksatamenikanorov: i think what you are looking for is a namespace for tags16:28
enikanorovSumitNaiksatam: yes, so these names may be grouped in the code16:28
SumitNaiksatamenikanorov: generic, service and driver16:28
enikanorovyep, we can call it a namespace, or enum16:28
*** ygbo has joined #openstack-neutron16:28
SumitNaiksatamenikanorov: not sure namespace and enum are the same16:29
enikanorovfor some reason i didn't think of it as of major problem as it's just a code organization issue to me16:29
SumitNaiksatamenikanorov: well if we do make this categorization, then i think its something we need to organize formally16:30
SumitNaiksatamenikanorov: organize in the design16:30
SumitNaiksatamenikanorov: i believe its more than just a code artifact16:30
enikanorovwell, for me it seems like a problem because i don't have a list of possible names in mind16:31
*** wendar_ is now known as wendar16:31
enikanorovso i can't say i can start it16:31
SumitNaiksatamenikanorov: but you have indentified the namespaces - generic, service and driver16:31
enikanorovi think we can have module for that where we gather tag names under different 'namespaces'16:31
*** sbfox has joined #openstack-neutron16:31
SumitNaiksatamenikanorov: yeah16:32
enikanorovalthough i'm not sure how do you think namespace map on the code16:32
enikanorovis that a list?16:32
*** leseb has joined #openstack-neutron16:32
SumitNaiksatamenikanorov: i am not talking about the tag name16:32
SumitNaiksatamenikanorov: i am talking about the types16:32
*** dave_tucker is now known as dave_tucker_zzz16:32
SumitNaiksatamenikanorov: i would imagine that the namespace is always hierarchical16:32
enikanorovi'm not sure why we even need to pay such attention to categorization16:33
SumitNaiksatamenikanorov: so the generic, genric/service, generic/service/driver16:33
SumitNaiksatamenikanorov: because you have mentioned: Developers need to distinguish tags they going to add by types16:33
enikanorovi think that's just a common sense to not duplicate possible tag names16:35
SumitNaiksatamenikanorov: having the hierarchical structure/namespace for the types will lend itself to matching16:35
enikanorovintroducing them in different places16:35
enikanorovi just don't see a reason to have tag type as an attribute of tag16:36
SumitNaiksatamenikanorov: well you can attribute anything to common sense16:36
SumitNaiksatamenikanorov: then there is no need for any organization16:36
enikanorovfrom admin user perspective only name of the tag matters16:37
*** leseb has quit IRC16:37
SumitNaiksatamenikanorov: so what if two drivers have tags with the same name but they are semantically different?16:37
*** leseb has joined #openstack-neutron16:37
enikanorovthat's exactly whan needs to be checked on review16:38
enikanorov*what16:38
SumitNaiksatamenikanorov: and how to do you expect the reviewer to validate that?16:38
*** sbfox has quit IRC16:38
*** sbfox has joined #openstack-neutron16:38
*** leenheer has joined #openstack-neutron16:38
enikanorovtag name must be meaning full and provide a hint of its 'level' (generic, service-specific, vendor-specific)16:39
enikanorovand then check against a common place16:39
SumitNaiksatamenikanorov: i think that is too subjective, and prone to error and misinterpretation16:39
enikanorovor, if some other vendor introduces the same - increase the level of tag to service-specific from vendor-specific16:39
enikanorovSumitNaiksatam: but that's how we do with all constants that we use across the code16:39
SumitNaiksatamenikanorov: if we are proposing a formal framework, we should do it the right way16:40
enikanorovtag name is just a string constant16:40
enikanorovif you see a new mechanism driver that introduces MIN_VXLAN_VNI, would not you tell submitter  to reuse existing?16:41
*** networkstatic has joined #openstack-neutron16:41
*** a_le has joined #openstack-neutron16:41
SumitNaiksatamenikanorov: but constant has a literal value which completely defines it16:41
*** tomoe_ has joined #openstack-neutron16:41
SumitNaiksatamenikanorov: this is not the case here16:42
enikanorovwhy?16:42
enikanorovi starting to feel we're discussing two different notions here16:42
SumitNaiksatamenikanorov: the literal value has a mapping on the driver side16:42
SumitNaiksatamenikanorov: lets take this discussion to the IRC meeting16:42
enikanorovi'd like to clarify it before the meeting16:43
SumitNaiksatamenikanorov: i need to step away16:43
enikanorovlet me give you an example16:43
enikanorovoh, ok16:43
SumitNaiksatamenikanorov: sorry, ttly16:44
*** armax has left #openstack-neutron16:45
*** jroovers has joined #openstack-neutron16:46
*** jroovers2 has quit IRC16:46
*** jroovers2 has joined #openstack-neutron16:48
*** suresh12 has joined #openstack-neutron16:50
*** leseb has quit IRC16:50
*** amaretskiy has quit IRC16:50
*** leseb has joined #openstack-neutron16:50
*** jroovers has quit IRC16:51
*** spandhe has quit IRC16:53
*** leenheer has quit IRC16:53
*** leseb has quit IRC16:54
*** ramishra has joined #openstack-neutron16:55
*** isviridov is now known as isviridov|away16:56
*** spandhe has joined #openstack-neutron16:58
*** ramishra has quit IRC16:59
*** dims_ has joined #openstack-neutron16:59
*** VijayB_ has joined #openstack-neutron17:00
openstackgerritRalf Haferkamp proposed a change to openstack/neutron: Don't convert numeric protocol values to int  https://review.openstack.org/10094217:00
*** rcurran has quit IRC17:00
*** VijayB has quit IRC17:01
*** pcm__ has joined #openstack-neutron17:01
*** shivharis has quit IRC17:02
*** slogan is now known as slogan_away17:02
*** jmsoares has quit IRC17:02
*** ranger81 has joined #openstack-neutron17:03
*** pcm_ has quit IRC17:04
openstackgerritHenry Gessau proposed a change to openstack/neutron-specs: DB migration refactoring  https://review.openstack.org/9573817:04
*** trinaths has quit IRC17:05
*** harlowja_away is now known as harlowja17:06
*** sweston has quit IRC17:07
*** amotoki has quit IRC17:07
*** rolledback has quit IRC17:10
*** salv-orlando has joined #openstack-neutron17:11
Sukhdevbanix: ping17:13
banixSukhdev: hi17:14
Sukhdevbanix: I want to chat with you regarding the bulk transaction bug that we discussed earlier17:15
banixSukhdev: yes sure17:15
Sukhdevbanix: wanted to see if you will be available later today17:15
Sukhdevbanix: have couple of things to catchup now, but, wanted to see your availability so that we can connect a bit later here on this channel?17:16
banixSukhdev: i am available omly from 4:30 to 5:30pm EST today17:16
banixSukhdev: would that work for you?17:16
*** thedodd has joined #openstack-neutron17:16
Sukhdevbanix: You are on Eastern time..17:17
*** crc32 has joined #openstack-neutron17:17
banixSukhdev: of course; that’s more fashionable these days :)17:17
Sukhdevbanix: I should be able to finish my things by 4:30pm - will ping you then…if not, perhaps tomorrow17:17
openstackgerritRossella Sblendido proposed a change to openstack/neutron: Remove SELECT FOR UPDATE lock from db_base_plugin  https://review.openstack.org/10096317:18
banixSukhdev: sounds good17:18
Sukhdevbanix: really, did not know that ….looks like that makes me old fashioned :-)17:18
banixSukhdev: or me a wishful thinking person :)17:18
Sukhdevbanix: ha ha…funny!!17:19
*** sballe__ has joined #openstack-neutron17:19
openstackgerritRossella Sblendido proposed a change to openstack/neutron: Remove SELECT FOR UPDATE lock from db_base_plugin (WIP)  https://review.openstack.org/10096317:19
*** rm_work is now known as rm_work|away17:20
*** jprovazn has joined #openstack-neutron17:21
*** vthapar has quit IRC17:21
*** ygbo has quit IRC17:21
*** vthapar has joined #openstack-neutron17:21
*** VijayB_ has quit IRC17:21
*** armax has joined #openstack-neutron17:21
*** luqas has quit IRC17:22
*** sballe_ has quit IRC17:23
*** os-new has quit IRC17:24
*** osnew has joined #openstack-neutron17:24
markmcclainrossella_s: hated to -2 the lock fix, but it does not work when there are multiple instances of neutron-server17:24
*** vthapar has quit IRC17:27
openstackgerritenikanorov proposed a change to openstack/neutron-specs: neutron-flavor-framework  https://review.openstack.org/9007017:27
*** pcm__ has quit IRC17:28
*** vivek-ebay has quit IRC17:28
*** jp_at_hp has quit IRC17:28
enikanorovmarkmcclain: can you give me a link to a review?17:30
rossella_smarkmcclain: that's why I sent the patch, so we can discuss... -2 is fine17:31
rossella_smarkmcclain: I have to go now...can I ping you later to talk about this?17:31
enikanorovrossella_s: may i look to this review?17:32
enikanorovi'm working on failures related to multiple processes17:32
rossella_smarkmcclain: I was thinking that in case of multiple neutron-server, the db will throw a conflict exception. If we have a logic to catch that and retry...maybe it could work17:33
*** pcm_ has joined #openstack-neutron17:34
rossella_senikanorov: https://review.openstack.org/#/c/10096317:34
enikanorovrossella_s: thanks17:34
*** jp_at_hp has joined #openstack-neutron17:34
rossella_senikanorov: no problem! thanks for your interest...gotta go now unfortunately. ttyl17:35
*** ijw has quit IRC17:36
markmcclainrossella_s: yes17:36
*** rm_work|away is now known as rm_work17:36
*** ijw has joined #openstack-neutron17:37
*** ranger81_ has joined #openstack-neutron17:37
*** ranger81 has quit IRC17:39
*** changbl has quit IRC17:40
*** alex_klimov has joined #openstack-neutron17:40
*** nati_ueno has joined #openstack-neutron17:41
*** alagalah has joined #openstack-neutron17:44
*** Sukhdev has quit IRC17:45
*** safchain has quit IRC17:47
*** zzelle has joined #openstack-neutron17:47
*** tong_ has joined #openstack-neutron17:52
*** SridharG has joined #openstack-neutron17:53
*** nati_ueno has quit IRC17:53
*** dims_ has quit IRC17:53
*** TrevorV has quit IRC17:53
*** nati_ueno has joined #openstack-neutron17:54
*** jp_at_hp has quit IRC17:54
*** ramishra has joined #openstack-neutron17:56
*** ranger81 has joined #openstack-neutron17:57
*** jecarey has joined #openstack-neutron17:58
*** salv-orlando has quit IRC18:00
*** SridharG has quit IRC18:00
*** ramishra has quit IRC18:00
*** igordcard has quit IRC18:00
*** vivek-ebay has joined #openstack-neutron18:01
*** dave_tucker_zzz is now known as dave_tucker18:01
*** TrevorV has joined #openstack-neutron18:01
*** ranger81_ has quit IRC18:01
*** dims_ has joined #openstack-neutron18:01
*** ranger81_ has joined #openstack-neutron18:02
*** VijayB_ has joined #openstack-neutron18:02
*** leenheer has joined #openstack-neutron18:04
*** moha_hunt has quit IRC18:05
*** ranger81 has quit IRC18:05
*** packet has quit IRC18:05
*** sbfox has quit IRC18:05
*** ijw has quit IRC18:07
*** ijw has joined #openstack-neutron18:08
*** rcarrillocruz has quit IRC18:08
*** rcarrillocruz has joined #openstack-neutron18:08
*** aburaschi has quit IRC18:10
*** moha_hunt has joined #openstack-neutron18:11
*** rolledback has joined #openstack-neutron18:11
openstackgerritDustin Lundquist proposed a change to openstack/neutron-specs: LBaaS refactor HAProxy namespace driver  https://review.openstack.org/10097718:11
*** geekinut1h is now known as geekinutah18:13
*** seizadi has quit IRC18:14
*** changbl has joined #openstack-neutron18:16
openstackgerritArvind Somya proposed a change to openstack/neutron-specs: ML2 Type drivers refactor to allow extensiblity  https://review.openstack.org/9181118:16
*** trad511 has quit IRC18:17
*** WackoRobie has quit IRC18:18
*** WackoRobie has joined #openstack-neutron18:18
*** packet has joined #openstack-neutron18:19
*** jroovers2 has quit IRC18:20
*** Demitar_ has joined #openstack-neutron18:20
*** WackoRobie has quit IRC18:20
*** WackoRobie has joined #openstack-neutron18:21
*** jlibosva has joined #openstack-neutron18:22
*** Demitar has quit IRC18:22
*** ramishra has joined #openstack-neutron18:23
*** ranger81_ has quit IRC18:26
*** ramishra has quit IRC18:27
*** [1]eezhova has quit IRC18:29
*** terryw has joined #openstack-neutron18:34
*** ranger81 has joined #openstack-neutron18:35
*** otherwiseguy has quit IRC18:35
*** dims_ has quit IRC18:36
*** jaypipes has joined #openstack-neutron18:37
openstackgerritHenry Gessau proposed a change to openstack/neutron-specs: DB migration refactoring  https://review.openstack.org/9573818:37
*** lukego has quit IRC18:37
openstackgerritDustin Lundquist proposed a change to openstack/neutron-specs: LBaaS refactor HAProxy namespace driver  https://review.openstack.org/10097718:38
*** samuelbercovici1 has joined #openstack-neutron18:39
*** samuelbercovici1 is now known as samuelbercovici18:39
*** dave_tucker is now known as dave_tucker_zzz18:42
*** dave_tucker_zzz is now known as dave_tucker18:43
*** jckasper has quit IRC18:43
*** doude has quit IRC18:46
openstackgerritKanzhe Jiang proposed a change to openstack/neutron-specs: Add spec for service-base-and-insertion  https://review.openstack.org/9312818:46
*** yamamoto has quit IRC18:48
*** beagles is now known as beagles_brb18:48
openstackgerritDoug Wiegley proposed a change to openstack/neutron: Ignore emacs checkpoint files  https://review.openstack.org/10099218:49
*** igordcard has joined #openstack-neutron18:52
*** lykinsbd has joined #openstack-neutron18:53
openstackgerritPhillip Toohill proposed a change to openstack/neutron-specs: Improving LBaaS reference implementation  https://review.openstack.org/10093118:54
*** jistr has quit IRC18:55
*** suresh12 has quit IRC18:58
*** ozialien has joined #openstack-neutron18:59
*** dims_ has joined #openstack-neutron19:00
*** german__ has quit IRC19:02
*** german__ has joined #openstack-neutron19:02
*** openstackgerrit has quit IRC19:06
*** openstackgerrit_ has joined #openstack-neutron19:08
*** openstackgerrit_ is now known as openstackgerrit19:08
*** jaypipes has quit IRC19:09
*** dims_ has quit IRC19:09
*** jlibosva has quit IRC19:10
zzelleDoug Wiegley proposed a change to openstack/neutron: Ignore emacs checkpoint files  https://review.openstack.org/100992 ????19:12
openstackgerritPhillip Toohill proposed a change to openstack/neutron-specs: Improving LBaaS reference implementation  https://review.openstack.org/10093119:12
*** terryw has quit IRC19:13
*** a_le has quit IRC19:13
*** alagalah has quit IRC19:15
*** suresh12 has joined #openstack-neutron19:17
*** leenheer has quit IRC19:17
*** julim has quit IRC19:18
*** ozialien has quit IRC19:18
*** SridharG has joined #openstack-neutron19:18
openstackgerritPhillip Toohill proposed a change to openstack/neutron-specs: Improving LBaaS reference implementation  https://review.openstack.org/10093119:20
*** ramishra has joined #openstack-neutron19:24
*** jaypipes has joined #openstack-neutron19:24
*** otherwiseguy has joined #openstack-neutron19:26
*** sbfox has joined #openstack-neutron19:27
*** terryw has joined #openstack-neutron19:28
*** ramishra has quit IRC19:29
*** otherwiseguy has quit IRC19:32
*** a_le has joined #openstack-neutron19:35
openstackgerritOpenStack Proposal Bot proposed a change to openstack/neutron: Updated from global requirements  https://review.openstack.org/9976219:36
openstackgerritBob Melander proposed a change to openstack/neutron: Adds router service plugin for CSR1kv  https://review.openstack.org/10100219:36
*** jecarey has quit IRC19:36
*** beagles_brb is now known as beagles19:37
openstackgerritBob Melander proposed a change to openstack/neutron: Adds router service plugin for CSR1kv  https://review.openstack.org/10100219:38
*** jroovers has joined #openstack-neutron19:40
openstackgerritBob Melander proposed a change to openstack/neutron: Adds router service plugin for CSR1kv  https://review.openstack.org/10100219:43
*** Zemeio has joined #openstack-neutron19:43
*** nlahouti has quit IRC19:44
openstackgerritBob Melander proposed a change to openstack/neutron: Adds router service plugin for CSR1kv  https://review.openstack.org/10100219:45
*** dguitarbite has quit IRC19:47
openstackgerritPhillip Toohill proposed a change to openstack/neutron-specs: Improving LBaaS reference implementation  https://review.openstack.org/10093119:47
openstackgerritBob Melander proposed a change to openstack/neutron: Adds router service plugin for CSR1kv  https://review.openstack.org/10100219:48
*** TrevorV has quit IRC19:49
*** bandarji has quit IRC19:50
*** rm_work is now known as rm_work|away19:51
*** julim has joined #openstack-neutron19:51
*** TrevorV has joined #openstack-neutron19:56
openstackgerritDustin Lundquist proposed a change to openstack/neutron-specs: LBaaS refactor HAProxy namespace driver  https://review.openstack.org/10097719:58
*** alagalah has joined #openstack-neutron20:01
*** rolledback has quit IRC20:02
*** leseb has joined #openstack-neutron20:02
*** bandarji has joined #openstack-neutron20:02
*** sbfox has quit IRC20:04
*** SridharG has quit IRC20:04
*** terryw has quit IRC20:04
*** sbfox has joined #openstack-neutron20:05
*** TrevorV has quit IRC20:05
*** lori is now known as lori|away20:05
*** TrevorV has joined #openstack-neutron20:07
rms_13A gentle reminder to review https://review.openstack.org/#/c/98532/20:07
rms_13to any core dev20:07
*** VijayB_ has quit IRC20:08
openstackgerritPhillip Toohill proposed a change to openstack/neutron-specs: Improving LBaaS reference implementation  https://review.openstack.org/10093120:09
openstackgerritA change was merged to openstack/neutron-specs: LBaaS refactor HAProxy namespace driver  https://review.openstack.org/10097720:10
openstackgerritenikanorov proposed a change to openstack/neutron: Move _filter_non_model_columns method to CommonDbMixin  https://review.openstack.org/10092620:11
*** rcarrill` has joined #openstack-neutron20:14
*** VijayB has joined #openstack-neutron20:15
*** jaypipes has quit IRC20:15
*** rcarrillocruz has quit IRC20:16
*** jobewan has quit IRC20:20
*** lukego has joined #openstack-neutron20:24
*** nlahouti has joined #openstack-neutron20:24
*** ramishra has joined #openstack-neutron20:25
*** dave_tucker is now known as dave_tucker_zzz20:26
openstackgerritPhillip Toohill proposed a change to openstack/neutron-specs: Improving LBaaS reference implementation  https://review.openstack.org/10093120:27
*** trad511 has joined #openstack-neutron20:28
*** safchain has joined #openstack-neutron20:29
*** nlahouti has quit IRC20:29
*** ramishra has quit IRC20:29
*** bandarji has quit IRC20:31
*** julim has quit IRC20:34
openstackgerritpritesh proposed a change to openstack/neutron: Remove vim editor configuration from comments and add hacking checks  https://review.openstack.org/9950820:36
*** VijayB has quit IRC20:36
*** lukego has quit IRC20:38
*** crc32 has quit IRC20:39
*** VijayB has joined #openstack-neutron20:40
*** jprovazn has quit IRC20:43
*** nlahouti has joined #openstack-neutron20:43
*** BradLsys has quit IRC20:47
*** otherwiseguy has joined #openstack-neutron20:49
*** regXboi has left #openstack-neutron20:50
openstackgerritpritesh proposed a change to openstack/neutron: Remove old quantum scripts  https://review.openstack.org/10079520:52
*** jp_at_hp has joined #openstack-neutron20:52
openstackgerritPhillip Toohill proposed a change to openstack/neutron-specs: Improving LBaaS reference implementation  https://review.openstack.org/10093120:52
*** doude has joined #openstack-neutron20:53
*** sbfox has quit IRC20:53
*** networkstatic has quit IRC20:56
*** sbfox has joined #openstack-neutron20:58
*** erecio has quit IRC20:59
*** mwagner_lap has quit IRC21:00
*** yamahata has joined #openstack-neutron21:00
*** lykinsbd has quit IRC21:01
_cjones_Hey guys. I don't usually ask "how to" questions here, but I'm stumped. I'm looking for the correct syntax to perform a "neutron router-update" to add a new route.21:04
*** bandarji has joined #openstack-neutron21:04
*** karimb has joined #openstack-neutron21:06
*** dims_ has joined #openstack-neutron21:06
zzelle_cjones_, it should be something like21:06
*** sbfox has quit IRC21:07
_cjones_zzelle: I got:  neutron router-update --request-format xml "{\"router\": {\"routes\": [{\"nexthop\":\"10.0.0.4\",\"destination\":\"192.168.1.0/24\"}]}}" 31eede22-a82c-4155-a100-31280703b06f21:07
_cjones_Getting: "Invalid values_specs 31eede22-a82c-4155-a100-31280703b06f"21:07
openstackgerritA change was merged to openstack/neutron: Brocade mechanism driver should be derived from ML2 plugin base class  https://review.openstack.org/9887221:08
zzelle_cjones_, neutron router-update my-router --routes type=dict list=true nexthop=10.0.0.2,destination=192.168.1.0/2421:08
_cjones_zzelle21:10
_cjones_OMG!21:10
_cjones_I suppose the "neutron help router-update" command needs to be updated . :)21:10
zzelle_cjones_, if you prefer to call me god .. it's fine21:10
zzelle_cjones_, nope it's a generic option21:11
*** sweston has joined #openstack-neutron21:12
*** VijayB has quit IRC21:12
*** tongli has joined #openstack-neutron21:12
zzelle_cjones_, "--%NAME% type=dict list=true %KEY0%=%VAL0%" generate {"%NAME%": [{"%KEY0%": "%VAL0%"}]}21:13
*** VijayB_ has joined #openstack-neutron21:13
_cjones_zzelle: Okay, God...  How was I supposed to decipher to do all of that from this: https://gist.github.com/cjones-/7269151383b84d2be89421:13
*** tongli has quit IRC21:13
*** sweston has quit IRC21:14
_cjones_zzelle: Thanks for the explanation.21:14
*** sweston has joined #openstack-neutron21:14
zzelle_cjones_, must of the time neutron something-update are less documented than neutron something-create21:14
zzelle_cjones_, so look at *-create help21:14
_cjones_zzeelle: Heh. I *had* done that. Unfortunately there's nothing in router-create pertaining to routes as from my understanding this and setting the G/W are all done through the update.21:15
*** paraa has quit IRC21:16
_cjones_zzelle: Thanks though. I really appreciate the help.21:17
*** samuelbercovici has quit IRC21:17
*** yamahata has quit IRC21:18
*** yamahata has joined #openstack-neutron21:18
yamahatabanix: ping?21:19
banixyamahata: hi21:19
yamahatabanix: hi.21:19
yamahatabanix: can you please check blueprint of physical-topology-extension?21:20
yamahataI added link table and link.id?21:20
*** dims_ has quit IRC21:20
yamahataand hopefully remove -1 in the wiki page?21:20
*** tong_ has quit IRC21:20
banixyamahata: believe it or not I was doing that right now :)21:20
yamahatabanix: cool!21:21
*** dims_ has joined #openstack-neutron21:21
banixyamahata: with respect to -1 in the wiki, you could have removed it yourself as you have updated a new patchset. Anyway will be comenting soon; have a couple of minor questions21:22
yamahatabanix: I see, thanks21:23
*** lykinsbd has joined #openstack-neutron21:24
*** ramishra has joined #openstack-neutron21:25
*** dims_ has quit IRC21:26
*** fifieldt has quit IRC21:27
*** safchain has quit IRC21:29
*** leenheer has joined #openstack-neutron21:29
*** dave_tucker_zzz is now known as dave_tucker21:29
*** ramishra has quit IRC21:30
*** garyk1 has joined #openstack-neutron21:30
*** garyk has quit IRC21:31
*** abhraut has joined #openstack-neutron21:33
abhrautping kevinbenton21:33
kevinbentonabhraut: pong21:34
abhrautyouve used GreenPools right?21:34
kevinbentonyeah21:34
abhrautto spawn threads and talk to the controller ?21:34
*** dkehn has quit IRC21:34
*** luqas has joined #openstack-neutron21:34
kevinbentonyes21:35
abhrautcool.. how do you test those things..21:35
abhrauti mean.. since we mock most of our controller side code..21:35
abhrauthow would i test when the pool is exhausted, threads wait .. and their behavior..21:36
*** dkehn has joined #openstack-neutron21:36
abhrauthave you thought of such test cases.. and IF such test cases make sense21:36
kevinbentonthose scenarios are pretty well defined. if the pool is exhausted the new spawn will just block until a slot becomes available21:37
kevinbentoni don’t think you need to write tests specifically for that behavior alone21:37
*** jordandh has joined #openstack-neutron21:37
abhrautyeah. the behavior is well documented.. im asking.. does a UT around it make sense?21:37
*** packet has quit IRC21:38
kevinbentonabhraut: only if you have special logic that is triggered when the pool becomes exhausted21:38
abhrautnot in this case.. would you like to take a look at the review?21:38
*** changbl has quit IRC21:38
kevinbentonhere is a test on ours that checks for our async behavior21:38
kevinbentonhttps://github.com/bigswitch/neutron/blob/master/neutron/tests/unit/bigswitch/test_restproxy_plugin.py#L10821:38
kevinbentonsure21:38
*** jordandh has quit IRC21:39
abhrautthat would be very helpful..21:39
*** jordandh has joined #openstack-neutron21:39
abhrautthanks.. i will add you to the list21:39
kevinbentonbetter link me directly to the review21:39
*** jordandh has left #openstack-neutron21:39
kevinbentoni have a big backlog right now because i’ve been busy so i might miss the notification for yours21:39
abhrautkevinbenton: link -> https://review.openstack.org/#/c/100388/21:39
kevinbentonthanks21:40
abhrautcool21:40
*** fifieldt has joined #openstack-neutron21:40
*** jordandh has joined #openstack-neutron21:40
*** jordandh has joined #openstack-neutron21:42
*** Abhishek has joined #openstack-neutron21:42
*** Abhishek has quit IRC21:42
*** Abhishek has joined #openstack-neutron21:43
*** jordandh has quit IRC21:43
*** jordandh has joined #openstack-neutron21:43
openstackgerritPhillip Toohill proposed a change to openstack/neutron-specs: Improving LBaaS reference implementation  https://review.openstack.org/10093121:44
kevinbentonabhraut: just posted a few comments21:45
*** VijayB_ has quit IRC21:45
kevinbentonabhraut: by calling wait() on every spawn you are effectively defeating the purpose of the greenthreads21:45
kevinbentonabhraut: wait() waits for it to finish and returns the value21:46
kevinbentonabhraut: only wait() when you need to actually guaruntee that it has finished at that point21:46
kevinbentonabhraut: and in most of those cases then you can just skip the greenpool all together21:47
abhrauti added waits because i need to know if an exception has occured21:47
*** jp_at_hp has quit IRC21:47
abhrautthis is not much of an async use case21:47
abhrautits more of limit the number of connections to the controller21:47
kevinbentonabhraut: oh, i see21:48
kevinbentonabhraut: in that case greenthreads might be a lot of overhead21:48
*** banix has quit IRC21:48
*** luqas has quit IRC21:48
abhrautis there a better alternative21:49
kevinbentonabhraut: might be easier to just keep a global counter in your connection code21:49
abhrautokay21:50
_cjones_zzelle: Last one. How would I go abouts removing the route I just updated?21:50
zzelle--routes action=clear21:50
*** a_le has quit IRC21:51
*** a_le has joined #openstack-neutron21:51
openstackgerritA change was merged to openstack/neutron: Ignore emacs checkpoint files  https://review.openstack.org/10099221:53
abhrautkevinbenton: i thought greenlets are quite cheap and have minimal overhead21:55
kevinbentonabhraut: yes, it’s not that. it just adds a lof of extra stuff the code21:56
kevinbentonabhraut: if you want to use them, i would create a module level greenpool in n1kv_client and keep the logic there21:57
abhrautkevinbenton: yeah .. that makes sense to me21:58
openstackgerritA change was merged to openstack/neutron: Updated from global requirements  https://review.openstack.org/9976221:58
*** Zemeio has quit IRC21:59
openstackgerritA change was merged to openstack/neutron: Imported Translations from Transifex  https://review.openstack.org/10042922:00
*** VijayB_ has joined #openstack-neutron22:01
openstackgerritSukhdev Kapur proposed a change to openstack/neutron-specs: Arista L3 Router Service Plugin  https://review.openstack.org/9712522:03
*** seizadi has joined #openstack-neutron22:04
*** Sukhdev has joined #openstack-neutron22:06
openstackgerritTerry Wilson proposed a change to openstack/neutron: Pass 'top' to remove_rule so that rule matching succeeds  https://review.openstack.org/10103922:07
*** ijw has quit IRC22:08
*** alex_klimov has quit IRC22:09
*** kbringard has quit IRC22:09
*** ijw has joined #openstack-neutron22:12
*** ghyoc has quit IRC22:13
*** VijayB_ has quit IRC22:13
*** dims_ has joined #openstack-neutron22:16
*** ijw has quit IRC22:16
*** dims_ has quit IRC22:19
*** dims_ has joined #openstack-neutron22:20
*** moha_hunt has quit IRC22:21
*** VijayB has joined #openstack-neutron22:23
openstackgerritPraneet Bachheti proposed a change to openstack/neutron: Opencontrail plug-in implementation for core resources  https://review.openstack.org/9663022:23
*** jroovers has quit IRC22:23
*** rm_work|away is now known as rm_work22:24
*** WackoRobie has quit IRC22:24
*** WackoRobie has joined #openstack-neutron22:25
*** VijayB has quit IRC22:25
*** VijayB has joined #openstack-neutron22:25
*** doude has quit IRC22:26
*** ramishra has joined #openstack-neutron22:26
*** karimb has quit IRC22:26
*** chuckC has quit IRC22:26
*** karimb has joined #openstack-neutron22:27
openstackgerritFranck Yelles proposed a change to openstack/neutron-specs: Support of host-routes and dns_nameservers via DHCP options for the Nuage Plugin  https://review.openstack.org/10104122:28
*** yamahata has quit IRC22:28
*** thedodd has quit IRC22:29
*** WackoRobie has quit IRC22:29
*** nlahouti has quit IRC22:29
*** VijayB has quit IRC22:29
*** seizadi1 has joined #openstack-neutron22:30
*** ramishra has quit IRC22:31
*** seizadi has quit IRC22:31
*** garyduan has joined #openstack-neutron22:31
*** armax has left #openstack-neutron22:31
*** lyxus has joined #openstack-neutron22:31
*** sbfox has joined #openstack-neutron22:32
*** bandarji has quit IRC22:34
rossella_smarkmcclain: ping22:36
*** ranger81_ has joined #openstack-neutron22:38
*** ranger81 has quit IRC22:40
openstackgerritMohammad Hanif proposed a change to openstack/neutron-specs: Spec for inter datacenter connectivity orchestration using mpls vpn  https://review.openstack.org/10104322:41
openstackgerritMohammad Hanif proposed a change to openstack/neutron-specs: Spec for Inter datacenter connectivity orchestration using MPLS VPN  https://review.openstack.org/10104422:41
*** banix has joined #openstack-neutron22:41
*** leseb has quit IRC22:42
*** VijayB_ has joined #openstack-neutron22:42
*** TrevorV has quit IRC22:42
mesteryotherwiseguy: I may have just crossed the line with my pedantic comment on your 1-line commit. :P22:43
*** leseb has joined #openstack-neutron22:43
*** otherwiseguy has quit IRC22:45
*** VijayB_ has quit IRC22:46
openstackgerritenikanorov proposed a change to openstack/neutron-specs: neutron-flavor-framework  https://review.openstack.org/9007022:47
*** mwagner_lap has joined #openstack-neutron22:47
*** leseb has quit IRC22:47
*** chuckC has joined #openstack-neutron22:49
*** leseb has joined #openstack-neutron22:50
*** rm_work is now known as rm_work|away22:50
*** mestery has quit IRC22:51
*** s3wong has quit IRC22:52
*** sballe__ has quit IRC22:52
*** leseb has quit IRC22:55
*** osnew has quit IRC22:56
*** german__ has quit IRC22:56
*** markmcclain has quit IRC22:56
openstackgerritA change was merged to openstack/neutron: NSX: revert queue extension name change  https://review.openstack.org/9948322:57
*** rolledback has joined #openstack-neutron22:59
*** banix has quit IRC23:01
*** sbalukoff has quit IRC23:02
*** jecarey has joined #openstack-neutron23:02
*** ozialien has joined #openstack-neutron23:03
*** igordcard has quit IRC23:03
*** ranger81 has joined #openstack-neutron23:03
openstackgerritAndy Shi proposed a change to openstack/neutron-specs: Dell ml2 Mechanism Driver  https://review.openstack.org/9249523:04
*** ranger81_ has quit IRC23:05
*** pcm_ has quit IRC23:09
*** mlavalle has quit IRC23:12
*** leseb has joined #openstack-neutron23:13
openstackgerritSoheil Eizadi proposed a change to openstack/neutron-specs: Pluggable IPAM Subsystem in Neutron  https://review.openstack.org/9796723:13
*** leseb has quit IRC23:14
*** leseb has joined #openstack-neutron23:15
*** dims_ has quit IRC23:17
*** vivek-ebay has quit IRC23:19
*** leseb has quit IRC23:19
*** rolledback has quit IRC23:20
*** prad_ has quit IRC23:20
karimbcan neutron-server run in active active fashion ?23:22
zzellekarimb, AFAIK yes23:25
karimbzzelle, thanx23:27
*** ramishra has joined #openstack-neutron23:27
openstackgerritKarthik proposed a change to openstack/neutron-specs: Design Spec For Brocade Vyatta L3 Plugin  https://review.openstack.org/10104823:27
*** ranger81 has quit IRC23:29
*** ranger81 has joined #openstack-neutron23:31
*** ramishra has quit IRC23:31
*** leenheer has quit IRC23:32
openstackgerritA change was merged to openstack/neutron: remove pep8 E122 exemption and correct style  https://review.openstack.org/9951223:32
openstackgerritPraneet Bachheti proposed a change to openstack/neutron: Opencontrail plug-in implementation for core resources  https://review.openstack.org/9663023:33
openstackgerritenikanorov proposed a change to openstack/neutron-specs: neutron-flavor-framework  https://review.openstack.org/9007023:33
*** trad511 has quit IRC23:34
*** otherwiseguy has joined #openstack-neutron23:35
*** leenheer has joined #openstack-neutron23:35
enikanorovnati_ueno: Hi, I've left a reply to your comment in https://review.openstack.org/#/c/100903/ Do you think we can separate whose items?23:38
enikanorov*those23:38
*** trad511 has joined #openstack-neutron23:39
*** xuhanp has joined #openstack-neutron23:41
lyxusHi, I just wrote a neutrspec and submitted and Jenkins is complainig about ERROR: Unexpected indentation. How can i run the same script23:41
kevinbentonlyxus: run tox23:42
openstackgerritTerry Wilson proposed a change to openstack/neutron: Pass 'top' to remove_rule so that rule matching succeeds  https://review.openstack.org/10103923:44
nati_uenoenikanorov: I prefer the patch with UT when it is merged.. How about co-authoring this patch with your friend?23:46
enikanorovnati_ueno: what if we merge UT first?23:47
enikanorovthe method is not covered by UT anyway23:47
nati_uenoenikanorov: I think we need UT for the bug you filed23:47
nati_uenoenikanorov: so the UT should fail23:47
nati_uenoenikanorov: so we can't merge UT first23:48
enikanorovok23:48
openstackgerritKarthik proposed a change to openstack/neutron-specs: Fixed the workflow diagram  https://review.openstack.org/10105223:49
*** yfujioka has joined #openstack-neutron23:49
*** sbfox has quit IRC23:51
openstackgerritKarthik proposed a change to openstack/neutron-specs: Design Spec For Brocade Vyatta L3 Plugin  https://review.openstack.org/10105223:51
*** Demitar_ has quit IRC23:53
*** otherwiseguy has quit IRC23:53
*** dims_ has joined #openstack-neutron23:56
*** zzelle has quit IRC23:56
*** sungju has joined #openstack-neutron23:59

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