Thursday, 2015-03-12

*** vishwanathj has quit IRC00:00
*** banix has joined #openstack-neutron00:01
*** tflynn has quit IRC00:01
*** tflynn_ has quit IRC00:02
*** Longgeek has joined #openstack-neutron00:03
*** annegentle has joined #openstack-neutron00:06
*** Longgeek has quit IRC00:07
openstackgerritSukhdev Kapur proposed openstack/neutron: Arista L3 Service Plugin decomposition  https://review.openstack.org/16198400:07
*** VijayB has quit IRC00:07
*** leenheer has quit IRC00:08
*** VijayB has joined #openstack-neutron00:12
*** tflynn has joined #openstack-neutron00:12
*** VijayB has quit IRC00:13
*** VijayB_ has joined #openstack-neutron00:14
*** bandarji has quit IRC00:15
*** boris-42 has joined #openstack-neutron00:15
shivharisarmax: hi00:15
armaxshivharis: hello00:15
*** xgerman has quit IRC00:16
shivharisarmax: you seem to have  a concern, please describe00:16
armaxshivharis: well, how clarify! :)00:16
armaxshivharis: I don’t understand what that plugin is for00:16
armaxshivharis: it’s a dummy class, no?00:16
shivharisarmax: yes this is dummy - the real import is done by the ml2 plugin which is now moved to networking_brocade00:17
openstackgerritMerged openstack/neutron: Add README and requirements.txt for VMware plugins  https://review.openstack.org/16324400:18
armaxshivharis: ok, so why do you keep this at all?00:18
shivharisarmax: the dummy is only to get folks off this in a clean manner00:18
armaxshivharis: besides ml2 != l300:18
armaxshivharis: nah, that’s not the right way at all00:18
armaxshivharis: why do you think this is clean?00:18
shivharisarmax: it could be nothing at all here as well00:18
armaxshivharis: it’s only masking an error in configuration00:19
armaxshivharis: it’s a time bomb, that’s what it is :)00:19
*** Longgeek has joined #openstack-neutron00:19
shivharisarmax: what is ur suggestion00:20
armaxshivharis: have you considered dropping it altogether?00:20
shivharisarmax: i could remove everything00:20
armaxshivharis: is that going to be replaced by this one?00:20
armaxhttps://review.openstack.org/#/c/159056/?00:20
shivharisarmax: no that is different00:21
*** melwitt has quit IRC00:21
armaxok00:21
*** mwagner_lap has joined #openstack-neutron00:21
shivharisarmax: the same stackforge has stuff for the vdx hardware as well, which is what will be used00:22
armaxso the BrocadeSVIPlugin is in stackforge, correct?00:22
shivharisexactly00:23
shivharisyes00:23
shivharisI can remove the class in the shim completely if that is what u recommend00:23
armaxshivharis: well, I think that your actions need to be consistent00:23
armaxshivharis: If you do what you do here: https://review.openstack.org/#/c/152383/17/neutron/plugins/ml2/drivers/brocade/mechanism_brocade.py00:24
armaxshivharis: it makes sense to do the same for the l3 service plugin00:24
shivharisi was being nice to the end customer - but consistency is a good argument you gave00:24
shivharisarmax: will that satisfy the concern, anything else u see?00:25
*** Longgeek has quit IRC00:26
armaxshivharis: I’d argue the opposite, you’re nasty to the end customer who see things starting and nothing working :)00:26
shivharis:-) hmmm.. :-)00:26
armaxshivharis: nothing else caught my eye, but I have been pretty congested00:26
*** banix has quit IRC00:26
armaxshivharis: so my eye is a bit watery :)00:26
shivharisarmax: anything else u see that is of concern - or is that the only issue? (i dont want it the start flooding..)00:27
armaxwhat flooding?00:28
shivharisok, i will take care of this in the next 10 mins00:28
shivhariswatery.. never mind00:28
armax:)00:28
armaxshivharis: ic00:28
shivharisarmax: thanks, it will be there shortly, no more class definition00:29
armaxshivharis: ok00:30
shivharisarmax: ok, ttyl00:30
armaxshivharis: thanks, ttyl00:30
*** shwetaap has joined #openstack-neutron00:30
*** annegentle has quit IRC00:31
*** shwetaap1 has joined #openstack-neutron00:33
*** absubram has joined #openstack-neutron00:33
*** natarajk1 has joined #openstack-neutron00:34
*** shwetaap has quit IRC00:37
*** annegentle has joined #openstack-neutron00:38
*** vishy has quit IRC00:41
*** vmtrooper has joined #openstack-neutron00:41
*** Sayaji has quit IRC00:42
*** vishy has joined #openstack-neutron00:43
*** Rajeev_ has quit IRC00:44
*** zzzeek has quit IRC00:44
*** tflynn has quit IRC00:45
*** vmtrooper has quit IRC00:46
*** tsg_ has quit IRC00:47
*** csoukup has quit IRC00:47
*** annegentle has quit IRC00:48
*** annegentle has joined #openstack-neutron00:49
*** tflynn has joined #openstack-neutron00:50
*** tiswanso has joined #openstack-neutron00:51
*** tiswanso has quit IRC00:54
*** tflynn has quit IRC00:54
openstackgerritHenry Gessau proposed openstack/neutron: Add Mellanox decomposition progress to chart  https://review.openstack.org/16165100:55
*** tiswanso has joined #openstack-neutron00:55
*** chlong has quit IRC00:55
openstackgerritShiv Haris proposed openstack/neutron: Brocade vendor code decomposition from neutron repo.  https://review.openstack.org/15238300:55
*** chlong has joined #openstack-neutron00:55
*** dannywilson has joined #openstack-neutron00:57
openstackgerritSalvatore Orlando proposed openstack/neutron: Support for reservations  https://review.openstack.org/16365900:58
*** patrickeast has joined #openstack-neutron00:58
*** nati_uen_ has quit IRC00:58
*** marun has joined #openstack-neutron01:00
dannywilsonI am seeing some weird behavior in neutron.  I am trying to associate a floatingip address to an instance.  The command succeeds without warning but I can't use the ip address, but the logs show an error saying "TooManyExternalNetworks: More than one external network exists".  Is this a known issue?01:00
*** _cjones_ has quit IRC01:02
*** topshare has joined #openstack-neutron01:02
*** igordcard_ has quit IRC01:03
*** topshare has quit IRC01:03
*** claired has quit IRC01:04
*** rwsu is now known as rwsu-afk01:04
*** sripriya has quit IRC01:05
*** marun has quit IRC01:05
*** shivharis has quit IRC01:06
*** shivharis has joined #openstack-neutron01:06
openstackgerritJoe Jiang proposed openstack/python-neutronclient: PEP8(version: 1.6.2) test fails in root directory of neutronclient.  https://review.openstack.org/16366301:06
*** topshare has joined #openstack-neutron01:07
*** apuimedo has quit IRC01:08
*** annegentle has quit IRC01:08
*** Bhavya has quit IRC01:10
*** VijayB_ has quit IRC01:12
*** SumitNaiksatam has quit IRC01:13
*** shivharis has quit IRC01:17
*** shivharis has joined #openstack-neutron01:17
*** VijayB_ has joined #openstack-neutron01:20
*** VijayB_ has quit IRC01:22
*** pal has joined #openstack-neutron01:26
*** markvoelker has quit IRC01:27
*** ivar-laz_ has joined #openstack-neutron01:27
*** markvoelker has joined #openstack-neutron01:27
*** ivar-lazzaro has quit IRC01:30
*** ivar-laz_ has quit IRC01:32
*** markvoelker has quit IRC01:32
*** topshare_ has joined #openstack-neutron01:33
*** nlahouti has quit IRC01:34
*** pal_ has joined #openstack-neutron01:35
*** topshare has quit IRC01:35
*** stanzgy_ has joined #openstack-neutron01:36
*** stanzgy has joined #openstack-neutron01:36
*** pal has quit IRC01:36
*** sbfox has joined #openstack-neutron01:38
openstackgerritTimothy Swanson proposed openstack/neutron: Add MTU selection & advertisement settings to Neutron config  https://review.openstack.org/15373301:39
*** sbfox has quit IRC01:39
openstackgerritKevin Benton proposed openstack/neutron: Immediately schedule net to a DHCP agt by default  https://review.openstack.org/16367201:42
*** clds_ has quit IRC01:42
*** popw has joined #openstack-neutron01:44
*** wwriverrat has left #openstack-neutron01:44
*** clds has joined #openstack-neutron01:44
*** natarajk1 has quit IRC01:47
*** sballe__ has quit IRC01:48
*** nati_ueno has joined #openstack-neutron01:50
*** Longgeek has joined #openstack-neutron01:52
*** pal_ has quit IRC01:52
*** VijayB has joined #openstack-neutron01:52
*** dims_ has quit IRC01:53
*** ajmiller has joined #openstack-neutron01:54
*** VijayB_ has joined #openstack-neutron01:54
*** VijayB has quit IRC01:54
*** ajmiller_ has joined #openstack-neutron01:55
*** ajmiller has quit IRC01:55
*** Longgeek has quit IRC01:56
*** salv-orlando has quit IRC01:58
*** VijayB_ has quit IRC01:58
*** yamahata has quit IRC01:59
*** armax has quit IRC02:01
*** shivharis has quit IRC02:01
*** shivharis has joined #openstack-neutron02:02
*** shivharis has quit IRC02:04
openstackgerrityalei wang proposed openstack/neutron-fwaas: Implementation of Mcafee NGFW Driver  https://review.openstack.org/15209302:05
HenryGmestery: still around?02:07
*** VijayB_ has joined #openstack-neutron02:07
*** s3wong has quit IRC02:07
*** armax has joined #openstack-neutron02:08
*** banix has joined #openstack-neutron02:11
*** nati_ueno has quit IRC02:11
openstackgerritMerged openstack/neutron: linuxbridge UT: Do not create the same instance in each cases  https://review.openstack.org/16282502:13
*** jckasper has quit IRC02:13
openstackgerritArmando Migliaccio proposed openstack/neutron: Test the shiny new WeightScheduler  https://review.openstack.org/16223102:14
*** ijw has quit IRC02:14
openstackgerritAssaf Muller proposed openstack/neutron: Stop using RPC namespace to unbreak rolling upgrades  https://review.openstack.org/16367602:15
openstackgerritAssaf Muller proposed openstack/neutron: Stop using RPC namespace to unbreak rolling upgrades  https://review.openstack.org/16367602:16
*** patrickeast has quit IRC02:18
*** marrusl has quit IRC02:18
*** yfried|afk is now known as yfried|pto02:20
*** ivar-lazzaro has joined #openstack-neutron02:23
*** _cjones_ has joined #openstack-neutron02:24
openstackgerritAssaf Muller proposed openstack/neutron: Replace keepalived notifier bash script with Python ip monitor  https://review.openstack.org/12538402:27
*** erkules_ has joined #openstack-neutron02:28
*** VijayB_ has quit IRC02:30
*** amuller has quit IRC02:30
*** vmtrooper has joined #openstack-neutron02:30
*** yfried|pto is now known as yfried|afk02:30
*** erkules has quit IRC02:31
*** marrusl has joined #openstack-neutron02:32
*** vishwanathj has joined #openstack-neutron02:32
openstackgerritMerged openstack/neutron: Add Mellanox decomposition progress to chart  https://review.openstack.org/16165102:33
*** vmtrooper has quit IRC02:35
openstackgerritPhillip Toohill proposed openstack/python-neutronclient: Updates pool session persistence options  https://review.openstack.org/16368202:38
*** chandankumar has joined #openstack-neutron02:40
*** leenheer has joined #openstack-neutron02:40
*** achanda has quit IRC02:42
*** ivar-lazzaro has quit IRC02:43
*** david-lyle is now known as david-lyle_afk02:45
*** yfried|afk is now known as yfried|pto02:50
*** Marga_ has quit IRC02:51
*** oomichi has joined #openstack-neutron02:52
*** marun has joined #openstack-neutron02:53
*** dims_ has joined #openstack-neutron02:54
*** shwetaap1 has quit IRC02:55
*** tflynn has joined #openstack-neutron02:57
marunkevinbenton: Sorry, had to take off.02:58
*** shwetaap has joined #openstack-neutron02:58
*** salv-orlando has joined #openstack-neutron02:59
marunkevinbenton: Having the l2 agent worry about dhcp seems strange.02:59
marunkevinbenton: But it saves a coordination step.02:59
*** dims_ has quit IRC02:59
*** yfried|pto is now known as yfried|afk03:00
marunkevinbenton: We're conflating 'port up' with 'port ready for vm' at the server currently.  If we treat those differently, I think it might make sense for the l2 agent to report on dhcp status since it can verify as close to the vm as possible that dhcp will be available.03:01
marunkevinbenton: The other option might be adding yet another agent that can report on that same state, but I'm not sure the added complexity is justified.03:02
*** tflynn has quit IRC03:02
*** zz_jgrimm is now known as jgrimm03:03
*** tflynn has joined #openstack-neutron03:06
*** popw1 has joined #openstack-neutron03:07
openstackgerritIsaku Yamahata proposed openstack/neutron: iptables firewall: add framework for iptables firewall functional test  https://review.openstack.org/16005103:08
*** popw has quit IRC03:08
openstackgerritMerged openstack/neutron: Arista L3 Service Plugin decomposition  https://review.openstack.org/16198403:09
*** nati_ueno has joined #openstack-neutron03:09
*** dboik_ has quit IRC03:10
*** dkehn has quit IRC03:10
openstackgerritMerged openstack/neutron: Imported Translations from Transifex  https://review.openstack.org/16330203:10
*** salv-orlando has quit IRC03:10
*** nati_ueno has quit IRC03:11
*** yfried|afk is now known as yfried|pto03:12
*** drjones has joined #openstack-neutron03:13
*** dkehn has joined #openstack-neutron03:15
*** _cjones_ has quit IRC03:16
*** salv-orlando has joined #openstack-neutron03:19
*** ivar-lazzaro has joined #openstack-neutron03:19
*** SumitNaiksatam has joined #openstack-neutron03:22
*** radez_g0n3 is now known as radez03:22
*** salv-orlando has quit IRC03:23
*** Networkn3rd has joined #openstack-neutron03:23
*** salv-orlando has joined #openstack-neutron03:24
*** tflynn has quit IRC03:26
*** aranjan has joined #openstack-neutron03:26
*** aranjan has quit IRC03:27
*** aranjan has joined #openstack-neutron03:27
*** aranjan has quit IRC03:28
*** aranjan has joined #openstack-neutron03:29
*** salv-orlando has quit IRC03:29
openstackgerritMerged openstack/neutron: linuxbridge UT: Mock get_interface_by_ip  https://review.openstack.org/16282603:30
*** chandankumar has quit IRC03:30
*** tflynn has joined #openstack-neutron03:31
*** dannywilson has quit IRC03:31
*** topshare_ has quit IRC03:33
*** topshare has joined #openstack-neutron03:35
*** nlahouti has joined #openstack-neutron03:36
*** ijw has joined #openstack-neutron03:36
*** Longgeek has joined #openstack-neutron03:37
*** yfried|pto is now known as yfried|afk03:38
*** zhhuabj has quit IRC03:39
*** yfried|afk is now known as yfried|pto03:41
*** topshare has quit IRC03:41
*** ijw has quit IRC03:41
*** banix has quit IRC03:41
*** nlahouti has quit IRC03:42
*** nlahouti has joined #openstack-neutron03:43
*** topshare has joined #openstack-neutron03:43
*** claudiub has quit IRC03:46
openstackgerritZengfa Gao proposed openstack/neutron: Use common agent.linux.utils.ensure_dir method  https://review.openstack.org/15410303:46
*** chandankumar has joined #openstack-neutron03:48
*** topshare has quit IRC03:53
*** Marga_ has joined #openstack-neutron03:53
openstackgerritSridar Kandaswamy proposed openstack/neutron-fwaas: FWaaS Insertion Model on Routers  https://review.openstack.org/15269703:54
*** tflynn has quit IRC03:54
*** tflynn has joined #openstack-neutron03:54
*** topshare has joined #openstack-neutron03:55
*** shwetaap1 has joined #openstack-neutron03:55
*** yfried|pto is now known as yfried|afk03:55
*** yfried|afk is now known as yfried|pto03:56
*** shwetaap has quit IRC03:57
*** drjones has quit IRC03:59
*** chandankumar has quit IRC04:00
openstackgerritKevin Benton proposed openstack/neutron: Schedule net to a DHCP agt on subnet create  https://review.openstack.org/16367204:00
*** SridharRamaswamy has joined #openstack-neutron04:00
*** Longgeek has quit IRC04:03
openstackgerritYAMAMOTO Takashi proposed openstack/neutron: OVS-agent: Introduce Ryu based OpenFlow driver  https://review.openstack.org/15394604:04
openstackgerritYAMAMOTO Takashi proposed openstack/neutron: OVS-agent: Separate ovs-ofctl using code as a driver  https://review.openstack.org/16024504:04
openstackgerritKevin Benton proposed openstack/neutron: Schedule net to a DHCP agt on subnet create  https://review.openstack.org/16367204:06
kevinbentonmarun: still around?04:08
marunkevinbenton: yes04:08
marunkevinbenton: thoughts?04:09
kevinbentonmarun: so how does the L2 agent on the compute node know that DHCP is ready?04:09
kevinbentonmarun: would we have logic like, 'if not.port_owner.startswith(neutron): poll for dhcp port'04:10
kevinbenton'once dhcp port is ready, set port status to up'04:10
marunkevinbenton: something like that, yeah04:10
kevinbentonmarun: that's just so crazy it might work!04:10
marunkevinbenton: as far as how to tell the dhcp port is ready, the only guaranteed way is to actually invoke dhcping04:11
kevinbentonmarun: right04:11
kevinbentonmarun: i chatted with salvatore about that a bit after you left04:11
kevinbentonmarun: seeing that the DHCP port is up is only half of it04:11
marunkevinbenton: right.04:11
kevinbentonmarun: it's sort of a sliding scale of complexity vs complete safety04:12
marunkevinbenton: and if we actually verified that dhcp was working before booting a vm it should reduce the potential failure domain04:12
marunkevinbenton: agreed04:12
kevinbentonmarun: can dhcping bind to an arbitrary tap port to send requests?04:12
marunkevinbenton: not sure, tbh.  If not, we could always implement a minimal client entirely in python.04:13
*** oomichi has quit IRC04:13
*** pal has joined #openstack-neutron04:14
*** chandankumar has joined #openstack-neutron04:14
*** shwetaap has joined #openstack-neutron04:14
openstackgerritYAMAMOTO Takashi proposed openstack/neutron: ofagent: Update after networking-ofagent release  https://review.openstack.org/16327304:14
kevinbentonmarun: what do you think about starting with that patch i just proposed?04:14
kevinbentonmarun: https://review.openstack.org/16367204:14
*** Marga_ has quit IRC04:15
marunkevinbenton: Seems reasonable04:16
marunkevinbenton: I think the proposed patch is an optimization if we verify dhcp is live before reporting to nova.04:16
*** Marga_ has joined #openstack-neutron04:16
marunkevinbenton: but I don't think it would hurt.04:16
kevinbentonmarun: yeah, i think this just buys us a little more room04:16
marunkevinbenton: what did salvatore have to say about why the port wasn't previously created on subnet creation?04:17
*** shwetaap1 has quit IRC04:17
*** jckasper has joined #openstack-neutron04:17
kevinbentonmarun: he said he thought it was done to save resources04:18
marunkevinbenton: which resources in particular?04:18
kevinbentonmarun: but he wasn't strongly sold on the argument so he was fine with a change04:18
kevinbentonmarun: on the dhcp agent04:18
kevinbentonmarun: spinning up an unnecessary dnsmasq process04:18
marunkevinbenton: hmmm04:18
marunkevinbenton: well, if it proves a problem the change can be reversed pretty easiliy04:19
marunkevinbenton: and a dhcp liveness check will ensure that vm's don't fail due to the previous race04:19
*** vmtrooper has joined #openstack-neutron04:19
kevinbentonmarun: so do you think i should go all out and try the dhcping full verification04:20
marunkevinbenton: I have mixed feelings.04:21
marunkevinbenton: It is clearly a potential failure mode04:21
marunkevinbenton: I don't have a good sense of how likely it actually is04:21
*** radez is now known as radez_g0n304:21
*** amotoki has joined #openstack-neutron04:22
marunkevinbenton: I would tend to work on the thing that is causing us the most pain.04:22
kevinbentonmarun: let's start with this and i'll see what kind of dhcp agent failures i still get when working on the scaling stuff04:22
marunkevinbenton: Is your sense that this is a serious enough problem to address before other issues?04:22
marunkevinbenton: fair enough04:22
marunkevinbenton: if you're seeing failure due to this problem, then it would seem a good problem to solve04:22
kevinbentonmarun: it was pretty rare to encounter, and i think this will make it even harder to hit04:23
kevinbentonmarun: the patch i just proposed, that is04:23
*** vmtrooper has quit IRC04:24
*** chandankumar has quit IRC04:24
marunkevinbenton: fair enough.04:24
*** topshare has quit IRC04:24
marunkevinbenton: well, I look forward to hearing how it goes04:24
*** topshare has joined #openstack-neutron04:24
*** devlaps has quit IRC04:25
*** gridinv_ has joined #openstack-neutron04:28
*** topshare has quit IRC04:28
*** ijw has joined #openstack-neutron04:31
*** achanda has joined #openstack-neutron04:31
*** zhhuabj has joined #openstack-neutron04:32
*** yatin has joined #openstack-neutron04:32
*** ivar-lazzaro has quit IRC04:33
*** ivar-lazzaro has joined #openstack-neutron04:34
*** SridharRamaswam1 has joined #openstack-neutron04:36
*** sthillma has joined #openstack-neutron04:37
*** ijw has quit IRC04:37
*** SridharRamaswamy has quit IRC04:38
*** ivar-lazzaro has quit IRC04:39
*** killer_prince is now known as lazy_prince04:39
*** Bhavya has joined #openstack-neutron04:40
*** jckasper has quit IRC04:41
openstackgerritZhang Hua proposed openstack/neutron-vpnaas: Functional tests of ipsec strongswan vpnaas driver  https://review.openstack.org/15856004:41
openstackgerritZhang Hua proposed openstack/neutron-vpnaas: VPNaaS breakage by refactoring commit  https://review.openstack.org/16284004:41
openstackgerritZhang Hua proposed openstack/neutron-vpnaas: IPsec strongSwan driver implemention  https://review.openstack.org/14439104:41
*** marun has quit IRC04:47
*** topshare has joined #openstack-neutron04:47
*** salv-orlando has joined #openstack-neutron04:49
*** lazy_prince has quit IRC04:49
*** junbo has joined #openstack-neutron04:50
*** marun has joined #openstack-neutron04:50
*** yfried|pto is now known as yfried|afk04:50
*** lazy_prince has joined #openstack-neutron04:51
*** neeti has joined #openstack-neutron04:53
*** lazy_prince is now known as killer_prince04:53
*** trinaths has joined #openstack-neutron04:55
*** killer_prince is now known as lazy_prince04:58
*** junbo has left #openstack-neutron04:59
*** salv-orlando has quit IRC05:00
*** yamahata has joined #openstack-neutron05:04
*** wwriverrat_ has joined #openstack-neutron05:04
*** achanda has quit IRC05:04
*** david-ly_ has joined #openstack-neutron05:04
*** aleksandr_null_ has joined #openstack-neutron05:04
*** tflynn has quit IRC05:04
*** lazy_pri- has joined #openstack-neutron05:05
*** kevinbenton_ has joined #openstack-neutron05:05
*** blogan_ has joined #openstack-neutron05:06
*** ijw has joined #openstack-neutron05:06
*** mestery_ has joined #openstack-neutron05:07
*** anilvenkata has joined #openstack-neutron05:07
*** SumitNaiksatam has quit IRC05:07
*** shadyabh1 has joined #openstack-neutron05:08
*** rook-tmp_ has joined #openstack-neutron05:08
*** mestery has quit IRC05:08
*** terryw has joined #openstack-neutron05:09
*** shaohe_robot1 has joined #openstack-neutron05:09
*** jyuso has quit IRC05:09
*** lazy_prince has quit IRC05:09
*** lazy_pri- is now known as lazy_prince05:09
*** david-lyle_afk has quit IRC05:09
*** jyuso has joined #openstack-neutron05:09
*** otherwiseguy has quit IRC05:09
*** beagles has quit IRC05:09
*** shaohe_robot has quit IRC05:09
*** mikal_ has joined #openstack-neutron05:09
*** aleksandr_null has quit IRC05:09
*** kevinbenton has quit IRC05:09
*** blogan has quit IRC05:09
*** shadyabhi has quit IRC05:09
*** frickler has quit IRC05:09
*** mikal has quit IRC05:09
*** frickler has joined #openstack-neutron05:09
*** raginbajin has quit IRC05:09
*** rook-tmp has quit IRC05:09
*** b3nt_pin has joined #openstack-neutron05:10
*** b3nt_pin is now known as Guest2707405:10
*** yfried|afk is now known as yfried|pto05:11
*** topshare has quit IRC05:11
*** ijw has quit IRC05:12
*** raginbajin has joined #openstack-neutron05:13
*** absubram has quit IRC05:15
*** britthouser has joined #openstack-neutron05:16
openstackgerritZhang Hua proposed openstack/neutron-vpnaas: Functional tests of ipsec strongswan vpnaas driver  https://review.openstack.org/15856005:16
openstackgerritZhang Hua proposed openstack/neutron-vpnaas: IPsec strongSwan driver implemention  https://review.openstack.org/14439105:16
*** SridharRamaswam1 has quit IRC05:17
*** britthouser has quit IRC05:19
*** marun has quit IRC05:19
*** chandankumar has joined #openstack-neutron05:20
*** yfried|pto has quit IRC05:20
*** topshare has joined #openstack-neutron05:21
*** marun has joined #openstack-neutron05:22
*** topshare_ has joined #openstack-neutron05:23
*** shwetaap has quit IRC05:23
*** britthouser has joined #openstack-neutron05:24
trinathsHi all, Very Good Morning :)05:25
*** markvoelker has joined #openstack-neutron05:25
*** gridinv_ has quit IRC05:25
*** topshare has quit IRC05:26
*** pradipta has joined #openstack-neutron05:26
*** Murali has joined #openstack-neutron05:27
*** achanda has joined #openstack-neutron05:28
*** yuanying has joined #openstack-neutron05:30
*** markvoelker has quit IRC05:30
*** reed has quit IRC05:30
*** leenheer has quit IRC05:33
*** britthouser has quit IRC05:34
*** tflynn has joined #openstack-neutron05:36
*** chandankumar has quit IRC05:36
*** enikanorov has joined #openstack-neutron05:40
*** leenheer has joined #openstack-neutron05:40
*** britthouser has joined #openstack-neutron05:40
*** harlowja_ is now known as harlowja_away05:41
*** chandankumar has joined #openstack-neutron05:41
*** enikanorov__ has quit IRC05:41
*** obondarev_ has joined #openstack-neutron05:42
*** leenheer has quit IRC05:42
*** obondarev has quit IRC05:42
*** marun has quit IRC05:42
*** yog_ has joined #openstack-neutron05:45
*** Longgeek has joined #openstack-neutron05:45
*** britthouser has quit IRC05:46
openstackgerritYAMAMOTO Takashi proposed openstack/neutron: OVS-agent: Introduce Ryu based OpenFlow driver  https://review.openstack.org/15394605:46
openstackgerritYAMAMOTO Takashi proposed openstack/neutron: OVS-agent: Separate ovs-ofctl using code as a driver  https://review.openstack.org/16024505:46
*** yamamoto has joined #openstack-neutron05:47
*** britthouser has joined #openstack-neutron05:50
*** chandankumar has quit IRC05:52
*** yog_ has quit IRC05:55
*** wuhg has joined #openstack-neutron05:56
*** adelinatuvenie has joined #openstack-neutron06:00
*** anilvenkata has quit IRC06:04
*** ijw has joined #openstack-neutron06:06
*** yshafrir has joined #openstack-neutron06:07
*** Murali_ has joined #openstack-neutron06:08
*** vmtrooper has joined #openstack-neutron06:08
*** Murali has quit IRC06:08
*** Murali_ is now known as Murali06:08
openstackgerritHong Hui Xiao proposed openstack/neutron: Always run dnsmasq as root  https://review.openstack.org/16156706:11
*** yog_ has joined #openstack-neutron06:11
*** ijw has quit IRC06:11
openstackgerritOpenStack Proposal Bot proposed openstack/neutron: Imported Translations from Transifex  https://review.openstack.org/16370806:12
*** rook-tmp_ has quit IRC06:12
*** numan has joined #openstack-neutron06:12
*** vmtrooper has quit IRC06:13
*** aranjan_ has joined #openstack-neutron06:14
openstackgerritSridar Kandaswamy proposed openstack/neutron-fwaas: FWaaS Insertion Model on Routers  https://review.openstack.org/15269706:15
*** Longgeek has quit IRC06:18
*** anilvenkata has joined #openstack-neutron06:21
*** nlahouti has quit IRC06:23
*** yog_ has quit IRC06:24
*** afazekas is now known as __afazekas06:25
*** markvoelker has joined #openstack-neutron06:26
*** yamahata has quit IRC06:26
*** leenheer has joined #openstack-neutron06:27
*** markvoelker has quit IRC06:32
*** dims_ has joined #openstack-neutron06:32
*** leenheer has quit IRC06:32
*** chandankumar has joined #openstack-neutron06:34
*** anilvenkata has quit IRC06:35
*** lazy_prince is now known as killer_prince06:35
*** yog_ has joined #openstack-neutron06:36
*** killer_prince is now known as lazy_prince06:37
*** oda-g has quit IRC06:37
*** dims_ has quit IRC06:38
*** itzikb has joined #openstack-neutron06:42
*** irenab has joined #openstack-neutron06:43
*** setuid has quit IRC06:43
*** setuid has joined #openstack-neutron06:46
openstackgerritRyan Tidwell proposed openstack/neutron: WIP Subnet allocation from a subnet pool  https://review.openstack.org/15759706:47
*** gildub has quit IRC06:49
*** rook-tmp has joined #openstack-neutron06:50
*** yfried|pto has joined #openstack-neutron06:51
*** shaohe_robot1 is now known as shaohe_feng06:55
*** yfried|pto has quit IRC06:55
*** yfried|pto has joined #openstack-neutron06:56
*** anilvenkata has joined #openstack-neutron06:57
*** yfried|pto is now known as yfried|afk06:57
*** yfried|afk is now known as yfried|pto07:00
*** yfried|pto is now known as yfried07:00
*** nmagnezi_ has joined #openstack-neutron07:01
*** kawa2014 has joined #openstack-neutron07:02
*** gongysh has joined #openstack-neutron07:05
*** bobmel has joined #openstack-neutron07:06
*** armax has quit IRC07:06
*** ijw has joined #openstack-neutron07:06
*** kawa2014 has quit IRC07:07
*** Marga_ has quit IRC07:07
*** kawa2014 has joined #openstack-neutron07:07
openstackgerritMamta Prabhu proposed openstack/neutron: IBM SDN-VE Plugin decomposition  https://review.openstack.org/13047107:09
*** bobmel_ has quit IRC07:09
*** achanda has quit IRC07:11
*** vthapar has joined #openstack-neutron07:11
*** ijw has quit IRC07:11
*** kawa2014 has quit IRC07:12
*** kawa2014 has joined #openstack-neutron07:12
*** lazy_prince has quit IRC07:15
*** lazy_prince has joined #openstack-neutron07:16
*** watanabe_isao has joined #openstack-neutron07:17
*** SridharG1 has joined #openstack-neutron07:17
*** _cjones_ has joined #openstack-neutron07:19
*** anshul has joined #openstack-neutron07:20
*** _cjones_ has quit IRC07:21
*** achanda has joined #openstack-neutron07:24
*** anshul has quit IRC07:26
*** anshul has joined #openstack-neutron07:26
*** salv-orlando has joined #openstack-neutron07:27
*** markvoelker has joined #openstack-neutron07:28
*** evgenyf has joined #openstack-neutron07:32
*** lazy_prince has quit IRC07:33
*** markvoelker has quit IRC07:33
*** lazy_prince has joined #openstack-neutron07:34
*** yfried is now known as yfried|afk07:34
*** lazy_prince is now known as killer_prince07:35
*** yfried|afk is now known as yfried07:35
*** salv-orlando has quit IRC07:35
*** nyechiel_ has joined #openstack-neutron07:36
openstackgerritItzik Brown proposed openstack/neutron: Adding a cleanup for 'fip-' and 'snat-' namespaces in netns_cleanup  https://review.openstack.org/16340307:36
*** bjornar_ has joined #openstack-neutron07:38
*** salv-orlando has joined #openstack-neutron07:39
*** killer_prince is now known as lazy_prince07:39
*** salv-orl_ has joined #openstack-neutron07:42
*** tflynn has quit IRC07:42
*** sluo_wfh has joined #openstack-neutron07:42
*** salv-orlando has quit IRC07:43
*** achanda has quit IRC07:44
*** anilvenkata has quit IRC07:46
*** salv-orl_ has quit IRC07:46
*** terryw has quit IRC07:48
*** ChrisNBlum has joined #openstack-neutron07:49
*** safchain has joined #openstack-neutron07:51
*** lazy_prince has quit IRC07:52
*** otherwiseguy has joined #openstack-neutron07:53
*** lazy_prince has joined #openstack-neutron07:53
*** sluo_wfh has quit IRC07:53
*** lazy_prince is now known as killer_prince07:54
*** jlanoux has joined #openstack-neutron07:54
*** vmtrooper has joined #openstack-neutron07:57
*** killer_prince is now known as lazy_prince07:57
*** scheuran has joined #openstack-neutron07:59
openstackgerritRomil Gupta proposed openstack/neutron: Ml2 Mechanism Driver for OVSvApp Solution  https://review.openstack.org/16024708:01
*** salv-orlando has joined #openstack-neutron08:01
*** vmtrooper has quit IRC08:02
*** Longgeek has joined #openstack-neutron08:03
*** kickinz1|afk is now known as kickinz108:03
*** jistr has joined #openstack-neutron08:03
*** roeyc has joined #openstack-neutron08:04
*** salv-orlando has quit IRC08:06
*** ijw has joined #openstack-neutron08:06
openstackgerritHong Hui Xiao proposed openstack/neutron: Always run dnsmasq as root  https://review.openstack.org/16156708:06
*** afazekas_ has joined #openstack-neutron08:06
*** kawa2014 has quit IRC08:08
*** pask81 has joined #openstack-neutron08:08
*** sudipto has joined #openstack-neutron08:09
*** apuimedo has joined #openstack-neutron08:10
*** gildub has joined #openstack-neutron08:11
*** nyechiel__ has joined #openstack-neutron08:11
*** matrohon has joined #openstack-neutron08:11
*** nyechiel_ has quit IRC08:12
*** ijw has quit IRC08:12
*** Longgeek has quit IRC08:12
*** yuanying has quit IRC08:13
*** pask81 has quit IRC08:15
*** pradipta has quit IRC08:16
*** adelinatuvenie has quit IRC08:16
*** sluo_wfh has joined #openstack-neutron08:18
*** ronis has joined #openstack-neutron08:20
*** yfried is now known as yfried|afk08:20
*** pask81 has joined #openstack-neutron08:21
openstackgerritKevin Benton proposed openstack/neutron: Schedule net to a DHCP agt on subnet create  https://review.openstack.org/16367208:21
*** markvoelker has joined #openstack-neutron08:29
*** watanabe_isao has quit IRC08:32
*** aranjan_ has quit IRC08:34
*** markvoelker has quit IRC08:34
*** salv-orlando has joined #openstack-neutron08:34
*** aranjan has joined #openstack-neutron08:35
openstackgerritHirofumi Ichihara proposed openstack/neutron: Return 404 when executing net-list-on-dhcp-agent with invalid agent_id  https://review.openstack.org/15904608:36
*** erkules_ is now known as erkules08:40
*** erkules has quit IRC08:40
*** erkules has joined #openstack-neutron08:40
*** jschwarz has joined #openstack-neutron08:40
*** yfried|afk is now known as yfried08:40
*** kawa2014 has joined #openstack-neutron08:41
openstackgerritenikanorov proposed openstack/neutron: Refactor retry mechanism used in some DB operations  https://review.openstack.org/14926108:46
openstackgerritenikanorov proposed openstack/neutron: Set default of api_workers to number of CPUs  https://review.openstack.org/14049308:46
*** gongysh has quit IRC08:49
*** yfried is now known as yfried|afk08:50
*** ygbo has joined #openstack-neutron08:51
*** yfried|afk is now known as yfried08:54
*** moshele has joined #openstack-neutron08:55
openstackgerritpritesh proposed openstack/neutron: Adding VLAN Transparency support for ML2 along with REST API changes  https://review.openstack.org/15842009:00
*** neeti has quit IRC09:05
openstackgerritHirofumi Ichihara proposed openstack/neutron: Return 404 when executing net-list-on-dhcp-agent with invalid agent_id  https://review.openstack.org/15904609:05
*** neeti has joined #openstack-neutron09:05
*** ijw has joined #openstack-neutron09:06
*** ChrisNBlum has quit IRC09:06
*** ijw has quit IRC09:11
*** gongysh has joined #openstack-neutron09:13
matrohonrossella_s : hi09:13
*** nyechiel has joined #openstack-neutron09:13
*** lazy_prince is now known as killer_prince09:15
*** nyechiel__ has quit IRC09:15
*** kerpan has joined #openstack-neutron09:15
openstackgerritKevin Benton proposed openstack/neutron: Schedule net to a DHCP agt on subnet create  https://review.openstack.org/16367209:17
*** yamamoto has quit IRC09:21
*** Bhavya has quit IRC09:22
*** topshare_ has quit IRC09:26
*** stanzgy_ has quit IRC09:26
*** stanzgy_ has joined #openstack-neutron09:26
*** anilvenkata has joined #openstack-neutron09:28
rossella_smatrohon: hi09:29
*** yog_ has quit IRC09:30
*** markvoelker has joined #openstack-neutron09:30
*** ihrachyshka has joined #openstack-neutron09:32
jschwarzihrachyshka, congratz, friend! :)09:33
openstackgerritHirofumi Ichihara proposed openstack/neutron: Fix l3_agentschedulers_db for consistency of code  https://review.openstack.org/16374009:34
ihrachyshkajschwarz, tnx09:34
* ihrachyshka is a bit scared of possessing +2 hammer. I need to learn to use it with due care.09:34
*** Bhavya has joined #openstack-neutron09:35
*** markvoelker has quit IRC09:35
*** jpena has joined #openstack-neutron09:36
*** aepifanov has joined #openstack-neutron09:36
*** aranjan has quit IRC09:38
*** yog_ has joined #openstack-neutron09:39
openstackgerritMerged openstack/neutron: Stop using RPC namespace to unbreak rolling upgrades  https://review.openstack.org/16367609:39
*** aranjan has joined #openstack-neutron09:40
*** Guest27074 is now known as b3nt_pin09:41
*** kawa2014 has quit IRC09:43
*** kawa2014 has joined #openstack-neutron09:44
*** vmtrooper has joined #openstack-neutron09:45
amotokiihrachyshka: congrats!09:47
ihrachyshkaamotoki, thanks, will do my best09:48
*** yfried is now known as yfried|afk09:49
*** yfried|afk is now known as yfried09:50
*** vmtrooper has quit IRC09:51
*** yamamoto has joined #openstack-neutron09:51
*** dims__ has joined #openstack-neutron09:54
*** Longgeek has joined #openstack-neutron09:54
*** moha_hunt has joined #openstack-neutron09:56
trinathsihrachyshka: Congratulations..10:02
*** ijw has joined #openstack-neutron10:06
openstackgerritAkihiro Motoki proposed openstack/python-neutronclient: Copy functional tests from tempest cli  https://review.openstack.org/16284810:08
*** killer_prince is now known as lazy_prince10:09
*** carlp has quit IRC10:10
*** ijw has quit IRC10:11
*** ajo has quit IRC10:12
*** jp_at_hp has joined #openstack-neutron10:13
*** Longgeek has quit IRC10:14
*** jlanoux has quit IRC10:18
*** salv-orl_ has joined #openstack-neutron10:18
*** lazy_pri- has joined #openstack-neutron10:18
*** imcsk8 is now known as imcsk8|zZz10:18
*** ajo has joined #openstack-neutron10:18
*** lazy_prince has quit IRC10:18
*** lazy_pri- is now known as lazy_prince10:18
*** stanzgy has quit IRC10:19
*** stanzgy has joined #openstack-neutron10:20
*** salv-orlando has quit IRC10:20
*** gongysh has quit IRC10:21
*** mlavalle has quit IRC10:21
*** _cjones_ has joined #openstack-neutron10:22
*** salv-orl_ has quit IRC10:22
openstackgerritIhar Hrachyshka proposed openstack/neutron: Migrate to oslo.log  https://review.openstack.org/15963810:23
*** lazy_pri- has joined #openstack-neutron10:24
ihrachyshkayamamoto, btw thanks for your comments, that are usually very detailed10:25
*** lazy_prince has quit IRC10:26
*** lazy_pri- is now known as lazy_prince10:26
*** _cjones_ has quit IRC10:26
*** lazy_prince is now known as killer_prince10:27
openstackgerrityalei wang proposed openstack/neutron: iptables firewall: add framework for iptables firewall functional test  https://review.openstack.org/16005110:28
openstackgerrityalei wang proposed openstack/neutron: Add portsecurity extension support  https://review.openstack.org/12655210:28
yamamotoihrachyshka: np10:29
openstackgerritPraveen Kumar SM proposed openstack/neutron: DHCP Service LoadBalancing Scheduler  https://review.openstack.org/13701710:31
*** markvoelker has joined #openstack-neutron10:31
*** killer_prince is now known as lazy_prince10:33
*** yamamoto has quit IRC10:35
*** markvoelker has quit IRC10:36
*** jistr has quit IRC10:36
*** safchain has quit IRC10:43
*** yog_ has quit IRC10:44
kerpanWhat's the best practice to implement an old abandon blueprint started by someone else? Contact the author? Start a new one? (The design is 'drafting' and there is some code waiting for review with no activity for more then 1 year)10:47
*** ParsectiX_ has joined #openstack-neutron10:47
openstackgerritIhar Hrachyshka proposed openstack/neutron: Add the rebinding chance in _bind_port_if_needed  https://review.openstack.org/16226010:48
openstackgerritKevin Benton proposed openstack/neutron: Reload DHCP interface when port is updated  https://review.openstack.org/16375810:48
ParsectiX_Hello guys. I'm trying to create a router and attach it to a network10:48
ParsectiX_neutron router-create router_210:48
ParsectiX_neutron router-interface-add router_2 <UUID of the subnet>10:48
ParsectiX_but I'm getting Bad request10:49
ParsectiX_The subnet must have DHCP enabled ?10:49
*** stanzgy has quit IRC10:50
*** pc_m has joined #openstack-neutron10:51
*** stanzgy_ has quit IRC10:51
*** skraynev has quit IRC10:52
*** jlanoux has joined #openstack-neutron10:52
*** skraynev has joined #openstack-neutron10:52
*** skraynev has quit IRC10:53
*** jistr has joined #openstack-neutron10:53
*** skraynev has joined #openstack-neutron10:53
*** yog_ has joined #openstack-neutron11:00
*** skraynev_ has joined #openstack-neutron11:03
*** ijw has joined #openstack-neutron11:06
amotokiParsectiX_: there is no need to enable DHCP for a subnet.11:07
amotokiParsectiX_: what message did you get in BadRequest response?11:07
*** chandankumar has quit IRC11:09
*** ijw has quit IRC11:11
*** chandankumar has joined #openstack-neutron11:16
*** aix has joined #openstack-neutron11:19
*** safchain has joined #openstack-neutron11:19
*** yog_ has quit IRC11:21
*** bjornar_ has quit IRC11:23
*** yuanying has joined #openstack-neutron11:23
*** yuanying has quit IRC11:28
*** yfried is now known as yfried|afk11:31
*** markvoelker has joined #openstack-neutron11:33
ParsectiX_amotoki: Bad Request (HTTP 400) (Request-ID:11:33
*** hichihara has quit IRC11:34
*** vmtrooper has joined #openstack-neutron11:34
amotokiParsectiX_: no detail message? Some versions of neutronclient behaves so. IIRC, in this case neutron -v router-interface-add would show the detail message.11:35
*** markvoelker has quit IRC11:37
*** sudipto has quit IRC11:38
*** nihilifer has quit IRC11:39
ParsectiX_<amotoki> Thanks for the info11:39
*** nihilifer has joined #openstack-neutron11:39
*** vmtrooper has quit IRC11:39
*** yfried|afk is now known as yfried11:42
*** scheuran_ has joined #openstack-neutron11:46
*** kickinz1 is now known as kickinz1|afk11:48
*** scheuran has quit IRC11:49
*** kickinz1|afk is now known as kickinz111:54
openstackgerritHenry Gessau proposed openstack/neutron: WIP: Drop support for SQL Schema Downgrades  https://review.openstack.org/16378711:55
*** gildub has quit IRC11:57
*** aix has quit IRC11:59
*** Longgeek has joined #openstack-neutron11:59
*** tsg_ has joined #openstack-neutron11:59
raornhi12:00
*** salv-orlando has joined #openstack-neutron12:00
raornI am trying to backport two commits into stable/icehouse12:00
raorngit review decided to squash them into one, but this is cherry-picks...12:01
HenryGraorn: which commits?12:01
raornhttps://review.openstack.org/163783 https://review.openstack.org/16378412:02
raorndhcp in v6-only networks12:02
HenryGraorn: Have you tried the "Cherry Pick To" button in the gerrit UI?12:03
*** yfried is now known as yfried|afk12:03
raornHenryG: there are two different comits and I tried git review -X12:03
raornon both master commits12:04
*** yfried|afk is now known as yfried12:04
*** moha_hunt has quit IRC12:04
*** mwagner_lap has quit IRC12:04
HenryGraorn: I have only done backports on already merged patches. Maybe check with ihrachyshka12:05
*** nyechiel_ has joined #openstack-neutron12:05
raornboth these patches are merged and even backported to juno12:06
*** ijw has joined #openstack-neutron12:06
raorni don't think it's right to squash these two patches, since this is cherry-picks or two different commits. but decided to ask here.12:07
*** nyechiel has quit IRC12:07
ihrachyshkaraorn, hm, do we even have ipv6 supported in icehouse?12:07
*** salv-orlando has quit IRC12:08
*** baoli has joined #openstack-neutron12:08
*** salv-orlando has joined #openstack-neutron12:08
raornihrachyshka: even in havana12:11
ihrachyshkaraorn, really? are you sure? afaik ipv6 attrs were disabled for icehouse12:11
raornonly dhcpv6-statefull supported12:11
*** ijw has quit IRC12:12
ihrachyshkaraorn, are you sure we speak about upstream releases and not some local changes in some distro?12:12
openstackgerritJohn Davidge proposed openstack/neutron: Implement Neutron IPv6 Prefix Delegation  https://review.openstack.org/15869712:13
raornopenstack knows nothing about SLAAC12:13
ihrachyshkaraorn, "With the IceHouse release of OpenStack, only IPv6 Upstream Provider Network support with VLANs is feasible."12:14
ihrachyshkathat's from http://techs.enovance.com/7199/journey-of-ipv6-in-openstack12:14
*** Bhavya has quit IRC12:14
raornihrachyshka: I build openstack myself from gihtub repos, but our local changes are minimal.12:14
*** markvoelker has joined #openstack-neutron12:15
ihrachyshkaso how do you create a ipv6 subnet while https://review.openstack.org/#/q/I69a1a571b5beb566641200e60b84f0716c1ec138,n,z is merged in icehouse?12:15
raornihrachyshka: we only have provider networks and IPv6 (-only and dualstack) works fine since havana12:15
ihrachyshkaah right, so provider networks. ok, looking.12:16
*** marun has joined #openstack-neutron12:17
*** kawa2014 has quit IRC12:19
*** Longgeek has quit IRC12:19
*** salv-orlando has quit IRC12:20
openstackgerritvikram.choudhary proposed openstack/neutron: Time Sync As Service  https://review.openstack.org/16379312:21
openstackgerritMoshe Levi proposed openstack/neutron: Move mlnx agent to be under ml2/drivers/mlnx  https://review.openstack.org/16269312:21
*** ferest has joined #openstack-neutron12:21
*** skraynev has quit IRC12:23
*** skraynev has joined #openstack-neutron12:23
*** lazy_prince is now known as killer_prince12:24
*** ferest has quit IRC12:25
*** ajmiller_ has quit IRC12:26
*** bjornar_ has joined #openstack-neutron12:28
raornihrachyshka: I guess "Hide ipv6 subnet API attributes" creates some problems with IPv6, but I'm not shure what is wrong. mayme my view is limited by provider networks...12:29
*** pal has quit IRC12:29
*** tiswanso has quit IRC12:29
ihrachyshkaraorn, sorry if my questions are dumb, but do you really run dhcp agent for provider networks?12:30
*** dims__ has quit IRC12:35
*** dims_ has joined #openstack-neutron12:36
ihrachyshkaraorn, ok, I think I get how your setup looks like. makes sense.12:37
raornihrachyshka: yes. we are give VLAN and set of subnets (real addresses). "provider" does routing and RA, we only can manage flags in announces12:38
*** sudipto has joined #openstack-neutron12:39
*** Longgeek has joined #openstack-neutron12:39
ihrachyshkaraorn, so back to your original problem with patch squashing... looking at the patches, I don't see any problems. These two patches are properly linked in a list of dependencies. What's your problem with them?12:40
*** Marga_ has joined #openstack-neutron12:40
*** moha_hunt has joined #openstack-neutron12:40
*** txomon|fon has quit IRC12:41
*** marun has quit IRC12:41
raornihrachyshka: no problem. this is the fist time I backport several commits at once, better ask ;-)12:42
*** txomon|fon has joined #openstack-neutron12:43
*** amuller has joined #openstack-neutron12:44
*** jpena is now known as jpena|lunch12:44
*** elmiko has left #openstack-neutron12:45
*** radez_g0n3 is now known as radez12:48
*** txomon|fon has quit IRC12:49
*** nyechiel_ has quit IRC12:50
*** pavel_bondar has joined #openstack-neutron12:51
*** mwagner_lap has joined #openstack-neutron12:52
*** sudipto has quit IRC12:52
*** amotoki has quit IRC12:57
*** aix has joined #openstack-neutron12:58
*** yfried_ has joined #openstack-neutron13:00
*** yfried__ has joined #openstack-neutron13:03
*** dboik has joined #openstack-neutron13:03
*** yfried has quit IRC13:03
*** jschwarz has quit IRC13:03
*** yshafrir has quit IRC13:04
*** setuid has quit IRC13:05
*** setuid has joined #openstack-neutron13:05
*** dboik_ has joined #openstack-neutron13:05
*** yfried_ has quit IRC13:05
*** ijw has joined #openstack-neutron13:06
*** raildo has joined #openstack-neutron13:06
*** yatin has quit IRC13:07
*** mestery_ has quit IRC13:08
*** dboik has quit IRC13:08
*** nyechiel_ has joined #openstack-neutron13:09
*** nyechiel__ has joined #openstack-neutron13:11
*** dboik_ has quit IRC13:11
*** ijw has quit IRC13:12
*** yamamoto has joined #openstack-neutron13:13
*** nyechiel_ has quit IRC13:14
*** aveiga has joined #openstack-neutron13:17
*** mestery has joined #openstack-neutron13:20
*** shwetaap has joined #openstack-neutron13:23
*** vmtrooper has joined #openstack-neutron13:23
*** Longgeek has quit IRC13:24
*** jckasper has joined #openstack-neutron13:25
*** cference has joined #openstack-neutron13:27
*** vmtrooper has quit IRC13:29
*** dboik has joined #openstack-neutron13:32
*** dboik has quit IRC13:32
*** cference has quit IRC13:32
*** dboik has joined #openstack-neutron13:32
*** julien_ZTE has joined #openstack-neutron13:32
*** tiswanso has joined #openstack-neutron13:34
*** cference has joined #openstack-neutron13:35
openstackgerritMerged openstack/neutron: Return 404 when executing net-list-on-dhcp-agent with invalid agent_id  https://review.openstack.org/15904613:36
openstackgerritJohn Davidge proposed openstack/neutron: Implement Neutron IPv6 Prefix Delegation  https://review.openstack.org/15869713:36
*** Murali has quit IRC13:36
*** annegentle has joined #openstack-neutron13:37
*** Longgeek has joined #openstack-neutron13:37
*** kbringard has joined #openstack-neutron13:39
*** shwetaap1 has joined #openstack-neutron13:42
*** kawa2014 has joined #openstack-neutron13:44
*** adelinatuvenie has joined #openstack-neutron13:45
*** shwetaap has quit IRC13:45
*** bjornar_ has quit IRC13:46
*** Marga_ has quit IRC13:49
*** anilvenkata has left #openstack-neutron13:50
*** Marga_ has joined #openstack-neutron13:50
*** zhhuabj has quit IRC13:51
*** anshul has quit IRC13:54
*** pal has joined #openstack-neutron13:57
*** trinaths1 has joined #openstack-neutron13:57
*** trinaths has quit IRC13:57
*** neeti has quit IRC13:59
*** baoli has quit IRC14:00
*** sudipto has joined #openstack-neutron14:01
*** jpena|lunch is now known as jpena14:01
*** baoli_ has joined #openstack-neutron14:02
*** apuimedo has quit IRC14:02
*** baoli_ has quit IRC14:02
openstackgerritElena Ezhova proposed openstack/neutron: [WIP]: Add reset methods to WsgiWorker and RPCWorker  https://review.openstack.org/16173214:03
*** r-daneel has joined #openstack-neutron14:03
*** apuimedo has joined #openstack-neutron14:03
*** baoli_ has joined #openstack-neutron14:04
*** ajmiller has joined #openstack-neutron14:06
*** ijw has joined #openstack-neutron14:06
openstackgerritZongKai LI proposed openstack/neutron: Remove router binding with router-interface-delete  https://review.openstack.org/16345414:06
*** johnbelamaric has joined #openstack-neutron14:07
*** ParsectiX_ has quit IRC14:07
*** yamamoto has quit IRC14:07
*** pal has quit IRC14:08
*** irenab has quit IRC14:08
*** Marga_ has quit IRC14:08
*** chandankumar has quit IRC14:09
*** ijw has quit IRC14:12
*** amotoki has joined #openstack-neutron14:12
*** carlp has joined #openstack-neutron14:14
*** thomasem_ has quit IRC14:14
openstackgerritenikanorov proposed openstack/neutron: Set default of api_workers to number of CPUs  https://review.openstack.org/14049314:14
*** yfried__ has quit IRC14:14
*** thomasem has joined #openstack-neutron14:18
*** aepifanov has quit IRC14:19
*** yamamoto has joined #openstack-neutron14:19
*** criptoverso has joined #openstack-neutron14:19
*** adelinatuvenie has quit IRC14:19
*** yfried__ has joined #openstack-neutron14:20
*** yfried__ has quit IRC14:21
*** topshare has joined #openstack-neutron14:23
*** julien_ZTE has quit IRC14:24
*** blues-man has joined #openstack-neutron14:25
*** julien_ZTE has joined #openstack-neutron14:26
*** dsundarraj has joined #openstack-neutron14:26
*** yfried has joined #openstack-neutron14:27
*** csoukup has joined #openstack-neutron14:31
*** popw1 has quit IRC14:31
*** jprovazn has joined #openstack-neutron14:31
*** prad has joined #openstack-neutron14:34
*** yfried is now known as yfried|afk14:35
*** yfried|afk is now known as yfried14:35
*** yfried has quit IRC14:35
*** marun has joined #openstack-neutron14:35
*** achanda has joined #openstack-neutron14:36
*** adelinatuvenie has joined #openstack-neutron14:39
*** seizadi has joined #openstack-neutron14:40
openstackgerritRob Pothier proposed openstack/neutron: Add MTU selection to ML2  https://review.openstack.org/15631814:41
*** tflynn has joined #openstack-neutron14:41
*** tflynn has quit IRC14:42
*** apuimedo is now known as apuimedo|away14:42
*** r-daneel has quit IRC14:42
*** jschwarz has joined #openstack-neutron14:43
*** tflynn has joined #openstack-neutron14:43
*** achanda has quit IRC14:44
*** r-daneel has joined #openstack-neutron14:45
*** topshare has quit IRC14:47
openstackgerritstephen-ma proposed openstack/neutron: Router is not unscheduled when the last VM is deleted  https://review.openstack.org/15929614:48
*** aveiga has quit IRC14:49
*** carl_baldwin has joined #openstack-neutron14:49
*** apuimedo|away has quit IRC14:49
*** chandankumar has joined #openstack-neutron14:50
*** tflynn has quit IRC14:50
*** aveiga has joined #openstack-neutron14:50
*** johnbelamaric has quit IRC14:51
matrohonrkukura : hi?14:52
*** numan has quit IRC14:53
*** thedodd has joined #openstack-neutron14:53
*** aveiga has quit IRC14:53
openstackgerritBrian Bowen proposed openstack/neutron: Fix missing spaces in strings split across lines.  https://review.openstack.org/16385914:54
*** Longgeek has quit IRC14:55
*** pal has joined #openstack-neutron14:55
*** xgerman has joined #openstack-neutron14:57
*** wwriverrat_ has left #openstack-neutron14:58
*** dboik_ has joined #openstack-neutron14:58
openstackgerritPaul Michali proposed openstack/neutron: Fix missing spaces in strings split across lines.  https://review.openstack.org/16385914:58
*** mlavalle has joined #openstack-neutron14:58
*** yfried has joined #openstack-neutron14:59
*** ajmiller has quit IRC14:59
*** tiswanso has quit IRC14:59
openstackgerritPaul Michali proposed openstack/neutron: Remove dead code  https://review.openstack.org/16358615:00
*** baoli_ has quit IRC15:00
*** dboik has quit IRC15:00
*** claudiub has joined #openstack-neutron15:01
openstackgerritMiguel Angel Ajo proposed openstack/neutron: Add a netns-cleanup functional test  https://review.openstack.org/16386015:01
openstackgerritMiguel Angel Ajo proposed openstack/neutron: Fix netns-cleanup broken by ProcessMonitor refactor  https://review.openstack.org/16340115:02
openstackgerritMiguel Angel Ajo proposed openstack/neutron: Add a netns-cleanup functional test  https://review.openstack.org/16386015:02
openstackgerritMiguel Angel Ajo proposed openstack/neutron: Add a netns-cleanup functional test  https://review.openstack.org/16386015:04
ajo... sorry for the noise...15:04
trinaths1ajo: :) okay15:05
ajotrinaths1 :)15:06
*** ijw has joined #openstack-neutron15:06
ajoihrachyshka, amuller,  marun  ^15:06
ajolet me know how it looks15:06
*** tiswanso has joined #openstack-neutron15:07
*** baoli has joined #openstack-neutron15:08
*** moha_hunt has quit IRC15:09
*** ajmiller has joined #openstack-neutron15:11
*** ijw has quit IRC15:11
openstackgerritRomil Gupta proposed openstack/neutron: Added Flag for default External Networks type in ML2  https://review.openstack.org/14954615:12
*** chandankumar has quit IRC15:12
mesterymarun HenryG: The experimental job passed for the functional job for the client, this is ready to merge now: https://review.openstack.org/#/c/162848/15:12
*** vmtrooper has joined #openstack-neutron15:12
mesterymarun HenryG: It's a straight move of hte existing tests into the client repo15:12
openstackgerritAdelina Tuvenie proposed openstack/neutron: OVS agent support on Hyper-V  https://review.openstack.org/16059615:14
*** zhhuabj has joined #openstack-neutron15:14
*** trinaths1 has quit IRC15:15
*** yamahata has joined #openstack-neutron15:17
*** prad has quit IRC15:17
*** vmtrooper has quit IRC15:17
rkukuramatrohon: hi15:18
matrohonrkukura : hi15:18
*** pritesh has joined #openstack-neutron15:18
matrohonrkukura : I left a comment on DVR tech debt bug15:19
*** adelinatuvenie has quit IRC15:19
*** zhhuabj has quit IRC15:19
rkukuramatrohon: OK, I’ll look15:19
matrohonrkukura : I just wanted to know if there is still a chance that a patch make it for k315:19
*** SridharG1 has left #openstack-neutron15:19
openstackgerritJohn Davidge proposed openstack/neutron: Implement Neutron IPv6 Prefix Delegation  https://review.openstack.org/15869715:20
rkukuramatrohon: I’m hoping so. I made some progress last week, but haven’t been able to get back to it this week.15:20
rkukuramatrohon: But plan to work on it today15:20
matrohonrkukura : ok, cool!15:20
*** dannywilson has joined #openstack-neutron15:20
*** dsundarraj has quit IRC15:21
matrohonrkukura : this patch add tech debt to : https://review.openstack.org/#/c/141114/15:21
matrohonrkukura : but we move forward to have a chance to have a patch for k15:22
ihrachyshkamestery, hey15:22
*** dannywil_ has joined #openstack-neutron15:22
*** dannywilson has quit IRC15:23
matrohonrkukura : BTW rossella_s proposed to rebind the port when get_device_details is received : https://review.openstack.org/#/c/163178/15:23
*** david-ly_ is now known as david-lyle15:23
rkukuramatrohon: I thought it already did that if not already bound.15:23
matrohonrkukura : it address the case when the port is already bound, typically in Live-migration15:24
*** Murali has joined #openstack-neutron15:24
*** chandankumar has joined #openstack-neutron15:25
matrohonrkukura : I think it's a good idea, this would avoid a nova patch15:26
rkukuramatrohon: The get_device_details calling plugin.get_bound_port_context should result in rebinding if the host has changed.15:26
*** aranjan has quit IRC15:26
rkukuramatrohon: Nova updates the port’s binding:host_id on migration, right?15:26
*** ChuckC has joined #openstack-neutron15:26
matrohonrkukura : hummm... this is not the case in our experiments...15:26
rossella_srkukura: it doesn't bind it if the host change because the vif type is not VIF_TYPE_UNBOUND15:27
matrohonrkukura : yes, at least during Juno15:27
rossella_srkukura but if the port update from nova is processed after the get_device_detail we have a problem15:27
*** rpothier has joined #openstack-neutron15:28
rkukuramatrohon, rossella_s: Anything that changes the port’s host_id value should change invalidate the binding.15:28
rossella_srkukura: I totally agree, that's why I proposed that the port binding should be updated if the host change, regardless of the port vif type15:29
matrohonrkukura, rossella_s : so you have to think about how get_device_details is legitimate to modify the host15:29
rkukuramatrohon, rossella_s: BTW, my patch will be changing the way binding states (unbound vs. binding failed vs. bound)  are represented in the DB, so if there is a case not covered, please let me know15:29
rkukurarossella_s: Am I wrong that the only way the host_id can change is through a port_update?15:29
rossella_srkukura I didn't have the chance to review your patch, will look for it15:30
rkukuraJust receiving an RPC from a different host should not change the host_id, IMHO.15:30
rkukurarossella_s: Its not posted yet15:30
matrohonrkukura : so you consider get_device_detail RPC msg should not be allowed to change the binding?15:31
rkukuramatrohon: It can change the binding - i.e. cause a new binding attempt if the previous attempt failed.15:31
rossella_srkukura: right now if a port comes up on an l2 agent, it sends a get_device_details and the port is bound to that agent15:31
rkukuramatrohon: But nova should be doing a port_update to change the host_id before the port gets created on the new host during a migration, right?15:32
rkukurarossella_s: I really don’t think get_device_details should be changing the host_id. The REST call should be the only way to do that.15:32
matrohonrkukura : we can't be sure of that15:32
*** rwsu-afk is now known as rwsu15:33
*** chandankumar has quit IRC15:33
rossella_srkukura: there's no order guarantee, right? if the port update is processed after the messages sent by the l2 agent, we can't set the port state correctly15:33
matrohonrossella_s : +115:33
*** shwetaap1 has quit IRC15:34
rkukuramatrohon: Are you saying nova might not update the host_id first, or just that the neutron process handling the RPC might have stale state?15:34
matrohonrkukura : last time I check yes...15:34
*** reed has joined #openstack-neutron15:34
rossella_srkukura: I'd like to understand your point. If the get_device_details is allowed to set the host binding in case of a "new" port, why shouldn't it update the binding in case of a migration?15:34
rkukuramatrohon: The port update is synchronous, so nova shouldn’t re-plug the VIF until after the update REST call has returned.15:35
matrohonrkukura : that's why I reported this bug, but it was while ago : https://bugs.launchpad.net/neutron/+bug/127416015:35
openstackLaunchpad bug 1274160 in neutron "port_update from nova could potentially occurs after get_device_details/update_device_up" [Medium,In progress] - Assigned to Mathieu Rohon (mathieu-rohon)15:35
*** setuid has quit IRC15:35
*** prad has joined #openstack-neutron15:35
*** setuid has joined #openstack-neutron15:36
rkukuramatrohon: I hadn’t seen that. Is this really a nova bug?15:36
*** shwetaap has joined #openstack-neutron15:36
matrohonrkukura : I first though about a nova bug but sdague removed it because this bug hasn't been seen, just a potential race to confirm15:37
*** itzikb has quit IRC15:37
rkukuramatrohon, rossella_s: I’ve considered the REST calls to be the only way to change normal “input” attributes like binding:host_id.15:38
*** Murali has quit IRC15:38
rkukuramatrohon, rossella_s: I’m concerned if we allow the get_device_details RPC to update the host_id, we could potentially have race conditions between L2 agents on the two hosts during a migtration. The old host could possible to a get_device_details RPC for the port at about the same time as the new host, and either could win.15:40
*** _cjones_ has joined #openstack-neutron15:40
openstackgerritZongKai LI proposed openstack/neutron: Remove router binding with router-interface-delete  https://review.openstack.org/16345415:41
rkukuramatrohon, rossella_s: I wish we could discuss these kinds of issues at the ML2 IRC meetings.15:41
openstackgerritSandhya Dasu proposed openstack/neutron: Cisco UCS Manager ML2 Mechanism Driver  https://review.openstack.org/15543615:42
*** seizadi has quit IRC15:42
matrohonrkukura, rossella_s :  yep, but unfortunately I'm available for every ML2 meetings15:42
mlavallepc_m: do you want to talk now or would you rather wait for carl_baldwin to be back and we all talk together?15:42
pc_mdoesn't matter to me. I'm adding some items in the bug right now.15:42
*** Swami_ has joined #openstack-neutron15:42
matrohonrossella_s : do you think the race rkukura mentioned could happen15:42
*** aranjan has joined #openstack-neutron15:43
rossella_smatrohon, rkukura I don't see how that race could happen. get_device_details is sent by the l2 agent when a new device is discovered. So the old l2 agent won't send it15:43
*** zilberstein has joined #openstack-neutron15:43
zilbersteinGood evening #openstack-neutron15:44
HenryGmarun: ping15:44
rossella_smatrohon, rkukura the only possibility is a port going up and then down on the old host before being migrated...but not sure how likely it is15:44
zilbersteinwhat interface_driver I should use in dhcp_agent.ini if I use ofagent,l2population with openvswitch ?15:44
rkukurarossella_s, matrohon: Couldn’t the old host’s agent just be coming up at the same time as the migration, or maybe the VM was migrated immediately after it was created?15:44
rkukurarossella_s, matrohon: Seems a migration might be triggered by a compute host being down, but it could recover during the migration.15:45
*** tflynn has joined #openstack-neutron15:46
*** armax has joined #openstack-neutron15:46
rkukurarossella_s, matrohon: I think its safer for it to be nova’s responsibility to make sure the port_update is called when it decides to migrate the VM.15:46
*** ajo has quit IRC15:47
*** aranjan has quit IRC15:47
*** sudipto has quit IRC15:48
matrohonrkukura : the scenario where the migration is triggered by a host going down makes sense15:48
*** ajo has joined #openstack-neutron15:48
rkukurarossella_s, matrohon: I do think get_device_details should check the host_id, and return an error if the agent making the call isn’t the current host. I think I saw a recent patch for this.15:49
rossella_srkukura: you read my mind. That's what I was going to suggest15:49
rossella_srkukura: I am working on that, maybe you've seen one of my patches15:49
matrohonrossella_s : do you have the link to this patch?15:50
rkukurarossella_s: I do try to look at them ;)15:50
rossella_srossella_s, matrohon do we agree that the best would be to return an error if the host_id doesn't match so that we give the time to the neutron server to process the port update?15:50
*** tflynn has quit IRC15:50
HenryGajo: ping15:51
ajohi HenryG15:51
ajohow're you doing?15:51
HenryGajo: good thanks, how about you?15:51
*** dsundarraj has joined #openstack-neutron15:51
ajoHenryG, very well, playing a bit with IPv6 ;) all my home exposed via v6... I feel naked ;D15:52
ajoI will need to firewall a bit, I guess ;D15:52
*** wwriverrat has joined #openstack-neutron15:52
mlavallepc_m: Plese read the comment I made here: https://review.openstack.org/#/c/147744/18/neutron/agent/l3/dvr_snat_ns.py. It is a little long. My conclusion, is that snat namepsaces were sometimes created implicitely prior to the refactoring and now we have made that evident after the refactoring15:52
*** ihrachyshka has quit IRC15:52
matrohonrkukura, rossella_s : I agree, this won't fix the potential race but I think it will fix this error : https://bugs.launchpad.net/neutron/+bug/1274160/comments/715:53
openstackLaunchpad bug 1274160 in neutron "port_update from nova could potentially occurs after get_device_details/update_device_up" [Medium,In progress] - Assigned to Mathieu Rohon (mathieu-rohon)15:53
mlavallepc_m: I need to relocate to the office, but I will ping you later in a bit after you have read that comment. is that ok?15:53
HenryGajo: I am looking at https://review.openstack.org/15251215:53
pc_mok15:53
ajoHenryG, opening15:53
*** wwriverrat has left #openstack-neutron15:53
pc_mmlavalle: will loook at it15:53
matrohonrkukura, rossella_s : If we don't modify the port status of course15:53
ajoHenryG, how does it look did you find anything strange? :)15:53
HenryGajo: The fix is good. No problems there.15:53
rossella_smatrohon: there are several patches for the l2 agent, but I don't exactly check the host id, not sure if Bob meant the patch by somebody else...15:53
mlavallepc_m: it is not excatly the problem you are dealing with, but I think it is all related15:53
*** vivek-ebay has joined #openstack-neutron15:54
ajoHenryG: thanks for looking at it :)15:54
rkukurarossella_s: The patch I’m thinking of (or imagining) change get_device_details in neutron-server to check the host_id.15:54
rossella_smatrohon: if you are interested just check the patch owned by me...it's still WIP but I am trying to modify the way the ovs agent handles arrors15:54
HenryGajo: I ended up with many nits on the tests. Nothing I want to block on, but I wonder if you want to have a chance to read them before I approve?15:54
ajosure HenryG , makes sense15:55
rossella_srkukura: I haven't seen it then...if you find it please share :)15:55
ajomay be I can sort them out tomorrow morning15:55
rkukurarossella_s, matrohon: Do we know for sure that nova may re-plug the port before the update_port has returned?15:55
matrohonrkukura : may be you're talking about my current proposal : https://review.openstack.org/#/c/163178/4/neutron/plugins/ml2/rpc.py15:55
matrohonrkukura : I'll recheck15:55
*** mlavalle has quit IRC15:55
rkukuramatrohon: that’s the one15:56
ajoHenryG, so feel free to publish the comments without approving, and I will fix :)15:56
*** Longgeek has joined #openstack-neutron15:56
rossella_srkukura, matrohon mistery solved :) ...shall we continue the discussion on the review directly?15:56
matrohonrkukura : but by the time I reported the bug, the race could be seen with a debugger that slows down nova15:56
rkukurarossella_s: makes sense, but feel free to ping me on IRC15:57
HenryGajo: thanks. Also marun might have something to say, maybe.15:57
*** pal_ has joined #openstack-neutron15:57
ajoack, marun  ^  https://review.openstack.org/15251215:58
matrohonrkukura : fine, I'll try to summarize this discussion on gerrit, and propose a patch that returns an error when the port is not bound to host submitted by get_device_details15:58
marunajo: ack15:58
rossella_smatrohon, rkukura thanks for the nice discussion!15:58
matrohonrossella_s : thanks for your review!15:59
rossella_smatrohon you can't return an error right now, the l2 agent will resync everything15:59
rossella_smatrohon: my pleasure!15:59
HenryGajo: comments posted15:59
*** pal has quit IRC15:59
*** ronis has quit IRC15:59
*** vthapar has quit IRC15:59
rossella_smatrohon: I mean right now the code in the l2 agent is dummy, in case of error it's going to resync. I am changing that...after the agent behaves better in case of error, you can modify get_device_details to return an error if the host id doesn't match16:00
*** nlahouti has joined #openstack-neutron16:00
*** thedodd has quit IRC16:00
*** roeyc has quit IRC16:01
*** carl_baldwin has quit IRC16:01
matrohonrossella_s : ok, so right now, returning an error will trigger a loop of RPC get_device_details call?16:01
*** aveiga has joined #openstack-neutron16:02
rossella_smatrohon right now, the error will trigger a resync in the OVS agent, which will send get_device_details again and if it gets another error...well loop16:03
rossella_smatrohon: it will probably loop till the port update is processed...that's very bad16:03
*** andreykurilin has quit IRC16:03
*** prad has quit IRC16:04
rossella_smatrohon: if there are many devices, the resync is expensive16:04
matrohonrossella_s : ok I'd better to propose my patch on top of yours16:04
*** killer_prince is now known as lazy_prince16:04
rossella_smatrohon: if you are not in an hurry :)16:04
ajothanks HenryG , I commented back, I will fix, but I disagree about two points, one in the iptables_firewall @staticmethod thing, and the other about duplicating set and list (line 1485 @ test), the 2nd one not very strong, but check if what I said makes sense. :)16:05
matrohonrossella_s : which one of your patch introduces error handling in l2 agent?16:05
ajoHenryG if you have a second, please check my comments, and thanks for the review, not all were nits ;)16:05
*** ijw has joined #openstack-neutron16:06
HenryGajo: I certainly did not expect agreement to all my opinions! Thanks for looking, I will check your comments.16:06
rossella_smatrohon: https://review.openstack.org/#/c/15783916:07
ajo:D HenryG  :)16:07
rossella_smatrohon: I have to update it a little bit, anyway you get the general idea there16:07
matrohonrossella_s : ok thanks16:08
*** nmagnezi_ has quit IRC16:08
*** claudiub has quit IRC16:09
*** scheuran_ has quit IRC16:09
*** tiswanso has quit IRC16:09
matrohonrossella_s : my patch is mainly motivated by errors seen will running this patch : https://review.openstack.org/#/c/141114/16:09
matrohonrossella_s : we would love to have l2pop compatible with HA router in k, so we are more or less in a hurry :)16:10
*** dboik_ has quit IRC16:10
*** tiswanso has joined #openstack-neutron16:10
*** dboik has joined #openstack-neutron16:10
rossella_smatrohon: I understand, well you know that having dependencies doesn't speed up16:11
*** chandankumar has joined #openstack-neutron16:11
*** ijw has quit IRC16:12
*** ParsectiX has joined #openstack-neutron16:12
*** vivek-eb_ has joined #openstack-neutron16:12
*** dsundarraj has quit IRC16:12
*** setuid has quit IRC16:13
matrohonrossella_s : of course, but there is another alternative proposed for 141114, so I'll move forward with dependencies16:13
openstackgerritvishwanath jayaraman proposed openstack/neutron-fwaas: Refactoring related to 'FWaaS insertion Model on Routers'  https://review.openstack.org/16188116:13
rossella_smatrohon just read your comment on that patch. I think 2 is a good option16:14
*** yfried has quit IRC16:14
*** setuid has joined #openstack-neutron16:14
*** vivek-ebay has quit IRC16:16
*** tsg_ has quit IRC16:17
*** setuid has quit IRC16:18
*** claudiub has joined #openstack-neutron16:20
*** setuid has joined #openstack-neutron16:21
*** dsundarraj has joined #openstack-neutron16:22
*** bandarji has joined #openstack-neutron16:23
*** prad has joined #openstack-neutron16:23
*** seizadi has joined #openstack-neutron16:24
*** seizadi has quit IRC16:25
*** seizadi has joined #openstack-neutron16:25
*** rkukura has quit IRC16:25
*** dsundarraj has quit IRC16:25
*** vivek-eb_ has quit IRC16:26
*** Longgeek has quit IRC16:26
openstackgerritZengfa Gao proposed openstack/python-neutronclient: Add Neutron subnetpool API  https://review.openstack.org/15961816:27
*** afazekas_ has quit IRC16:27
*** vivek-ebay has joined #openstack-neutron16:27
*** pritesh has quit IRC16:28
*** baoli has quit IRC16:28
*** jlanoux has quit IRC16:28
*** irenab has joined #openstack-neutron16:28
*** vivek-ebay has quit IRC16:28
*** matrohon has quit IRC16:30
*** salv-orlando has joined #openstack-neutron16:31
openstackgerritMerged openstack/python-neutronclient: Copy functional tests from tempest cli  https://review.openstack.org/16284816:32
enikanorovsalv-orlando: hi! so... how do you think, who's +A do we need? :) https://review.openstack.org/#/c/149261/16:32
*** bjornar_ has joined #openstack-neutron16:36
*** absubram has joined #openstack-neutron16:37
*** rkukura has joined #openstack-neutron16:38
*** baoli has joined #openstack-neutron16:38
*** chandankumar has quit IRC16:38
*** zhhuabj has joined #openstack-neutron16:38
claudiubarmax: hello16:38
openstackgerritMerged openstack/python-neutronclient: Remove unused AlreadyAttachedClient  https://review.openstack.org/16341916:38
*** zzzeek has joined #openstack-neutron16:40
openstackgerritZongKai LI proposed openstack/neutron: fix check_ports_exist_on_l3agent in no subnet case  https://review.openstack.org/15933816:40
*** SridharRamaswamy has joined #openstack-neutron16:41
* zilberstein loses connectivity when neutron starts16:42
zilbersteinWhat to check ?16:42
larskszilberstein: start with why your connectivity is gone...routing problem? Address got deleted?  iptables blocking traffic?  Also, you probably want to move this to #openstack; this channel is primarily for development questions.16:44
*** jistr has quit IRC16:45
zilbersteinlarsks, no, routering table hasnt changed in linux. What is strange is that when I stop neutron-ofagent, it works again16:45
*** vivek-ebay has joined #openstack-neutron16:45
zilbersteinand ovs-vsctl shows br-int and its only port eth0 that I added16:45
zilbersteinas it should be16:45
openstackgerritSridar Kandaswamy proposed openstack/neutron-fwaas: Adds FWaaS service plugin for Cisco CSR1kv  https://review.openstack.org/15228216:46
*** pal_ has quit IRC16:47
*** yamahata has quit IRC16:47
*** Miouge has joined #openstack-neutron16:48
*** yamamoto has quit IRC16:48
*** carl_baldwin has joined #openstack-neutron16:48
*** Marga_ has joined #openstack-neutron16:52
openstackgerritCarl Baldwin proposed openstack/neutron: Introduce External IPAM Interface  https://review.openstack.org/13433916:53
carl_baldwinmestery: ^ working around new pylint requirement.16:53
*** aranjan has joined #openstack-neutron16:53
*** leenheer has joined #openstack-neutron16:54
*** ParsectiX has quit IRC16:54
*** ParsectiX has joined #openstack-neutron16:54
*** Marga__ has joined #openstack-neutron16:55
*** Swami_ has quit IRC16:55
*** Marga_ has quit IRC16:57
*** sbfox has joined #openstack-neutron16:57
*** VijayB_ has joined #openstack-neutron16:58
*** Marga__ has quit IRC16:58
*** aranjan has quit IRC16:58
*** ihrachyshka has joined #openstack-neutron16:58
*** aranjan has joined #openstack-neutron16:58
*** Marga_ has joined #openstack-neutron16:58
*** mwagner_lap has quit IRC16:59
*** seizadi has quit IRC16:59
*** SridharRamaswamy has quit IRC16:59
*** seizadi has joined #openstack-neutron16:59
*** afazekas has joined #openstack-neutron16:59
*** yfauser has joined #openstack-neutron17:00
*** vmtrooper has joined #openstack-neutron17:01
*** yfauser has left #openstack-neutron17:02
*** kickinz1 is now known as kickinz1|afk17:03
*** jlanoux has joined #openstack-neutron17:04
*** harlowja_away is now known as harlowja_17:06
*** ijw has joined #openstack-neutron17:06
*** vmtrooper has quit IRC17:06
*** Swami has joined #openstack-neutron17:07
*** VijayB_ has quit IRC17:07
*** chandankumar has joined #openstack-neutron17:08
*** yamahata has joined #openstack-neutron17:09
*** ijw has quit IRC17:11
*** VijayB_ has joined #openstack-neutron17:13
*** aranjan has quit IRC17:14
*** ronis has joined #openstack-neutron17:16
*** aranjan has joined #openstack-neutron17:17
*** ygbo has quit IRC17:17
*** aranjan has quit IRC17:18
ihrachyshkaHenryG, hey. up to a challenge? ;)17:19
HenryGihrachyshka: not the oslo.log patch? ;)17:19
ihrachyshkaok, moving on...17:20
ihrachyshka:)17:20
HenryGihrachyshka: hit me17:20
ihrachyshkawell... yeah, the oslo.log: https://review.openstack.org/#/c/159638/17:20
HenryGihrachyshka: Ouch! ;)17:20
ihrachyshkaI will need two cores to get it in the queue before something breaks it (and that can be anything)17:21
HenryGihrachyshka: sure, I will do it. I have been keeping track of it so it should not be too bad.17:21
ihrachyshkaHenryG, even if you track patches like that, the number of files is just... unmanageable17:22
ihrachyshkaHenryG, thanks a lot, I owe you17:22
*** sbalukoff has quit IRC17:22
*** melwitt has joined #openstack-neutron17:22
*** aranjan has joined #openstack-neutron17:23
ajoHenryG, your list vs set comment makes sense, thanks for responding so fast ;)17:23
*** aix has quit IRC17:25
*** aranjan has quit IRC17:25
*** kawa2014 has quit IRC17:25
*** britthouser has quit IRC17:26
*** aranjan has joined #openstack-neutron17:26
*** britthouser has joined #openstack-neutron17:26
*** devlaps has joined #openstack-neutron17:26
*** evgenyf has quit IRC17:29
*** pask81 has quit IRC17:29
*** wuhg has quit IRC17:29
*** jp_at_hp has quit IRC17:31
*** aranjan_ has joined #openstack-neutron17:32
*** aranjan has quit IRC17:32
*** achanda has joined #openstack-neutron17:32
*** blues-man has quit IRC17:32
*** ivar-lazzaro has joined #openstack-neutron17:33
*** tflynn has joined #openstack-neutron17:35
*** sripriya has joined #openstack-neutron17:37
*** lazy_prince is now known as killer_prince17:41
*** yfried has joined #openstack-neutron17:42
*** sudipto has joined #openstack-neutron17:43
*** aranjan_ has quit IRC17:44
*** shwetaap1 has joined #openstack-neutron17:45
*** aranjan has joined #openstack-neutron17:47
*** mrsmith has joined #openstack-neutron17:47
*** pritesh has joined #openstack-neutron17:47
*** shwetaap has quit IRC17:48
*** natarajk has joined #openstack-neutron17:51
*** safchain has quit IRC17:52
*** Marga_ has quit IRC17:53
*** claudiub has quit IRC17:54
*** aranjan has quit IRC17:56
*** Marga_ has joined #openstack-neutron17:56
*** aranjan has joined #openstack-neutron17:56
pc_mcarl_baldwin: hi17:57
carl_baldwinpc_m: hi17:57
pc_mcarl_baldwin: Wondering if you had time to chat about the bug reported with DVR and namespace for VPN17:58
*** SumitNaiksatam has joined #openstack-neutron17:59
carl_baldwinpc_m: Yes, what is the latest?17:59
*** annegentle has quit IRC18:00
*** yfried is now known as yfried|afk18:00
*** shivharis has joined #openstack-neutron18:00
pc_mcarl_baldwin: Well, in VPN, when creating a process manager for OpenSwan, it expects there to be a namespace for the router.18:00
*** SridharRamaswamy has joined #openstack-neutron18:01
carl_baldwinRight.18:01
pc_mThis call happens when VPN service is created, or when router added (I guess for case of router add taking longer than service create).18:01
pc_mIn the past, this was always true, but is not, now for DVR.18:02
*** igordcard_ has joined #openstack-neutron18:02
pc_mNow, for the latter case, which could happen with just a router add and no service, it would not create the process.18:03
*** dims__ has joined #openstack-neutron18:03
*** dims__ has quit IRC18:03
*** bjornar_ has quit IRC18:04
pc_mHowever, it calls code as part of the "sync" that will try to delete the process for any processes where there is no VPN service.18:04
carl_baldwinpc_m: Just need 3-5 minutes to clean out a workspace to start looking at it.  Bear with me.18:04
*** dims__ has joined #openstack-neutron18:04
pc_mOK18:04
*** flaviof has quit IRC18:04
*** flaviof has joined #openstack-neutron18:05
*** mwagner_lap has joined #openstack-neutron18:06
*** seizadi has quit IRC18:06
*** ijw has joined #openstack-neutron18:06
*** seizadi1 has joined #openstack-neutron18:06
*** jlanoux has quit IRC18:06
pc_mcarl_baldwin: Let em know when you're ready, and I'll get all the links to the code for the issue.18:06
*** fredlhsu has joined #openstack-neutron18:07
*** dims_ has quit IRC18:07
carl_baldwinpc_m: ok, ready18:07
* carl_baldwin reading back...18:07
pc_mHere's the sequence that causes the issue....18:08
openstackgerritCarl Baldwin proposed openstack/neutron: Move internal port processing to router classes  https://review.openstack.org/15451318:08
pc_mcarl_baldwin: Router is created, and notify() calls this: https://github.com/openstack/neutron-vpnaas/blob/master/neutron_vpnaas/services/vpn/vpn_service.py#L5718:08
pc_mIt will add the router to the driver, and do a sync().  https://github.com/openstack/neutron-vpnaas/blob/master/neutron_vpnaas/services/vpn/device_drivers/ipsec.py#L77218:09
*** jpena has quit IRC18:09
*** dims__ has quit IRC18:09
pc_mFor the sync, nothing will happen for _sync_vpn_processes(), but it will try to get rid of processes without VPN.18:09
pc_mNote process is an object that manages the OpenSwan process.18:10
pc_mFor some reason, that module first creates the process, if it doesn't exist: https://github.com/openstack/neutron-vpnaas/blob/master/neutron_vpnaas/services/vpn/device_drivers/ipsec.py#L79518:10
pc_mAnd that's the issue, as there is no namespace.18:10
pc_mNow, there is a fix to that. Here: https://review.openstack.org/#/c/162840/8/neutron_vpnaas/services/vpn/device_drivers/ipsec.py18:11
*** shivharis has quit IRC18:11
pc_mIt is part of another fix, where I suggested this line be removed (couldn't see why it was there). It would avoid the problem18:11
*** shivharis has joined #openstack-neutron18:11
*** Longgeek has joined #openstack-neutron18:11
*** ijw has quit IRC18:12
pc_mcarl_baldwin: However, the question is, would we have an issue with the namespace not being set, if a router is created and a VPN connection added immediately?18:12
*** annegentle has joined #openstack-neutron18:12
*** fredlhsu has quit IRC18:12
* SridharRamaswamy pulls a chair to learn his router -> vpn flow18:12
carl_baldwinpc_m: I’m following along, just trying to soak it in still18:13
pc_mcarl_baldwin: So it creates the router entry, does a sync operation, and tries to create the process, so it can delete it...duh18:13
pc_mThe fix out for review on another item, would prevent this issue too, IMHO.18:14
pc_mHowever, I'm concerned we could create a router and VN connection, and maybe not have the namespace in time, for a normal VPN create flow.18:15
*** mlavalle has joined #openstack-neutron18:15
*** nyechiel__ has quit IRC18:15
*** annegentle has quit IRC18:15
carl_baldwinpc_m: It doesn’t look right to “ensure_process” just to destroy the router.18:15
*** iwi has joined #openstack-neutron18:15
*** annegentle has joined #openstack-neutron18:15
pc_mexactly, so when this fix was out for review, I asked to have that pulled. No idea why it was there.18:15
carl_baldwinThe normal VPN flow happens on “after_router_added”, right?18:15
iwiHi there18:16
*** baoli has quit IRC18:16
pc_mcarl_baldwin: Yes, you'd add the router, and then once added, the VPN service could be created and connection created.18:16
pc_mcarl_baldwin: Wondering if it is always guaranteed that the namespace will be there by then.18:16
iwiis it possible to use an external network as a normal tenant network (so for both - DHCP for VMs and Floating IPs)18:16
iwi?18:16
carl_baldwinpc_m: Then the namespace will be there.18:16
HenryGihrachyshka: done, now you need one more.18:17
*** kawa2014 has joined #openstack-neutron18:17
pc_mcarl_baldwin: If so, then I think the fix out will kill two birds with one stone.18:17
*** achanda has quit IRC18:17
ihrachyshkaHenryG, thanks, let's see who hasn't hidden under a bench yet...18:17
carl_baldwinpc_m: Still speaking of this fix:  https://review.openstack.org/#/c/162840 ?18:18
pc_mcarl_baldwin: yes18:18
ihrachyshkamestery, up to adventure?.. ;) I need someone brave to dive into oslo.log migration patch to push it in gate.18:19
pc_mcarl_baldwin: It clears up some other issues with a recent refactoring that I had done, but changes this area too (as we need to delete the process, but not the router we created, when we delete a service).18:19
*** jschwarz has quit IRC18:20
*** yfried|afk is now known as yfried18:20
*** Marga_ has quit IRC18:20
pc_mSridharRamaswamy: I've got some questions for you too, once done here, if you're still around.18:20
SridharRamaswamypc_m: sure18:21
carl_baldwinpc_m: Reviewing it now.18:21
pc_mcarl_baldwin: Thanks. So will the namespace be there shortly after the after_router_added() notify?18:21
mlavallecarl_baldwin, pc_m: I see you are weel underway in your discussion about the namespace issue. Don't want to get in the way. Please ping me if you think I might add something18:22
*** achanda has joined #openstack-neutron18:23
*** tsg_ has joined #openstack-neutron18:23
pc_mmlavalle: yeah, sorry forgot to ping you too. If you can scroll up, you'll see we have a fix. Though I do have a question on when the namespace is ready.18:23
carl_baldwinpc_m: The namespace will be created for a new router before notifying.18:23
*** achanda has quit IRC18:23
pc_mcarl_baldwin: Hmmm. Then wondering why there was no namespace.18:24
openstackgerritMerged openstack/neutron: Fix non-existent self.local_subnets in DvrRouter class  https://review.openstack.org/15762118:24
*** achanda has joined #openstack-neutron18:24
*** kawa2014 has quit IRC18:24
*** rushil_ has quit IRC18:24
*** kawa2014 has joined #openstack-neutron18:24
pc_mSridharRamaswamy: For the Brocade VPN plugin I had some questions about out of tree code...18:26
SridharRamaswamypc_m: sure, i replied after your discussion w/ armax18:26
SridharRamaswamypc_m: I think you logged off by then18:26
pc_mSridharRamaswamy: (may have missed) . The code has an import of networking_brocade.18:26
*** afazekas has quit IRC18:26
pc_mSridharRamaswamy: How does the package get installed, say for devstack or UT use?18:27
SridharRamaswamypcm:  We got that working by mocking the networking_brocade imports. See #link https://github.com/openstack/neutron-vpnaas/blob/master/neutron_vpnaas/tests/unit/services/vpn/device_drivers/test_vyatta_ipsec.py18:27
*** sudipto has quit IRC18:28
pc_mSridharRamaswamy: Ah. Makes sense. Then for devstack, do you have a devstack plugin to do the install?18:28
SridharRamaswamypc_m: this is was standard suggestion for all the decomposed plugins - to get their UT working *without* doing the import18:28
SridharRamaswamypc_m: No. Our CI scripts does add a separate entry point to invoke out vyatta-vpn-agent. But that happens only in our CI env18:30
*** yfried is now known as yfried|afk18:30
*** yfried|afk is now known as yfried18:31
*** Longgeek has quit IRC18:31
pc_mSridharRamaswamy: So for functional tests, you could create a devstack plugin to load the package, I guess.18:31
carl_baldwinpc_m: If I understood correctly, there was no namespace because it wasn’t a new router, it was a router being cleaned up.18:31
pc_mcarl_baldwin: Oh, do you mean in that test case they were cleaning up a router?18:32
Swamipcm: that might be true18:32
pc_mcarl_baldwin: Why would it call after_router_added (and not after/before_router_removed)?18:33
carl_baldwinpc_m: I thought that a different router was added which notified.  Then a sync routers was called that was trying to destroy another router, not the one that was just added.18:34
pc_mcarl_baldwin: Ah, ok that would make sense.18:34
*** dims has joined #openstack-neutron18:35
SridharRamaswamypc_m: Yes, we have devstack patches in the CI env. We can augment that if needed to get the functional tests going.18:35
openstackgerritZhang Hua proposed openstack/neutron-vpnaas: Functional tests of ipsec strongswan vpnaas driver  https://review.openstack.org/15856018:36
openstackgerritZhang Hua proposed openstack/neutron-vpnaas: IPsec strongSwan driver implemention  https://review.openstack.org/14439118:36
*** claudiub has joined #openstack-neutron18:36
pc_mSridharRamaswamy: Thanks for the info. We may need to do the same in Cisco code.18:37
*** numan has joined #openstack-neutron18:37
*** numan has quit IRC18:37
SridharRamaswamypc_m: sounds good18:38
*** Sukhdev has joined #openstack-neutron18:38
Sukhdevmestery: Ping18:39
*** skraynev has quit IRC18:39
Sukhdevmestery: Can you please tag this for K3? - https://bugs.launchpad.net/neutron/+bug/142996818:40
openstackLaunchpad bug 1429968 in neutron "In Arista ML2 delete tenant if no resources associated with it" [Undecided,Fix committed] - Assigned to Sukhdev Kapur (sukhdev-8)18:40
pc_mcarl_baldwin: Thanks for approving. What do we do about the bug filed? Have the submitter check and close, if fixed?18:40
carl_baldwinpc_m: Sounds like a good plan.18:40
Swamipc_m: what bug are you talking about here.18:41
pc_mcarl_baldwin: I updated bug.18:41
pc_mSwami: 143107718:41
*** sbalukoff has joined #openstack-neutron18:42
pc_mSwami: Is that the one you filed?18:42
Swamipc_m: Yes this is the one that I filed.18:42
SwamiI will check this with your patch and then update the LP.18:42
pc_mSwami: Thanks!18:42
pc_mSwami: I think it may resolve the issue.18:43
pc_mcarl_baldwin: If you get a chance, look at https://review.openstack.org/#/c/163586/. Easy peasy review and removes dead code related to this namespace thing.18:44
*** claudiub has quit IRC18:47
carl_baldwinpc_m: Done18:47
*** SridharRamaswamy is now known as sridhar_ram18:48
pc_mSridharRamaswamy: Can you test the latest StrongSwan patch (61)? The previous one looked good, other than changes interfered with running connections. I think Zhang has a fix for that. I'm going to re-test as well.18:48
pc_mcarl_baldwin: Thank you. Don't you wish they all were that easy? :)18:48
carl_baldwinpc_m: :)18:48
sridhar_rampc_m: sure, will give it a spin18:49
*** sbfox has quit IRC18:49
pc_msridhar_ram: THanks. Looks like we're close to getting this in...18:49
*** vmtrooper has joined #openstack-neutron18:50
*** Marga_ has joined #openstack-neutron18:50
enikanorov_mestery: hi, who could be a core to give +A https://review.openstack.org/#/c/149261/ ? This has raised a lot of discussions, but i think we may want to have a fix for the issue and move on with enabling multiple api and rpc workers  by default18:52
*** vmtrooper has quit IRC18:55
*** thedodd has joined #openstack-neutron18:55
openstackgerritCarl Baldwin proposed openstack/neutron: Move internal port processing to router classes  https://review.openstack.org/15451318:56
sridhar_rampc_m: i agree, but state transition from pending-create --> down --> active still bothers me. I'm going to check if that got fixed in the latest PS.19:00
pc_msridhar_ram: Yeah, I mentioned that I prefer PENDING_CREATE - > ACTIVE, but it will take more complex logic to check that, because of the way StrongSwan reports status.19:03
pc_msridhar_ram: I mentioned that I'd be OK with doing that as a bug fix later.19:03
*** yfried is now known as yfried|afk19:04
*** shivharis has quit IRC19:04
sridhar_rampc_m: okay19:04
*** shivharis has joined #openstack-neutron19:05
pc_msridhar_ram: it reports CONNECTING, when connecting, but for a period of time, it reports INSTALLED and CONNECTING, so need to check multiple status lines.19:05
sridhar_rampc_m: I see19:05
*** kawa2014 has quit IRC19:05
*** bobmel_ has joined #openstack-neutron19:06
*** yfried|afk is now known as yfried19:06
pc_msridhar_ram: I'm wondering if openswan drops traffic on a config change (adding a second connection). I think it will.19:06
*** ijw has joined #openstack-neutron19:06
*** yfauser has joined #openstack-neutron19:07
*** yfauser has left #openstack-neutron19:07
*** bobmel has quit IRC19:09
*** tgohad has joined #openstack-neutron19:09
*** carl_baldwin has quit IRC19:10
*** Marga_ has quit IRC19:11
*** ijw has quit IRC19:11
*** seizadi1 has quit IRC19:12
*** tsg_ has quit IRC19:13
*** seizadi has joined #openstack-neutron19:18
*** chandankumar has quit IRC19:19
*** seizadi has quit IRC19:19
*** ParsectiX has quit IRC19:22
*** crc32 has joined #openstack-neutron19:22
*** ParsectiX has joined #openstack-neutron19:22
*** shivharis has quit IRC19:23
openstackgerritBrian Bowen proposed openstack/neutron: Fix missing spaces in strings split across lines.  https://review.openstack.org/16385919:25
*** claudiub has joined #openstack-neutron19:26
*** leenheer has quit IRC19:30
*** leenheer has joined #openstack-neutron19:33
*** chandankumar has joined #openstack-neutron19:34
*** YorikSar has joined #openstack-neutron19:34
*** jorgem has joined #openstack-neutron19:35
YorikSarotherwiseguy: Do you know if there's a consensus about whether we should enable daemon mode by default?19:35
*** ParsectiX has quit IRC19:36
otherwiseguyYorikSar: A lot of people have asked that. I tend to be cautious about such things. I mean, technically even using rootwrap at all is non-default.19:36
otherwiseguySince root_helper defaults to 'sudo'.19:37
YorikSarotherwiseguy: Is it? Oh...19:37
otherwiseguyBut rootwrap daemon should be faster than sudo too...19:37
YorikSarotherwiseguy: Ok, then the question should be whether we want devstack to configure daemon mode by default.19:38
YorikSarotherwiseguy: Or we'll have to run 2 jobs: one for daemon mode, another for rootwrap case.19:38
*** pc_m has quit IRC19:38
*** sbfox has joined #openstack-neutron19:38
*** _cjones_ has quit IRC19:39
otherwiseguyYorikSar: If it was up to me, I'd say yes just from the perspective that 'rootwrap' code doesn't really change a lot and is unlikely to break, also even neutron will use rootwrap and rootwrap daemon both (because of long-running processes), so both will get exercised.19:39
*** openstackgerrit has quit IRC19:40
otherwiseguyOthers may like the 'do both' approach, but I hate doubling effort for testing.19:40
YorikSarotherwiseguy: I guess this discussion will be held in devstack CR19:40
*** openstackgerrit has joined #openstack-neutron19:41
YorikSarotherwiseguy: Yeah. Besides rootwrap is well-tested (including daemon mode).19:41
otherwiseguyYorikSar: so does 158805 look good now that the devstack patch tests are passing?19:42
*** aranjan has quit IRC19:42
YorikSarotherwiseguy: Let's wait for another round... As I said it's very strange that lbaas counldn't start daemon while other services could.19:43
YorikSar(I mean dhcp and l3 agents)19:43
*** leenheer has quit IRC19:43
*** leenheer has joined #openstack-neutron19:44
*** chandankumar has quit IRC19:45
*** Marga_ has joined #openstack-neutron19:45
*** Marga_ has quit IRC19:50
*** yfried is now known as yfried|afk19:51
*** marun has quit IRC19:54
*** Padmini has joined #openstack-neutron19:58
*** _cjones_ has joined #openstack-neutron19:59
*** imcsk8|zZz is now known as imcsk820:01
*** melwitt_ has joined #openstack-neutron20:02
*** mikal_ is now known as mikal20:02
*** seizadi has joined #openstack-neutron20:02
*** melwitt has quit IRC20:02
*** melwitt_ is now known as melwitt20:02
*** nlahouti has quit IRC20:03
*** seizadi has quit IRC20:03
*** seizadi has joined #openstack-neutron20:03
*** setuid has quit IRC20:03
*** baoli has joined #openstack-neutron20:03
*** sbfox has quit IRC20:04
*** baoli has quit IRC20:04
*** setuid has joined #openstack-neutron20:05
*** baoli has joined #openstack-neutron20:05
*** aranjan has joined #openstack-neutron20:05
*** jgrimm is now known as zz_jgrimm20:07
*** leenheer has quit IRC20:08
*** matrohon has joined #openstack-neutron20:10
*** yfried|afk is now known as yfried20:11
*** aepifanov has joined #openstack-neutron20:11
*** sbfox has joined #openstack-neutron20:11
*** yfauser has joined #openstack-neutron20:12
*** rushil has joined #openstack-neutron20:13
*** yfauser has left #openstack-neutron20:13
*** seizadi has quit IRC20:13
*** seizadi has joined #openstack-neutron20:14
*** zz_jgrimm is now known as jgrimm20:14
*** Longgeek has joined #openstack-neutron20:16
*** amuller has quit IRC20:16
*** chandankumar has joined #openstack-neutron20:16
ihrachyshkaarmax, hey. may I ask you to check oslo.log migration patch? It needs the 2nd +2: https://review.openstack.org/159638 (CI failures are expected and were communicated to 3party maintainers)20:17
openstackgerritMichael Smith proposed openstack/neutron: delay fip namespace cleanup/destroy  https://review.openstack.org/15175820:18
*** sambetts has quit IRC20:18
openstackgerritJerome Bertolino proposed openstack/neutron: Update Cisco decomposition URL  https://review.openstack.org/16395020:19
*** jorgem has quit IRC20:20
*** sambetts has joined #openstack-neutron20:20
*** yfried is now known as yfried|afk20:21
*** Longgeek has quit IRC20:23
*** jorgem has joined #openstack-neutron20:23
*** rpothier has quit IRC20:25
*** Marga_ has joined #openstack-neutron20:35
*** jorgem has quit IRC20:37
*** Marga_ has quit IRC20:37
*** jorgem has joined #openstack-neutron20:38
*** Marga_ has joined #openstack-neutron20:38
*** Sukhdev has quit IRC20:38
*** vmtrooper has joined #openstack-neutron20:39
*** vmtrooper has quit IRC20:44
armaxihrachyshka: question20:46
ihrachyshkaarmax, shoot20:47
armaxihrachyshka: do we know of the uptake of the change for the vendor repos?20:47
armaxihrachyshka: i.e. have people started moving away from common/log to oslo log, as we preemptively done for the aas repos?20:48
*** carl_baldwin has joined #openstack-neutron20:48
ihrachyshkaarmax, some of them already did, like ryu or brocade. I expect others to follow once they actually notice breakage.20:48
ihrachyshkaarmax, the complecity of the fix is low - import changes.20:49
armaxihrachyshka: ok20:49
*** VijayB_ has quit IRC20:49
*** chandankumar has quit IRC20:49
ihrachyshkain very rear case - an import path change, or similar. but I would expect no vendor libs will really need even that20:49
armaxihrachyshka: let’s make sure we bring that up again during hte next irc meeting20:49
armaxihrachyshka: we need to evangelize that wiki link of yours relentelessy :)20:50
ihrachyshkaok.20:50
*** thedodd has quit IRC20:50
ihrachyshkaarmax, I should have also reach out their 3party team meeting on Mon to communicate the issue. though I realized that late in Tue.20:50
armaxihrachyshka: it’s okay20:51
armaxihrachyshka: we’ll get more proactive next time20:51
ihrachyshkayeah, but next time I will try to be more vocal :)20:51
HenryGarmax: ihrachyshka: working on a patch for ours20:52
*** nlahouti has joined #openstack-neutron20:52
*** cference has quit IRC20:52
armaxHenryG: great,20:52
armaxihrachyshka: let me go over the oslo patch and I’ll nudge it in20:52
ihrachyshkaarmax, thanks sir :)20:53
*** thedodd has joined #openstack-neutron20:53
armaxihrachyshka: is there a defined schedule as to when these changes are best brought in?20:53
armaxihrachyshka: or is it to whim of the oslo releases?20:53
openstackgerritvishwanath jayaraman proposed openstack/neutron-fwaas: Refactoring related to 'FWaaS insertion Model on Routers'  https://review.openstack.org/16188120:54
HenryGarmax: you will need to upgrade your glasses prescription when you are done looking at the oslo.log patch :)20:54
*** thedodd has quit IRC20:54
*** marun has joined #openstack-neutron20:54
ihrachyshkaarmax, well, neutron team decides what's in each cycle, we do it on our own schedule.20:54
armaxHenryG: I change a pair a day20:54
dhellmannarmax: it's up to the projects, but we like to have adoption of new libs finished by the end of the cycle after we release the lib20:54
dhellmannarmax: what ihrachyshka said :-)20:54
*** yfried|afk is now known as yfried20:55
armaxok, so technically we can line them all up and do them, for instance during the RC window?20:55
ihrachyshkaarmax, though I would vote for the more adoptions the better (except those we are scared e.g. due to position in the cycle)20:55
*** leenheer has joined #openstack-neutron20:55
ihrachyshkaarmax, I wouldn't do that that late in the cycle. there are glitches and bugs that may be introduced during adoption.20:56
armaxbut then things will be fairly quite and fixes would be done more easily?20:56
ihrachyshkaarmax, I would say, kilo-3 mile should be the last stone to consider any new merges. we also have dep freeze that is kinda related.20:57
*** jprovazn has quit IRC20:57
ihrachyshkabecause each adoption means a new dep20:57
armaxihrachyshka: ok, fair point20:57
openstackgerritPradeep Kilambi proposed openstack/neutron: Expose Rest Api access to mtu attributes  https://review.openstack.org/15492120:57
openstackgerritPradeep Kilambi proposed openstack/neutron: Add MTU selection to ML2  https://review.openstack.org/15631820:57
openstackgerritPradeep Kilambi proposed openstack/neutron: Advertise mtu over dhcp  https://review.openstack.org/15587420:57
openstackgerritPradeep Kilambi proposed openstack/neutron: Add MTU selection & advertisement settings to Neutron config  https://review.openstack.org/15373320:57
armaxihrachyshka: so, still close to the end20:57
ihrachyshka+ doing it at rc time is a bit uncomfortable for those who package stuff I guess (too many work at the end of the cycle)20:58
dhellmannarmax, ihrachyshka: you wouldn't want to do them during the RC window because you shouldn't be adding dependencies at that point20:58
dhellmannthe feature freeze date is also a requirements freeze date20:58
*** raildo has quit IRC20:58
ihrachyshkadhellmann, right, dep freeze20:58
*** ajo has quit IRC20:58
*** aranjan has quit IRC20:59
*** dboik_ has joined #openstack-neutron20:59
claudiubarmax: hello20:59
ihrachyshkaclaudiub, hey man, I had a question to you!20:59
*** sbfox has quit IRC20:59
claudiubihrachyshka: hello. :)20:59
*** ajmiller_ has joined #openstack-neutron21:00
armaxdhellmann, ihrachyshka: makes sense, close to the third milestone it is21:00
armaxclaudiub: hello21:00
ihrachyshkaclaudiub, in RDO, we traditionally packaged hyperv agent in a separate package. but as I understand it, there is no real reason to ship it at all, since RDO is not meant to run on Windows :) so are we safe to kill the agent during build process?21:00
dhellmannarmax: well, *by* the 3rd milestone. You could also do it earlier. :-)21:00
*** mrsmith has quit IRC21:01
*** VijayB_ has joined #openstack-neutron21:01
ihrachyshkaarmax, I don't actually think it's easier to introduce patches like that near 3rd milestone. it's exactly the time you get lots of reviews and updates and fixes, because everyone rushes to push his stuff in the tree that cycle.21:01
claudiubarmax: so, about the decomposition... well, you were saying that it should be moved to neutron/plugins/hyperv... well, I'd argue that it is more of an agent, since there is nothing else that can bind the neutron ports in hyper-v.. the reason it was in neutron/plugins (other than the fact that everyone was there) was because it had a HyperVNeutronPlugin, which we removed in favor of Ml2Plugin21:02
*** dboik has quit IRC21:02
kevinbenton_rossella_s: hey21:02
*** ronis has quit IRC21:02
*** leenheer has quit IRC21:02
claudiubarmax: plus, having some naming issues, which I cannot solve. :D21:02
*** natarajk1 has joined #openstack-neutron21:03
armaxdhellmann, ihrachyshka: I was only thinking whether it would be beneficial if we were more specific about the times these changes are introduced21:03
*** harlowja_ has quit IRC21:03
armaxclaudiub: I am not sure I follow21:03
*** ajmiller has quit IRC21:03
claudiubihrachyshka: yeah, the hyper-v agent only runs on windows and has windows specific packages that have nothing to do with linux (wmi, pywin32, win32com)21:03
armaxclaudiub: let’s take the discussion on the review page itself21:03
*** natarajk has quit IRC21:04
claudiubihrachyshka: just one thing though21:04
*** harlowja has joined #openstack-neutron21:04
claudiubihrachyshka: if there are RDO users that will use Hyper-V nodes, they will probably use Ml2Plugin to bind them, with hyperv mechanism.. for that to work, networking_hyperv package will be needed21:04
*** moshele has quit IRC21:05
claudiubihrachyshka: since the mech driver will be imported from there21:05
*** seizadi has quit IRC21:05
claudiubihrachyshka: other than that, nothing else is needed. :)21:05
claudiubarmax: Ok.21:05
armaxclaudiub: review # again?21:05
armax:)21:05
claudiubhttps://review.openstack.org/#/c/153259/21:05
ihrachyshkaarmax, I agree I should be more specific about when patches are to be published for review and expected to be merged21:06
ihrachyshkaclaudiub, right. we don't kill ml2 driver. so probably we'll kill agent, but keep the openstack-neutron-hyperv package to pull in the needed dependency (if we get to packaging it)21:06
armaxihrachyshka: yeah, maybe we can announce when they are about to be introduced so that we can set expectations21:06
armaxihrachyshka: not it’s a small thing really21:07
ihrachyshkaarmax, I announced expected adoptions this cycle, but milestone wise, not specific dates. I'll consider being more specific, sounds like a good thing for everyone21:07
*** padkrish has joined #openstack-neutron21:08
armaxihrachyshka: ok, I am pretty sure you did and I did not pay attention ;)21:08
*** yfried is now known as yfried|afk21:08
*** vivek-ebay has quit IRC21:09
*** leenheer has joined #openstack-neutron21:12
*** blogan_ is now known as blogan21:12
openstackgerritCarl Baldwin proposed openstack/neutron: Move external port processing to router classes  https://review.openstack.org/15849521:13
*** thedodd has joined #openstack-neutron21:14
armaxclaudiub: ping21:15
claudiubarmax: sorry, was posting on the review. :) https://review.openstack.org/#/c/153259/21:16
claudiubi hope I explained myself a little bit better. :)21:17
*** mlavalle has quit IRC21:17
armaxclaudiub: ok21:17
* armax reading21:17
*** yfried|afk is now known as yfried21:17
claudiubarmax: PS: forgot to mention, the hyper-v agent is an L2 agent. :)21:19
openstackgerritTimothy Swanson proposed openstack/neutron: Add MTU selection & advertisement settings to Neutron config  https://review.openstack.org/15373321:19
armaxclaudiub: ok I still don’t understand your argument, I’ll try to be more specific on the review as to what I mean21:19
claudiubarmax: I understand what you mean. :)21:20
*** Marga_ has quit IRC21:20
claudiubarmax: but honestly, neutron.plugins.hyperv.agent.agent.HyperVNeutronAgent is such an ugly name... :D21:20
*** tiswanso has quit IRC21:21
claudiubarmax: while neutron.agent.hyperv.HyperVNeutronAgent would be much better...21:21
*** aranjan has joined #openstack-neutron21:21
*** ajo has joined #openstack-neutron21:21
armaxclaudiub: I am afraid your argument doens’t fly21:22
armaxclaudiub: we’re not writing poems or novels21:22
armaxclaudiub: it’s a place for a class is not appropriate, I don’t think we should be conditionated by the fully qualified name21:23
*** leenheer has quit IRC21:23
*** amuller has joined #openstack-neutron21:24
*** Marga_ has joined #openstack-neutron21:24
*** yfauser has joined #openstack-neutron21:24
*** shwetaap1 has quit IRC21:24
*** yfauser has left #openstack-neutron21:25
*** tim_o has joined #openstack-neutron21:26
*** shwetaap has joined #openstack-neutron21:27
openstackgerritPhillip Toohill proposed openstack/python-neutronclient: Updating lbaas cli for TLS  https://review.openstack.org/16140421:27
tim_ois mestery or emagma here?21:27
salv-orlandoarmax: so won't you take my patch for refactoring code in alternate rhyme?21:28
armaxsalv-orlando: um probably not21:28
*** aranjan_ has joined #openstack-neutron21:28
*** aranjan has quit IRC21:28
matrohonarmax, dougwig : hi21:29
*** tgohad has quit IRC21:29
*** jorgem has quit IRC21:29
tim_ojust trying to get ahold of someone to look at the CI log file21:30
salv-orlandoarmax: what about refactoring methods into sonnets?21:30
armaxsalv-orlando: all I can accept is a limerick21:30
*** kevinbenton_ is now known as kevinbenton21:30
*** jorgem has joined #openstack-neutron21:31
matrohonarmax, dougwig : as you validated https://review.openstack.org/#/c/148318/, could you please have an eye on : https://review.openstack.org/#/c/163478/ ?21:31
*** SridharRamaswamy has joined #openstack-neutron21:31
openstackgerritAbishek Subramanian proposed openstack/neutron: Support IPv6 Router  https://review.openstack.org/15628321:32
*** sridhar_ram has quit IRC21:32
*** sripriya has quit IRC21:33
*** sripriya has joined #openstack-neutron21:33
dougwigmatrohon: commented21:34
matrohondougwig : thanks21:34
matrohondouwig : I was betting that somebody would ask for UT :)21:35
matrohondougwig ^21:36
dougwiggood bet.  :)21:36
openstackgerritAbishek Subramanian proposed openstack/neutron: Support IPv6 Router  https://review.openstack.org/15628321:36
*** leenheer has joined #openstack-neutron21:37
openstackgerritEd Bak proposed openstack/neutron: Break calls to get_devices_details_list into subsets  https://review.openstack.org/16359421:38
matrohondougwig : however do you consider that extending the client the way I described in the bug report is the good way? personally, I find it very useful!21:38
*** Marga_ has quit IRC21:38
*** iwi has left #openstack-neutron21:39
openstackgerritAbishek Subramanian proposed openstack/neutron: Support IPv6 Router  https://review.openstack.org/15628321:39
*** Marga_ has joined #openstack-neutron21:39
dougwigmatrohon: seemed reasonable to me21:39
matrohondougwig : fine I'll submit UT asap, thanks21:40
openstackgerritJuergen Brendel proposed openstack/neutron: ARP spoofing patch: Low level ebtables integration  https://review.openstack.org/14113021:40
openstackgerritJuergen Brendel proposed openstack/neutron: ARP spoofing patch: Low level ebtables integration  https://review.openstack.org/14113021:43
*** dboik_ has quit IRC21:43
*** dboik has joined #openstack-neutron21:44
*** Sukhdev has joined #openstack-neutron21:44
*** ihrachyshka has quit IRC21:44
Sukhdevmestery: are you there?21:45
*** sbfox has joined #openstack-neutron21:46
*** amotoki has quit IRC21:49
*** oda-g has joined #openstack-neutron21:49
armaxHenryG: ping21:51
*** prad has quit IRC21:52
dougwigsalv-orlando: i want to see a gerrit review of sonnets.21:52
*** kbringard has quit IRC21:53
*** annegentle has quit IRC21:56
*** moha_hunt has joined #openstack-neutron21:56
*** pritesh has quit IRC22:01
*** SridharRamaswamy has quit IRC22:01
*** melwitt_ has joined #openstack-neutron22:01
*** melwitt_ has quit IRC22:02
*** harlowja has quit IRC22:02
*** melwitt has quit IRC22:03
*** SridharRamaswamy has joined #openstack-neutron22:04
*** padkrish has quit IRC22:04
*** apuimedo has joined #openstack-neutron22:04
*** padkrish has joined #openstack-neutron22:04
*** harlowja has joined #openstack-neutron22:04
*** marrusl has quit IRC22:08
*** Longgeek has joined #openstack-neutron22:08
*** marrusl has joined #openstack-neutron22:08
*** vivek-ebay has joined #openstack-neutron22:10
openstackgerritKevin Benton proposed openstack/neutron: Update contribute.rst with Big Switch decomp  https://review.openstack.org/16399622:10
openstackgerritClaudiu Belu proposed openstack/neutron: Hyper-V Agent decomposition  https://review.openstack.org/15325922:11
*** tsg has joined #openstack-neutron22:12
salv-orlandodougwig: I made an attempt a while ago.... not really a sonnet though: https://review.openstack.org/#/c/69579/1//COMMIT_MSG22:12
openstackgerritAbhishek Raut proposed openstack/neutron: Cisco Nexus1000V ML2 Mechanism Driver  https://review.openstack.org/15724322:12
*** aranjan_ has quit IRC22:12
*** vivek-ebay has quit IRC22:14
openstackgerritKevin Benton proposed openstack/neutron: Update contribute.rst with Big Switch decomp  https://review.openstack.org/16399622:15
claudiubarmax: uploaded a new patchset22:15
dougwigsalv-orlando: ha, love it.22:15
*** aranjan has joined #openstack-neutron22:17
amullerarmax: And you say I have too much time...22:17
*** shivharis has joined #openstack-neutron22:17
*** carl_baldwin has quit IRC22:18
openstackgerritNachi Ueno proposed openstack/neutron: OpenContrail plugin code split  https://review.openstack.org/16321822:18
apuimedoamuller: no weekend for you?22:18
amullerapuimedo: It's 6:20pm here right now :)22:19
apuimedoah yes... I keep forgetting that you are in the other side22:19
apuimedoso you do monday-friday now?22:19
amulleryeah22:20
*** aranjan has quit IRC22:20
amullerstill kind of confused when I see calendars starting on a Monday22:20
*** aranjan has joined #openstack-neutron22:20
apuimedoheh, for me it's the other way around, a lot of software starting the week no Sunday22:21
*** cference has joined #openstack-neutron22:21
amullerapuimedo: How's it going in Barcelona?22:22
*** yfried is now known as yfried|afk22:22
apuimedoamuller: going there too much for my taste22:22
*** luqas has joined #openstack-neutron22:22
apuimedoI enjoy more working from my town22:22
*** carl_baldwin has joined #openstack-neutron22:23
amullerapuimedo: Where's that?22:24
apuimedo1h commuting by bus22:25
apuimedoOlesa de Montserrat22:25
*** logan2 has quit IRC22:26
openstackgerritMerged openstack/neutron: Remove dead code  https://review.openstack.org/16358622:28
*** vmtrooper has joined #openstack-neutron22:28
armaxamuller: what did I do that doesn’t please you?22:28
amullerarmax: You claim I have too much free time, your team mate is writing sonnets in commit messages :)22:29
armaxamuller: true, I shouldn’t single you out like I did22:29
amullerheh, fair enough22:29
armaxamuller: but I blame salv-orlando all the time too22:31
*** Longgeek has quit IRC22:32
*** vmtrooper has quit IRC22:33
*** SridharRamaswamy has quit IRC22:35
*** tiswanso has joined #openstack-neutron22:35
*** csoukup has quit IRC22:35
*** melwitt has joined #openstack-neutron22:36
*** Marga_ has quit IRC22:36
*** SridharRamaswamy has joined #openstack-neutron22:38
*** thedodd has quit IRC22:39
openstackgerritpritesh proposed openstack/neutron: Adding VLAN Transparency support for ML2 along with REST API changes  https://review.openstack.org/15842022:40
*** tiswanso has quit IRC22:40
salv-orlandoamuller: what's your job title?22:41
amullercode monkey probably22:41
*** aranjan has quit IRC22:41
amullerwhat's yours?22:41
salv-orlandothat does not entitle to you to waste time.22:41
salv-orlandocheck it out for yourself: https://uk.linkedin.com/in/taturiello22:41
*** yfried|afk is now known as yfried22:42
* b3nt_pin apologizes for eavesdropping while rofl22:42
amullersalv-orlando: Yeah definitely going to do something about my title next time I go to print cards22:42
amullerWell played22:43
*** aranjan has joined #openstack-neutron22:44
*** dims_ has joined #openstack-neutron22:46
*** dims has quit IRC22:46
*** SridharRamaswamy has quit IRC22:47
*** SridharRamaswamy has joined #openstack-neutron22:47
*** aranjan has quit IRC22:47
*** aranjan_ has joined #openstack-neutron22:47
*** dboik has quit IRC22:48
*** dboik has joined #openstack-neutron22:48
*** r-daneel has quit IRC22:48
*** amotoki has joined #openstack-neutron22:49
*** ajo has quit IRC22:51
*** carl_baldwin has quit IRC22:51
*** aranjan_ has quit IRC22:52
*** yfried is now known as yfried|afk22:52
*** achanda has quit IRC22:52
openstackgerritAssaf Muller proposed openstack/neutron: Introduce RPC tests  https://review.openstack.org/16281122:53
amullerarmax: ^22:53
*** achanda has joined #openstack-neutron22:53
amullerChanged to addCleanup and made  some other simplifications22:53
kevinbentonsalv-orlando: here is the easy change from our discussion yesterday https://review.openstack.org/#/c/163672/22:53
*** carl_baldwin has joined #openstack-neutron22:54
*** amotoki has quit IRC22:55
*** sridhar_ram has joined #openstack-neutron22:55
*** baoli has quit IRC22:55
*** thomasem has quit IRC22:55
*** achanda_ has joined #openstack-neutron22:56
*** annegentle has joined #openstack-neutron22:56
*** achanda_ has quit IRC22:56
*** achanda_ has joined #openstack-neutron22:56
*** natarajk1 has left #openstack-neutron22:56
*** sridhar_ram has left #openstack-neutron22:57
*** achanda has quit IRC22:57
*** jorgem has quit IRC22:58
*** melwitt has quit IRC22:59
*** dims_ has quit IRC23:00
*** dims has joined #openstack-neutron23:00
*** apuimedo has quit IRC23:01
openstackgerritArmando Migliaccio proposed openstack/neutron: Start metadata agent without trying to connect db  https://review.openstack.org/15936523:01
*** annegentle has quit IRC23:02
openstackgerritShiv Haris proposed openstack/neutron: Brocade vendor code decomposition from neutron repo.  https://review.openstack.org/15238323:03
*** apuimedo has joined #openstack-neutron23:03
*** ajmiller_ has quit IRC23:04
*** melwitt has joined #openstack-neutron23:05
*** dims has quit IRC23:05
*** dboik has quit IRC23:05
*** shwetaap has quit IRC23:06
*** SridharRamaswamy has quit IRC23:08
*** achanda has joined #openstack-neutron23:08
*** SridharRamaswamy has joined #openstack-neutron23:08
*** hichihara has joined #openstack-neutron23:09
*** Marga_ has joined #openstack-neutron23:09
*** SridharRamaswamy has quit IRC23:09
*** matrohon has quit IRC23:10
*** Marga_ has quit IRC23:10
*** aranjan has joined #openstack-neutron23:10
*** Marga_ has joined #openstack-neutron23:10
*** achanda_ has quit IRC23:11
*** igordcard_ has quit IRC23:12
HenryGarmax: pong23:13
armaxHenryG: yo23:13
armaxHenryG: line https://github.com/openstack/neutron/blob/master/setup.cfg#L9123:14
armaxHenryG: not sure if we should have entry points to external code like this23:15
armaxHenryG: what do you reckon?23:15
HenryGarmax: I see, yes, there should be a shim in tree23:15
armaxor no entry point at all23:16
armaxthat could be added to the stackforge repo23:16
HenryGarmax: True. I can poke the owners to do one or the other.23:16
shivharisarmax: hi, need to clarify something. waiting for you to get free from the other conversation ...23:16
armaxshivharis: shoot23:16
shivharisarmax: refering to your comment in the patch-set, the 2 products are different may have different needs for invokation23:17
*** yamamoto has joined #openstack-neutron23:18
shivharisone depends on ml2 other does not23:18
armaxshivharis: that table is not the place to document vendor specific technologies23:18
armaxshivharis: I appreciate the differrence23:18
armaxshivharis: but we don’t need to capture those there23:18
armaxshivharis: imagine if all plugins start doing the same23:18
*** Marga_ has quit IRC23:18
armaxHenryG’s employer especially23:18
armaxthey have tons of different flavors23:18
*** Marga_ has joined #openstack-neutron23:19
armaxshivharis: can you imagine if we start documenting those there? The table is going to bloat out of proportion23:19
*** crc32 has quit IRC23:20
armaxshivharis: we should simply capture the flavor of plugin/driver being available in the repo23:20
armaxso that we know what to look for…as for the specific, user documentation and other resources are more appropriate tools to drive a user to know what options are available from a Brocade OpenStack solution23:21
shivharisi am at a loss, need clarification: should the other patch-set update the vyatta column?23:21
armaxshivharis: where?23:22
shivharisor should this distinction not be there at all?23:22
armaxyou mean change 15906?23:22
armaxsorry 159056?23:22
shivharisthere are 2 product and 2 service plugins23:22
armaxI get it23:22
shivharisyes 15905623:22
armaxall I am saying, wrt that table23:22
armaxwe simply do not need to care23:23
shivharisok, i'll remove the distinction23:23
armaxif you want to specify the various flavors of l3 plugins Brocade has23:23
armaxit’s best to put taht in the README23:23
armaxall we need to know from that table is that the L3 code that was once in the tree, is now avaialable out of tree23:24
armaxit’s that simple23:24
shivharissounds like a plan. will send update shortly. We will mainting this in the stackforge README.dm23:24
shivhariss/dm/md/23:24
armaxI think that’s best, thank you23:24
armaxshivharis: bear in mind the other comment on the entry point23:24
armaxif you plan on not having it, for consistency we’d want to remove the one from review 15905623:25
armaxso perhaps sync up with your colleague to get that sorted23:25
shivharisthat worries me23:25
shivharisthey may have a different requirement for invokation.. we depend on ml2 they do not23:25
*** natarajk1 has joined #openstack-neutron23:26
*** padkrish has quit IRC23:26
armaxshivharis: do you understand what entry points are?23:27
*** padkrish has joined #openstack-neutron23:27
*** julien_ZTE has quit IRC23:27
armaxshivharis: I am not sure I fully grasp the implications of your statement23:28
shivharisarmax: yes23:28
armaxok, so maybe I don't23:28
*** julien_ZTE has joined #openstack-neutron23:28
armaxwhat has the invocation have to do with anything?23:28
shivharisi am using that term instead of entry point23:29
*** yamamoto has quit IRC23:29
armaxshivharis: ok, leave it as you wish23:29
armaxwe’ll sort it out on the other patch23:30
shivharisarmax: ok23:31
sripriyaarmax:hi23:31
armaxas much as I am trying to explain myself23:31
armaxshivharis: it seems like I am unable to get my message across23:32
armaxsripriya: hi23:32
*** achanda has quit IRC23:32
*** padkrish has quit IRC23:32
sripriyawe are just following the published guidelines that was referred in 15905623:32
shivharisok, we have both plugins represented, shivharis and sripriya23:32
*** achanda has joined #openstack-neutron23:32
*** sbfox has quit IRC23:32
*** julien_ZTE has quit IRC23:33
armaxsripriya, shivharis: yes, however I see two different approaches for the same vendor; albeit they are both acceptable IMO23:33
armaxi’d rather see one or the other in the context of the same vendor23:33
armaxnow, shivharis’s l3 plugin is removed altogether, sripriya’s l3 plugin has a shim23:33
armaxto me this is confusing23:34
armaxas in the former case, obviosuly there’s no need of the entry point, in the latter there is23:34
armaxyou with me?23:35
shivharisarmax: i appreciate your point but the implementation/deployments are dissimilar23:35
sripriyaarmax: vyatta only has a new L3 plugin so we are doing a shim based on the guidelines23:35
*** absubram has quit IRC23:35
*** melwitt has quit IRC23:35
armaxshivharis: how so?23:36
shivharisarmax: whereas the vdx plugin has an ml2 component and an l3 plugin as well23:36
armaxshivharis: an l3 service plugin is an l3 service plugin23:36
armaxshivharis: ok, fine, keep it inconsistent23:36
*** mlavalle has joined #openstack-neutron23:36
shivhariscorrect, however devstack and other apps creating neutron.conf use a mechanism for this specific purpose23:37
armaxshivharis: if it was me, I wouldn’t do it in two separate ways, but I can’t force you23:37
armaxshivharis: be warned that if you came later and want to add the entry point, you’re sure gonna get shot down :)23:37
* armax kids23:38
shivharisarmax: :-), that exactly why I like your approach23:38
armaxshivharis: I think you’ll change your mind when I slap a -2 on your patches23:38
sripriyaarmax: just an FYI, we have few more new Brocade L3 plugins that are following vyatta approach: https://review.openstack.org/#/c/161489/7/setup.cfg23:39
shivharisarmax: you really want this changed?23:39
*** aepifanov has quit IRC23:39
*** sridhar_ram has joined #openstack-neutron23:39
armaxshivharis, sripriya the more the merrier, that makes me question why shivharis’ approach is an outlier23:40
armaxanyhow my energy in discussing this is ran out23:40
armaxthanks for listening for now :)23:40
*** david-lyle is now known as david-lyle_afk23:41
shivharisarmax: what do you want me to do?23:41
armaxmy suggestion would be to sort yourselves out so that the brocade’s contributions are cleaner23:41
armaxat this point, I am happy to let this go23:41
armaxI need to drink a gallon of water after all this talking :)23:42
shivharisok23:42
shivharisi will let this stand, in the background think about it23:43
shivharishowever, i would also want to chat with you why i am using this approach23:43
shivharis(at a later time)23:43
armaxshivharis: ok23:44
armaxsounds fair23:44
*** Sukhdev has quit IRC23:44
shivharisfair, thanks23:44
openstackgerritShiv Haris proposed openstack/neutron: Brocade vendor code decomposition from neutron repo.  https://review.openstack.org/15238323:47
*** sripriya has quit IRC23:50
*** dboik has joined #openstack-neutron23:52
*** sripriya has joined #openstack-neutron23:52
*** yamamoto has joined #openstack-neutron23:52
*** dboik_ has joined #openstack-neutron23:52
*** sripriya has quit IRC23:52
*** padkrish has joined #openstack-neutron23:53
*** shwetaap has joined #openstack-neutron23:54
*** reed has quit IRC23:55
*** ChuckC has quit IRC23:56
*** dboik has quit IRC23:56
*** annegentle has joined #openstack-neutron23:58
*** dannywil_ has quit IRC23:59

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