Wednesday, 2014-10-01

*** tflynn_ has joined #openstack-neutron00:01
*** dconde has quit IRC00:01
*** JayJ has quit IRC00:01
*** ivar-lazzaro has quit IRC00:01
*** otherwiseguy has quit IRC00:02
*** JayJ has joined #openstack-neutron00:02
*** achanda has quit IRC00:02
*** ivar-laz_ has quit IRC00:02
rkukurakevinbenton: will check00:03
rkukurakevinbenton, armax: I gave my +200:04
*** tflynn has quit IRC00:05
*** tflynn_ has quit IRC00:05
*** baoli has joined #openstack-neutron00:08
*** cjellick_ has quit IRC00:08
*** moha_hunt has quit IRC00:08
*** cjellick has joined #openstack-neutron00:08
*** ChuckC has quit IRC00:09
*** cjellick has quit IRC00:13
*** dims has joined #openstack-neutron00:14
*** Youcef_ has quit IRC00:14
*** AbsinthMind has joined #openstack-neutron00:15
*** obondarev_ has quit IRC00:15
*** obondarev has joined #openstack-neutron00:17
*** dims has quit IRC00:19
*** Youcef has joined #openstack-neutron00:26
*** sgordon_ has joined #openstack-neutron00:27
*** parkit has joined #openstack-neutron00:27
*** lori is now known as lori|away00:31
*** yamahata has joined #openstack-neutron00:31
*** nlahouti has quit IRC00:32
*** armax has quit IRC00:32
*** jckasper has joined #openstack-neutron00:32
*** parkit has quit IRC00:33
*** manishg has quit IRC00:34
*** otherwiseguy has joined #openstack-neutron00:36
*** mwagner_lap has quit IRC00:36
*** AbsinthMind has quit IRC00:36
*** padkrish has quit IRC00:37
*** JayJ has quit IRC00:37
*** Youcef has quit IRC00:38
*** mwagner_lap has joined #openstack-neutron00:38
*** jckasper has quit IRC00:40
*** jckasper has joined #openstack-neutron00:40
*** nlahouti has joined #openstack-neutron00:41
*** ChuckC has joined #openstack-neutron00:43
*** signed8bit is now known as signed8bit_ZZZzz00:43
*** signed8bit_ZZZzz has quit IRC00:43
*** JayJ has joined #openstack-neutron00:44
*** mlavalle has quit IRC00:49
*** parkit has joined #openstack-neutron00:49
*** chuckC_ has joined #openstack-neutron00:51
JayJrook: Can neutron ML2 be configured with flat netwroks?00:53
*** vivek-ebay has quit IRC00:57
*** otherwiseguy has quit IRC00:57
*** AbsinthMind has joined #openstack-neutron00:58
*** sbfox has joined #openstack-neutron01:02
*** vivek-ebay has joined #openstack-neutron01:02
*** dfarrell07 has joined #openstack-neutron01:03
rkukuraJayJ: flat networks can be used as provider networks, but not as tenant networks01:03
*** r-daneel has quit IRC01:04
*** vivek-ebay has quit IRC01:04
*** vivek-ebay has joined #openstack-neutron01:04
*** diegows has quit IRC01:05
*** suresh12_ has quit IRC01:06
*** suresh12 has joined #openstack-neutron01:06
*** baoli has quit IRC01:06
*** suresh12 has quit IRC01:06
JayJrkukura: I want to remove the L3 NAT agent as its a bottleneck in out deployment. No l3 NAT means no floating IP (I assume). We have multiple tenants. I want to configure each tenant with an flat provider network so the switching is done at physical router.01:06
*** suresh12 has joined #openstack-neutron01:06
JayJrkukura: What is the best way you suggest to approach this?01:07
*** sbfox has quit IRC01:08
JayJrkukura: main thing is to remove floating IP to DHCP NAT01:08
rkukuraJayJ: You’d need to do that by creating the flat networks as admin, setting their tenant_ids.01:08
*** carl_baldwin_m has quit IRC01:09
JayJrkukura: do you know of any docs? I did my share of searching but resulted nothing01:09
rkukuraJayJ: The reason flat tenant networks aren’t supported is that it doesn’t make much sense to have a pool of them.01:09
JayJrkukura: If admin creates flat network, then how does the instance talks to outside?01:11
rkukuraJayJ: I’d think creating provider networks (i.e. passing “—provider:network_type flat”) would be covered in the docs, but I’m not sure.01:11
rkukuraJayJ: You said you wanted to create a separate flat network for each tenant, right? And connect each to a physical router?01:12
JayJrkukura: yes01:12
JayJrkukura: That's correct01:12
rkukuraYou’d need to set the default gateway on the subnets to be the physical router’s gateway address.01:12
JayJrkukura: So I remove l3 agent01:13
rkukuraRight01:13
JayJrkukura: You mean for the provider net, right?01:13
*** AbsinthMind has quit IRC01:13
*** AbsinthMind has joined #openstack-neutron01:14
rkukuraJayJ: Right, you would not need to connect these flat provider networks to a neutron router.01:14
JayJrkukura: How is DHCP IP to external IP linked in that case?01:15
rkukuraJayJ: Do you really want to use separate flat networks for each tenant? Wouldn’t this mean your compute nodes need NICs connected to each of these?01:16
JayJrkukura: I am assuming there is an DHCP ip with flat networks?01:16
*** dkehnm has joined #openstack-neutron01:16
rkukuraJayJ: I’d recommend still using neutron’s DHCP on these networks. It should hand the correct gateway IP out with the leases.01:16
*** baoli has joined #openstack-neutron01:17
rkukuraYou might also want to consider a single shared external network (probably flat) that tenants can connect to directly.01:17
JayJrkukura: then how does it work? DHCP and provider flat network are going to be completely different subnet right?01:17
*** vivek-ebay has quit IRC01:17
*** sbfox has joined #openstack-neutron01:18
JayJrkukura: With single shared network, I need to have a flaoting IP. My understandign it flaoting IP to internal DHCP NAT is done by the l3 agent, which is our bottleneck01:18
rkukuraJayJ: The neutron DHCP server would listen on each of these flat networks.01:18
JayJrkukura: Ah, then that shoudl work01:19
rkukuraIt is possible to have a shared provider network that is connected directly to a physical router, and for VMs from multiple tenants to connect to that.01:19
*** parkit has quit IRC01:20
rkukuraJayJ: I’m still not clear how you’d have enough NICs on each compute node for each tenant to have its own flat network.01:20
*** otherwiseguy has joined #openstack-neutron01:20
rkukuraJayJ: It would be fine for a couple tenants, but wouldn’t scale.01:20
JayJrkukura: So if I create a flat provider network (with gateway on the pysical router) as admin, the tenants can connect to the same flat network and the instances get an IP on the same subnet.01:21
*** zhhuabj has quit IRC01:21
rkukuraJayJ: Yes. And if that network is shared, multiple tenants can use it.01:21
JayJrkukura: As far as NICs, I was thinkign to use VLANs (example: eth1.10, eth1.11 etc.)01:22
*** parkit has joined #openstack-neutron01:22
*** HenryG_ has joined #openstack-neutron01:22
rkukuraIf you are using VLANs on the router, you could use vlan provider (or tenant) networks.01:22
*** Sukhdev has joined #openstack-neutron01:22
JayJrkukura: Scalling is an issue. Agreed. We will have less than a few handful of tenants.01:23
*** sbfox has quit IRC01:24
*** HenryG has quit IRC01:24
*** geekinutah has quit IRC01:25
rkukuraJayJ: You should be able to do what you were talking about with vlan provider networks and avoid having to manually create and bridge the subinterfaces (i.e. eth1.10).01:25
JayJrkukura: I'm a bit confused with ML2 plugins also. If I set type_driver to say vlan or gre, and use mechanism_driver=openvswitch, Is it true that this will alow only tenant level islolation? The external traffic still has to go through a flaoting IP01:25
rkukuraJayJ: In ML2, the L2 configuration is pretty much orthogonal to what you do at L3.01:27
rkukuraJayJ: It doesn’t really matter what type of L2 isolation is used - you can still connect the network to neutron routers, or configure the subnet on the network with a physical router as the gateway. Of course getting a physical router to connect to a neutron GRE or VXLAN tunnel is probably not possible, since the router wouldn’t know about neutron’s tunnel endpoint management.01:29
JayJrkukura: in that case, if I setup vlan provider network with GW on the physical router, what would you suggest my type_driver and mechanism_driver be, so I can elimiate neutron L3 agent?01:31
JayJrkukura: I meant L2 agent (NAT)01:33
JayJrkukura: Let me ask this, I want to create a provider network and when I create an instance, I want to get an IP directly off of the provider net. How do I do that?01:35
*** dfarrell07 has quit IRC01:36
*** dims has joined #openstack-neutron01:36
*** parkit has quit IRC01:37
*** parkit has joined #openstack-neutron01:37
*** nlahouti has quit IRC01:38
*** changbl has joined #openstack-neutron01:40
*** ramishra has joined #openstack-neutron01:44
*** baoli has quit IRC01:47
*** sbfox has joined #openstack-neutron01:48
*** geekinutah has joined #openstack-neutron01:49
*** signed8bit has joined #openstack-neutron01:52
*** signed8bit is now known as signed8bit_ZZZzz01:53
*** signed8bit_ZZZzz is now known as signed8bit01:53
*** signed8bit has quit IRC01:54
*** ramishra has quit IRC01:55
*** ramishra has joined #openstack-neutron01:56
*** ramishra has quit IRC01:56
openstackgerritBrian Haley proposed a change to openstack/neutron: Change L2 OVS DVR Agent to handle initial DVR RPC failure  https://review.openstack.org/12391101:57
*** ramishra has joined #openstack-neutron01:57
JayJrkukura: Sorry I must have bored you with my stupid questions :) I need to tryout a few things to get better understanding02:00
openstackgerritAaron Rosen proposed a change to openstack/python-neutronclient: Leverage neutronclient.openstack.common.importutils import_class  https://review.openstack.org/11896902:01
*** amcrn has quit IRC02:02
*** vivek-ebay has joined #openstack-neutron02:07
*** spandhe has quit IRC02:08
*** chuckC_ has quit IRC02:11
*** yamamoto_ has joined #openstack-neutron02:12
*** sbfox has quit IRC02:13
*** geekinutah has quit IRC02:16
*** parkit has quit IRC02:16
*** JayJ has quit IRC02:18
*** dvorkinista has joined #openstack-neutron02:19
*** parkit has joined #openstack-neutron02:22
*** jobewan has joined #openstack-neutron02:22
*** parkit has quit IRC02:23
*** tflynn has joined #openstack-neutron02:24
*** tflynn has quit IRC02:24
*** tflynn has joined #openstack-neutron02:25
*** HenryG_ has quit IRC02:25
*** parkit has joined #openstack-neutron02:29
*** parkit has quit IRC02:31
*** parkit has joined #openstack-neutron02:34
*** VijayB has quit IRC02:35
*** carl_baldwin has joined #openstack-neutron02:37
*** Sukhdev has quit IRC02:39
*** tsg has joined #openstack-neutron02:40
*** HenryG has joined #openstack-neutron02:41
*** harlowja is now known as harlowja_away02:42
*** otherwiseguy has quit IRC02:42
*** suresh12 has quit IRC02:48
*** sbfox has joined #openstack-neutron02:49
*** dvorkinista has quit IRC02:55
*** parkit has quit IRC02:55
*** tflynn has quit IRC02:56
*** tflynn has joined #openstack-neutron02:56
*** shashankhegde has quit IRC02:59
*** dvorkinista has joined #openstack-neutron03:00
*** dvorkinista has quit IRC03:01
*** gildub has quit IRC03:01
*** otherwiseguy has joined #openstack-neutron03:05
*** dvorkinista has joined #openstack-neutron03:06
*** nlahouti has joined #openstack-neutron03:08
*** tflynn has quit IRC03:08
*** dsneddon has quit IRC03:08
*** sbfox has quit IRC03:09
*** dsneddon has joined #openstack-neutron03:09
*** tflynn has joined #openstack-neutron03:10
*** nlahouti has quit IRC03:12
*** nlahouti has joined #openstack-neutron03:12
*** afaranha has quit IRC03:17
*** dvorkinista has quit IRC03:18
*** samuelmz-awaw has quit IRC03:18
*** dvorkinista has joined #openstack-neutron03:19
*** dvorkinista has quit IRC03:20
*** sbfox has joined #openstack-neutron03:20
*** dvorkinista has joined #openstack-neutron03:22
*** dvorkinista has quit IRC03:24
*** dims has quit IRC03:33
*** ramishra has quit IRC03:34
*** tomoe has quit IRC03:39
*** gildub has joined #openstack-neutron03:46
openstackgerritIsaku Yamahata proposed a change to openstack/neutron: l3_db.py: refactor port creation/deleteion/update for routervm  https://review.openstack.org/12469903:49
*** amcrn has joined #openstack-neutron03:49
*** AbsinthMind has quit IRC03:51
*** yfujioka has joined #openstack-neutron03:52
*** parkit has joined #openstack-neutron03:52
*** mwagner_lap has quit IRC03:53
*** mwagner_lap has joined #openstack-neutron03:53
*** suresh12 has joined #openstack-neutron03:59
*** geekinutah has joined #openstack-neutron04:03
*** suresh12 has quit IRC04:03
*** tflynn has quit IRC04:06
*** dvorkinista has joined #openstack-neutron04:07
*** tflynn has joined #openstack-neutron04:07
*** jobewan has quit IRC04:07
*** dvorkinista has quit IRC04:07
*** jobewan has joined #openstack-neutron04:08
*** tflynn has quit IRC04:09
*** tflynn has joined #openstack-neutron04:10
*** tsg has quit IRC04:12
*** dvorkinista has joined #openstack-neutron04:13
*** tflynn has quit IRC04:16
*** tflynn has joined #openstack-neutron04:18
*** dvorkinista has quit IRC04:18
*** Sukhdev_ has joined #openstack-neutron04:25
*** armax has joined #openstack-neutron04:26
*** dsneddon has quit IRC04:27
*** dsneddon has joined #openstack-neutron04:27
*** SumitNaiksatam has quit IRC04:27
*** SumitNaiksatam has joined #openstack-neutron04:28
*** terryw has joined #openstack-neutron04:28
*** otherwiseguy has quit IRC04:29
*** dvorkinista has joined #openstack-neutron04:32
*** carl_baldwin has quit IRC04:33
*** parkit has quit IRC04:39
*** armax has quit IRC04:39
*** armax has joined #openstack-neutron04:40
*** armax has quit IRC04:44
*** terryw has quit IRC04:47
*** dvorkinista has quit IRC04:47
*** marun has joined #openstack-neutron04:48
*** dguitarbite has quit IRC04:55
*** marun has quit IRC04:55
*** carlp has quit IRC04:57
*** flaviof_zzz has quit IRC04:57
*** killer_prince is now known as lazy_prince05:00
*** carlp has joined #openstack-neutron05:02
*** samuelmz has joined #openstack-neutron05:06
*** Sukhdev_ has quit IRC05:06
*** vivek-ebay has quit IRC05:09
*** rakesh_hs has joined #openstack-neutron05:09
*** mandeep has joined #openstack-neutron05:09
*** mandeep has quit IRC05:10
*** Longgeek has joined #openstack-neutron05:11
*** suresh12 has joined #openstack-neutron05:12
*** afaranha has joined #openstack-neutron05:15
*** dvorkinista has joined #openstack-neutron05:16
*** yfauser has joined #openstack-neutron05:17
*** yfauser has left #openstack-neutron05:17
*** vthapar has joined #openstack-neutron05:18
*** dvorkini_ has joined #openstack-neutron05:20
*** carl_baldwin has joined #openstack-neutron05:21
*** shashankhegde has joined #openstack-neutron05:22
*** armax has joined #openstack-neutron05:22
*** dvorkinista has quit IRC05:23
*** yfujioka has quit IRC05:26
*** kopparam has joined #openstack-neutron05:28
*** yatin has joined #openstack-neutron05:30
*** flaviof_zzz has joined #openstack-neutron05:30
*** dguitarbite has joined #openstack-neutron05:32
*** dguitarbite has joined #openstack-neutron05:32
*** dguitarbite has quit IRC05:32
*** dguitarbite has joined #openstack-neutron05:32
*** feleouet has quit IRC05:32
*** ajo has joined #openstack-neutron05:38
*** yatin has quit IRC05:39
*** parkit has joined #openstack-neutron05:40
*** armax has quit IRC05:43
*** parkit has quit IRC05:44
*** dvorkini_ has quit IRC05:44
*** reed has quit IRC05:45
*** k4n0 has joined #openstack-neutron05:47
*** numan has joined #openstack-neutron05:49
*** irenab has joined #openstack-neutron05:49
*** shashankhegde has quit IRC05:49
*** yatin has joined #openstack-neutron06:04
*** tflynn has quit IRC06:06
*** carl_baldwin has quit IRC06:08
*** vivek-ebay has joined #openstack-neutron06:09
*** sergsh has left #openstack-neutron06:10
*** SumitNaiksatam has quit IRC06:10
*** SumitNaiksatam has joined #openstack-neutron06:11
*** suresh12 has quit IRC06:13
*** skolekonov has joined #openstack-neutron06:13
*** vivek-ebay has quit IRC06:14
*** shashankhegde has joined #openstack-neutron06:23
*** popw1 has joined #openstack-neutron06:26
*** popw has quit IRC06:26
*** evgenyf has joined #openstack-neutron06:27
*** shashankhegde has quit IRC06:28
*** jswiders has joined #openstack-neutron06:28
*** arosen-home1 has quit IRC06:28
*** SridharG has joined #openstack-neutron06:30
*** dims has joined #openstack-neutron06:33
*** dims has quit IRC06:38
*** SridharG has left #openstack-neutron06:45
*** SridharG has joined #openstack-neutron06:46
*** SridharG has quit IRC06:50
*** SridharG has joined #openstack-neutron06:51
*** jpena has joined #openstack-neutron06:51
*** salv-orlando has joined #openstack-neutron06:52
*** nmagnezi has joined #openstack-neutron06:56
*** pasquier-s has joined #openstack-neutron06:58
*** kopparam has quit IRC07:00
*** kopparam has joined #openstack-neutron07:01
*** akamyshnikova has joined #openstack-neutron07:03
*** amotoki has joined #openstack-neutron07:04
*** nyechiel has joined #openstack-neutron07:04
*** nlahouti1 has joined #openstack-neutron07:04
*** kopparam has quit IRC07:05
*** nlahouti has quit IRC07:06
*** amcrn has quit IRC07:11
*** karimb has joined #openstack-neutron07:11
*** nmagnezi has quit IRC07:12
*** nmagnezi has joined #openstack-neutron07:12
openstackgerritOpenStack Proposal Bot proposed a change to openstack/neutron: Updated from global requirements  https://review.openstack.org/12264607:12
*** oomichi has joined #openstack-neutron07:18
*** yfauser has joined #openstack-neutron07:18
*** yfauser has left #openstack-neutron07:18
*** nyechiel has quit IRC07:20
*** doude has joined #openstack-neutron07:23
*** suresh12 has joined #openstack-neutron07:24
*** jamiehannaford has joined #openstack-neutron07:25
*** suresh12 has quit IRC07:29
*** ramishra has joined #openstack-neutron07:31
*** safchain has joined #openstack-neutron07:32
*** kickinz1|afk is now known as kickinz107:38
*** ZZelle has quit IRC07:43
*** ZZelle has joined #openstack-neutron07:43
*** sbfox has quit IRC07:45
*** sbfox has joined #openstack-neutron07:47
*** kickinz1 is now known as kickinz1|afk07:47
*** ygbo has joined #openstack-neutron07:48
*** coolsvap|afk is now known as coolsvap07:48
*** gildub has quit IRC07:52
*** kickinz1|afk is now known as kickinz107:56
*** salv-orlando_ has joined #openstack-neutron07:56
*** salv-orlando has quit IRC07:58
*** salv-orlando_ is now known as salv-orlando07:58
*** jschwarz has joined #openstack-neutron07:58
*** kickinz1 is now known as kickinz1|afk08:01
*** kopparam has joined #openstack-neutron08:01
*** jistr has joined #openstack-neutron08:02
*** kickinz1|afk is now known as kickinz108:02
*** nyechiel has joined #openstack-neutron08:06
*** kopparam has quit IRC08:06
*** ihrachyshka has joined #openstack-neutron08:07
*** kickinz1 is now known as kickinz1|afk08:09
*** kopparam has joined #openstack-neutron08:14
*** ramishra has quit IRC08:15
*** ramishra has joined #openstack-neutron08:16
*** irenab has quit IRC08:17
*** ramishra has quit IRC08:18
*** ramishra has joined #openstack-neutron08:18
*** ramishra has quit IRC08:20
*** ramishra has joined #openstack-neutron08:21
openstackgerritSamer Deeb proposed a change to openstack/neutron: Fix sleep function call  https://review.openstack.org/12528308:23
openstackgerritRomil Gupta proposed a change to openstack/neutron: Schema enhancement to support MultiSegment Network  https://review.openstack.org/10354608:24
*** coolsvap is now known as coolsvap|afk08:24
*** lori|away is now known as lori08:26
*** feleouet has joined #openstack-neutron08:27
*** matrohon has joined #openstack-neutron08:27
openstackgerritRomil Gupta proposed a change to openstack/neutron: Schema enhancement to support MultiSegment Network  https://review.openstack.org/10354608:29
*** jlibosva has joined #openstack-neutron08:30
*** yamamoto_ has quit IRC08:31
openstackgerritJohn Schwarz proposed a change to openstack/python-neutronclient: Don't get keystone session if using noauth  https://review.openstack.org/12502208:31
*** kbyrne has quit IRC08:33
*** kbyrne has joined #openstack-neutron08:38
*** sergsh has joined #openstack-neutron08:39
ajoamotoki, salv-orlando : https://bugs.launchpad.net/neutron/+bug/137612808:39
ajoif you can, look at this,08:39
ajoI'm working on it, if you can, set milestone: RC2 and raise prio08:39
ajoihrachyshka, ^08:39
ihrachyshkayeah, that's a nasty bug. if we release Juno without it fixed we're doomed to insert dirty hacks to fallback to old values08:40
ihrachyshka(in downstream)08:40
ajoyes, or making shell scripts to translate from old pid files to new pid files,08:41
ajoor back on downgrades, not nice08:41
*** igordcard has joined #openstack-neutron08:41
ajoeven in some cases, translation can't be reverted when external_process is used for more affairs.08:42
*** NellyK has joined #openstack-neutron08:53
*** amuller has joined #openstack-neutron08:55
amullerkevinbenton: yo08:57
*** aepifanov has joined #openstack-neutron08:58
*** NellyK has quit IRC08:58
*** sambetts has joined #openstack-neutron09:06
openstackgerritCedric Brandily proposed a change to openstack/python-neutronclient: Correct 4xx/5xx response management in SessionClient  https://review.openstack.org/12528909:06
openstackgerritA change was merged to openstack/neutron: ML2: move L3 cleanup out of network transaction  https://review.openstack.org/12497509:08
*** dsneddon has quit IRC09:09
*** dsneddon has joined #openstack-neutron09:10
salv-orlandoajo: does the respawn result in loss of functionality?09:14
salv-orlandolike metadata does not work anymore?09:14
*** jp_at_hp has joined #openstack-neutron09:15
ajosalv-orlando, let me verify that, a few minutes09:16
*** afazekas has joined #openstack-neutron09:17
ajoI'm not sure if the new one failed to start, thus, not "bringing up" any code changes (code bug fixes, etc)09:17
ajothey should collide on port, at least,09:17
ajobut I remember they replicated... hmm, that was before PTO, ... well, checking and reporting in a minute09:18
*** oomichi has quit IRC09:20
openstackgerritAssaf Muller proposed a change to openstack/neutron: Improve ml2 port binding failure logging  https://review.openstack.org/12511509:22
*** jprovazn has joined #openstack-neutron09:26
*** ramishra_ has joined #openstack-neutron09:28
*** dsneddon has quit IRC09:30
*** ramishra has quit IRC09:30
*** dsneddon has joined #openstack-neutron09:31
*** mwagner_lap has quit IRC09:36
*** wendar has quit IRC09:38
*** wendar has joined #openstack-neutron09:38
*** dsneddon has quit IRC09:50
*** dsneddon has joined #openstack-neutron09:51
*** rotbeard has joined #openstack-neutron09:55
*** kopparam has quit IRC09:59
*** kopparam has joined #openstack-neutron09:59
*** kopparam has quit IRC10:01
*** Longgeek has quit IRC10:01
ajosalv-orlando, yes, it results in failure10:02
ajochecking http://169.254.169.254/2009-04-04/instance-id10:02
ajofailed 1/20: up 6.39. request failed10:02
ajofailed 2/20: up 9.10. request failed10:02
ajofailed 3/20: up 11.65. request failed10:02
ajofailed 4/20: up 13.85. request failed10:02
ajoI enabled metadata (disabled configdrive), checked out parent to change, started all neutron services, checked that it works,10:03
*** Longgeek has joined #openstack-neutron10:03
ajothen checked out master, restarted all neutron services, stopped instance, started instance and it fails10:03
*** ollivier has joined #openstack-neutron10:04
*** Longgeek has quit IRC10:04
ajonot sure of the exact failure mechanism, could be the unix socked disconnected from ns-metadata-proxy to neutron-metadata-server which get's restarted, thus the ns-metadata-proxy needs restart, but just guessing here10:04
*** Longgeek has joined #openstack-neutron10:04
ajoihrachyshka, ^ worse than it initially looked... metadata stops working until you kill all neutron-ns-metadata-proxy and restart l3-agent / dhcp-agent10:06
jschwarzajo++, nice find10:08
*** kopparam has joined #openstack-neutron10:10
*** dsneddon has quit IRC10:11
ajoI'd like to understand the exact failure mechanism10:12
*** Longgeek_ has joined #openstack-neutron10:13
*** dsneddon has joined #openstack-neutron10:13
*** pcm_ has joined #openstack-neutron10:14
*** pcm_ has quit IRC10:15
*** pcm_ has joined #openstack-neutron10:16
*** Longgeek_ has quit IRC10:16
*** Longgeek has quit IRC10:16
*** SridharG has left #openstack-neutron10:16
*** Longgeek has joined #openstack-neutron10:16
*** pasquier-s_ has joined #openstack-neutron10:17
*** xuhanp has joined #openstack-neutron10:17
*** mwagner_lap has joined #openstack-neutron10:20
*** pasquier-s has quit IRC10:20
*** Longgeek_ has joined #openstack-neutron10:22
*** Longgeek_ has quit IRC10:23
yamahataamotoki: ping?10:23
*** Longgeek_ has joined #openstack-neutron10:24
amotokiyamahata: pong10:24
yamahataamotoki: does recheck-nec bug ### work? or only recheck-nec?10:24
amotokiyamahata: only "recheck-nec" works now.10:26
*** Longgeek has quit IRC10:26
amotokiyamahata: which one failed?10:26
yamahataamotoki: got it. thanks.10:26
yamahataamotoki: https://review.openstack.org/#/c/124699/   seems glance-manage issue10:27
amotokiyamahata: I found it.10:27
yamahataamotoki: it seems to be fixed in jenkins. So I expect same with third party CIs10:28
amotokiyamahata: thanks. I just retriggered yours.10:28
*** Longgeek_ has quit IRC10:28
yamahataamotoki: thank you.10:28
*** Longgeek has joined #openstack-neutron10:28
salv-orlandoajo: I think the root cause is simple and two proxies should not exist anyway… if you want to dig deep in understanding how it fails go ahead, but I think we can just fix this problem by restoring the same naming scheme as icehouse10:29
*** SridharG has joined #openstack-neutron10:29
salv-orlandootherwise we will have to stop and restart all metadata proxies upon upgrade, which will likely require some ad-hoc operation10:29
salv-orlandoI don’t have a specific reason for using <uuid>/pid or <uuid>.pid - do you?10:30
ajosalv-orlando, I think it's good to keep the new naming scheme for new services10:30
*** Longgeek has quit IRC10:30
ajoto avoid an excesive listing in the "external" directory10:30
amotoki+1 from me too. (i just returned to my desk)10:30
ajoso pid files (per-service) keep contained inside their uuid dirs10:30
ajofor example10:30
*** Longgeek has joined #openstack-neutron10:31
ajouuid1/pid10:31
ajouuid1/pid.raddvd10:31
ajoradvd10:31
salv-orlandoyeah it’s cool. But it breaks the world.10:31
salv-orlandothis makes it uncool10:31
ajosalv-orlando, I have a patch ready10:31
salv-orlandowhat does this patch do?10:31
ajoto specify an specific pid path within the external directory10:31
*** rotbeard has quit IRC10:32
ajomay be we can use this optional parameter with "old" services,10:32
ajoand keep thew new convention to others10:32
ajoactually, only the metadata-proxy is affected10:32
ajobecause dhcp puts it's files in dhcp/uuid/pid10:32
ajoalso, this patch should help in later refactor (dhcp/processmonitor) to use external_process in dhcp agent10:33
ajoinstead of replicating10:33
*** Longgeek_ has joined #openstack-neutron10:33
amotokiajo: how about vpn-agent openswan?10:33
ajohmmmmm, too, right10:34
ajoamotoki++10:34
*** yamahata has quit IRC10:34
ajoamotoki, salv-orlando , if you find it right, I could try to propose this mixed solution, keeping the new services in-directory, and old services with the old naming convention10:36
*** Longgeek has quit IRC10:36
openstackgerritAlexey I. Froloff proposed a change to openstack/neutron: Only raise exception if no IP wass allocated in dual-stack network  https://review.openstack.org/12506110:36
ajootherwise, if the "external" directory gets too cluttered we will have performance problems due to the file system10:36
salv-orlandoIn principle I don’t like mixed stuff - but go ahead and push the code ;)10:37
salv-orlandoI take my scotch dry - always10:37
ajosalv-orlando, I understand your point of view10:37
ajohahaha10:37
*** bradjones has joined #openstack-neutron10:38
ajosalv-orlando, other option could be providing a migration tool for the pids (and future stuff)10:38
ajobad part is... upgrades are easy, downgrades.... can be impossible in some situations10:38
jschwarzamotoki, evening :) was wondering if you could take a look at a patch I made for neutronclient - a patch you approved some time ago (92390) broke noauth support so I made a fix for that (125022) :)10:38
amotokiajo: can we do that? I think we need to re-launch a process to change pid path.10:39
amotokiajo: "that" == "migration tool"10:39
ajoamotoki, for example, before agent restart: migration_tool --upgrade10:39
ajowould move all $external/*.pid to $external/*/pid10:39
ajowell, actually just in the middle would be the best10:40
ajostop / migrate-pids / start10:40
ajoall pid files are moved, and found by the agent at restart10:40
salv-orlandoamotoki: relaunching will surely fix the problem, but what ajo is trying to do is to move the uuid.pid files into uuid/pid so that upon restart the l3 agent won’t think those processes are dead10:41
ajoamotoki, but, for example,  I have a pending patch for Kilo, to refactor the dhcp agent, and plug in process monitor, that moves dnsmasq pid file from /var/neutron/dhcp/<uuid>/pid to /var/neutron/external/<uuid>/pid10:41
salv-orlandothat’s the “ad-hoc” thing that we’d need to do to ensure things work smoothly with the new naming convention10:42
ajoif we migrate that^, it can't be reverted10:42
ajosalv-orlando, correct10:42
salv-orlandoajio: one bit I do not understand… you’ve been referring to an “external” directory… which directory is that?10:43
amotokisalv-orlando: aio: I think I understand the case. what in my mind is a case where a running process maintains its pid file.10:43
ajosalv-orlando, neutron/agen/linux/external_process implementation10:43
amotokimetadata-proxy is not a case of this.10:43
ajoputs all the pid files in that /var/neutron/external/<uuid>...10:43
ajoit's actualy $state_path + /external/10:44
ajoamotoki, not metadata proxy, but neutron-ns-metadata-proxy10:44
ajothe one living in each namespace10:44
salv-orlandoajo:  understood. So the metata pid will be moved again as well right?10:44
salv-orlandoI meant the proxy pid10:45
ajosalv-orlando, not sure I understood your question10:45
amotokiajo: it seems my typo... ns-metadata-proxy. right.10:45
ajomoved by a "migration tool" you mean?10:45
salv-orlandothe ns-metadata-proxy process is a child process which will be managed by the child process monitor. Is that correct?10:45
ajosalv-orlando, for K, yes10:46
ajonot monitored for J yet :)10:46
salv-orlandoso it’s pid file will be moved into /var/neutron/external/<uuid> which is yet a different position compared to where we’re putting it in Juno10:46
ajosalv-orlando, correct, on current master10:46
ajoit moves from $external/<uuid>.pid to $external/<uuid>/pid10:47
salv-orlandobut we want to do <uuid>/pid in juno because it’s consistent with other stuff?10:47
salv-orlandolike radvdv10:47
salv-orlandomeh radvd10:47
ajo:)10:47
ajoI think radvdv does <uuid>/pid.radvd10:47
ajousing the new convention10:48
ajobut10:48
ajothe question would be10:48
ajoa simple patch like10:48
ajo<uiid>.pid.radvd10:48
ajoand keeping all in a dir,10:48
ajowould be ok?10:48
ajosalv-orlando, amotoki let me check something,10:48
salv-orlandoajo: ok - I’m just slightly confused… do your checks. But I reckon we’re probably making this problem more complex than what it has to be10:49
ajohow good/bad does FS scale for a 10k*2 file listing... that'd be ok for <10k nets10:49
ajosalv-orlando, I'm trying to go with your simpler suggestion (keeping the old convention)10:49
ajomay be there's no actual scaling problem10:50
salv-orlandoajo: but maybe we can also use the new convention… can’t we just assume in the code that the pidfiel can be available either in the new or old position10:50
*** diegows has joined #openstack-neutron10:50
ajoI prefer not keeping two conventions too10:50
salv-orlandook10:50
ajohmmm10:51
ajosalv-orlando, that sounds good too10:51
ajoso we look for the new, if not existing, check for the old... otherwise we create on the new10:51
salv-orlandomy point re your blueprint was anyway that if you’re going to move this stuff again for K - let’s keep the pidfiles where are now10:51
salv-orlandoand then let’s face the problem in kilo10:51
ajosalv-orlando++10:51
*** ramishra_ has quit IRC10:52
salv-orlandoajo: there is one more clarification i need10:52
ajosalv-orlando, sure10:52
ajoask10:52
salv-orlandothe metadata pid file is now in $state_path/<uuid>/pid10:52
salv-orlandowhere the uuid is usually a router id but in some cases could be a subnet id10:52
salv-orlandocorrect?10:52
ajocorrect, after the radvd change10:52
salv-orlandowhereas the radvd pidfile is called radvd.pid10:53
salv-orlandoand is in the same dir10:53
ajoyes, it could be other resource id, not sure how lbaas / vpnaas use it10:53
salv-orlandodon’t you see an inconsistency as well? like the metadata proxy pid file should be called ns-meta-proxy.pid?10:53
ajoyes10:53
ajousing a new convention10:53
ajoI would put all services under a named pi10:54
ajopid10:54
salv-orlandootherwise the vpn will start openswan call the pidfile “pid” too and overwrite the one for the meta proxy10:54
ajohmm, that's a deeper problem, let's check for the vpnaas / lbaas implementation10:55
ajosalv-orlando, may be you found an extra bug10:55
*** aix has joined #openstack-neutron10:55
*** Longgeek_ has quit IRC10:56
ajopff10:57
ajosalv-orlando, ha-proxy implementation, again, duplicates parts of external_process10:57
ajoand moves files to $state_path/lbaas10:57
ajothat's why it's not happening there10:57
ajo(collision I mean)10:57
ajoor moved pid10:58
ajougly10:58
*** ramishra has joined #openstack-neutron11:01
ajosalv-orlando, for openswan/ipsec it seems to be $state_path/ipsec/<uuid>/var/run/pluto11:03
ajoouch11:03
ajoalso, it's own implementation11:03
ajowe reinvent wheel everywhere11:03
* ajo is sad ;]11:04
jschwarzajo, I remember looking at the haproxy process flow a while back and figuring out what was the reason they didn't use ProcessMonitor11:04
jschwarzajo, I forgot it now though... :< sec11:04
ajojschwarz, external_process, Process monitor is new, yet not used by any11:05
ajojschwarz, it could be external_process wasn't available at that time?11:05
jschwarzajo, might as it was written a year ago11:05
jschwarzajo, but I remember finding a reason it couldn't be easily ported11:05
ajoit happened with dhcp, and I got it refactored to work with external_process11:05
jschwarzajo, ah yes11:07
jschwarzajo, haproxy receives '-p' arguments and it writes the PID of its own child processes to the pid11:07
jschwarzajo, then there's a kill_all function which knows to kill every pid in that file11:07
jschwarzajo, which is unlike the logic in external_process iirc?11:08
ajojschwarz, correct, but external_process could be extended to handle specific kills methods/callbacks11:09
ajono reason to reinvent wheels for that :)11:09
jschwarzajo, agreed - but I said "couldn't be easily ported"11:09
ajojschwarz, :)11:10
jschwarzajo, extending external_process != easy ;-)11:10
ajonahh11:10
jschwarzajo, there's also no way to execute processes in daemon mode11:10
ajomaintaining replicated code != easy :D11:10
jschwarzajo, need to write that now for the integration code11:10
jschwarzintegration testing code*11:10
ajojschwarz, what do you mean by "in daemon mode" ?11:11
ajodaemon mode generally drops pid, and detaches11:11
jschwarzajo, aye. there's a Daemon class thrown around somewhere in the agent/linux dir, but any agent/proxy process who wants to use it need to inherit from it, making a mess11:12
*** baoli has joined #openstack-neutron11:12
jschwarzajo, I intend to extend it a bit11:12
jschwarzajo, also reminds me I wanted to talk to you about extending ProcessMonitor a bit ;-D11:12
*** dims has joined #openstack-neutron11:12
ajojschwarz, sure11:12
ajoanytime you can :)11:13
jschwarzajo, will ping you later today I think. will let you finish up with salv first11:13
ajoamotoki, salv-orlando :11:15
ajowith 40k files:11:15
ajo[majopela@f20-devstack test]$ time ls *10000*11:15
ajotest1000011:15
ajoreal0m0.040s11:15
ajouser0m0.023s11:15
ajosys0m0.016s11:15
ajowith 1 file:11:15
ajo[majopela@f20-devstack t]$ time ls *10000*11:15
ajotest1000011:15
ajoreal0m0.001s11:15
ajouser0m0.001s11:15
ajosys0m0.001s11:15
ajobut let me try with 10000 files, because, that'd be more realistic (splitting pid files inside the <uuuid> dir, which also needs seek in the "external" dir)11:16
*** bradjones has quit IRC11:20
*** Longgeek has joined #openstack-neutron11:20
salv-orlandoajo: it’s trivial to understand that a single pid file would be a whole lot faster… but what I do not understand is whether you’re talking about something you want to do now for juno or later for kilo11:20
salv-orlandoajo: btw, you’d then have to add the time to search for a specifiic pid within the file, I guess - that might add some time11:20
ihrachyshkasalv-orlando: hey. there is also probably a regression introduced by switching to oslo.db: https://bugs.launchpad.net/oslo.db/+bug/1376211 though oslo.db is not strictly a part of integrated release, I guess we need to track it somehow11:20
ajono salv-orlando11:21
ihrachyshkaajo: does that 0.04s really influence anything?..11:21
*** Longgeek has quit IRC11:21
ajoI'm trying to compare the FS overhead of <uuuid>/x.pid scheme11:21
ajovs <uuid>.x.pid scheme11:21
*** Longgeek has joined #openstack-neutron11:22
ajo[.$x].pìd  would be more accurate11:22
ajoihrachyshka, probably not11:22
ajoI believe it's quite negligible11:22
ajoand we could put them all together to keep the old convention for now11:22
ajoand avoid having two11:22
ajowell11:22
ajoif we had to look for those 4*10k files11:23
*** pasquier-s_ has quit IRC11:23
ajothat'd become:11:23
ihrachyshkaajo: yes, that patch for two schemes in parallel sucks in that it's not clear why we even need both11:23
ihrachyshkaif it's possible to stick to one scheme, that would be great11:23
ajo640seconds11:23
salv-orlandoihrachyshka: we might constraint nuetron to use at least an oslo.db version where this bug is fixed, but then this means that we will make neutron RC-1 depend on a oslo.db release11:23
ajoif only counting the sys part11:23
salv-orlandoperhaps something like that might be handled with release notes, considering is a bit of a corner case.11:23
ihrachyshkasalv-orlando: I guess we should push oslo.db team somehow to fix that before Juno :)11:24
ajoanyway11:24
ihrachyshkaand then bump11:24
ajolooking at those numbers, <uuid>/[x].pid doesn't solve that performance issue either11:24
*** bradjones has joined #openstack-neutron11:24
ihrachyshkasalv-orlando: not really a corner case. each reboot ends up with neutron and other services dead11:25
openstackgerritRossella Sblendido proposed a change to openstack/neutron: Functional test IPAM DB operation  https://review.openstack.org/12413611:25
ihrachyshkasalv-orlando: at least in RH world, we don't depend on mysql start before invoking neutron11:25
*** rakesh_hs has quit IRC11:25
ihrachyshkasalv-orlando: this is partly because we don't know which db is to use11:25
ajoso salv-orlando / ihrachyshka I would just stay with the old convention, and think about performance with something definitive, not an intermediate change (I'm considering the current <uuid>/pid intermediate and not definite when looking at performance)11:25
salv-orlandoajo: I don’t want to sound like a nagger, but at this stage I would worry more about ensuring metadata works correctly after an upgrade, and leave the scalability enhancement for a later patch. Unless you’ve found out that scalability now is terrible.11:25
ajosalv-orlando, I totally agree with you11:26
ajoafter my experiments11:26
ihrachyshkasalv-orlando++11:26
ihrachyshkano need for premature optimization11:26
ajothe new naming scheme from radvd patch it's not actually optimizing11:26
ajoso let's drop it11:26
salv-orlandoihrachyshka, ajo: I’d love to optimize, but we’re also pushing to get a RC-1 out of the door.11:26
ajosure11:26
ajoI agree, not the moment for that11:27
ajoand probably there's not easy solution11:27
salv-orlandoihrachyshka: do you think it’s reasonable to expect a oslo.db fix and new release in 24 hours?11:27
ajoprobably splitting uuid in subfolders, but let's think about that for K/H11:27
ihrachyshkasalv-orlando: no way!11:27
ihrachyshka:)11:27
ihrachyshkasalv-orlando: why 24h?11:27
salv-orlandobecause mestery said he would like to do a RC-1 by tomorrow ;)11:28
ihrachyshkasalv-orlando: ah, RC-1 is fine to be broken ;)11:28
ihrachyshkasalv-orlando: let's target final release though!11:28
* jschwarz lols11:28
*** Longgeek has quit IRC11:28
*** Longgeek has joined #openstack-neutron11:28
salv-orlandoihrachyshka: cool. I’ll bring that up with mestery and markmcclain when they’re up.11:28
salv-orlandoDo you have already alerted rpodoliaka?11:29
ihrachyshkasalv-orlando: can we add neutron to the bug and set a milestone for that? or is it overkill?11:29
ihrachyshkanot yet, I'll do it now11:29
salv-orlandoihrachyshka: we can add neutron, sure. For the milestone...11:29
salv-orlandowe don’t have yet either a final juno or RC-2 label I’m afraid11:30
*** sgordon_ has quit IRC11:30
*** lazy_prince is now known as killer_prince11:31
*** feleouet has quit IRC11:31
*** feleouet has joined #openstack-neutron11:31
*** vthapar has quit IRC11:33
*** baoli has quit IRC11:35
ihrachyshkasalv-orlando: ok, let's see whether oslo team is going to add their own milestone for the issue11:37
*** aix has quit IRC11:40
*** aix has joined #openstack-neutron11:41
openstackgerritAnn Kamyshnikova proposed a change to openstack/neutron: Test  https://review.openstack.org/12532811:44
*** pasquier-s has joined #openstack-neutron11:47
openstackgerritJohn Schwarz proposed a change to openstack/python-neutronclient: Don't get keystone session if using noauth  https://review.openstack.org/12502211:49
*** tsg has joined #openstack-neutron11:53
*** Longgeek has quit IRC11:55
*** pasquier-s has quit IRC11:55
*** tnurlygayanov has quit IRC11:57
*** baoli has joined #openstack-neutron12:01
*** aix has quit IRC12:01
*** sgordon_ has joined #openstack-neutron12:08
*** tsg has quit IRC12:08
*** kopparam_ has joined #openstack-neutron12:09
*** kopparam has quit IRC12:11
*** yatin has quit IRC12:12
openstackgerritAssaf Muller proposed a change to openstack/neutron: Make sudo check in ip_lib.IpNetnsCommand.execute optional  https://review.openstack.org/12533712:12
*** pasquier-s has joined #openstack-neutron12:13
*** aix has joined #openstack-neutron12:14
*** baoli has quit IRC12:14
*** ramishra has quit IRC12:19
*** baoli has joined #openstack-neutron12:19
*** JayJ has joined #openstack-neutron12:20
*** cnesa15 has joined #openstack-neutron12:20
openstackgerritAssaf Muller proposed a change to openstack/neutron: Bulk up L3 HA update_router_status RPC  https://review.openstack.org/12533912:20
*** dims has quit IRC12:21
*** dims has joined #openstack-neutron12:21
*** carol-b has joined #openstack-neutron12:24
*** kopparam_ has quit IRC12:26
*** igordcard has quit IRC12:27
*** parkit has joined #openstack-neutron12:31
*** julim has joined #openstack-neutron12:36
*** yamahata has joined #openstack-neutron12:38
*** igordcard has joined #openstack-neutron12:39
*** pcm__ has joined #openstack-neutron12:41
*** pcm__ has quit IRC12:42
*** Longgeek has joined #openstack-neutron12:42
*** Longgeek has quit IRC12:42
*** Longgeek has joined #openstack-neutron12:43
*** Longgeek_ has joined #openstack-neutron12:46
*** Longgeek_ has quit IRC12:47
*** Longgeek_ has joined #openstack-neutron12:48
pcm_carol-b: Hi carol12:48
*** tsg has joined #openstack-neutron12:49
*** Longgeek has quit IRC12:49
carol-bpcm_; test_ack12:50
pcm_carol-b: Here is where you can post questions about the bug you are trying to fix...12:51
*** skolekonov has quit IRC12:53
*** rook-roll is now known as rook12:54
*** thomasem has joined #openstack-neutron12:56
*** nus has quit IRC12:58
*** amotoki has quit IRC13:00
openstackgerritAnn Kamyshnikova proposed a change to openstack/neutron: Allow to set ipv6_ra_mode,ipv6_address_mode to None  https://review.openstack.org/12532813:02
*** kbringard has joined #openstack-neutron13:03
*** geekinutah has quit IRC13:04
*** rpothier has joined #openstack-neutron13:07
*** evgenyf has quit IRC13:07
*** rotbeard has joined #openstack-neutron13:18
openstackgerritSalvatore Orlando proposed a change to openstack/neutron: NSX: Remove logic for creating chained logical switches  https://review.openstack.org/12534813:19
*** baoli has quit IRC13:19
*** yfried|afk has quit IRC13:19
*** yfried|afk has joined #openstack-neutron13:20
openstackgerritMiguel Angel Ajo proposed a change to openstack/neutron: Fix pid file location to avoid I->J changes that break metadata.  https://review.openstack.org/12534913:21
*** ramishra has joined #openstack-neutron13:21
ajosalv-orlando, ihrachyshka , https://review.openstack.org/#/c/125349/ tell me how does it look, checking for unit testing changes (or extension) atm.13:23
*** moha_hunt has joined #openstack-neutron13:23
*** garyduan has joined #openstack-neutron13:29
*** gduan has quit IRC13:30
*** afazekas has quit IRC13:30
*** afazekas has joined #openstack-neutron13:33
*** ramishra has quit IRC13:34
*** ramishra has joined #openstack-neutron13:35
*** signed8bit has joined #openstack-neutron13:38
openstackgerritRossella Sblendido proposed a change to openstack/neutron: Add unique constraints in IPAvailabilityRange  https://review.openstack.org/12350413:39
*** ramishra has quit IRC13:39
*** jaypipes has joined #openstack-neutron13:42
openstackgerritMiguel Angel Ajo proposed a change to openstack/neutron: Fix pid file location to avoid I->J changes that break metadata.  https://review.openstack.org/12534913:45
ajojschwarz, ihrachyshka , salv-orlando , ^13:45
*** armax has joined #openstack-neutron13:45
jschwarzajo, iglob? :P13:45
ajojschwarz, iglob?13:46
jschwarzajo, ihar made a comment someplace about maybe using glob.iglob13:46
jschwarzinstead of glob.glob13:46
ajohmm let me check13:46
*** dbite has joined #openstack-neutron13:49
*** parkit has quit IRC13:50
openstackgerritEdward Hope-Morley proposed a change to openstack/neutron: Adds optional timeout for neutronclient  https://review.openstack.org/6118313:52
openstackgerritRomil Gupta proposed a change to openstack/neutron: Stale VXLAN and GRE tunnel port deletion  https://review.openstack.org/12100013:52
*** bradjones has quit IRC13:53
openstackgerritMiguel Angel Ajo proposed a change to openstack/neutron: Fix pid file location to avoid I->J changes that break metadata.  https://review.openstack.org/12534913:53
jschwarzajo++13:54
ajo:D I had missed ihrachyshka comment13:55
ajook, it looks like ready now13:56
*** kopparam has joined #openstack-neutron13:58
*** krtaylor has quit IRC14:01
*** shashankhegde has joined #openstack-neutron14:04
*** SridharG has left #openstack-neutron14:06
jschwarzajo++14:07
jschwarzwill look at the code when I finish my current task14:07
*** parkit has joined #openstack-neutron14:11
*** AbsinthMind has joined #openstack-neutron14:11
*** parkit has quit IRC14:12
*** mrsnivvel has quit IRC14:12
*** jlibosva has quit IRC14:13
*** nlahouti has joined #openstack-neutron14:13
*** nmagnezi has quit IRC14:14
*** jswiders has left #openstack-neutron14:15
*** geekinutah has joined #openstack-neutron14:16
*** nlahouti has quit IRC14:16
*** openstackgerrit has quit IRC14:18
*** kopparam has quit IRC14:18
amullerajo: I predict a few functional tests will fail on that patch14:18
*** kopparam has joined #openstack-neutron14:19
ajoamuller, I checked those :)14:19
ajoamuller, which ones?14:19
ajofor the firts patchset it happened, because there was an actual error on the code14:19
ajobut fixed on ps2/314:19
*** openstackgerrit has joined #openstack-neutron14:19
*** kopparam has quit IRC14:23
*** parkit has joined #openstack-neutron14:24
*** parkit has quit IRC14:25
*** krtaylor has joined #openstack-neutron14:25
*** kopparam has joined #openstack-neutron14:27
*** pasquier-s has quit IRC14:27
amullerajo: did you run all the functional tests with your patch?14:27
amullerjschwarz: https://review.openstack.org/#/c/124136/6/neutron/tests/functional/db/test_ipam.py14:29
*** mrsnivvel has joined #openstack-neutron14:30
*** annegentle has joined #openstack-neutron14:30
ajoamuller, yes14:30
amullerajo: Okie14:30
*** dboik has joined #openstack-neutron14:32
jschwarzrossella_s, ping14:32
rossella_sjschwarz: pong14:33
jschwarzrossella_s, I'm looking at https://review.openstack.org/#/c/124136/6 and have a few questions :)14:34
rossella_sjschwarz: shoot :)14:34
jschwarzrossella_s, I understand you're writing functional tests which use a DB and that you clean the DB before each test14:35
jschwarzdid you consider using SqlTestCase (in neutron/tests/unit/testlib_api.py?)14:35
amullerajo: I'm really surprised that everything passes, I'd expect unit tests that assert strings like some/path/pid14:36
amullerajo: for keepalived14:37
*** jamiehan_ has joined #openstack-neutron14:37
amullerajo: If you give Jenkins enough time we'll know where the failures are =D14:37
*** dhellmann has joined #openstack-neutron14:38
*** dbite has quit IRC14:38
rossella_sjschwarz: yes...the problem was that I don't want to use sqlite but to test using mysql or postgres14:38
*** dhellmann has quit IRC14:38
jschwarzrossella_s, why?14:38
rossella_sjschwarz: why I don't want to use sqlite?14:38
jschwarzrossella_s, yeah, but it was a silly question14:39
rossella_sjschwarz: no question is silly :) Unit tests already use sqlite...I wanted to go a step further14:39
*** jamiehannaford has quit IRC14:40
jschwarzrossella_s, so why can't you use SqlTestCase with mysql or postgres?14:40
jschwarzrossella_s, I'd expect the functions there to work for all of those DB types14:41
*** dhellmann has joined #openstack-neutron14:41
*** jamiehan_ has quit IRC14:41
jschwarzrossella_s, I'm asking because I'm starting to write something similar to what you're doing here for integration tests, so I'm curious about the process you went through the code writing14:42
ihrachyshkaamuller: ain't no body got time for that14:42
*** parkit has joined #openstack-neutron14:43
rossella_sjschwarz: integration tests? interesting :) ...so let me explain better because I fear I was a bit misleading. So I could use SqlTestCase but that wouldn't spare much code. If you see in my functional tests in the setup I do the following things 1) override the database connection string 2) create the test DB and add clean up 3) create the models14:44
rossella_sso if I use SqlTestCase I have to do 1) and 2) anyway and before calling SqlTestCase.setup because otherwise it won't use the database connection that I want to use14:46
*** otherwiseguy has joined #openstack-neutron14:46
*** ramishra has joined #openstack-neutron14:46
jschwarzrossella_s, I see. We plan to wipe the DB clean between each test so they will have minimum interaction between them14:46
jschwarzrossella_s, you mean because of that 'db_api.get_engine()' part? why won't it use the DB connection you want?14:46
*** markmcclain has joined #openstack-neutron14:47
rossella_sjschwarz: yes exactly. I mean you have to call that line after having override the config db connection string14:47
*** markmcclain has quit IRC14:47
*** sgordon_ has quit IRC14:47
*** openstackgerrit has quit IRC14:47
*** markmcclain has joined #openstack-neutron14:48
rossella_sjschwarz: it's possible to use SqlTestCase ...my thought was just to try to keep things simple. So I didn't want to have SqlTestCase as super class just to spare one line :)14:48
jschwarzrossella_s, so you can override the config db connection string in the __init__ and then use SqlTestCase's as is (with the added(?) bonus of wiping DB between each test)14:48
*** jamiehannaford has joined #openstack-neutron14:49
*** openstackgerrit has joined #openstack-neutron14:49
jschwarzrossella_s, the dsvm-functional test suite runs ./stack.sh which creates a mysql DB and initializes it, right?14:49
jschwarzrossella_s, so I think you can skip 2) since the DB scheme already exists14:50
rossella_sjschwarz: if my test inherits from SqlTestCase, I could simply override the connection string and then call super.setUp...it should work14:50
jschwarzrossella_s, :)14:50
*** alexpilotti has joined #openstack-neutron14:50
rossella_sjschwarz: the point is that I am using a test DB that it's created in the test. That's why I don't need the clean up of SqlTestCase14:50
rossella_sand that's why I need 2)14:51
*** nyechiel has quit IRC14:51
rossella_sjschwarz: I prefer using a test db so that the test is isolated14:51
*** matrohon has quit IRC14:52
jschwarzrossella_s, ack14:52
rossella_sjschwarz: :) ...does it make sense?14:52
*** parkit has quit IRC14:52
jschwarzrossella_s, yes, now I see that14:52
*** ihrachyshka has quit IRC14:53
rossella_sjschwarz: thanks for your reviews btw...good luck with the integration test!14:53
openstackgerritRossella Sblendido proposed a change to openstack/neutron: Remove locking from network and subnet delete op  https://review.openstack.org/11536014:54
jschwarzrossella_s, thanks14:54
jschwarzrossella_s, I think you can make the code a lot simpler btw14:56
*** dave_tucker_zzz is now known as dave_tucker14:56
jschwarzrossella_s, you can move all the stuff that involves the config overriding to DBTestFramework.__init__, and then use a similar structure to what salv used for SqlTestCase for the DB creation and drop14:56
*** dave_tucker is now known as dave_tucker_zzz14:56
*** dboik has quit IRC14:57
*** dboik has joined #openstack-neutron14:57
*** dhellmann_ has quit IRC14:57
*** thedodd has joined #openstack-neutron14:57
rossella_sjschwarz: I am sure the code can be simplified...anyway I don't really see your point. The config overriding and db creation is in the setup method...why move it to init? Also not sure what structure you are referring to...in  SqlTestCase there's no DB creation14:59
*** yamahata has quit IRC14:59
rossella_sjschwarz: anyway please feel free to comment directly on the review, I think it's easier because you can point directly at the code15:00
jschwarzrossella_s, in the future, integration tests will probably want to do exactly what you're doing, so writing the code as generic as possible now will help save patchsets later :>15:00
*** shashankhegde has quit IRC15:00
jschwarzrossella_s, I'll write my thoughts inline then.15:01
*** dhellmann_ has joined #openstack-neutron15:01
rossella_sjschwarz: agreed, I like generic code. I will read your comments later then ;)15:01
*** carl_baldwin has joined #openstack-neutron15:02
*** chuckC_ has joined #openstack-neutron15:02
jschwarzrossella_s, thank you very much for your time :)15:02
*** openstackgerrit has quit IRC15:02
rossella_sjschwarz: actually thank you :) let's try to collaborate to make the code as re-usable as possible. I don't know much about the integration tests, anyway it looks interesting15:03
jschwarzrossella_s, the writing is ongoing (early stages of framework writing) so any thoughts will be helpful15:03
*** evgenyf has joined #openstack-neutron15:04
rossella_sjschwarz: is there some link, review that I can look at?15:04
*** openstackgerrit has joined #openstack-neutron15:04
jschwarzrossella_s, it is as early as I started writing it today ;-)15:05
* rook waves at rkukura!!15:05
jschwarzrossella_s, when i have something up i'll try to remember adding you as a reviewr15:05
*** dhellmann_ has quit IRC15:05
rkukurarook:  :)15:06
rossella_sjschwarz: great, thanks!15:06
*** dhellmann_ has joined #openstack-neutron15:06
*** ramishra has quit IRC15:06
*** ramishra has joined #openstack-neutron15:07
*** mdorman has joined #openstack-neutron15:08
*** Sukhdev has joined #openstack-neutron15:08
*** cjellick has joined #openstack-neutron15:09
*** mwagner_lap has quit IRC15:09
*** sergsh has left #openstack-neutron15:10
*** ramishra has quit IRC15:11
*** geekinutah has quit IRC15:12
*** parkit has joined #openstack-neutron15:13
*** parkit has quit IRC15:14
*** test_ has joined #openstack-neutron15:16
openstackgerritAssaf Muller proposed a change to openstack/neutron: Replace keepalived notifier bash script with Python  https://review.openstack.org/12538415:17
*** mrsnivvel has quit IRC15:17
*** mitz_ has quit IRC15:18
*** xgerman has joined #openstack-neutron15:18
openstackgerritRomil Gupta proposed a change to openstack/neutron: Stale VXLAN and GRE tunnel port deletion  https://review.openstack.org/12100015:18
jschwarzrossella_s, done :)15:18
*** jobewan has quit IRC15:19
rossella_sjschwarz: :)15:22
*** mitz_ has joined #openstack-neutron15:23
*** carol-b is now known as cbouch_15:24
*** geekinutah has joined #openstack-neutron15:24
*** karimb has quit IRC15:25
*** test_ has quit IRC15:25
*** mrsnivvel has joined #openstack-neutron15:25
*** sbfox has quit IRC15:26
*** dhellmann_ has quit IRC15:26
*** cbouch__ has joined #openstack-neutron15:27
*** cbouch__ has quit IRC15:27
*** dhellmann_ has joined #openstack-neutron15:28
*** nlahouti has joined #openstack-neutron15:30
*** absubram has joined #openstack-neutron15:30
*** cbouch_ has quit IRC15:30
*** SridharRamaswamy has joined #openstack-neutron15:32
*** sdake has joined #openstack-neutron15:32
*** cbouch has joined #openstack-neutron15:32
*** nyechiel has joined #openstack-neutron15:33
*** dhellmann has quit IRC15:34
*** dhellmann_ is now known as dhellmann15:34
*** karimb has joined #openstack-neutron15:37
*** lori is now known as lori|away15:37
*** karimb has quit IRC15:37
*** karimb has joined #openstack-neutron15:37
*** annegentle has quit IRC15:38
*** annegentle has joined #openstack-neutron15:38
openstackgerritMatthew Treinish proposed a change to openstack/neutron: Add db migrations test framework with data  https://review.openstack.org/12494615:39
*** igordcard has quit IRC15:42
*** dvorkinista has joined #openstack-neutron15:42
*** mikedillion has joined #openstack-neutron15:42
*** xuhanp has quit IRC15:43
*** otherwiseguy has quit IRC15:46
*** jorgem has joined #openstack-neutron15:46
*** devvesa has joined #openstack-neutron15:47
jschwarzrossella_s, messed around with the code I pastebin'd for you on your patch - the __init__ part is no good because config_parse should be called before the db connection overriding part. Calling self.parse_config() manually beforehand fixes it15:49
*** tnadeau has joined #openstack-neutron15:51
*** otherwiseguy has joined #openstack-neutron15:51
*** numan has quit IRC15:54
*** a_le has quit IRC15:55
*** amotoki has joined #openstack-neutron15:59
*** geekinutah has quit IRC16:00
*** manishg has joined #openstack-neutron16:00
*** irenab has joined #openstack-neutron16:02
*** chuckC_ has quit IRC16:03
*** padkrish has joined #openstack-neutron16:03
*** afazekas has quit IRC16:04
*** k4n0 has quit IRC16:05
*** annegentle has quit IRC16:05
*** dfarrell07 has joined #openstack-neutron16:06
*** aepifanov has quit IRC16:06
jschwarzamotoki, ping16:09
amotokijschwarz: hi16:09
jschwarzamotoki, morning for you, is it? :)16:09
amotokijschwarz: late night in my side.16:10
openstackgerritJaume Devesa proposed a change to openstack/neutron-specs: Add dynamic routing spec  https://review.openstack.org/12540116:10
jschwarzamotoki, sounds like fun. can't sleep?16:10
amotokijschwarz: no, i am in ML2 meeting and then go to bed :)16:11
*** thomasem has quit IRC16:11
jschwarzamotoki, dedication ftw :)16:11
jschwarzamotoki, I replied on the inline question you asked regarding the noauth patch16:11
*** parkit has joined #openstack-neutron16:12
*** mestery has joined #openstack-neutron16:12
jschwarzamotoki, mainly wanted to see if you have any ideas about checking the noauth's coverage.. I looked around that code and it seems like there are quite a few places where neutronclient tries to contact keystone16:12
*** ygbo has quit IRC16:12
openstackgerritJaume Devesa proposed a change to openstack/neutron-specs: BGP dynamic routing  https://review.openstack.org/12540116:13
jschwarzamotoki, also I don't think it's possible to check if neutronclient doesn't talk to keystone in the elimination process since there could always be another place where keystone is contacted and we missed it16:13
amotokijschwarz: honestly i don't have good idea now. How did you notice that noauth does not work?16:13
*** JayJ_ has joined #openstack-neutron16:14
jschwarzamotoki, we're writing the framework for integration tests so we tried to make neutronclient work with keystone off :)16:15
amotokijschwarz: My quesiton in your review is just a question and it is not a blocker completely.16:15
*** rook_ has joined #openstack-neutron16:15
jschwarzamotoki, ack16:15
*** parkit has quit IRC16:17
*** dvorkinista has quit IRC16:17
*** med_ has quit IRC16:17
*** mordred has quit IRC16:17
*** ianw_pto has quit IRC16:17
*** rotbeard has quit IRC16:17
*** rpothier has quit IRC16:17
*** JayJ has quit IRC16:17
*** harlowja_away has quit IRC16:17
*** rharwood has quit IRC16:17
*** tcammann has quit IRC16:17
*** Apsu has quit IRC16:17
*** rook has quit IRC16:17
*** openstackgerrit has quit IRC16:18
*** tcammann has joined #openstack-neutron16:18
*** Apsu has joined #openstack-neutron16:18
*** med_ has joined #openstack-neutron16:18
*** med_ has quit IRC16:18
*** med_ has joined #openstack-neutron16:18
*** mordred has joined #openstack-neutron16:18
*** rotbeard has joined #openstack-neutron16:18
*** sgordon_ has joined #openstack-neutron16:18
jschwarzamotoki, sounds like there isn't an easy way to do this right now16:18
amotokijschwarz: agree16:19
*** rharwood has joined #openstack-neutron16:19
jschwarzamotoki, but probably once the integration tests are merged for K this will be checked (since if it doesn't work all the integration tests will fail)16:19
*** openstackgerrit has joined #openstack-neutron16:19
jschwarzamotoki, a loopy way but a way indeed ;->16:19
*** ianw_pto has joined #openstack-neutron16:19
*** amuller has quit IRC16:20
*** samuelmz has quit IRC16:20
jschwarzamotoki, I'm off to home so I wrote my thoughts on the patch. let me know if you come up with any ideas about this :)16:23
jschwarzgood night all16:23
amotokijschwarz: good night16:23
*** thomasem has joined #openstack-neutron16:23
*** parkit has joined #openstack-neutron16:24
*** diegows has quit IRC16:26
*** marun has joined #openstack-neutron16:27
*** sambetts has quit IRC16:27
*** jschwarz has quit IRC16:29
*** thomasem has quit IRC16:30
*** marun has quit IRC16:34
*** tnadeau_ has joined #openstack-neutron16:35
*** tnadeau has quit IRC16:35
*** tnadeau_ is now known as tnadeau16:35
openstackgerritMiguel Angel Ajo proposed a change to openstack/neutron: Fix pid file location to avoid I->J changes that break metadata.  https://review.openstack.org/12534916:36
*** mestery has quit IRC16:37
*** geekinutah has joined #openstack-neutron16:37
*** evgenyf has quit IRC16:38
*** absubram has quit IRC16:39
*** padkrish has quit IRC16:40
*** s3wong has joined #openstack-neutron16:40
*** devvesa has quit IRC16:41
*** jamiehannaford has quit IRC16:41
*** amotoki has quit IRC16:42
*** devvesa_ has joined #openstack-neutron16:43
*** vivek-ebay has joined #openstack-neutron16:43
*** diegows has joined #openstack-neutron16:47
*** Longgeek_ has quit IRC16:47
*** devvesa_ has quit IRC16:48
*** thomasem has joined #openstack-neutron16:49
*** safchain has quit IRC16:50
*** kopparam has quit IRC16:51
*** sbalukoff has quit IRC16:51
*** SumitNaiksatam has quit IRC16:52
*** kopparam has joined #openstack-neutron16:52
*** vivek-ebay has quit IRC16:52
*** irenab has quit IRC16:55
*** tsg has quit IRC16:55
*** suresh12 has joined #openstack-neutron16:56
*** kopparam has quit IRC16:56
*** thomasem_ has joined #openstack-neutron16:57
*** thomasem has quit IRC16:57
*** rotbeard has quit IRC16:58
*** thomasem_ has quit IRC16:58
*** moha_hunt has quit IRC17:00
*** sbfox has joined #openstack-neutron17:00
*** thedodd has quit IRC17:00
*** parkit has quit IRC17:00
*** moha_hunt has joined #openstack-neutron17:01
*** mwagner_lap has joined #openstack-neutron17:02
*** thomasem has joined #openstack-neutron17:02
*** dvorkinista has joined #openstack-neutron17:04
*** ollivier has quit IRC17:05
*** reed has joined #openstack-neutron17:05
*** nyechiel has quit IRC17:05
*** padkrish has joined #openstack-neutron17:07
*** tflynn has joined #openstack-neutron17:08
*** Sukhdev has quit IRC17:08
*** tsg has joined #openstack-neutron17:08
*** radez_g0n3 is now known as radez17:10
*** mlavalle has joined #openstack-neutron17:10
*** akamyshnikova has quit IRC17:10
*** annegentle has joined #openstack-neutron17:11
*** jistr has quit IRC17:13
*** a_le has joined #openstack-neutron17:13
*** a_le has quit IRC17:13
*** parkit has joined #openstack-neutron17:14
*** harlowja has joined #openstack-neutron17:15
*** parkit has quit IRC17:15
*** a_le has joined #openstack-neutron17:16
*** sbfox has quit IRC17:16
*** sbfox1 has joined #openstack-neutron17:16
*** reed has quit IRC17:16
*** tsg has quit IRC17:16
*** sbfox1 has quit IRC17:17
*** sbfox has joined #openstack-neutron17:17
*** baoli has joined #openstack-neutron17:17
*** jp_at_hp has quit IRC17:17
*** SumitNaiksatam has joined #openstack-neutron17:19
*** sbfox1 has joined #openstack-neutron17:20
*** padkrish has quit IRC17:21
*** sbfox has quit IRC17:21
*** kopparam has joined #openstack-neutron17:23
*** killer_prince has quit IRC17:23
*** nyechiel has joined #openstack-neutron17:28
*** jobewan has joined #openstack-neutron17:28
*** spandhe has joined #openstack-neutron17:29
*** Youcef has joined #openstack-neutron17:30
*** karimb has quit IRC17:31
*** kopparam has quit IRC17:33
*** doude has quit IRC17:36
*** killer_prince has joined #openstack-neutron17:36
*** killer_prince is now known as lazy_prince17:37
*** sbfox1 has quit IRC17:44
*** parkit has joined #openstack-neutron17:46
*** parkit has quit IRC17:50
*** padkrish has joined #openstack-neutron17:51
*** cnesa15 has quit IRC17:51
armaxajo: ping17:52
*** VijayB_ has joined #openstack-neutron17:52
*** thedodd has joined #openstack-neutron17:52
*** sbalukoff has joined #openstack-neutron17:56
*** radez is now known as radez_g0n317:56
*** tflynn has quit IRC17:59
*** dvorkinista has quit IRC18:00
*** a_le has quit IRC18:00
*** padkrish has quit IRC18:00
*** a_le_ has joined #openstack-neutron18:00
*** tflynn has joined #openstack-neutron18:00
*** parkit has joined #openstack-neutron18:00
*** dboik has quit IRC18:01
*** padkrish has joined #openstack-neutron18:03
*** parkit has quit IRC18:03
*** aix has quit IRC18:03
openstackgerritmark mcclain proposed a change to openstack/neutron: remove linuxbridge plugin  https://review.openstack.org/12362418:03
*** dconde has joined #openstack-neutron18:03
*** dvorkinista has joined #openstack-neutron18:04
*** ChuckC has quit IRC18:05
*** kopparam has joined #openstack-neutron18:07
*** otherwiseguy has quit IRC18:07
*** SridharRamaswamy has quit IRC18:07
*** SridharRamaswamy has joined #openstack-neutron18:08
openstackgerritArmando Migliaccio proposed a change to openstack/neutron: Fix pid file location to avoid I->J changes that break metadata  https://review.openstack.org/12534918:08
openstackgerritmark mcclain proposed a change to openstack/neutron: remove linuxbridge plugin  https://review.openstack.org/12362418:08
armaxajo: your patch https://review.openstack.org/#/c/125349 breaks on pep8 validation but it was trivial enough and I took care of it18:08
*** amcrn has joined #openstack-neutron18:10
*** amcrn has quit IRC18:10
*** htrmeira has joined #openstack-neutron18:14
*** sbfox has joined #openstack-neutron18:14
*** jobewan has quit IRC18:15
*** radez_g0n3 is now known as radez18:15
*** zhiyan has quit IRC18:16
*** carl_baldwin has quit IRC18:17
*** zhiyan has joined #openstack-neutron18:17
*** reed has joined #openstack-neutron18:18
*** annegent_ has joined #openstack-neutron18:19
*** test_ has joined #openstack-neutron18:19
*** kopparam has quit IRC18:20
*** carl_baldwin has joined #openstack-neutron18:20
*** dvorkini_ has joined #openstack-neutron18:20
*** annegentle has quit IRC18:21
*** kopparam has joined #openstack-neutron18:22
JayJ_Guys I'm looking gor a guide to setup neutron flat network. Anyone have a good link? I want flat nework so I I eliminate NAT (l2 agent) for performance reasons.18:22
*** amcrn has joined #openstack-neutron18:22
JayJ_Gusy and Gals, I mean :)18:22
*** shashankhegde has joined #openstack-neutron18:23
JayJ_Guys and Gals :) Third time is charm18:23
*** dvorkinista has quit IRC18:23
*** jprovazn has quit IRC18:24
anteayaJayJ_: thanks for being inclusive, I'd help you if I knew the answer18:25
*** ChuckC has joined #openstack-neutron18:25
*** ChuckC has quit IRC18:25
JayJ_anteaya: Thanks...18:25
*** ChuckC has joined #openstack-neutron18:25
anteayakevinbenton: I do believe that the last round of vArmour CI falling over came across your radar, any objection from you for fulfilling garyduan's request? http://lists.openstack.org/pipermail/third-party-announce/2014-October/000054.html18:26
anteayaJayJ_: :D18:26
*** chuckC_ has joined #openstack-neutron18:27
*** baoli has quit IRC18:28
*** baoli has joined #openstack-neutron18:29
*** kickinz1|afk is now known as kickinz118:30
*** devvesa has joined #openstack-neutron18:32
*** Sukhdev has joined #openstack-neutron18:33
*** jobewan has joined #openstack-neutron18:33
*** devvesa has quit IRC18:36
*** networkstatic has joined #openstack-neutron18:40
*** dvorkini_ has quit IRC18:42
*** carl_baldwin has quit IRC18:42
*** markmcclain has quit IRC18:47
*** kopparam has quit IRC18:47
*** moha_hunt has quit IRC18:50
*** kickinz1 is now known as kickinz1|afk18:50
*** thedodd has quit IRC18:54
*** nyechiel has quit IRC18:56
*** jdaggett has quit IRC18:56
*** marun has joined #openstack-neutron18:58
*** jdaggett has joined #openstack-neutron18:58
*** annegent_ has quit IRC18:59
*** annegentle has joined #openstack-neutron18:59
*** baoli has quit IRC19:02
*** sbfox has quit IRC19:03
*** parkit has joined #openstack-neutron19:03
*** carl_baldwin has joined #openstack-neutron19:06
*** test_ has quit IRC19:06
*** radez is now known as radez_g0n319:07
*** parkit has quit IRC19:08
*** tsg has joined #openstack-neutron19:11
*** shashankhegde has quit IRC19:12
*** shashankhegde has joined #openstack-neutron19:13
*** cbouch has quit IRC19:19
*** networkstatic has quit IRC19:19
ChuckCcarl_baldwin: one more followup on that question19:20
*** dfarrell07 has quit IRC19:21
carl_baldwinChuckC: sure19:21
ChuckCcarl_baldwin: the reason I noticed an issue with where I do this is because it caused a unit test to fail that I think you added recently19:21
carl_baldwinChuckC: Which one?19:21
*** alexpilotti has quit IRC19:21
ChuckCcarl_baldwin: test_update_port_rpc_outside_transaction19:22
ChuckCcarl_baldwin: it barfs on looking up the l3plugin19:22
carl_baldwinChuckC: Yes, let me look to refresh.19:22
carl_baldwinCould you put the trace in pastebin or something?19:23
ChuckCcarl_baldwin: give me a minute to repro19:24
*** otherwiseguy has joined #openstack-neutron19:24
carl_baldwinnp19:24
*** jpena has quit IRC19:24
ChuckCcarl_baldwin: http://paste.openstack.org/show/117561/19:26
*** suresh12 has quit IRC19:29
ChuckCcarl_baldwin: looks like the test is just not set up to handle service plugins19:29
carl_baldwinChuckC: I was just able to reproduce the failure with your code.  Now I can look in to it a bit.19:31
*** Youcef_ has joined #openstack-neutron19:34
*** Youcef has quit IRC19:34
carl_baldwinChuckC: Yes, the unit test is not expecting the use of manager here.  It would need to be mocked out.19:34
ChuckCmock out a few things, I guess.  OK, let me spend some time on that.19:36
carl_baldwinBesides that, I think dvr_vmarp_table_update should be done outside of the context of the transaction.  In fact, I have a placeholder patch to move the other call to this method outside.  https://review.openstack.org/#/c/12288019:36
*** radez_g0n3 is now known as radez19:36
carl_baldwinI keep meaning to get back to that.19:37
*** nlahouti has quit IRC19:37
openstackgerritCarl Baldwin proposed a change to openstack/neutron: WIP Try to remove RPC notification for DVR from delete_port  https://review.openstack.org/12288019:38
ChuckCcarl_baldwin: if you move it out of the transaction for delete, won't the vmarp update fail to get the port data from the plugin?19:38
*** suresh12 has joined #openstack-neutron19:38
carl_baldwinThere are many loose ends in that patch.  It is just a place-holder to show what I mean to accomplish.19:39
ChuckCl3_dvr_db.py:57119:39
ChuckCcarl_baldwin: ok19:39
carl_baldwinChuckC: The point is that the notification should not be sent while inside the transaction.19:40
ChuckCcarl_baldwin: agree19:40
*** f13o_f13o has joined #openstack-neutron19:41
ChuckCcarl_baldwin: will do the same in my change19:41
*** f13o_f13o has quit IRC19:41
carl_baldwinChuckC: I might be able to clean that change up tomorrow and address these questions unless you would like to work on it.19:42
ChuckCcarl_baldwin: were you thinking of passing a port dict to dvr_vmarp_table_update?19:42
ChuckCcarl_baldwin: sure, I'll be working in similar code anyway19:42
carl_baldwinChuckC: Feel free to take over that patch then.  Passing a port dict is something I have considered.  Obviously, I never came to a conclusion.19:43
ChuckCcarl_baldwin: ok, will do19:44
carl_baldwinChuckC: I’ll change my -2 to a WIP tag so that progress can be made.19:44
ChuckCcarl_baldwin: that change has a dependency19:44
carl_baldwinChuckC: thanks19:45
carl_baldwinChuckC: I thought I just rebased the dependency away.  Did you reload?19:45
ChuckCcarl_baldwin: no, the dependency is gone now!19:45
carl_baldwinThe dependency had already merged I thought.19:45
ChuckCcarl_baldwin: ok19:46
openstackgerritEd Bak proposed a change to openstack/neutron: Don't fail when trying to unbind a router  https://review.openstack.org/12486519:47
*** SridharRamaswamy has quit IRC19:48
ChuckCcarl_baldwin: as for taking over the patch, do I just git review -d and then submit some code change?19:48
carl_baldwinChuckC: Yes.19:48
ChuckCcarl_baldwin: many thanks19:49
carl_baldwinChuckC:  You can also reset the author to you and leave me as a co-author if you want.19:49
ChuckCcarl_baldwin: ok19:49
*** rkukura has quit IRC19:51
*** parkit has joined #openstack-neutron19:52
ChuckCcarl_baldwin: was a bug ever submitted for that change?19:52
carl_baldwinChuckC: Probably not.19:52
carl_baldwinIt was a hasty post while I was in the middle of a bunch of other stuff.19:52
*** thomasem has quit IRC19:53
ChuckCack19:53
carl_baldwinChuckC: In fact, I completely forgot about it until now.  :)19:53
ChuckC:)19:53
*** parkit has quit IRC19:53
*** thomasem has joined #openstack-neutron19:55
*** parkit has joined #openstack-neutron19:55
*** snowgust has joined #openstack-neutron19:57
*** clarkb has joined #openstack-neutron19:57
*** VijayB_ has quit IRC19:59
*** thomasem has quit IRC19:59
*** tsg has quit IRC19:59
*** lcurtis has joined #openstack-neutron20:02
*** lcurtis has left #openstack-neutron20:03
*** rdo has quit IRC20:05
*** rdo has joined #openstack-neutron20:06
*** padkrish has quit IRC20:08
*** lori|away is now known as lori20:10
*** ivar-lazzaro has joined #openstack-neutron20:11
*** padkrish has joined #openstack-neutron20:11
*** ivar-lazzaro has quit IRC20:12
*** ivar-lazzaro has joined #openstack-neutron20:13
*** signed8bit has quit IRC20:14
*** a_le_ has quit IRC20:15
*** a_le has joined #openstack-neutron20:16
*** baoli_ has joined #openstack-neutron20:18
*** baoli_ has quit IRC20:19
*** JayJ_ has quit IRC20:19
*** baoli_ has joined #openstack-neutron20:19
*** JayJ_ has joined #openstack-neutron20:20
*** signed8bit has joined #openstack-neutron20:21
*** devvesa has joined #openstack-neutron20:21
*** vivek-ebay has joined #openstack-neutron20:22
*** VijayB_ has joined #openstack-neutron20:24
*** devvesa has quit IRC20:25
*** padkrish has quit IRC20:25
*** annegentle has quit IRC20:26
*** igordcard has joined #openstack-neutron20:27
*** annegentle has joined #openstack-neutron20:27
*** terryw has joined #openstack-neutron20:31
*** Sukhdev has quit IRC20:33
*** otherwiseguy has quit IRC20:33
*** sbfox has joined #openstack-neutron20:34
*** nlahouti has joined #openstack-neutron20:40
*** annegent_ has joined #openstack-neutron20:42
*** annegentle has quit IRC20:43
*** annegent_ has quit IRC20:46
*** annegentle has joined #openstack-neutron20:46
*** dims_ has joined #openstack-neutron20:46
*** cjellick has quit IRC20:47
*** dvorkinista has joined #openstack-neutron20:47
openstackgerritBrian Haley proposed a change to openstack/neutron: Change L2 OVS DVR Agent to handle initial DVR RPC failure  https://review.openstack.org/12391120:47
*** dims_ has quit IRC20:48
*** dims_ has joined #openstack-neutron20:48
*** moha_hunt has joined #openstack-neutron20:48
*** dims_ has quit IRC20:49
*** padkrish has joined #openstack-neutron20:49
*** dims_ has joined #openstack-neutron20:49
*** dims has quit IRC20:50
*** mikedillion has quit IRC20:51
*** SridharRamaswamy has joined #openstack-neutron20:51
*** ZZelle_ has joined #openstack-neutron20:53
openstackgerritEd Bak proposed a change to openstack/neutron: Don't fail when trying to unbind a router  https://review.openstack.org/12486520:54
*** dims_ has quit IRC20:54
kevinbentonanteaya: i’m fine with vArmour being re-enabled20:59
*** marun has quit IRC21:00
*** annegentle has quit IRC21:09
*** radez is now known as radez_g0n321:10
*** snowgust has quit IRC21:15
*** pcm_ has quit IRC21:16
*** a_le has quit IRC21:18
*** signed8bit has quit IRC21:20
*** a_le has joined #openstack-neutron21:20
*** signed8bit has joined #openstack-neutron21:20
*** pcm_ has joined #openstack-neutron21:21
*** dvorkinista has quit IRC21:24
*** rook_ is now known as rook21:25
*** htrmeira has quit IRC21:29
openstackgerritCarl Baldwin proposed a change to openstack/neutron: Change L2 OVS DVR Agent to handle initial DVR RPC failure  https://review.openstack.org/12391121:30
*** padkrish has quit IRC21:37
*** padkrish has joined #openstack-neutron21:38
*** manishg has quit IRC21:39
*** sbfox has quit IRC21:39
*** sgordon_ has quit IRC21:41
*** sbfox has joined #openstack-neutron21:42
*** a_le has quit IRC21:44
*** a_le has joined #openstack-neutron21:45
*** sbfox1 has joined #openstack-neutron21:45
*** vivek-ebay has quit IRC21:45
*** vivek-ebay has joined #openstack-neutron21:46
*** sbfox has quit IRC21:47
*** AbsinthMind has quit IRC21:49
*** sballe has quit IRC21:50
*** sballe has joined #openstack-neutron21:50
*** gildub has joined #openstack-neutron21:52
*** mordred has quit IRC21:54
*** mordred has joined #openstack-neutron21:54
*** kbringard has quit IRC21:55
*** tsg has joined #openstack-neutron21:56
*** thedodd has joined #openstack-neutron21:58
*** VijayB_ has quit IRC21:59
*** vivek-ebay has quit IRC21:59
*** dims has joined #openstack-neutron22:02
*** dconde has quit IRC22:03
*** vivek-ebay has joined #openstack-neutron22:03
*** cjellick has joined #openstack-neutron22:05
*** parkit has quit IRC22:08
*** devvesa has joined #openstack-neutron22:09
*** mikedillion has joined #openstack-neutron22:10
*** devvesa has quit IRC22:14
*** manishg has joined #openstack-neutron22:17
*** annegentle has joined #openstack-neutron22:18
*** signed8bit has quit IRC22:19
*** manishg_ has joined #openstack-neutron22:21
*** manishg has quit IRC22:21
*** manishg_ is now known as manishg22:21
*** jobewan has quit IRC22:22
*** VijayB has joined #openstack-neutron22:24
*** geekinutah has quit IRC22:25
*** amcrn_ has joined #openstack-neutron22:26
*** spandhe has quit IRC22:29
*** spandhe has joined #openstack-neutron22:29
*** amcrn has quit IRC22:30
*** annegentle has quit IRC22:32
*** sbfox1 has quit IRC22:33
*** parkit has joined #openstack-neutron22:39
*** baoli_ has quit IRC22:40
JayJ_Any idea why ovs-vsctl del-br brex command might hang?22:41
JayJ_I am trying to delete it in the hopes of recreating22:41
anteayakevinbenton: thank you22:43
*** parkit has quit IRC22:43
*** parkit has joined #openstack-neutron22:48
*** JayJ_ has quit IRC22:49
*** baoli has joined #openstack-neutron22:55
*** baoli has quit IRC23:02
*** pcm_ has quit IRC23:03
*** mlavalle has quit IRC23:04
*** carl_baldwin has quit IRC23:05
*** mikedillion has quit IRC23:09
*** dvorkinista has joined #openstack-neutron23:10
*** a_le has quit IRC23:15
*** ivar-laz_ has joined #openstack-neutron23:16
*** a_le has joined #openstack-neutron23:16
*** dvorkinista has quit IRC23:17
*** ivar-laz_ has quit IRC23:17
*** dvorkinista has joined #openstack-neutron23:17
*** ivar-laz_ has joined #openstack-neutron23:17
*** ivar-lazzaro has quit IRC23:19
*** dvorkinista has quit IRC23:22
*** yamahata has joined #openstack-neutron23:24
*** armax has quit IRC23:26
*** geekinutah has joined #openstack-neutron23:27
*** armax has joined #openstack-neutron23:28
*** mestery has joined #openstack-neutron23:28
*** ZZelle_ has quit IRC23:29
*** JayJ_ has joined #openstack-neutron23:29
openstackgerritArmando Migliaccio proposed a change to openstack/neutron: Fix pid file location to avoid I->J changes that break metadata  https://review.openstack.org/12534923:31
*** thedodd has quit IRC23:32
*** yamahata has quit IRC23:32
*** yamahata has joined #openstack-neutron23:33
*** jorgem has quit IRC23:35
*** a_le has quit IRC23:36
*** a_le has joined #openstack-neutron23:37
openstackgerritA change was merged to openstack/neutron: fix dvr snat bindings for external-gw-clear  https://review.openstack.org/12379523:41
openstackgerritKevin Benton proposed a change to openstack/neutron: Batch ports from security groups RPC handler  https://review.openstack.org/12447823:46
*** s3wong has quit IRC23:47
kevinbentonarosen: ping23:49
arosenpng23:49
arosenpong*23:49
kevinbentonare you concerned something will modify the default in this?23:50
kevinbentonhttps://review.openstack.org/#/c/123483/2/neutron/extensions/l3.py23:50
*** popw1 has quit IRC23:50
kevinbentonarosen: ^^23:51
arosenkevinbenton:  repliexd23:53
*** geekinutah has quit IRC23:53
*** mikedillion has joined #openstack-neutron23:55
*** sbfox has joined #openstack-neutron23:55
kevinbentonarosen: right. just wanted to make sure that’s what you were worried about. I will update the patch23:56
*** devvesa has joined #openstack-neutron23:58
*** terryw has quit IRC23:58
*** sbfox has quit IRC23:59
openstackgerritKevin Benton proposed a change to openstack/neutron: Allow reading a tenant router's external IP  https://review.openstack.org/12348323:59
*** parkit has quit IRC23:59

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