Friday, 2015-03-13

*** yamamoto has quit IRC00:00
*** leenheer has quit IRC00:00
*** annegentle has quit IRC00:02
*** markvoelker has quit IRC00:03
*** achanda has quit IRC00:03
*** Padmini has quit IRC00:04
*** padkrish has quit IRC00:04
*** padkrish has joined #openstack-neutron00:05
*** tim_o has quit IRC00:05
openstackgerritAssaf Muller proposed openstack/neutron: Move Unix domain socket helpers to a common place  https://review.openstack.org/16403200:06
*** vivek-ebay has joined #openstack-neutron00:07
*** Swami has quit IRC00:08
*** markvoelker has joined #openstack-neutron00:08
openstackgerritAssaf Muller proposed openstack/neutron: Replace keepalived notifier bash script with Python ip monitor  https://review.openstack.org/12538400:08
amullercarl_baldwin: marun: I split up the gigantic patch, moved some trivial stuff off to a preceding patch00:08
carl_baldwinamuller: ack.  Sometimes that is what you've got to do.00:09
*** padkrish has quit IRC00:09
amullercarl_baldwin: it only took like 20 minutes, really not bad00:10
*** vivek-eb_ has joined #openstack-neutron00:12
*** vivek-ebay has quit IRC00:12
*** xgerman has quit IRC00:12
*** reed has joined #openstack-neutron00:14
*** Marga_ has quit IRC00:15
*** amuller has quit IRC00:15
*** padkrish has joined #openstack-neutron00:16
*** vmtrooper has joined #openstack-neutron00:17
*** VijayB_ has quit IRC00:17
*** Longgeek has joined #openstack-neutron00:18
*** moha_hunt has quit IRC00:18
*** shwetaap1 has joined #openstack-neutron00:19
*** claudiub has quit IRC00:20
hichiharacarl_baldwin: Could you review a patch? I guess the bug is critical issue to Neutron L3. https://review.openstack.org/#/c/161947/00:21
*** vmtrooper has quit IRC00:22
carl_baldwinhichihara:  I will have a look.00:22
*** shwetaap has quit IRC00:23
hichiharacarl_baldwin: Thank you! :-)00:24
*** watanabe_isao has joined #openstack-neutron00:24
watanabe_isaoGood morning to all.:-)00:24
*** sripriya has joined #openstack-neutron00:25
*** jp_at_hp has joined #openstack-neutron00:26
*** Marga_ has joined #openstack-neutron00:27
*** killer_prince has quit IRC00:28
*** dims has joined #openstack-neutron00:32
*** Longgeek has quit IRC00:32
*** VijayB_ has joined #openstack-neutron00:34
*** killer_prince has joined #openstack-neutron00:35
*** killer_prince is now known as lazy_prince00:35
*** SumitNaiksatam has quit IRC00:39
*** tflynn has quit IRC00:40
*** yamamoto has joined #openstack-neutron00:41
*** pal has joined #openstack-neutron00:42
*** marun has quit IRC00:42
*** jaypipes has quit IRC00:45
*** radez is now known as radez_g0n300:51
*** aranjan has quit IRC00:51
*** topshare has joined #openstack-neutron00:52
*** tiswanso has joined #openstack-neutron00:52
*** harkirat has joined #openstack-neutron00:53
*** mwagner_lap has quit IRC00:53
*** tiswanso has quit IRC00:53
*** tiswanso has joined #openstack-neutron00:54
*** liusheng has quit IRC00:56
*** annegentle has joined #openstack-neutron00:58
*** kevinbenton has quit IRC00:59
openstackgerritAbhishek Raut proposed openstack/neutron: Cisco Nexus1000V ML2 Mechanism Driver  https://review.openstack.org/15724301:00
*** evgenyf has joined #openstack-neutron01:01
HenryGarmax: ping01:02
armaxHenryG: pong01:02
HenryGarmax: Question about entry points ...01:03
*** ijw has joined #openstack-neutron01:03
*** annegentle has quit IRC01:03
armaxHenryG: shoot01:03
HenryGIf a package is added as a dependency, do all its entry points get added to the "global" set of entry points?01:04
armaxgood question, I am not sure01:04
HenryGs/added/installed/01:05
armaxI would be saying no01:05
*** ijw has quit IRC01:05
*** gildub has joined #openstack-neutron01:05
armaxhow would you ensure uniqueness?01:05
*** ijw has joined #openstack-neutron01:05
HenryGRight, that's what I thought. So how to "bring in" or access the entry points from a package?01:06
armaxHenryG: why would you want to do that?01:07
*** _cjones_ has quit IRC01:07
*** baoli has joined #openstack-neutron01:07
HenryGarmax: Due to lack of experience I am probably thinking about it wrong, but this is what I see ...01:09
*** Kennan has joined #openstack-neutron01:09
*** _cjones_ has joined #openstack-neutron01:09
*** bandarji has quit IRC01:09
*** SumitNaiksatam has joined #openstack-neutron01:09
HenryGBefore the split, a console script "vendor-foo" was available when neutron was installed, due to it being in setup.cfg01:10
HenryGNow if the entry is removed from neutron's setup.cfg, but put in vendor's setup.cfg, how will the script get installed?01:11
HenryG(And a side-question, if it was named "neutron-vendor-foo" before the split, does it now have to drop the "neutron-" prefix"?)01:13
*** rwsu is now known as rwsu-afk01:13
HenryGI guess this is all moot if a shim is kept in neutron?01:13
armaxthe console script is different01:14
armaxit is indeed made available when the package is installed01:15
HenryGorly? So how are conflicts avoided?01:15
armaxyou can have multiple console scripts with the same name but pointing to different locations01:17
armaxmaybe the last installed win, I’d need to checkj01:17
armaxI am only speculating at this point01:17
HenryGsounds like fun01:17
*** _cjones_ has quit IRC01:18
HenryGI don't see it being an issue though01:18
*** nlahouti has quit IRC01:18
HenryGarmax: So the vendor entrypoint can be removed from neutron?01:18
armaxI think so01:19
HenryGarmax: thanks, much appreciated01:19
armaxnp01:19
*** apuimedo has quit IRC01:20
openstackgerritJoe Jiang proposed openstack/python-neutronclient: PEP8(version: 1.6.2) test fails in root directory of neutronclient.  https://review.openstack.org/16366301:21
yamamotoHenryG: fyi ofagent works that way https://github.com/stackforge/networking-ofagent/blob/master/setup.cfg#L3901:23
*** enikanorov__ has joined #openstack-neutron01:24
*** enikanorov_ has quit IRC01:26
*** shivharis has quit IRC01:28
*** shivharis has joined #openstack-neutron01:28
openstackgerritMerged openstack/neutron: Migrate to oslo.log  https://review.openstack.org/15963801:28
*** jp_at_hp has quit IRC01:29
openstackgerritCarl Baldwin proposed openstack/neutron: Introduce External IPAM Interface  https://review.openstack.org/13433901:29
*** cference has quit IRC01:30
*** evgenyf has quit IRC01:30
*** stanzgy has joined #openstack-neutron01:30
*** stanzgy_ has joined #openstack-neutron01:31
openstackgerritMerged openstack/neutron: Don't start transaction during floating IP delete  https://review.openstack.org/16003601:31
openstackgerritshihanzhang proposed openstack/neutron: Fix incorrect comments  https://review.openstack.org/16404701:31
*** shivharis has quit IRC01:32
*** VijayB_ has quit IRC01:32
*** yamahata has quit IRC01:33
openstackgerritMerged openstack/neutron: ofagent: Update after networking-ofagent release  https://review.openstack.org/16327301:34
*** sridhar_ram has quit IRC01:34
*** VijayB has joined #openstack-neutron01:35
*** natarajk1 has quit IRC01:39
*** amotoki has joined #openstack-neutron01:39
*** trad511 has quit IRC01:40
*** ChuckC has joined #openstack-neutron01:42
*** baoli has quit IRC01:42
*** baoli has joined #openstack-neutron01:43
*** amotoki has quit IRC01:44
openstackgerritItsuro Oda proposed openstack/neutron: Keep existing services available on agents with admin_state_up False  https://review.openstack.org/14703201:45
*** popw has joined #openstack-neutron01:45
*** ChuckC has quit IRC01:46
*** ChuckC has joined #openstack-neutron01:47
*** harkirat has quit IRC01:50
HenryGarmax: So our repo is now broken with the oslo.log patch in neutron01:52
armaxya01:52
openstackgerritYAMAMOTO Takashi proposed openstack/neutron: OVS-agent: Introduce Ryu based OpenFlow driver  https://review.openstack.org/15394601:52
openstackgerritYAMAMOTO Takashi proposed openstack/neutron: OVS-agent: Separate ovs-ofctl using code as a driver  https://review.openstack.org/16024501:52
*** lazy_prince has quit IRC01:53
HenryGarmax: I am fixing it, but expect torches and pitchforks tomorrow from the angry mob :)01:53
*** Marga_ has quit IRC01:53
armaxwhy?01:53
armaxthat was announced across seven seas01:54
openstackgerritArmando Migliaccio proposed openstack/neutron: Decouple L3 and service plugins during DVR router migration  https://review.openstack.org/15474901:54
HenryGarmax: I am just kidding. Aren't there always some who don't hear announcements?01:54
armaxHenryG: oh, totally01:54
armaxHenryG: I am angry01:55
HenryGarmax: Hopefully not enough to form a mob01:55
*** luqas has quit IRC01:55
armaxthat patch caused me needless merge conflicts01:55
HenryGarmax: Why did you approve it? ;) And how many end-of-cycle mad rushes have you been through and not gotten angry? ;)01:56
armaxHenryG: I was coerced01:57
*** kevinbenton has joined #openstack-neutron01:57
*** tsg has quit IRC01:57
kevinbentoncarl_baldwin: hey, is there anything in the l2 agent that will periodically request port info to protect against lost notificaitons from the server?01:58
*** annegentle has joined #openstack-neutron01:59
carl_baldwinkevinbenton:  I don't think so.02:00
openstackgerritArmando Migliaccio proposed openstack/neutron: Break coupling between ML2 and L3 during delete operation  https://review.openstack.org/15547002:00
openstackgerritArmando Migliaccio proposed openstack/neutron: Break coupling between ML2 and L3 during create/update operations  https://review.openstack.org/15377502:00
carl_baldwinkevinbenton: I haven't been in the L2 agent much lately.02:00
kevinbentoncarl_baldwin: do you know anyone that has? :)02:01
kevinbentoncarl_baldwin: it seems like something that we should have02:01
kevinbentoncarl_baldwin: similar to the l3 agents sync_routers task02:01
*** padkrish has quit IRC02:01
*** padkrish has joined #openstack-neutron02:02
*** Marga_ has joined #openstack-neutron02:03
*** annegentle has quit IRC02:04
*** sbfox has joined #openstack-neutron02:04
*** gongysh has joined #openstack-neutron02:06
*** vmtrooper has joined #openstack-neutron02:06
*** padkrish has quit IRC02:06
*** ivar-lazzaro has quit IRC02:06
*** harlowja is now known as harlowja_away02:07
*** padkrish has joined #openstack-neutron02:09
*** vmtrooper has quit IRC02:11
*** Madkiss has quit IRC02:11
*** Madkiss has joined #openstack-neutron02:12
*** Madkiss has quit IRC02:12
*** Madkiss has joined #openstack-neutron02:12
openstackgerritHong Hui Xiao proposed openstack/neutron: Always run dnsmasq as root  https://review.openstack.org/16156702:13
carl_baldwinkevinbenton: I'm not sure the sync_routers task fits that bill either.  It only syncs if full sync is set which is only on error or restart.02:13
*** killer_prince has joined #openstack-neutron02:14
*** killer_prince is now known as lazy_prince02:14
*** dims has quit IRC02:14
*** VijayB has quit IRC02:16
*** marun has joined #openstack-neutron02:16
*** mlavalle has quit IRC02:17
*** padkrish has quit IRC02:18
*** Longgeek has joined #openstack-neutron02:18
kevinbentoncarl_baldwin: oh okay02:19
kevinbentoncarl_baldwin: i thought there was a periodic full sync02:19
kevinbentonsounds like i'm way off02:19
*** annegentle has joined #openstack-neutron02:19
kevinbentonwe have no protection against lost messages from the server then, right?02:19
*** vivek-eb_ has quit IRC02:19
carl_baldwinkevinbenton:  Well, sync_routers is called periodically.  But, the first thing it does is check if full_sync is True and exits otherwise.02:20
carl_baldwinkevinbenton:  I don't think so.02:20
carl_baldwin... except when the messages are lost on agent restart.02:20
openstackgerritJoe Jiang proposed openstack/python-neutronclient: Set specify IP address when use 'router-gateway-set' subcommand.  https://review.openstack.org/16333202:21
kevinbentoncarl_baldwin: so agent restart is okay because that always triggers a full sync anyway, right?02:21
carl_baldwinkevinbenton:  correct02:21
kevinbentoncarl_baldwin: yeah, i was just thinking of situations where we might have flaky servers02:22
*** marun has quit IRC02:22
*** ijw has quit IRC02:22
carl_baldwinkevinbenton:  If I recall, marun has talked a bit about detecting out of order and missing messages.  My memories are vague.02:23
kevinbentoncarl_baldwin: ok. i do remember him talking about agent consistency in the context of something else as well02:24
kevinbentoncarl_baldwin: i'll get a hold of him at some point02:24
*** annegentle has quit IRC02:25
*** annegentle has joined #openstack-neutron02:26
*** erkules_ has joined #openstack-neutron02:26
openstackgerritDane LeBlanc proposed openstack/neutron: Support Dual-Stack Gateway Ports on Neutron Routers  https://review.openstack.org/14906802:27
*** ijw has joined #openstack-neutron02:28
*** erkules has quit IRC02:29
*** ijw has quit IRC02:29
*** ijw has joined #openstack-neutron02:30
*** baoli has quit IRC02:32
*** baoli has joined #openstack-neutron02:32
*** reed has quit IRC02:33
*** sbfox has quit IRC02:33
*** dims has joined #openstack-neutron02:34
*** Longgeek has quit IRC02:35
openstackgerritSridar Kandaswamy proposed openstack/neutron-fwaas: FWaaS Insertion Model on Routers  https://review.openstack.org/15269702:35
*** pal has quit IRC02:39
*** dims has quit IRC02:39
*** vivek-ebay has joined #openstack-neutron02:41
*** mlavalle has joined #openstack-neutron02:41
openstackgerritKevin Benton proposed openstack/neutron: Reload DHCP interface when its port is updated  https://review.openstack.org/16375802:41
*** devlaps has quit IRC02:43
*** achanda has joined #openstack-neutron02:44
*** annegentle has quit IRC02:46
*** leenheer has joined #openstack-neutron02:47
*** sbalukoff has quit IRC02:47
openstackgerritRob Pothier proposed openstack/neutron: Add MTU selection to ML2  https://review.openstack.org/15631802:48
openstackgerritRob Pothier proposed openstack/neutron: Add MTU selection & advertisement settings to Neutron config  https://review.openstack.org/15373302:48
*** carl_baldwin has quit IRC02:48
openstackgerritTerry Wilson proposed openstack/neutron: Add rootwrap daemon mode support  https://review.openstack.org/15880502:53
openstackgerritAssaf Muller proposed openstack/neutron: Move Unix domain socket helpers to a common place  https://review.openstack.org/16403202:53
openstackgerritKevin Benton proposed openstack/neutron: Revert "Revert "Remove port from ovsdb after its deletion""  https://review.openstack.org/16016002:54
*** baoli has quit IRC02:54
*** amotoki has joined #openstack-neutron02:55
openstackgerritDane LeBlanc proposed openstack/neutron: IPv6 SLAAC subnet create should update ports on net  https://review.openstack.org/16062202:58
*** amotoki has quit IRC03:00
openstackgerritAssaf Muller proposed openstack/neutron: Send notification to controller about HA router state change  https://review.openstack.org/12618803:00
openstackgerritAssaf Muller proposed openstack/neutron: Expose ha_state per router to agent binding via API  https://review.openstack.org/12618903:00
openstackgerritAssaf Muller proposed openstack/neutron: Move Unix domain socket helpers to a common place  https://review.openstack.org/16403203:00
openstackgerritAssaf Muller proposed openstack/neutron: Replace keepalived notifier bash script with Python ip monitor  https://review.openstack.org/12538403:00
openstackgerritAssaf Muller proposed openstack/neutron: Reuse nova batch notifier  https://review.openstack.org/12801303:00
openstackgerritAssaf Muller proposed openstack/neutron: Remove unused L3 HA RPC method  https://review.openstack.org/15537003:00
*** TrevorV has quit IRC03:04
*** blogan has quit IRC03:05
*** pmath has quit IRC03:05
*** ptoohill has quit IRC03:06
*** absubram has joined #openstack-neutron03:09
*** TrevorV has joined #openstack-neutron03:11
*** absubram has quit IRC03:11
*** ptoohill has joined #openstack-neutron03:11
*** blogan has joined #openstack-neutron03:12
*** absubram has joined #openstack-neutron03:12
*** natarajk has joined #openstack-neutron03:15
*** harkirat has joined #openstack-neutron03:15
openstackgerritAssaf Muller proposed openstack/neutron: Refactor retry mechanism used in some DB operations  https://review.openstack.org/14926103:16
openstackgerritAssaf Muller proposed openstack/neutron: Set default of api_workers to number of CPUs  https://review.openstack.org/14049303:16
*** jaypipes has joined #openstack-neutron03:18
openstackgerritDane LeBlanc proposed openstack/neutron: Stop sending gratuitous arp when ip version is 6  https://review.openstack.org/16079903:19
*** ptoohill has quit IRC03:20
*** blogan has quit IRC03:20
*** sbfox has joined #openstack-neutron03:21
*** TrevorV has quit IRC03:22
*** tsg has joined #openstack-neutron03:22
*** TrevorV has joined #openstack-neutron03:23
*** aranjan has joined #openstack-neutron03:23
*** ptoohill has joined #openstack-neutron03:23
*** pmath has joined #openstack-neutron03:23
*** blogan has joined #openstack-neutron03:24
*** ptoohill has quit IRC03:26
*** pmath has quit IRC03:26
*** TrevorV has quit IRC03:26
*** blogan has quit IRC03:27
*** sripriya has quit IRC03:29
*** Longgeek has joined #openstack-neutron03:31
*** lazy_prince is now known as killer_prince03:31
*** leenheer has joined #openstack-neutron03:32
*** tiswanso has quit IRC03:32
*** gongysh has quit IRC03:33
openstackgerritAssaf Muller proposed openstack/neutron: Replace keepalived notifier bash script with Python ip monitor  https://review.openstack.org/12538403:33
*** TrevorV has joined #openstack-neutron03:35
*** ptoohill has joined #openstack-neutron03:35
*** blogan has joined #openstack-neutron03:36
*** ijw has quit IRC03:36
*** diegows has quit IRC03:38
*** zzzeek has quit IRC03:40
*** ajmiller has joined #openstack-neutron03:41
*** Longgeek has quit IRC03:41
*** watanabe_isao has quit IRC03:42
*** pmath has joined #openstack-neutron03:43
*** gongysh has joined #openstack-neutron03:45
*** ijw has joined #openstack-neutron03:51
openstackgerritAbhishek Raut proposed openstack/neutron: Cisco Nexus1000V ML2 Mechanism Driver  https://review.openstack.org/15724303:52
*** Longgeek has joined #openstack-neutron03:52
*** ijw has quit IRC03:53
*** ijw has joined #openstack-neutron03:53
openstackgerritRob Pothier proposed openstack/neutron: Add MTU selection to ML2  https://review.openstack.org/15631803:53
*** harkirat is now known as innosam03:54
*** innosam is now known as harkirat03:54
*** jckasper has quit IRC03:54
*** vmtrooper has joined #openstack-neutron03:55
*** vmtrooper has quit IRC03:59
*** nijaba has quit IRC04:01
*** watanabe_isao has joined #openstack-neutron04:01
*** liusheng has joined #openstack-neutron04:06
*** nijaba has joined #openstack-neutron04:07
*** nijaba has quit IRC04:07
*** nijaba has joined #openstack-neutron04:07
*** vivek-ebay has quit IRC04:10
*** amotoki has joined #openstack-neutron04:11
*** sbalukoff has joined #openstack-neutron04:11
*** yog_ has joined #openstack-neutron04:12
*** amotoki has quit IRC04:15
*** Marga_ has quit IRC04:16
*** pal has joined #openstack-neutron04:16
*** tflynn has joined #openstack-neutron04:19
*** nlahouti has joined #openstack-neutron04:19
*** tflynn has quit IRC04:19
*** tflynn has joined #openstack-neutron04:20
*** yatin has joined #openstack-neutron04:21
*** nlahouti has quit IRC04:21
*** jckasper has joined #openstack-neutron04:22
*** nlahouti has joined #openstack-neutron04:22
*** vivek-ebay has joined #openstack-neutron04:22
openstackgerritwatanabe isao proposed openstack/neutron: Improve validate of remove_router_interface  https://review.openstack.org/15197304:23
*** vivek-eb_ has joined #openstack-neutron04:25
*** vivek-ebay has quit IRC04:27
*** dboik_ has quit IRC04:34
*** Marga_ has joined #openstack-neutron04:37
*** killer_prince is now known as lazy_prince04:37
*** chandankumar has joined #openstack-neutron04:39
*** aranjan has quit IRC04:40
*** natarajk has quit IRC04:43
*** natarajk has joined #openstack-neutron04:44
openstackgerrityalei wang proposed openstack/neutron: iptables firewall: add framework for iptables firewall functional test  https://review.openstack.org/16005104:45
openstackgerrityalei wang proposed openstack/neutron: Add portsecurity extension support  https://review.openstack.org/12655204:45
*** natarajk has quit IRC04:46
*** Longgeek has quit IRC04:47
*** natarajk has joined #openstack-neutron04:47
openstackgerritMerged openstack/neutron: Update contribute.rst with Big Switch decomp  https://review.openstack.org/16399604:47
*** tflynn has quit IRC04:48
*** sbfox has quit IRC04:49
*** YorikSar has quit IRC04:50
*** tflynn has joined #openstack-neutron04:51
*** SumitNaiksatam has quit IRC04:51
*** natarajk has quit IRC04:52
*** SumitNaiksatam has joined #openstack-neutron04:52
*** natarajk has joined #openstack-neutron04:53
*** sripriya has joined #openstack-neutron04:53
*** vthapar has joined #openstack-neutron04:53
*** natarajk has quit IRC04:54
*** natarajk has joined #openstack-neutron04:54
*** sudipto has joined #openstack-neutron04:56
*** gridinv_ has joined #openstack-neutron04:57
*** tflynn has quit IRC04:58
*** tflynn has joined #openstack-neutron04:58
*** aranjan has joined #openstack-neutron04:59
*** leenheer has quit IRC04:59
openstackgerritDane LeBlanc proposed openstack/neutron: Support Dual-Stack Gateway Ports on Neutron Routers  https://review.openstack.org/14906805:00
*** vivek-eb_ has quit IRC05:01
*** YorikSar has joined #openstack-neutron05:01
*** ijw has quit IRC05:05
*** YorikSar has quit IRC05:06
*** ivar-lazzaro has joined #openstack-neutron05:10
*** coolsvap|afk is now known as coolsvap05:14
openstackgerritYAMAMOTO Takashi proposed openstack/neutron: OVS-agent: Introduce Ryu based OpenFlow driver  https://review.openstack.org/15394605:14
*** ivar-lazzaro has quit IRC05:17
*** shwetaap has joined #openstack-neutron05:18
*** shwetaap1 has quit IRC05:20
*** sbfox has joined #openstack-neutron05:28
*** sudipto has quit IRC05:30
*** Bhavya has joined #openstack-neutron05:31
*** chandankumar has quit IRC05:36
*** chandankumar has joined #openstack-neutron05:40
*** anshul has joined #openstack-neutron05:40
harkiratHi05:40
harkiratML2 question: can there be more than one driver in ml2 listening to port status05:41
*** harkirat is now known as innosam05:42
*** vmtrooper has joined #openstack-neutron05:43
*** Murali has joined #openstack-neutron05:44
innosamHi, a question on scalability. Does enabling notifications in nova and neutron have scalability impact05:44
*** Longgeek has joined #openstack-neutron05:48
openstackgerritPraveen Kumar SM proposed openstack/neutron: DHCP Service LoadBalancing Scheduler  https://review.openstack.org/13701705:48
*** vmtrooper has quit IRC05:48
*** sbfox has quit IRC05:48
*** chandankumar has quit IRC05:52
*** ajo has joined #openstack-neutron05:53
*** sudipto has joined #openstack-neutron05:54
*** YorikSar has joined #openstack-neutron05:54
*** gridinv_ has quit IRC05:55
openstackgerritAman Kumar proposed openstack/neutron: Added config variable for External Network type in ML2  https://review.openstack.org/14954605:55
*** aranjan has quit IRC05:58
*** sbfox has joined #openstack-neutron05:58
openstackgerritItsuro Oda proposed openstack/neutron: Make floatingip reachable from the same network  https://review.openstack.org/16194705:59
*** mlavalle has quit IRC06:00
*** sripriya has quit IRC06:00
*** numan has joined #openstack-neutron06:01
*** yfried|afk is now known as yfried06:02
*** absubram has quit IRC06:04
*** natarajk has quit IRC06:04
*** mlavalle has joined #openstack-neutron06:07
*** achanda has quit IRC06:10
*** markvoelker has quit IRC06:11
*** markvoelker has joined #openstack-neutron06:12
*** shwetaap has quit IRC06:12
*** achanda has joined #openstack-neutron06:13
openstackgerritLi Yingjun proposed openstack/neutron-vpnaas: API extension for PPTP VPN  https://review.openstack.org/16408706:15
*** markvoelker has quit IRC06:16
*** rook-tmp_ has joined #openstack-neutron06:18
*** Sukhdev has joined #openstack-neutron06:18
*** rook-tmp has quit IRC06:20
*** Longgeek has quit IRC06:21
openstackgerritOpenStack Proposal Bot proposed openstack/neutron: Imported Translations from Transifex  https://review.openstack.org/16370806:21
*** amotoki has joined #openstack-neutron06:23
*** yatin has quit IRC06:23
*** armax has quit IRC06:25
*** aepifanov has joined #openstack-neutron06:25
*** Murali_ has joined #openstack-neutron06:25
openstackgerritMerged openstack/neutron: Revert "Revert "Remove port from ovsdb after its deletion""  https://review.openstack.org/16016006:26
*** Murali has quit IRC06:27
*** Murali_ is now known as Murali06:28
*** achanda has quit IRC06:28
*** ajo has quit IRC06:31
*** ajo has joined #openstack-neutron06:32
*** gongysh has quit IRC06:32
*** achanda has joined #openstack-neutron06:34
*** yfried is now known as yfried|afk06:35
*** SridharG1 has joined #openstack-neutron06:35
*** yatin has joined #openstack-neutron06:37
*** skraynev has joined #openstack-neutron06:37
*** achanda has quit IRC06:38
*** gildub has quit IRC06:38
*** lazy_prince has quit IRC06:40
*** trinaths has joined #openstack-neutron06:40
openstackgerritRomil Gupta proposed openstack/neutron: Ml2 Mechanism Driver for OVSvApp Solution  https://review.openstack.org/16024706:41
*** markvoelker has joined #openstack-neutron06:42
*** Sukhdev has quit IRC06:43
*** markvoelker has quit IRC06:47
*** innosam has quit IRC06:47
*** ronis has joined #openstack-neutron06:48
*** pal has quit IRC06:49
*** chandankumar has joined #openstack-neutron06:52
openstackgerritLi Yingjun proposed openstack/neutron-vpnaas: PPTP VPN agent support  https://review.openstack.org/16409206:52
*** pal has joined #openstack-neutron06:53
openstackgerritLi Yingjun proposed openstack/neutron-vpnaas: PPTP VPN agent support  https://review.openstack.org/16409206:53
*** coolsvap is now known as coolsvap|afk06:54
*** yfried|afk is now known as yfried06:55
*** afazekas has joined #openstack-neutron07:04
*** yfried is now known as yfried|afk07:05
*** scheuran has joined #openstack-neutron07:05
*** bobmel has joined #openstack-neutron07:06
openstackgerritMamta Prabhu proposed openstack/neutron: IBM SDN-VE Plugin decomposition  https://review.openstack.org/13047107:07
*** coolsvap has joined #openstack-neutron07:09
*** bobmel_ has quit IRC07:09
*** SumitNaiksatam has quit IRC07:09
openstackgerritZhang Hua proposed openstack/neutron-vpnaas: Functional tests of ipsec strongswan vpnaas driver  https://review.openstack.org/15856007:10
openstackgerritZhang Hua proposed openstack/neutron-vpnaas: IPsec strongSwan driver implemention  https://review.openstack.org/14439107:10
*** ronis has quit IRC07:12
*** aranjan has joined #openstack-neutron07:12
*** nlahouti has quit IRC07:13
*** oda-g has quit IRC07:15
*** gridinv_ has joined #openstack-neutron07:18
*** Longgeek has joined #openstack-neutron07:18
*** gildub has joined #openstack-neutron07:20
*** yatin has quit IRC07:22
*** dims has joined #openstack-neutron07:23
*** gridinv_ has quit IRC07:23
*** sbfox has quit IRC07:24
*** sripriya has joined #openstack-neutron07:27
*** dims has quit IRC07:27
*** sripriya has quit IRC07:31
*** jamielennox has quit IRC07:31
*** vmtrooper has joined #openstack-neutron07:32
*** jamielennox has joined #openstack-neutron07:34
*** ijw has joined #openstack-neutron07:34
*** ijw has quit IRC07:35
*** ijw has joined #openstack-neutron07:36
*** vmtrooper has quit IRC07:37
*** ijw has quit IRC07:38
*** ijw has joined #openstack-neutron07:39
*** yfried|afk is now known as yfried07:39
*** jamielennox has quit IRC07:39
*** Longgeek has quit IRC07:41
*** Longgeek has joined #openstack-neutron07:41
openstackgerritMerged openstack/neutron: Use common agent.linux.utils.ensure_dir method  https://review.openstack.org/15410307:41
*** markvoelker has joined #openstack-neutron07:43
*** topshare has quit IRC07:45
*** topshare has joined #openstack-neutron07:46
*** Bhavya has quit IRC07:46
*** ajo has quit IRC07:47
*** apuimedo has joined #openstack-neutron07:47
*** markvoelker has quit IRC07:48
*** yfried is now known as yfried|afk07:49
*** kickinz1|afk is now known as kickinz107:50
*** pal has quit IRC07:52
openstackgerritMerged openstack/neutron: Refactor retry mechanism used in some DB operations  https://review.openstack.org/14926107:54
*** jamielennox|away has joined #openstack-neutron07:55
*** pal has joined #openstack-neutron07:55
*** jamielennox|away is now known as jamielennox07:55
*** vivek-ebay has joined #openstack-neutron07:57
*** coolsvap has quit IRC07:59
*** vthapar has quit IRC07:59
*** vivek-ebay has quit IRC08:02
*** Bhavya has joined #openstack-neutron08:02
*** kawa2014 has joined #openstack-neutron08:03
*** topshare has quit IRC08:04
*** topshare has joined #openstack-neutron08:06
*** kawa2014 has quit IRC08:08
*** chlong has quit IRC08:09
*** jistr has joined #openstack-neutron08:12
*** safchain has joined #openstack-neutron08:19
*** kawa2014 has joined #openstack-neutron08:20
*** matrohon has joined #openstack-neutron08:21
*** vishwanathj has quit IRC08:21
*** openstackgerrit has quit IRC08:21
*** openstackgerrit has joined #openstack-neutron08:22
*** apostal has joined #openstack-neutron08:23
*** jpena has joined #openstack-neutron08:25
*** topshare has quit IRC08:25
*** topshare has joined #openstack-neutron08:26
*** adelinatuvenie has joined #openstack-neutron08:28
*** pal has quit IRC08:28
*** ihrachyshka has joined #openstack-neutron08:30
*** cui has joined #openstack-neutron08:32
*** vthapar has joined #openstack-neutron08:33
*** killer_prince has joined #openstack-neutron08:35
*** killer_prince is now known as lazy_prince08:35
*** pal has joined #openstack-neutron08:36
*** numan has quit IRC08:37
*** YorikSar has quit IRC08:37
*** jlanoux has joined #openstack-neutron08:38
*** kawa2014 has quit IRC08:39
*** kawa2014 has joined #openstack-neutron08:40
kevinbentonamotoki: ping08:41
*** obondarev_ has quit IRC08:42
*** obondarev has joined #openstack-neutron08:42
*** YorikSar has joined #openstack-neutron08:43
*** aranjan has quit IRC08:44
*** markvoelker has joined #openstack-neutron08:44
*** aranjan has joined #openstack-neutron08:45
*** yfried|afk is now known as yfried08:46
*** aranjan has quit IRC08:46
*** aranjan has joined #openstack-neutron08:46
*** jistr has quit IRC08:46
*** stackmore has joined #openstack-neutron08:47
*** ajo has joined #openstack-neutron08:47
*** ihrachyshka has quit IRC08:48
*** markvoelker has quit IRC08:48
*** sbelous_ has joined #openstack-neutron08:50
*** aranjan has quit IRC08:51
*** yfauser has joined #openstack-neutron08:53
*** yfauser has left #openstack-neutron08:54
sbelous_yamamoto: Hi! may I ask you to check comments at https://review.openstack.org/#/c/154116/ ? And if it ok, can you review it, please?08:55
*** amotoki has quit IRC08:57
*** erkules_ is now known as erkules08:57
*** erkules has quit IRC08:57
*** erkules has joined #openstack-neutron08:57
yamamotosbelous_: will look08:59
*** ronis has joined #openstack-neutron09:01
*** luqas has joined #openstack-neutron09:03
*** sbelous_ has quit IRC09:03
*** sbelous_ has joined #openstack-neutron09:04
*** jistr has joined #openstack-neutron09:04
*** yfried is now known as yfried|afk09:05
openstackgerritYAMAMOTO Takashi proposed openstack/neutron: OVS-agent: Introduce Ryu based OpenFlow driver  https://review.openstack.org/15394609:06
yamamotosbelous: commented09:06
*** Miouge_ has joined #openstack-neutron09:06
*** pask81 has joined #openstack-neutron09:07
*** sbelous_ has quit IRC09:08
*** Miouge has quit IRC09:08
*** Miouge_ is now known as Miouge09:08
*** SridharG1 has left #openstack-neutron09:09
*** apostal has quit IRC09:10
*** ygbo has joined #openstack-neutron09:12
*** sudipto has quit IRC09:12
*** watanabe_isao has quit IRC09:12
*** hichihara has quit IRC09:16
*** Longgeek has quit IRC09:16
*** yfried|afk is now known as yfried09:18
*** sbelous has quit IRC09:19
*** amotoki has joined #openstack-neutron09:20
*** vmtrooper has joined #openstack-neutron09:21
*** numan has joined #openstack-neutron09:21
*** sbelous has joined #openstack-neutron09:21
amotokiKennan: pong09:22
amotokiKennan: sorry09:22
amotokikevinbenton: hi, what?09:22
sbelousyamamoto: thanks!09:24
openstackgerritMerged openstack/neutron: Set default of api_workers to number of CPUs  https://review.openstack.org/14049309:25
*** vmtrooper has quit IRC09:26
*** yfried is now known as yfried|afk09:28
*** SridharG1 has joined #openstack-neutron09:29
*** pradip_vedams has joined #openstack-neutron09:30
*** pradip_vedams has quit IRC09:30
*** pradip_vedams has joined #openstack-neutron09:31
pradip_vedamshi all, while doing devstak for neutron i am getting some error like fail to create internal network can any one help me to resolve this problem http://paste.openstack.org/show/192053/09:34
*** sanjjain has joined #openstack-neutron09:37
*** dims has joined #openstack-neutron09:42
*** markvoelker has joined #openstack-neutron09:45
*** sanjjain has left #openstack-neutron09:45
*** yfried|afk is now known as yfried09:46
*** dims has quit IRC09:47
*** aranjan has joined #openstack-neutron09:47
*** markvoelker has quit IRC09:49
*** Bhavya has quit IRC09:51
*** aepifanov has quit IRC09:52
*** dims has joined #openstack-neutron09:54
*** trinaths has left #openstack-neutron09:56
*** aranjan has quit IRC09:57
*** SridharG has quit IRC09:58
*** SridharG has joined #openstack-neutron10:00
*** bobmel has quit IRC10:01
*** bobmel has joined #openstack-neutron10:02
*** luqas has quit IRC10:04
*** moshele has joined #openstack-neutron10:05
*** yfried is now known as yfried|afk10:06
*** ihrachyshka has joined #openstack-neutron10:08
*** sluo_wfh has quit IRC10:08
*** yamamoto has quit IRC10:11
*** igordcard_ has joined #openstack-neutron10:13
*** jp_at_hp has joined #openstack-neutron10:15
*** moshele has quit IRC10:15
*** Bhavya has joined #openstack-neutron10:16
*** Longgeek has joined #openstack-neutron10:17
*** yog_ has quit IRC10:18
*** skraynev has quit IRC10:19
*** skraynev has joined #openstack-neutron10:19
*** aranjan has joined #openstack-neutron10:23
*** sudipto has joined #openstack-neutron10:26
*** innosam has joined #openstack-neutron10:26
innosamHi10:26
innosamIs there any one who can help me with registering to the AMQP queue using kombu10:27
innosami am not able to register to q-agent-notifier-port-update_fanout10:27
*** yfried|afk is now known as yfried10:29
openstackgerritAdelina Tuvenie proposed openstack/neutron: OVS agent support on Hyper-V  https://review.openstack.org/16059610:29
*** pc_m has joined #openstack-neutron10:29
*** stanzgy has quit IRC10:31
*** txomon|fon has joined #openstack-neutron10:31
*** stanzgy_ has quit IRC10:31
openstackgerritMiguel Angel Ajo proposed openstack/neutron: Add a netns-cleanup functional test  https://review.openstack.org/16386010:31
*** Longgeek has quit IRC10:32
*** cui has quit IRC10:35
*** aranjan has quit IRC10:35
*** luqas has joined #openstack-neutron10:35
openstackgerritMiguel Angel Ajo proposed openstack/neutron: Fix netns-cleanup broken by ProcessMonitor refactor  https://review.openstack.org/16340110:36
*** moha_hunt has joined #openstack-neutron10:36
openstackgerritMiguel Angel Ajo proposed openstack/neutron: Add a netns-cleanup functional test  https://review.openstack.org/16386010:36
*** yamamoto has joined #openstack-neutron10:37
*** luqas has quit IRC10:40
*** Miouge has quit IRC10:41
*** lazy_prince has quit IRC10:42
*** aix has joined #openstack-neutron10:44
*** markvoelker has joined #openstack-neutron10:46
*** markvoelker has quit IRC10:50
*** topshare_ has joined #openstack-neutron10:53
*** jamielennox has quit IRC10:55
*** topshare has quit IRC10:55
*** yfried is now known as yfried|afk10:58
*** killer_prince has joined #openstack-neutron10:58
*** killer_prince is now known as lazy_prince10:58
*** harlowja_away has quit IRC10:59
*** aranjan has joined #openstack-neutron11:01
*** jamielennox|away has joined #openstack-neutron11:02
*** jamielennox|away is now known as jamielennox11:02
*** Bhavya has quit IRC11:06
*** vmtrooper has joined #openstack-neutron11:10
*** puck has quit IRC11:12
*** jamielennox has quit IRC11:14
*** vmtrooper has quit IRC11:15
*** puck has joined #openstack-neutron11:15
openstackgerritJohn Davidge proposed openstack/neutron: Implement Neutron IPv6 Prefix Delegation  https://review.openstack.org/15869711:16
*** yfried|afk is now known as yfried11:18
*** aranjan has quit IRC11:20
*** vthapar has quit IRC11:20
*** pask81 has quit IRC11:23
*** jamielennox|away has joined #openstack-neutron11:24
*** jamielennox|away is now known as jamielennox11:24
*** lazy_prince is now known as killer_prince11:30
*** moha_hunt has quit IRC11:30
*** innosam has quit IRC11:32
*** Miouge has joined #openstack-neutron11:34
*** pal has quit IRC11:35
*** Bhavya has joined #openstack-neutron11:37
*** baoli has joined #openstack-neutron11:41
*** sudipto has quit IRC11:41
*** baoli has quit IRC11:44
*** baoli has joined #openstack-neutron11:44
openstackgerritZongKai LI proposed openstack/neutron: fix check_ports_exist_on_l3agent in no subnet case  https://review.openstack.org/15933811:45
*** aranjan has joined #openstack-neutron11:46
*** markvoelker has joined #openstack-neutron11:46
*** topshare_ has quit IRC11:47
*** moha_hunt has joined #openstack-neutron11:48
*** markvoelker has quit IRC11:51
*** mwagner_lap has joined #openstack-neutron11:57
*** diegows has joined #openstack-neutron11:57
*** yfried is now known as yfried|afk12:02
*** aranjan has quit IRC12:04
*** igordcard_ has quit IRC12:05
*** markvoelker has joined #openstack-neutron12:07
*** irenab has quit IRC12:10
*** irenab has joined #openstack-neutron12:11
*** Longgeek has joined #openstack-neutron12:15
*** radez_g0n3 is now known as radez12:22
*** Marga_ has quit IRC12:24
*** chlong has joined #openstack-neutron12:24
*** pc_m has left #openstack-neutron12:28
*** pc_m has joined #openstack-neutron12:28
*** raildo has joined #openstack-neutron12:28
*** yfauser has joined #openstack-neutron12:29
*** adelinatuvenie has quit IRC12:29
*** yfauser has left #openstack-neutron12:29
*** ajmiller has quit IRC12:29
*** Longgeek has quit IRC12:31
*** ferest has joined #openstack-neutron12:32
openstackgerritSergey Belous proposed openstack/neutron: Reduce db calls count in get_devices_details_list  https://review.openstack.org/15411612:36
*** yamamoto has quit IRC12:37
*** chandankumar has quit IRC12:40
*** yamamoto has joined #openstack-neutron12:41
*** Networkn3rd has quit IRC12:43
openstackgerritMathieu Rohon proposed openstack/python-neutronclient: Add commands from extensions to available commands  https://review.openstack.org/16347812:43
*** gildub has quit IRC12:50
openstackgerritPaul Michali proposed openstack/neutron-vpnaas: Fix functional test breakage from DevStack change  https://review.openstack.org/16416412:52
*** Murali has quit IRC12:55
*** vmtrooper has joined #openstack-neutron12:59
*** jpena is now known as jpena|lunch13:00
*** b3nt_pin is now known as seagulls13:01
*** aranjan has joined #openstack-neutron13:01
*** jpena|lunch is now known as jpena13:02
*** Miouge_ has joined #openstack-neutron13:03
*** mlavalle has quit IRC13:03
*** Miouge has quit IRC13:04
*** Miouge_ is now known as Miouge13:04
*** vmtrooper has quit IRC13:04
*** Bhavya has quit IRC13:04
*** aranjan has quit IRC13:05
openstackgerritMerged openstack/neutron: Break coupling between ML2 and L3 during create/update operations  https://review.openstack.org/15377513:07
*** mlavalle has joined #openstack-neutron13:07
*** ijw has quit IRC13:09
*** radez is now known as radez_g0n313:10
openstackgerritrcurran proposed openstack/neutron: ML2 cisco_nexus MD: sync config and models with vendor repo  https://review.openstack.org/16304013:13
*** markvoelker has quit IRC13:16
*** markvoelker has joined #openstack-neutron13:17
*** scheuran has quit IRC13:19
*** dboik has joined #openstack-neutron13:21
*** Marga_ has joined #openstack-neutron13:23
*** yfried|afk has quit IRC13:24
openstackgerritSergey Belous proposed openstack/neutron: Reduce db calls count in get_devices_details_list  https://review.openstack.org/15411613:25
*** tiswanso has joined #openstack-neutron13:26
*** yamamoto has quit IRC13:27
*** bobmel has quit IRC13:27
*** bobmel has joined #openstack-neutron13:28
*** popw1 has joined #openstack-neutron13:29
openstackgerritAman Kumar proposed openstack/neutron: Added config variable for External Network type in ML2  https://review.openstack.org/14954613:30
openstackgerritTimothy Swanson proposed openstack/neutron: Add MTU selection & advertisement settings to Neutron config  https://review.openstack.org/15373313:31
*** popw has quit IRC13:32
numankevinbenton, Hi, when you get some time can you please review it. You had +2'd it in the previous patch set13:33
numankevinbenton, https://review.openstack.org/#/c/142870/13:34
*** TrevorV|Home has joined #openstack-neutron13:36
*** setuid has quit IRC13:40
*** kbringard has joined #openstack-neutron13:42
*** thomasem has joined #openstack-neutron13:43
*** setuid has joined #openstack-neutron13:43
*** TrevorV|Home has quit IRC13:45
*** thomasem has quit IRC13:45
*** thomasem has joined #openstack-neutron13:46
*** crose has joined #openstack-neutron13:48
*** mikehn_ has quit IRC13:48
*** mikehn has joined #openstack-neutron13:49
*** dboik has quit IRC13:50
*** dboik has joined #openstack-neutron13:50
openstackgerritZhang Hua proposed openstack/neutron-vpnaas: Functional tests of ipsec strongswan vpnaas driver  https://review.openstack.org/15856013:51
*** Miouge has quit IRC13:51
*** amuller has joined #openstack-neutron13:52
*** Miouge has joined #openstack-neutron13:53
*** ferest has quit IRC13:53
*** Rajeev has joined #openstack-neutron13:55
*** dansmith is now known as superdan13:59
*** julim has quit IRC14:01
*** apuimedo has quit IRC14:01
*** aranjan has joined #openstack-neutron14:02
*** Miouge has quit IRC14:02
*** radez_g0n3 is now known as radez14:02
*** blinky42 has quit IRC14:02
*** jprovazn has joined #openstack-neutron14:03
*** txomon|fon has quit IRC14:04
*** kfox1111 has joined #openstack-neutron14:04
kfox1111Please don't let Kilo come without this bug fix: https://review.openstack.org/#/c/157167/14:04
kfox1111Juno had a serous bug creap up for us, and I had to fix it. It would be a huge shame if kilo was allowed to go out the door with the same regression. :/14:05
*** apuimedo has joined #openstack-neutron14:06
*** r-daneel has joined #openstack-neutron14:06
*** aranjan has quit IRC14:06
*** dims has quit IRC14:06
amullerkfox1111: I added some cores to that review14:07
*** jaypipes is now known as leakypipes14:07
kfox1111thank you. :)14:07
*** dims has joined #openstack-neutron14:07
openstackgerritMiguel Angel Ajo proposed openstack/neutron: Refactor _remove_unused_security_group_info  https://review.openstack.org/15251214:08
ajoHenryG, mestery  ^14:08
mesterythanks ajo!14:09
ajoyou're welcome ;)14:09
amullerkfox1111: btw since you have 1 tenant network connected to 2 routers, the tenant network has its default gateway as 1 of the routers right? So basically all outgoing traffic is going out through one of the routers, but incoming traffic comes through both?14:10
*** TrevorV|Home has joined #openstack-neutron14:10
amuller(Trying to figure out the setup and its use case)14:10
amullerthis is with OVS + L3 agent?14:10
*** yamamoto has joined #openstack-neutron14:11
*** shwetaap has joined #openstack-neutron14:11
kfox1111amuller: yes.14:11
kfox1111one is the internet external network.14:12
kfox1111one, is, for example, a bmc network. /24.14:12
kfox1111the vm's can talk on both networks. the internet or the private one.14:12
amullercan you connect the VMs to two tenant networks?14:12
kfox1111or how about this example. we have a cloud that has two networks. the internet network, and a public internal network.14:13
kfox1111all tenants can put a floating ip on either the internet network, or the shared private one.14:13
*** jp_at_hp has quit IRC14:13
kfox1111If they don't want their vm accessable from the internet, but want to share a resource (LDAP server), they put a floating ip on the shared private.14:13
*** Miouge has joined #openstack-neutron14:13
kfox1111the bug is specifically in that case, where it decides to put the shared private networks floating ip on the internet's router by mistake.14:14
kfox1111the code was mistakenly removed in a cleanup in juno. :/14:15
amullerIs there another way to achieve your use case? separate tenant networks, security groups?14:15
kfox1111no.14:15
*** dboik has quit IRC14:15
*** Longgeek has joined #openstack-neutron14:15
*** amotoki has quit IRC14:16
*** csoukup has joined #openstack-neutron14:16
*** dboik has joined #openstack-neutron14:16
*** ChuckC_ has joined #openstack-neutron14:16
*** ChuckC has quit IRC14:17
kfox1111in the case of bmc access, it truely is an external network I want vm's to be able to reach. they may be launching an ftp server for example to upgrade bmc's or other firmware.14:17
amullerkfox1111: Sorry I'm not following. If you have 2 tenant networks, 2 routers, 2 external networks. VMs connect to both tenant networks and get a floating IP only in one14:17
kfox11111 tenant network.14:17
amullerI'm saying, if you had 214:17
*** afazekas has quit IRC14:18
amulleryou wouldn't hit this bug, why wouldn't that hit your use case? Is there an issue I'm missing?14:18
openstackgerritMerged openstack/neutron: Fix incorrect comments  https://review.openstack.org/16404714:18
kfox1111that could be made to work. just more work for everyone.14:18
kfox1111this use to work, and someone broke it. I would just like to get it fixed.14:19
kfox1111the unit test case I provided clearly shows juno has the bug and current trunk to. :/14:19
amullerI'm not disputing the regression14:19
amullerJust asking if there's a work around14:19
*** aepifanov has joined #openstack-neutron14:20
kfox1111Not one that doesn't require rewriting a lot of templates.14:20
kfox1111the easier work around was writing the patch and patching our cloud. :/14:20
kfox1111it was unstable for a week until I found the regression though. :/14:20
*** SridharG has quit IRC14:20
kfox1111but hopefully the unit test will ensure it never happens again.14:21
*** dboik has quit IRC14:21
*** dboik has joined #openstack-neutron14:21
*** dims is now known as dimsum__14:21
*** carl_baldwin has joined #openstack-neutron14:22
*** ChuckC_ has quit IRC14:23
*** popw has joined #openstack-neutron14:25
*** kawa2014 has quit IRC14:26
*** kawa2014 has joined #openstack-neutron14:26
*** popw1 has quit IRC14:27
openstackgerritShiv Haris proposed openstack/neutron: Brocade vendor code decomposition from neutron repo.  https://review.openstack.org/15238314:28
openstackgerritEd Bak proposed openstack/neutron: Break calls to get_devices_details_list into subsets  https://review.openstack.org/16359414:33
*** Nakato has quit IRC14:33
*** Nakato has joined #openstack-neutron14:34
*** thedodd has joined #openstack-neutron14:36
*** Networkn3rd has joined #openstack-neutron14:36
openstackgerritCarl Baldwin proposed openstack/neutron: Move internal port processing to router classes  https://review.openstack.org/15451314:40
openstackgerritEd Bak proposed openstack/neutron: Break calls to get_devices_details_list into subsets  https://review.openstack.org/16359414:41
obondarevcarl_baldwin: ping14:42
carl_baldwinobondarev:  pong14:43
obondarevcarl_baldwin: a quick question on DVR14:43
obondarevcarl_baldwin: https://github.com/openstack/neutron/blob/master/neutron/plugins/openvswitch/agent/ovs_dvr_neutron_agent.py#L11014:43
obondarevcarl_baldwin: deriving from dvr_rpc.DVRAgentRpcApiMixin seems confusing and wrong, what do you think?14:44
*** stackmore has quit IRC14:44
*** stackmore has joined #openstack-neutron14:45
*** pal has joined #openstack-neutron14:45
obondarevcarl_baldwin: dvr_rpc.DVRAgentRpcApiMixin should be used in plugin I believe14:45
amullerahhhh, rebase hell14:45
obondarevcarl_baldwin: and is actually used14:45
obondarevcarl_baldwin: just wanted to confirm :)14:46
*** amotoki has joined #openstack-neutron14:46
carl_baldwinobondarev: looking...14:46
*** shaohe_feng1 has joined #openstack-neutron14:46
carl_baldwinamuller: Are you talking about the oslo logging stuff?14:46
amullercarl_baldwin: first that, now some of my patches are again in merge conflict, less than 8 hours after I resolved the previous merge conflict14:47
amullerbut yeah the timing of the oslo logging graduation was unfortunate...14:47
*** alaski is now known as lascii14:48
*** vmtrooper has joined #openstack-neutron14:48
*** thedodd has quit IRC14:48
*** Longgeek has quit IRC14:49
carl_baldwinobondarev:  I wonder if Rajeev could provide some insight here.14:50
obondarevcarl_baldwin: ok, I'l just put on gerrit and we'll figure out14:51
openstackgerritSandhya Dasu proposed openstack/neutron: Cisco UCS Manager ML2 Mechanism Driver  https://review.openstack.org/15543614:52
*** shivharis has joined #openstack-neutron14:52
carl_baldwinobondarev:  Let me know what the gerrit link is.14:52
obondarevcarl_baldwin: sure14:52
*** thedodd has joined #openstack-neutron14:53
*** vmtrooper has quit IRC14:53
*** PadminiB has joined #openstack-neutron14:54
*** SridharG1 has quit IRC14:54
*** SridharG has joined #openstack-neutron14:56
Rajeevobondarev: this class provides ovs support routines for the DVR code14:56
openstackgerritKyle Mestery proposed openstack/neutron: Change from core reviewers to maintainers  https://review.openstack.org/16420814:56
*** pal_ has joined #openstack-neutron14:57
*** popw has quit IRC14:57
*** tflynn has quit IRC14:57
*** pal has quit IRC14:57
obondarevRajeev: yeah but dvr_rpc.DVRAgentRpcApiMixin is for plugin use, right?14:57
Rajeevobondarev: yes14:57
*** carl_baldwin_ has joined #openstack-neutron14:58
obondarevRajeev: so no need for OVSDVRNeutronAgent(dvr_rpc.DVRAgentRpcApiMixin)14:58
*** carl_baldwin has quit IRC14:58
*** carl_baldwin_ is now known as carl_baldwin14:58
*** ronis has quit IRC14:58
*** leenheer has joined #openstack-neutron14:58
*** crose has quit IRC14:58
*** leenheer has quit IRC15:00
Rajeevobondarev: give me a minute15:01
openstackgerritAssaf Muller proposed openstack/neutron: Move Unix domain socket helpers to a common place  https://review.openstack.org/16403215:02
openstackgerritAssaf Muller proposed openstack/neutron: Replace keepalived notifier bash script with Python ip monitor  https://review.openstack.org/12538415:02
otherwiseguyHenryG: I think all of the kinks on daemon mode and environment variables are worked out. https://review.openstack.org/#/c/158805/ YorikSar has a devstack patch using that uses it and all the tests pass there as well.15:02
*** radez is now known as radez_g0n315:02
openstackgerritOleg Bondarev proposed openstack/neutron: Remove unneeded DVRAgentRpcApiMixin from OVSDVRNeutronAgent  https://review.openstack.org/16421215:03
HenryGajo: Thanks, LGTM. Let's hope zuul can merge it.15:03
*** aranjan has joined #openstack-neutron15:03
HenryGotherwiseguy: looking15:03
obondarevRajeev: carl_baldwin: https://review.openstack.org/#/c/164212/15:03
carl_baldwinobondarev: ack15:03
HenryGotherwiseguy: I have not been following that :(15:04
HenryGotherwiseguy: Let me see if I can catch up15:04
Rajeevobondarev: some of the methods are used by the agent to get the DVR MAC that is to be used by the agent15:04
otherwiseguyHenryG: Thanks! There's a little change in how we handle addl_env because of some changes in oslo_rootwrap, but we don't ever use addl_env and run_as_root=True at the same time anyway, so no other code changes because of it.15:04
Rajeevobondarev: you may want to add Vivek in if you think this needs to change15:05
otherwiseguyWe only use addl_env once in the code, and it is in a sanity check, but run directly. So it'll just run as env LC_ALL=C whatever now.15:05
ajothanks HenryG looking at your final comments,15:05
*** radez_g0n3 is now known as radez15:05
HenryGotherwiseguy: So that is the final patch for the BP?15:06
amullercarl_baldwin: https://review.openstack.org/164032 should be good to go15:06
ajoHenryG, isn't your default the same?15:06
ajo     if foo is None:15:06
ajo          foo = some_default15:06
otherwiseguyHenryG: Yes. (Finally!)15:06
carl_baldwinamuller: ack15:06
ajoyou won't be able to pass foo=None, in that case is changed to some_default too?15:06
*** thedodd has quit IRC15:06
ajoI'm missing something :)15:06
ajoHenryG ^15:06
ajo:)15:06
HenryGajo: what about foo={}15:06
HenryGajo: empty dict15:07
ajohmm15:07
*** aranjan has quit IRC15:07
*** ChuckC_ has joined #openstack-neutron15:07
ajoHenryG true!15:08
pavel_bondarhi carl_baldwin15:08
salv-orlandocarl_baldwin, pavel_bondar, PadminiB: aloha15:08
obondarevRajeev: seems it's DVRServerRpcApi which agent needs, not DVRAgentRpcApiMixin15:08
HenryGajo: The code in this case is fine because no caller currently tries to pass empty dict15:08
carl_baldwinpavel_bondar: salv-orlando PadminiB: hi15:08
stackmoreHosung here as well15:08
pc_mdougwig: ping?15:08
*** xgerman has joined #openstack-neutron15:08
pavel_bondarhi salv-orlando15:08
PadminiBHello Carl, Pavel, Slavatore15:09
salv-orlandoone of pavel_bondar's patches puts all the IPAM patches together. I count 5 of them all together, stacked as follows:15:09
HenryGajo: But in general, I hope you see what I mean? Perhaps my educational diatribe comment taught something? :)15:09
ajoHenryG, yeah, but good as a general pattern, thankgs for the teaching :)15:09
ajoabsolutely, it did, Thanks a lot :)15:09
pc_mcarl_baldwin: dougwig: Can either of you quickly review and approve https://review.openstack.org/#/c/164164/? Need driver core. This is fixing breakage.15:09
salv-orlandocarl_baldwin's interface, salv-orlando reference driver, pavel_bondar driver load mechanism, pavel_bondar db base refactoring15:10
ajoHenryG++15:10
salv-orlandosorry it's 4 patches not 515:10
ajowe need a karma bot ;D15:10
*** _cjones_ has joined #openstack-neutron15:10
pavel_bondarand this patch is #link https://review.openstack.org/#/c/153236/, it on the top of everything15:10
shivharisHenryG: hi15:10
salv-orlandoif by "integration" we mean putting eveyrthing together, then the strategy for me is to use the topmost patch, and then do debugging15:10
*** Marga_ has quit IRC15:10
salv-orlandothis has the disadvantage that if you fix an issue you have to apply the patch in a staged fashion to the 4 patches which make the blueprint.15:11
salv-orlandobut for me this is relatively easy, I don't know what carl_baldwin and pavel_bondar think about it15:11
pavel_bondarshould do anything additional to making all test passes in top most patch?15:12
carl_baldwinsalv-orlando: It can be a bit tricky to get all of the patches put together unless they are all rebased to the latest version of their respective dependencies.  I’ve done this before and can help manage the dependencies.15:13
salv-orlandopavel_bondar: I think we need tweaks in the other patches, not anything exceptional15:13
*** jgrimm has quit IRC15:13
*** jprovazn has quit IRC15:13
salv-orlandoyeah carl_baldwin that's a fairly tiring aspect but it can be easily scripted15:14
*** jgrimm has joined #openstack-neutron15:14
salv-orlandoon the other hand we might be proactive and do the rebases upstream15:14
salv-orlandofor instance if I push a new patchset of mine I can automaticaly rebase pavel's patches15:14
salv-orlandoI don't think pavel_bondar would mind if I push another patchset on his behalf15:14
shivharisHenryG: ping15:14
openstackgerritMiguel Angel Ajo proposed openstack/neutron: Add a netns-cleanup functional test  https://review.openstack.org/16386015:15
pavel_bondarsalv-orlando, yes, that is ok, you have latest version of my code15:15
carl_baldwinsalv-orlando: He must be aware that you’re doing it if he’s also working on a new version of his patch.  But yes, we can do that.15:15
Rajeevobondarev: I think you are right, I will add Vivek to the gerrit because he is most familiar with this code.15:15
salv-orlandocarl_baldwin: my opinion is that it's simpler to manage conflicts rather than coordinate among ourselves.15:16
carl_baldwinsalv-orlando: The real problems come if pavel_bondar puts up a rebased version of one of your older patch sets.15:16
salv-orlandocarl_baldwin: in that case I could rebase, couldn't I?15:16
*** jpena has quit IRC15:17
obondarevRajeev: thanks15:17
carl_baldwinsalv-orlando: Yes, you can rebase the correct version of your patch on top.15:17
salv-orlandocarl_baldwin, pavel_bondar: now this can ensure we work on our patches and then unit test pass.15:18
carl_baldwinsalv-orlando: I agree that we should work independently and resolve conflicts.  Just pointed out where I’ve seen things go wrong silently.15:18
Rajeevobondarev: welcome15:18
salv-orlandowhat about the gate? The neutorn dev community and infra teams would hardly accept us to default the neutorn db ipam driver15:19
HenryGotherwiseguy: Just curious, have you seen the privilege separation patch that gus proposed?15:19
*** kickinz1 is now known as kickinz1|afk15:19
salv-orlandoand I probably won't even dare to ask15:19
HenryGshivharis: pong15:19
carl_baldwinsalv-orlando: pavel_bondar:  I don’t think we’re at a point where we can talk about switching the gate to this driver.15:20
otherwiseguyHenryG: I took a quick look at it. Seemed to me that integrating it with rootwrap daemon mode would be interesting. It already has the daemon, communication protocol, etc.15:20
shivharisHenryG: it appear that oslo changes are merged. this may affect stackforge vendor plugins/drivers15:20
otherwiseguySet up filters based on code that you actually want to allow to run, etc.15:21
otherwiseguyBut haven't looked it it that closely, yet.15:21
*** yfried|afk has joined #openstack-neutron15:21
*** dannywilson has joined #openstack-neutron15:21
*** aepifanov has quit IRC15:21
salv-orlandocarl_baldwin: ack. But we want to gate somehow using this driver. I was thinking of a non-voting job for Kilo.15:21
shivharisHenryG: any suggestions on how to transition smoothly15:21
salv-orlandoand then perhaps switch the voting jobs to the new driver in Liberty15:21
HenryGshivharis: Not "may affect", but "definitely affects". This has been communicated.15:22
salv-orlandocarl_baldwin: unless you reckon that at this stage is fairly pointless doing so and we might be satisfied with unit tests alone15:22
carl_baldwinsalv-orlando: Agree, that would be very valuable.15:22
*** sripriya has joined #openstack-neutron15:22
HenryGshivharis: Basically you do the same in your repo as the oslo.log patch15:22
HenryGshivharis: Here is what we did for our repo this morning, https://review.openstack.org/16407215:23
carl_baldwinsalv-orlando: We need some gate feedback, you’re right about that.  Maybe just a patch on top of the chain that enables the driver so that the check queue can run on it.15:23
salv-orlandocarl_baldwin: that would give us a way of checking on-demand.15:23
shivharisHenryG: yup, that is needed now.15:24
shivharisHenryG: thanks15:24
salv-orlandocarl_baldwin: better than nothing. Do you think the proposal for a non-voting job to run with every patch will just bounce?15:24
carl_baldwinsalv-orlando: I think we should propose both.  It is easy to propose.  Maybe the single patch could help us flush out easy failures early.15:25
carl_baldwinsalv-orlando: Maybe we can turn on the non-voting job when we’re feeling serious about switching over to the new driver.15:26
mesterypc_m: Is this the most current location for VPNaaS API documentation: https://wiki.openstack.org/wiki/Neutron/VPNaaS#REST_API15:27
salv-orlandocarl_baldwin: yes of course. I was taking that for granted. There's no point in gating on something you know it does not work15:27
salv-orlandocarl_baldwin: as we're talking about a patch to enable the driver, we might want also to discuss how we manage upgrades to pluggable ipam15:27
salv-orlandounless there are some other topics concerning testing.15:28
*** dtynan has left #openstack-neutron15:28
*** sripriya has quit IRC15:29
*** tsg has quit IRC15:30
carl_baldwinpavel_bondar: What else do you have?15:30
pavel_bondarsalv-orlando: do we plan to allow upgrades to pluggable ipam on existing setup in this release cycle?15:30
salv-orlandopavel_bondar: not necessarily. But what's our upgrade story?15:30
salv-orlandoin Kilo it will be just for greenfield.15:31
salv-orlandothat is to say - you start a new deployment with the IPAM driver15:31
carl_baldwinsalv-orlando: +1 greenfield only for now.15:31
salv-orlandobecause if you have an existing deployment and you  switch you will be in a mess15:31
openstackgerritBrent Eagles proposed openstack/neutron: Refactor RESOURCE_ATTRIBUTE_MAP cleanup  https://review.openstack.org/15667015:31
*** jpena has joined #openstack-neutron15:32
salv-orlandoso when we'll let people switch, how are we going to do it? I have two options, but I would first like to know if you've already got something in mind15:32
*** YorikSar has quit IRC15:32
*** david-lyle_afk is now known as david-lyle15:33
pavel_bondarsalv-orlando: what you proposed in a ML a while ago sound good for me about migrations15:33
carl_baldwinpavel_bondar: salv-orlando:  I must have missed the migration stuff in the ML.15:33
pavel_bondarsalv-orlando: i.e. to have driver defined per subnetpool in database15:34
salv-orlandopavel_bondar: ah right the allocator15:34
salv-orlandothe other option would be a one-shot migration ml2-style15:34
salv-orlandocarl_baldwin: do you need a link to the ml post?15:35
*** sripriya has joined #openstack-neutron15:35
*** zzzeek has joined #openstack-neutron15:35
carl_baldwinsalv-orlando: If you have it handy, I can read through it quickly.15:36
*** dane_leblanc has joined #openstack-neutron15:36
*** yfried|afk has quit IRC15:36
salv-orlandocarl_baldwin: I think it's this one: http://lists.openstack.org/pipermail/openstack-dev/2015-March/058527.html15:37
*** adelinatuvenie has joined #openstack-neutron15:37
carl_baldwinsalv-orlando: I appologize that I started reading that one and got distracted part-way through.15:38
openstackgerritSergey Belous proposed openstack/neutron: Reduce db calls count in get_devices_details_list  https://review.openstack.org/15411615:38
baolicarl_baldwin, haleyb: quick question about DVR. Is it currently for v4 only? we are looking at the snat_redirect_add and remove methods.15:39
*** thedodd has joined #openstack-neutron15:39
carl_baldwinbaoli: yes15:39
*** reed has joined #openstack-neutron15:39
*** jpena has quit IRC15:40
salv-orlandocarl_baldwin: no worries. Basically we either have, for every subnet or subnetpool, a declaration of a driver to use, or we skip this part and we just migrate all the current stuff to neutron_db_ipam15:40
*** Miouge has quit IRC15:40
salv-orlandothen if somebody uses drivers different from neutorn_db_ipam they either provide their own conversion script or the driver will work for new deployment only15:41
haleybbaoli: we had to address some issues with IPv6/DVR just to get it to work properly, but all routing should be via the centralized router on the network node15:41
pc_mmestery: Well, no not really. The external API docs were created, and shown at the top of that page.15:42
carl_baldwinsalv-orlando:  I think the allocator idea could help us ease in to using the driver.  It has some appeal.15:42
openstackgerritSandhya Dasu proposed openstack/neutron: Cisco UCS Manager ML2 Mechanism Driver  https://review.openstack.org/15543615:42
dane_leblanccarl_baldwin, haleyb: Is it guaranteed that there will only be one fixed_ip for the call to snat_redirect_add: self._snat_redirect_add(ri, sn_port['fixed_ips'][0]15:42
*** jorgem has joined #openstack-neutron15:42
pc_mmestery: My understanding is though, that the apis were redone using WADL, but VPN was not included in that. I mentioned it to Edgar a few weeks ago. Didin't hear back from him.15:42
pc_mcarl_baldwin: ping15:43
*** dboik_ has joined #openstack-neutron15:43
*** dboik has quit IRC15:43
baolicarl_baldwin: with multiple prefix change, the caller assuming fixed_ip[0] for the gateway is not correct any more. Would it make sense to change the signature as _snat_redirect_add(self, ri, gateway, v4_cidr, sn_int) now that we have multiple ip_cidrs per port?15:43
carl_baldwinsalv-orlando: I imagine that one day we’ll want to deprecate and remove the baked-in IPAM code.  Without full migration, could we ever do that?15:43
carl_baldwinpc_m: pong15:43
baolihaleyb: ok, for v6, all the traffic should go to the centralized router. So redirect needs to programed for v6 as well?15:44
pc_mwondering if you had time later today to discuss some ideas I had about refactoring the *aaS code.15:44
*** nlahouti has joined #openstack-neutron15:44
pc_mcarl_baldwin: ^^15:44
openstackgerritRob Pothier proposed openstack/neutron: Add MTU selection to ML2  https://review.openstack.org/15631815:45
*** moha_hunt has quit IRC15:46
carl_baldwinpc_m: I might have some this afternoon.15:46
pc_mcarl_baldwin: cool. Let me know if you get some time.15:46
carl_baldwinpc_m: will do15:47
*** ajo has quit IRC15:48
*** ChuckC_ has quit IRC15:49
*** Swami has joined #openstack-neutron15:49
salv-orlandocarl_baldwin: decent point. Should we instead just ignore use cases when one wants to use multiple drivers? Is that something even remotely possible?15:49
salv-orlandoFrnakly for me that's the same question as asking when one would use multiple plugins15:49
*** thedodd has quit IRC15:49
salv-orlandowhen the answer is yes - but then you'll probably have different neutron instances too15:49
*** Miouge has joined #openstack-neutron15:50
*** trinaths has joined #openstack-neutron15:51
*** armax has joined #openstack-neutron15:52
*** leenheer has joined #openstack-neutron15:52
pavel_bondarcarl_baldwin, salv-orlando: just to confirm, does any new tasks goes into my queue in terms of integration testing? Right now I am already debugging and fixing bugs in my code based on failed tests15:52
carl_baldwinsalv-orlando: Not sure.  We may want more feedback about multiple drivers use cases.15:53
*** trinaths1 has joined #openstack-neutron15:53
*** ChuckC has joined #openstack-neutron15:53
salv-orlandopavel_bondar: I think there's nothing more for you ;)15:53
pavel_bondarsalv-orlando: thanks:)15:53
*** logan2 has joined #openstack-neutron15:54
*** marun has joined #openstack-neutron15:54
salv-orlandocarl_baldwin: my opinion is that probably we won't have a use case for multiple drivers unless we start going into the flavour rabbit hole15:54
salv-orlandocarl_baldwin: but what I know for sure is that we do not need them know15:54
carl_baldwinpavel_bondar: My question is whether existing tests have sufficient coverage to give us confidence in the new driver.  I’ll put up a patch to turn on the new driver by default so it gets exercised.15:54
salv-orlandoso I would like to work on this simple db conversion script.15:54
salv-orlandoI think the community would put that as a requirement15:54
carl_baldwinsalv-orlando: +1 for that.  I think that is a good direction to move.15:55
pavel_bondarcarl_baldwin: my patch already has new driver by default, so a lot of test failures 1,5 k reported by jenkins15:55
*** ajo has joined #openstack-neutron15:55
*** jpena has joined #openstack-neutron15:55
*** trinaths has quit IRC15:55
*** amotoki has quit IRC15:56
carl_baldwinpavel_bondar: ok, I misunderstood the scope of that patch.  It probably shouldn’t merge like that.15:56
*** thedodd has joined #openstack-neutron15:57
carl_baldwinTurning on the driver as default should eventually be taken out of it so that we can set up a non-voting job for it after it merges.15:57
carl_baldwin… so that can eventually graduate to a voting job.15:57
salv-orlandocarl_baldwin: I think pavel_bondar knows that... he's switching it on for testing15:57
haleybdane_leblanc: i guess the most likely answer is that there could be multiple fixed_ip's there15:57
*** yamamoto has quit IRC15:57
carl_baldwinsalv-orlando: understood and that is fine.15:57
pavel_bondarcarl_baldwin: it looks like right way to have new driver turned off by default in my patch and build new patch on top of that which just turn it on15:57
dane_leblanchaleyb: Should we add a check so that the redirection is only done for the IPv4 addresses?15:58
carl_baldwinpavel_bondar: Right, but you can leave it the way it is now for the time being.15:58
pavel_bondarcarl_baldwin: ok15:58
haleybbaoli: i will need to get back to you later today, need to run out15:58
*** natarajk has joined #openstack-neutron15:58
baolihaleyb: sure. And sorry for more questions: should a deployer use DVR at all in dual stack or v6 only setup?15:59
*** Sukhdev has joined #openstack-neutron15:59
*** Miouge has quit IRC15:59
*** amotoki has joined #openstack-neutron16:00
salv-orlandocarl_baldwin, pavel_bondar: it's time for me to jump into another meeting16:00
salv-orlandoping me on irc for anything else16:00
openstackgerritEd Bak proposed openstack/neutron: Break calls to get_devices_details_list into subsets  https://review.openstack.org/16359416:00
carl_baldwinsalv-orlando: Thanks.  I think we made good progress.  1)  We are testing the driver with Pavel’s patches.  2)  Will work on migration  3) Not supporting mutliple drivers for now.16:01
carl_baldwin4) Greenfield-only deployments in Kilo.16:01
salv-orlandocarl_baldwin: can yous end a summary update on the ML?16:01
carl_baldwinsalv-orlando: Yes, I will do that.16:02
carl_baldwinsalv-orlando: pavel_bondar:  Anything I missed?16:02
*** Miouge has joined #openstack-neutron16:02
*** mlavalle has quit IRC16:03
pavel_bondarcarl_baldwin: migration is planned for Liberty, nothing to do for kilo-3, right?16:03
carl_baldwinpavel_bondar: Right on migration.16:03
*** aranjan has joined #openstack-neutron16:04
pavel_bondarcarl_baldwin: ok, sounds good16:04
*** Miouge has quit IRC16:04
carl_baldwinpavel_bondar: though vendors can still provide migrations to their own drivers to support brown-field deployments.16:05
*** natarajk has quit IRC16:08
pavel_bondarcarl_baldwin: ok16:09
*** natarajk has joined #openstack-neutron16:09
*** aranjan has quit IRC16:10
*** amotoki has quit IRC16:10
*** PadminiB has quit IRC16:12
*** SridharG has left #openstack-neutron16:14
*** rwsu-afk is now known as rwsu16:15
mesterypc_m: Lets see if we can get those API docs done, I've had some people ask about them, woudl be good to get them converted an available, thanks!16:15
*** natarajk has quit IRC16:16
*** shivharis has quit IRC16:16
*** dannywilson has quit IRC16:16
*** gsagie has joined #openstack-neutron16:18
*** dannywilson has joined #openstack-neutron16:18
*** dannywilson has quit IRC16:19
*** vishwanathj has joined #openstack-neutron16:20
*** dannywilson has joined #openstack-neutron16:20
*** amotoki has joined #openstack-neutron16:22
*** criptoverso has quit IRC16:23
openstackgerritArmando Migliaccio proposed openstack/neutron: Break coupling between ML2 and L3 during delete operation  https://review.openstack.org/15547016:24
ihrachyshkaajo, hey16:24
ajohey ihrachyshka  ;)(16:24
ajo:-)16:24
ihrachyshkaajo, I think I know how to solve netns_cleanup issue with CLI args in tests without moving register call out of setup_conf()16:24
ajohmmm16:24
ajoihrachyshka how? :)16:24
ihrachyshkaajo, may you try locally to redefine setup_config for the test (that's defined in BaseTestCase) to call self.config_parse(args='--force') maybe?16:25
ihrachyshkathat one seems to be applicable for cases like that16:25
ihrachyshkawell, self.config_parse(args=['--force']) it is16:26
*** amotoki has quit IRC16:27
ajohmmm16:29
ajolet me look16:29
ajosounds interesting16:29
*** Rajeev has quit IRC16:30
*** sbfox has joined #openstack-neutron16:30
ajoahh16:31
ihrachyshkaajo, you may need to preregister options before super() call though16:31
ajoI could override setup_config16:31
ajoand insert my own args16:31
ihrachyshkaajo, right. it seems it's left exactly to be able to override16:31
*** sbfox has quit IRC16:31
*** sbfox has joined #openstack-neutron16:32
openstackgerritAssaf Muller proposed openstack/neutron: Move Unix domain socket helpers to a common place  https://review.openstack.org/16403216:32
openstackgerritAssaf Muller proposed openstack/neutron: Replace keepalived notifier bash script with Python ip monitor  https://review.openstack.org/12538416:32
ajovery nice ihrachyshka , thanks for the suggestion16:32
ihrachyshkaajo, does it work?16:32
amullercarl_baldwin: Sorry I had to fix an issue with a unit test in this patch https://review.openstack.org/16403216:32
*** gsagie has quit IRC16:32
ajoI will try soon ihrachyshka , I've been left with two monkeys alone, so I'm making a pause until later in the night16:33
ajobut I guess it will work16:33
*** dannywilson has quit IRC16:33
*** Longgeek has joined #openstack-neutron16:34
amullercarl_baldwin: thanks16:34
amullermarun: I split off the unix domain socket stuff https://review.openstack.org/16403216:34
*** stackmore has quit IRC16:35
*** aranjan has joined #openstack-neutron16:35
*** dboik_ has quit IRC16:36
*** dboik has joined #openstack-neutron16:37
*** vmtrooper has joined #openstack-neutron16:37
*** mlavalle has joined #openstack-neutron16:37
*** mlavalle has quit IRC16:38
*** sripriya has quit IRC16:39
*** melwitt has joined #openstack-neutron16:41
*** vmtrooper has quit IRC16:42
pc_mmestery: Sure. Are you expecting me to convert them? If so, I need some guidance, as I don't know what is needed exactly.16:42
openstackgerritSwaminathan Vasudevan proposed openstack/neutron: DVR qrouters not created on compute Node when vm added  https://review.openstack.org/14914316:43
*** sridhar_ram has joined #openstack-neutron16:44
*** Marga_ has joined #openstack-neutron16:46
*** YorikSar has joined #openstack-neutron16:47
*** matrohon has quit IRC16:49
*** dsneddon is now known as dsneddon_afk16:49
*** mestery has quit IRC16:50
*** bjornar_ has joined #openstack-neutron16:51
*** mestery has joined #openstack-neutron16:51
*** safchain has quit IRC16:51
*** gsagie has joined #openstack-neutron16:51
*** kerpan has quit IRC16:52
*** absubram has joined #openstack-neutron16:53
*** ygbo has quit IRC16:55
*** Longgeek has quit IRC16:56
*** harlowja has joined #openstack-neutron16:57
*** vishwana_ has joined #openstack-neutron16:57
*** sridhar_ram1 has joined #openstack-neutron16:58
*** sridhar_ram has quit IRC16:58
*** vishwanathj has quit IRC16:59
*** aranjan has quit IRC16:59
*** anshul has quit IRC16:59
*** TrevorV|Home has quit IRC17:01
*** ivar-lazzaro has joined #openstack-neutron17:02
*** moha_hunt has joined #openstack-neutron17:02
*** Networkn3rd has quit IRC17:03
*** jlanoux has quit IRC17:03
ajoihrachyshka, it works!! ;)17:04
ihrachyshkawootwoot17:04
*** gridinv_ has joined #openstack-neutron17:04
*** sripriya has joined #openstack-neutron17:05
*** _cjones_ has quit IRC17:05
*** padkrish has joined #openstack-neutron17:05
*** ivar-laz_ has joined #openstack-neutron17:05
*** Networkn3rd has joined #openstack-neutron17:06
*** ivar-lazzaro has quit IRC17:07
*** Networkn3rd has quit IRC17:09
*** leenheer has quit IRC17:10
*** sridhar_ram1 has quit IRC17:10
*** sridhar_ram has joined #openstack-neutron17:10
*** sbfox has quit IRC17:12
*** trinaths1 has quit IRC17:12
pc_mmarun: hi17:12
openstackgerritMiguel Angel Ajo proposed openstack/neutron: Add a netns-cleanup functional test  https://review.openstack.org/16386017:13
openstackgerritSandhya Dasu proposed openstack/neutron: Cisco UCS Manager ML2 Mechanism Driver  https://review.openstack.org/15543617:13
*** gsagie has quit IRC17:13
ajoihrachyshka, there it is ^ :)17:13
*** Networkn3rd has joined #openstack-neutron17:13
*** trinaths has joined #openstack-neutron17:13
ajoihrachyshka, ahh, I left a comment in the wrong place17:14
ajo1sec17:14
openstackgerritMiguel Angel Ajo proposed openstack/neutron: Add a netns-cleanup functional test  https://review.openstack.org/16386017:15
ajoihrachyshka shall be good now :)17:15
*** Networkn3rd has quit IRC17:15
*** Miouge has joined #openstack-neutron17:15
marunpc_m: hi17:16
ajoneed to go, have a good day/weekend ;)17:16
ajoI need.. ;)17:16
pc_mmarun: Have some time for some functional test related questions?17:16
*** ajo has quit IRC17:16
*** Trevor_V has joined #openstack-neutron17:16
marunpc_m: I'll do my best17:17
pc_mmarun: Yesterday, this merged upstream: https://review.openstack.org/#/c/164164/. As a result, it changed the path for a devstack file.17:17
pc_mmarun: To get VPN functional tests working, I did: https://review.openstack.org/#/c/164164/17:17
*** _cjones_ has joined #openstack-neutron17:18
pc_mmarun: Wondering though, the VPN gate hook is different than neutron (it was cloned from it a while back).17:18
pc_mmarun: Should we be updating the gate hook?17:18
pc_mmarun: Ref: https://github.com/openstack/neutron-vpnaas/blob/master/neutron_vpnaas/tests/contrib/gate_hook.sh17:18
pc_mRef https://github.com/openstack/neutron/blob/master/neutron/tests/contrib/gate_hook.sh17:19
*** Networkn3rd has joined #openstack-neutron17:19
*** ronis has joined #openstack-neutron17:19
*** tsg_ has joined #openstack-neutron17:19
pc_mmarun: Looks like some of the setup for neutron is moved into a configure_for_func_testing.sh.17:19
pc_mmarun: Do you think the simple hack I did was fine for now, or should we be reworking the gate hook?17:20
pc_mmarun: I ask, becuase FWaaS is doing the same thing... and will have the same problem17:20
*** Networkn3rd has quit IRC17:21
openstackgerritHenry Gessau proposed openstack/neutron: Remove vendor entry point  https://review.openstack.org/16427017:21
HenryGarmax: mestery: ^^17:21
*** dsneddon_afk is now known as dsneddon17:22
*** kawa2014 has quit IRC17:22
marunpc_m: looking17:24
marunpc_m: I think your hack is fine for now17:24
HenryGmarun: Me next :) Question about the api job, and adding an API test17:25
marunpc_m: The configure script is intended to ensure that a developer can reproduce the same env the gate will see17:25
marunpc_m: i.e. install db's and the functional rootwrap filter17:26
*** Networkn3rd has joined #openstack-neutron17:26
*** Networkn3rd has quit IRC17:26
marunpc_m: so it has a different purpose from just getting the gate to work17:26
*** Networkn3rd has joined #openstack-neutron17:26
marunHenryG: shoot17:26
pc_mmarun: Thanks. just didn't know if we needed to refactor that yet. Also, I have a commit for the post_gate_hook.sh that is waiting to be used.17:26
pc_mRef: https://review.openstack.org/#/c/154213/17:27
*** numan has quit IRC17:27
*** trinaths has quit IRC17:27
*** gsagie has joined #openstack-neutron17:27
pc_mmarun: Wondering if I should rebase and push it again or if we still need to wait for some other dependency (and if so, which)17:27
marunpc_m: I have patches up to try to refactor run-tox.sh for reuse17:27
marunpc_m: they haven't received any infra attention thus far, so we're stuck with what we have for now17:28
marunpc_m: https://review.openstack.org/#/c/161414/17:28
openstackgerritZhang Hua proposed openstack/neutron-vpnaas: Functional tests of ipsec strongswan vpnaas driver  https://review.openstack.org/15856017:28
openstackgerritSandhya Dasu proposed openstack/neutron: Cisco UCS Manager ML2 Mechanism Driver  https://review.openstack.org/15543617:28
openstackgerritZhang Hua proposed openstack/neutron-vpnaas: IPsec strongSwan driver implemention  https://review.openstack.org/14439117:28
marunpc_m: and https://review.openstack.org/#/c/154131/17:28
pc_mmarun: Thanks, I'll keep an eye on them.17:28
HenryGmarun: Is the api job broken and is there a fix in the works (or do you need help)?17:29
*** natarajk has joined #openstack-neutron17:30
HenryGmarun: And, can I add an API test now or are we not there yet?17:30
marunHenryG: we're not there yet.  I was talking to mtreinish yesterday about freezing neutron api test development in tempest17:31
pc_mmarun: So does that mean we need to hold off on moving API tests to the *aaS repos?17:31
*** jistr has quit IRC17:31
marunpc_m: we can move and run them all we want, but until development is frozen in tempest we can only copy them in, not modify them or add new ones.17:32
*** achanda has joined #openstack-neutron17:33
marunHenryG: The api job should be working, but patches that aren't rebased against master might be failing.17:33
pc_mmarun: OK, so I should probably wait.17:33
marunpc_m: We're copying all neutron tests, including *aas, and once we take over maintenance in the neutron repo it will be time to look at migrating the *aas tests into their respective repos.17:34
*** s3wong has joined #openstack-neutron17:34
*** tflynn has joined #openstack-neutron17:34
openstackgerritSandhya Dasu proposed openstack/neutron: Cisco UCS Manager ML2 Mechanism Driver  https://review.openstack.org/15543617:34
pc_mmarun: OK. Let me know when we can move VPN tests over and I can help out.17:34
*** saju_m has joined #openstack-neutron17:35
*** gridinv_ has quit IRC17:36
marunpc_m: will o17:36
marundo17:36
pc_mmarun: thanks for the useful info!17:37
*** jorgem has quit IRC17:37
*** vishwana_ has quit IRC17:37
*** vishwanathj has joined #openstack-neutron17:38
*** adelinatuvenie has quit IRC17:40
*** Marga_ has quit IRC17:40
*** Marga_ has joined #openstack-neutron17:41
*** bjornar_ has quit IRC17:42
*** VijayB_ has joined #openstack-neutron17:44
*** jorgem has joined #openstack-neutron17:44
*** VijayB_ has quit IRC17:44
*** VijayB_ has joined #openstack-neutron17:45
*** ChuckC has quit IRC17:45
*** Marga_ has quit IRC17:46
*** prad has joined #openstack-neutron17:47
*** signed8bit has joined #openstack-neutron17:48
*** jobewan has joined #openstack-neutron17:49
*** Sukhdev has quit IRC17:52
*** Longgeek has joined #openstack-neutron17:53
*** sudipto has joined #openstack-neutron17:54
openstackgerritMerged openstack/neutron: Refactor _remove_unused_security_group_info  https://review.openstack.org/15251217:58
*** jpena has quit IRC17:59
*** rpothier has joined #openstack-neutron17:59
*** sripriya has quit IRC18:00
*** sripriya has joined #openstack-neutron18:00
*** alexpilotti has joined #openstack-neutron18:00
*** bandarji has joined #openstack-neutron18:00
openstackgerritSam Betts proposed openstack/neutron: Add an internal DHCPv6 client for Prefix Delegation  https://review.openstack.org/16138118:01
*** alexpilotti has quit IRC18:01
*** alexpilotti has joined #openstack-neutron18:02
*** sudipto has quit IRC18:03
*** natarajk1 has joined #openstack-neutron18:04
*** natarajk has quit IRC18:05
*** ajmiller has joined #openstack-neutron18:05
*** leenheer has joined #openstack-neutron18:06
openstackgerritAngela Smith proposed openstack/neutron: Add L3 router plugin shim for Brocade MLX  https://review.openstack.org/16148918:07
openstackgerritAngela Smith proposed openstack/neutron: Add ML2 VLAN mechanism driver for Brocade MLX and ICX switches.  https://review.openstack.org/15470918:07
openstackgerritRyan Tidwell proposed openstack/neutron: Basic subnetpool CRUD  https://review.openstack.org/14869818:07
*** shivharis has joined #openstack-neutron18:09
shivharisarmax: hi18:10
openstackgerritMaru Newby proposed openstack/neutron: Set TEMPEST_CONFIG_DIR in the api tox env  https://review.openstack.org/16429618:19
*** fandi has joined #openstack-neutron18:20
*** fandi has quit IRC18:21
*** sbfox has joined #openstack-neutron18:21
*** Longgeek has quit IRC18:22
*** rpothier has quit IRC18:23
openstackgerritJuergen Brendel proposed openstack/neutron: ARP spoofing patch: Low level ebtables integration  https://review.openstack.org/14113018:23
*** sripriya has quit IRC18:26
*** vmtrooper has joined #openstack-neutron18:26
*** sripriya has joined #openstack-neutron18:26
*** pal_ has quit IRC18:29
*** Marga_ has joined #openstack-neutron18:29
*** vmtrooper has quit IRC18:31
openstackgerritAssaf Muller proposed openstack/neutron: Move Unix domain socket helpers to a common place  https://review.openstack.org/16403218:31
openstackgerritAssaf Muller proposed openstack/neutron: Replace keepalived notifier bash script with Python ip monitor  https://review.openstack.org/12538418:31
*** mlavalle has joined #openstack-neutron18:33
*** Sukhdev has joined #openstack-neutron18:35
openstackgerritRyan Tidwell proposed openstack/neutron: Basic subnetpool CRUD  https://review.openstack.org/14869818:37
pc_mmestery: ping18:38
*** wpward has joined #openstack-neutron18:38
*** wpward has left #openstack-neutron18:39
haleybdane_leblanc, baoli: sorry for bugging-out, was out of office.  But to answer your question we do need to call _snat_redirect_add() for IPv6 addresses because it calls 'ip rule add...' to make sure packets get to the router18:40
*** apuimedo has quit IRC18:41
*** drjones has joined #openstack-neutron18:42
*** shaohe_feng1 has quit IRC18:42
*** shaohe_feng1 has joined #openstack-neutron18:43
*** _cjones_ has quit IRC18:43
*** natarajk has joined #openstack-neutron18:43
*** natarajk1 has quit IRC18:44
*** VijayB_ has quit IRC18:44
HenryGmarun: I think the api job is broken. Sorry I can't offer better info because I can't figure out why.18:44
HenryGmarun: It's doing the "test run didn't actually run any tests" thing again.18:44
*** ijw has joined #openstack-neutron18:45
HenryGmarun: This patch https://review.openstack.org/156670 was just rebased and the api job exhibits the issue.18:45
marunHenryG: it does look broken after all18:46
marunHenryG: changes to tempest-lib, most likely18:46
marunHenryG: I'm thinking we're going to have to pin tempest-lib to avoid accidental breakage18:47
openstackgerritRyan Tidwell proposed openstack/neutron: Basic subnetpool CRUD  https://review.openstack.org/14869818:49
openstackgerritMaru Newby proposed openstack/neutron: Update api tests from tempest  https://review.openstack.org/16430118:49
HenryGmarun: makes sense to me, I thought the pin had already been done18:49
*** drjones has quit IRC18:49
marunHenryG: We talked about it, but we can't pin anything in the requirements file if it's not pinned globally18:49
*** _cjones_ has joined #openstack-neutron18:50
marunHenryG: Our only option until that changes is to pin in tox.ini18:50
*** saju_m has quit IRC18:53
openstackgerritAssaf Muller proposed openstack/neutron: Introduce RPC tests  https://review.openstack.org/16281118:54
*** apuimedo has joined #openstack-neutron18:55
*** ivar-laz_ has quit IRC18:56
*** lykinsbd has joined #openstack-neutron18:56
*** liusheng has quit IRC18:56
*** ivar-lazzaro has joined #openstack-neutron18:56
*** liusheng has joined #openstack-neutron18:57
*** rpothier has joined #openstack-neutron18:57
*** ivar-laz_ has joined #openstack-neutron18:58
HenryGmarun: so every time you copy the tests from tempest, also pin tempest-lib in tox.ini?18:58
*** ivar-laz_ has quit IRC18:58
marunHenryG: I think so, yeah.18:59
HenryGmarun: Should I expect an update to https://review.openstack.org/164301 ?18:59
*** ivar-laz_ has joined #openstack-neutron19:00
marunHenryG: yes19:01
*** ivar-lazzaro has quit IRC19:01
baolihaleyb: thanks for coming back to us. Does that mean _snat_redirect_add() should be called once for v4, and once for v6, in dual stack case?19:03
marunHenryG: Although, I don't think that tempest-lib could have broken us :/19:03
marunHenryG: There hasn't been a new release from pypi19:03
*** VijayB has joined #openstack-neutron19:03
HenryGmarun: hmmm19:03
HenryGmarun: I poked around a little in the logs looking for an import failure but didn't see one. Have you found the cause of the breakage?19:04
marunHenryG: I'll revert back to master and see if I can reproduce19:05
*** bobmel_ has joined #openstack-neutron19:06
haleybbaoli: i think it is now, 'for p in ri.internal_ports'19:06
marunHenryG: is there some magic to getting tox to spit out errors?19:07
marunHenryG: I usually just revert to using testtools.run19:07
*** rushil has quit IRC19:07
*** rpothier has quit IRC19:07
HenryGmarun: I do the same :(19:07
*** rpothier has joined #openstack-neutron19:08
*** bobmel has quit IRC19:09
*** rushil has joined #openstack-neutron19:11
openstackgerritMerged openstack/neutron: Move internal port processing to router classes  https://review.openstack.org/15451319:11
*** thedodd has quit IRC19:12
*** ivar-laz_ has quit IRC19:16
*** ivar-lazzaro has joined #openstack-neutron19:17
*** gsagie has quit IRC19:23
*** rkukura_ has joined #openstack-neutron19:24
*** Swami has quit IRC19:24
*** rkukura has quit IRC19:25
*** rkukura_ is now known as rkukura19:25
*** amuller has quit IRC19:26
*** sridhar_ram has quit IRC19:26
*** yfauser has joined #openstack-neutron19:27
*** yfauser has left #openstack-neutron19:29
baolihaleyb, ok, I see. thanks.19:29
*** aepifanov has joined #openstack-neutron19:31
*** dannywilson has joined #openstack-neutron19:33
openstackgerritZengfa Gao proposed openstack/python-neutronclient: Add Neutron subnetpool API  https://review.openstack.org/15961819:34
openstackgerritZengfa Gao proposed openstack/python-neutronclient: Add Neutron subnetpool API  https://review.openstack.org/15961819:36
*** rpothier has quit IRC19:37
*** dannywilson has quit IRC19:38
*** sballe__ has joined #openstack-neutron19:40
*** radez is now known as radez_g0n319:40
*** imcsk8 has quit IRC19:42
*** imcsk8 has joined #openstack-neutron19:42
*** s3wong has quit IRC19:43
*** sballe__ is now known as sballe19:44
*** padkrish_ has joined #openstack-neutron19:44
*** padkrish has quit IRC19:45
*** padkrish has joined #openstack-neutron19:46
*** s3wong has joined #openstack-neutron19:46
*** ajo has joined #openstack-neutron19:46
*** padkrish_ has quit IRC19:49
*** Longgeek has joined #openstack-neutron19:50
*** padkrish has quit IRC19:50
*** padkrish has joined #openstack-neutron19:51
openstackgerritAbhishek Raut proposed openstack/neutron: Cisco Nexus1000V ML2 Mechanism Driver  https://review.openstack.org/15724319:52
marunHenryG, armax: it's an in-tree logging change :/19:52
pc_marmax: ping19:52
marunwe should make this job vote asap, we're already getting broken19:52
armaxmarun: we knew that the oslo log change was going to cause a bit of a bumpy road ahead19:53
dougwigmarun: we pinned tempest via tox.ini in neutron-lbaas19:53
armaxmarun: or am I missing your point?19:53
*** thedodd has joined #openstack-neutron19:53
marunarmax: fine that we have breakage, but if the job were voting the proposer would have had to fix it instead of us19:54
armaxfair enough19:54
*** padkrish_ has joined #openstack-neutron19:54
armaxmarun: I guess I am to blame as I approve the patch19:54
marunarmax: no blame for a non-voting job :)19:55
armaxmarun: well, there should be :)19:55
*** padkrish has quit IRC19:56
maruna voting job? definitely19:56
openstackgerritZengfa Gao proposed openstack/python-neutronclient: Add Neutron subnetpool API  https://review.openstack.org/15961819:56
armaxmarun: non-voting jobs should be given the same attention as the voting ones…I have been preaching this for the dvr job for a while :)19:56
marunonce we get it working we should propose that19:56
marunarmax: can't disagree more19:56
marunarmax: if it's not voting, it's essentially optional to pay attention19:56
marunif you care, great.  if you don't, for whatever reason, that's fine too.19:56
armaxit escaped me looking at the api job as it’s still going through the teething phase19:56
*** ajmiller_ has joined #openstack-neutron19:56
marunarmax: not sure if should make it voting ahead of a tempest-lib pin19:57
marunarmax: we're going to have to fight to pin, from what I'm hearing.19:57
armaxmarun: indeed, but that doesn’t mean we should stop looking at it while that happens19:57
openstackgerritSumit Naiksatam proposed openstack/neutron-fwaas: Provide hooks for FWaaS repo functional gate  https://review.openstack.org/16147919:58
*** padkrish_ has quit IRC19:59
*** padkrish has joined #openstack-neutron19:59
*** ajmiller has quit IRC20:00
*** rpothier has joined #openstack-neutron20:00
marunHenryG, armax: https://review.openstack.org/#/c/164301/ should fix the breakage, since tempest has updated to reflect the change in oslo_log20:00
armaxmarun: thanks20:01
HenryGmarun: on it20:01
armaxI was looking at https://review.openstack.org/#/c/159638/20:01
armaxbut I couldn’t see where we missed the oslo log change20:01
armaxseems that it was taken care of here: https://review.openstack.org/#/c/159638/15/neutron/tests/api/contrib/clients.py20:02
dougwigarmax: if it doesn't vote, i don't even notice it.  broken third-party CI taught me that.20:02
marunarmax: ah, he was manually updating neutron/tests/tempest :(20:02
marunarmax: missing policy doc, my fault.20:02
armaxdougwig: that’s fine, but that’s not my practice :)20:02
openstackgerritpritesh proposed openstack/python-neutronclient: Adding VLAN Transparency support to neutronclient  https://review.openstack.org/16204020:03
armaxmarun: ah shoot20:03
marunarmax: oh, wait20:03
marunarmax: no, that is our copy of clients.20:03
dougwigarmax: but we already knew that you are exceptional.  :)20:03
marunarmax: but there are lots of updates to neutron/tests/tempest in that change20:03
armaxdougwig: nah20:03
armaxmarun: ok, so it’s not the entirety of neutron/tempest/api that’s read-only?20:04
marunarmax: it is the entirety20:05
marunarmax: neutron/tests/tempest should not be modified by hand20:05
* armax looks at the copy script20:05
openstackgerritZengfa Gao proposed openstack/python-neutronclient: Add Neutron subnetpool API  https://review.openstack.org/15961820:05
marunarmax: it's neutron/tests/api/contrib that contains customizations20:05
armaxmarun: ok got it, writing on the wall next to my screen20:05
armaxmarun: so it’s everything in here: https://github.com/openstack/neutron/blob/master/tools/copy_api_tests_from_tempest.sh#L2520:06
marunarmax: It's my fault, there should have been docs and dissemination before we merged.20:06
*** achanda has quit IRC20:06
marunarmax: those are the paths we copy from tempest20:06
marunarmax: the only path to know in neutron is neutron/tests/tempest20:06
armaxno-one’s fault, these things take time to sink in20:06
armaxbut those paths are mirrored to neutron, aren’t they?20:07
*** rpothier has quit IRC20:07
marunarmax: throwing it in the tree with only word-of-mouth instruction wasn't smart though :/20:07
marunarmax: those paths are copied to neutron/tests/tempest, yeah.20:07
armaxit was on the commit message, iirc, even if it was in the devref20:07
armaxI doubt that ihrachyshka, or anyone else would’ve had the time to read it and assimilate it20:08
armaxit’s worth doing it for sure20:08
marunarmax: it would have complicated things if he had known, honestly.20:08
armaxbut that wouldn’t have prevented the accidentanl breakage20:08
openstackgerritCarl Baldwin proposed openstack/neutron: Introduce External IPAM Interface  https://review.openstack.org/13433920:08
pc_marmax: Have some questions for you on callback mechanism. Have a few mins?20:08
marunarmax: we'd probably have had to copy from tempest and modify everything else in one change20:08
armaxI was going to lunch :(20:08
*** yamahata has joined #openstack-neutron20:09
armaxor at least that was my hope20:09
armax:)20:09
armaxpc_m: shoot20:09
pc_marmax: Do the callback methods need to be standalone methods, or can they be member methods?20:09
armaxmarun: yea, a bit of a double hop20:09
pc_marmax: talking abotu neutron/callbacks/*20:10
*** achanda has joined #openstack-neutron20:11
armaxpc_m: at the moment, we weakref the callback, if I recall correctly that doesn’t work for class methods20:11
pc_marmax:  Giving it a try (as an experiment) and getting a weakly-referenced object no longer exists.20:11
armaxpc_m: there’s an imporvement in weakref to overcome this limitation but I think it’s only py320:11
pc_marmax: That explains it.20:11
armaxpc_m: we could get rid of the weakref, but I was hesitant in case that caused wierd memory leaks we so hard fought against20:12
pc_marmax: Any idea of a workaround? I was thinking I could use the trigger to get to the instance, but it's a bit round about.20:12
pc_marmax: Yeah I saw some of the discussion, but it was over my head a bit :)20:13
*** pritesh has joined #openstack-neutron20:13
armaxpc_m: it depdends on the context20:14
armaxpc_m: some of the callback examples that I put in place needed module functions anyway to avoid causing a potential breakage to vendor plugins20:14
armaxpc_m: what are you up to?20:15
pc_marmax: Example I'm trying is metadata proxy... for router create/delete callback20:15
*** vmtrooper has joined #openstack-neutron20:15
armaxyou got a working code that I could look at?20:15
pc_marmax: Plan is to change the *aaS callback mechanism from using event_observer to callbacks.20:15
armaxpc_m: right, I thought we mentioned we were going to let the l3 restructuring bp complete and worry about later?20:16
pc_mI have it in place, but used self.after_router_added in the subscribe.20:16
pc_marmax: Yeah, I had a bit of time and this part is pretty stable. Wanted to clean up VPN in this area.20:17
pc_marmax: I could access the metadata proxy via the trigger (l3_agent).20:18
*** Longgeek has quit IRC20:18
armaxpc_m: ok, tag me on the review and I’d be happy to have a look when I can; I also wanted to do the work, but I was spending my energy moving the other stuff forward, this can probably wait a bit20:18
armaxpc_m: that would be what the trigger is for20:18
*** VijayB has quit IRC20:18
armaxpc_m: but I’d need to see the code to really get what you mean20:18
pc_marmax: Yeah, I'm hankering to clean up VPNaaS and the whole advanced_services stuff.20:19
armaxpc_m: ok20:19
pc_mI have it in my workspace. I'll get it going and do a WIP to view...20:19
*** vmtrooper has quit IRC20:20
*** sbfox has quit IRC20:22
*** achanda has quit IRC20:25
openstackgerritZengfa Gao proposed openstack/python-neutronclient: Add Neutron subnetpool API  https://review.openstack.org/15961820:26
openstackgerritCarl Baldwin proposed openstack/neutron: Move external port processing to router classes  https://review.openstack.org/15849520:26
*** Trevor__V has joined #openstack-neutron20:35
*** leenheer has left #openstack-neutron20:35
*** Sukhdev has quit IRC20:35
*** alexpilotti has quit IRC20:37
*** Trevor_V has quit IRC20:38
*** padkrish has quit IRC20:38
openstackgerritZengfa Gao proposed openstack/python-neutronclient: Add Neutron subnetpool API  https://review.openstack.org/15961820:39
*** rushil has quit IRC20:42
marunHenryG, armax: mtreinish has agreed to allow us to redirect api test development to neutron so long as we're doing the policing.20:44
maruni.e. we'll need to have to track the tempest queue and block changes to the neutron api tests20:44
*** irenab_ has joined #openstack-neutron20:45
*** Trevor__V has quit IRC20:45
kevinbentonmestery: ICMP ECHO REQUEST20:45
marunWe'll need policy defined in devref before we start enforcement, but we're not blocked by anything external at least.20:46
HenryGmarun: why can't we delete api/network from tempest?20:47
*** Swami has joined #openstack-neutron20:47
*** irenab has quit IRC20:47
*** irenab_ is now known as irenab20:47
marunHenryG: we have some hurdles to cross first20:49
marunHenryG: https://wiki.openstack.org/wiki/QA/Tempest-test-removal20:49
marunHenryG: We need to ensure the in-tree tests can target stable branches, and that defcore/revstack have a way of consuming the tests20:50
marunHenryG: again, I need to publish a plan so this all makes sense20:50
* marun isn't a very good project manager20:50
marunrevstack -> refstack20:51
*** sridhar_ram has joined #openstack-neutron20:51
HenryGmarun: You are doing a great job. We will trudge through this becuase we need it.20:52
openstackgerritTerry Wilson proposed openstack/neutron: Add native OVSDB implmentation of OVSDB API  https://review.openstack.org/15276120:52
* HenryG pokes around in gerrit for a good way to watch changes to networking api tests in tempest20:53
marunHenryG: It's not actually too bad...20:55
marunhttps://review.openstack.org/#/q/project:openstack/tempest+file:%255Etempest/api/network/.*+status:open,n,z20:55
marunHenryG: I'll make sure to include this query in the proposed documentation20:55
marunHenryG: Most of the changes are ancient anyway20:55
HenryGmarun: nice, thanks20:56
shivharisarmax: h20:57
shivharisarmax: hi20:57
marunHenryG: gerrit queries are pretty neat, though I never use them often enough to not have to consult the docs.20:57
*** achanda has joined #openstack-neutron20:57
HenryGmarun: yes, the file query especially always has me trying variations before i get it right20:58
openstackgerritRyan Tidwell proposed openstack/neutron: Introduce External IPAM Interface  https://review.openstack.org/13433920:58
openstackgerritRyan Tidwell proposed openstack/neutron: Basic subnetpool CRUD  https://review.openstack.org/14869820:58
marunHenryG: my brain always confuses glob vs actual regex20:58
marunHenryG: gerrit is using actual regex, fwiw20:58
*** melwitt has quit IRC20:58
*** pradip_vedams has quit IRC20:59
*** apuimedo has quit IRC20:59
marunHenryG: and seems to have an implicit $20:59
*** pradip_vedams has joined #openstack-neutron20:59
marunHenryG: such that submatches aren't performed if a wildcard isn't on the end20:59
HenryGmarun: yes, and it needs to start with ^20:59
*** melwitt has joined #openstack-neutron21:00
shivharismarun: hi21:00
marunshivharis: hi21:00
*** harlowja has quit IRC21:00
*** apuimedo has joined #openstack-neutron21:00
*** melwitt has quit IRC21:00
shivharismarun: i am probably seeing some new behavior in neutron, maybe i am wrong wanted to ask you21:00
*** apuimedo has quit IRC21:00
*** apuimedo has joined #openstack-neutron21:00
shivharismarun: when i start neutron, by run the neutron-server, i do a ps and see 17 processes is that a new change?21:01
shivharisi mean 17 neutron-server process, i make sure there were none before i start21:02
shivharismaybe just my issue?21:02
*** s3wong has quit IRC21:02
*** s3wong_ has joined #openstack-neutron21:03
*** irenab has quit IRC21:03
*** sridhar_ram has quit IRC21:03
kevinbentonrossella_s: ping21:03
*** sridhar_ram has joined #openstack-neutron21:03
*** raildo has quit IRC21:05
*** harlowja has joined #openstack-neutron21:06
marunshivharis: look for api_workers and rpc_workers in your config21:06
marunshivharis: by default iirc it will devote a process per core21:06
*** melwitt has joined #openstack-neutron21:07
shivharismarun: i took the drastic step of rebooting my machine - it is still coming up21:07
marunshivharis: I'm afraid I have to run21:08
marunshivharis: you'll need to look at your neutron.conf to fix if that's not what you want21:09
marunapi_workers21:09
*** ajo has quit IRC21:09
shivharismarun: np, you given me a good pointer21:09
*** gridinv_ has joined #openstack-neutron21:09
shivharismarun: thanks21:10
*** apuimedo has quit IRC21:10
*** marun has quit IRC21:14
*** diegows has quit IRC21:14
*** csoukup has quit IRC21:15
*** pc_m has quit IRC21:16
*** Networkn3rd has quit IRC21:18
HenryGarmax: ping, https://review.openstack.org/16304021:19
*** ajo has joined #openstack-neutron21:23
*** VijayB has joined #openstack-neutron21:24
openstackgerritRyan Tidwell proposed openstack/neutron: Basic subnetpool CRUD  https://review.openstack.org/14869821:25
*** jorgem has quit IRC21:26
*** achanda has quit IRC21:27
*** zui_m has joined #openstack-neutron21:28
*** sripriya has quit IRC21:30
*** ronis has quit IRC21:31
*** kbringard has quit IRC21:33
*** achanda has joined #openstack-neutron21:33
*** jgrimm_ has joined #openstack-neutron21:34
*** lascii is now known as alaski21:35
*** jgrimm- has joined #openstack-neutron21:36
*** jgrimm has quit IRC21:36
armaxHenryG: pong21:38
*** jgrimm_ has quit IRC21:39
*** ijw has quit IRC21:40
*** tiswanso has quit IRC21:41
*** s3wong_ has quit IRC21:42
*** sridhar_ram has quit IRC21:42
*** marun has joined #openstack-neutron21:44
*** sridhar_ram has joined #openstack-neutron21:44
*** sambetts has quit IRC21:45
HenryGarmax: thanks21:45
*** dboik has quit IRC21:46
*** prad has quit IRC21:46
*** sambetts has joined #openstack-neutron21:47
*** s3wong has joined #openstack-neutron21:48
*** Zengfa_ has quit IRC21:59
*** shivharis has quit IRC21:59
*** shivharis has joined #openstack-neutron21:59
*** Miouge has quit IRC22:00
*** rwsu has quit IRC22:00
*** s3wong has quit IRC22:00
*** carl_baldwin has quit IRC22:01
*** shivharis has quit IRC22:01
*** SumitNaiksatam has joined #openstack-neutron22:01
*** shivharis has joined #openstack-neutron22:01
*** carl_baldwin has joined #openstack-neutron22:02
*** s3wong has joined #openstack-neutron22:02
*** Longgeek has joined #openstack-neutron22:03
*** vmtrooper has joined #openstack-neutron22:04
*** harlowja has quit IRC22:04
*** tiswanso has joined #openstack-neutron22:04
*** harlowja has joined #openstack-neutron22:06
*** harlowja has quit IRC22:06
*** harlowja has joined #openstack-neutron22:06
ajohttps://twitter.com/DesignUXUI/status/576432203560685568/photo/122:08
ajoROFL22:08
*** vmtrooper has quit IRC22:08
ajoXD22:08
ajosorry for the offtopic22:08
ajoit's reality....22:09
openstackgerritKevin Benton proposed openstack/neutron: Reload DHCP interface when its port is updated  https://review.openstack.org/16375822:10
kevinbentonajo: ha!22:11
ajox)22:11
*** killer_prince has quit IRC22:11
*** mriedem has joined #openstack-neutron22:14
mriedemnew gate bug https://bugs.launchpad.net/neutron/+bug/143206522:14
openstackLaunchpad bug 1432065 in neutron "test_ports fails with "StaleDataError: UPDATE statement on table 'ports' expected to update 1 row(s); 0 were matched."" [Undecided,New]22:14
mriedemin last 24 hours22:14
mriedemhttps://review.openstack.org/#/c/149261/ looks related22:14
*** killer_prince has joined #openstack-neutron22:14
*** csoukup has joined #openstack-neutron22:15
*** irenab has joined #openstack-neutron22:15
*** killer_prince is now known as lazy_prince22:15
*** melwitt has quit IRC22:15
*** melwitt has joined #openstack-neutron22:16
*** gridinv_ has quit IRC22:16
*** gridinv_ has joined #openstack-neutron22:17
*** moshele has joined #openstack-neutron22:18
*** ihrachyshka has quit IRC22:19
*** moshele has quit IRC22:19
*** melwitt has quit IRC22:19
*** melwitt has joined #openstack-neutron22:19
*** gridinv_ has quit IRC22:21
*** sgran has quit IRC22:25
*** Longgeek has quit IRC22:25
salv-orlandomriedem: I and armax approved it because if we didn't we would have never known what monster it did uncover.22:27
salv-orlandobut we were aware we might be doing something insane22:27
armaxas a matter of fact we did it fearing the worst22:27
openstackgerritRyan Tidwell proposed openstack/neutron: WIP Subnet allocation from a subnet pool  https://review.openstack.org/15759722:28
*** melwitt has quit IRC22:28
salv-orlandoI mean if the gate is super broken because of this we can revert - otherwise I'd just look into the failures and identify the root cause of the race22:28
kevinbentonsalv-orlando, mriedem, armax: keep in mind this merged as well22:33
kevinbentonhttps://review.openstack.org/#/c/160160/522:33
kevinbentonwhich was causing deadlocks before on the port table22:33
armaxsalv-orlando: let’s revert then22:33
*** rwsu has joined #openstack-neutron22:33
*** zhhuabj has quit IRC22:35
*** pradip_rawat has joined #openstack-neutron22:35
*** VijayB has quit IRC22:35
*** lykinsbd has quit IRC22:35
mriedemsalv-orlando: kevinbenton: armax: check the bug details, there are 3 traces in the logs, the first and most prevalant doesn't appear to be causing failures, but it's super ugly22:36
mriedemthe 3rd and final trace with the db deadlock when deleting the subnet is what's killing tempest on teardown22:36
salv-orlandomriedem: doing that. Armax, do you think we might be happy with this trace reported by mriedem, and use it to work on a fix?22:36
armaxlet’s go ahead and revert, to be safe22:36
salv-orlandook22:36
openstackgerritSalvatore Orlando proposed openstack/neutron: Revert "Set default of api_workers to number of CPUs"  https://review.openstack.org/16436322:37
armaxit’s the weekend, and we don’t want to leave anything broken22:37
salv-orlandoarmax: all your for a +222:37
mtreinisharmax: bah, what kinda attitude is that? :)22:37
armaxmtreinish: it’s friday!22:37
armax:)22:37
*** pradip_vedams has quit IRC22:38
*** pradip_rawat is now known as pradip_vedams22:38
*** tiswanso has quit IRC22:39
mriedemsalv-orlando: is https://review.openstack.org/#/c/164363/ supposed to be for the bug in question?22:39
mriedemif so, can we get that in the commit message?22:39
*** tiswanso has joined #openstack-neutron22:39
kevinbentonwe had an impressive number of merges within a 4 hour window that could have caused this :)22:40
armaxmriedem: you’re right I posted a comment22:40
armaxbut the commit message would be more polite22:40
*** sgran has joined #openstack-neutron22:42
HenryGarmax: you can edit the commit msg right there in the gerrit UI. Go ahead, I will +222:43
*** thedodd has quit IRC22:43
HenryGThen mriedem will be happy22:43
mriedemi would be happy22:44
mriedem:)22:44
*** ajo has quit IRC22:44
armaxcan I?22:45
armaxI can!22:45
openstackgerritArmando Migliaccio proposed openstack/neutron: Revert "Set default of api_workers to number of CPUs"  https://review.openstack.org/16436322:46
armaxHenryG: do22:46
armaxne22:46
HenryGarmax: +2, ship it22:47
armaxHenryG: well, sink it22:47
HenryGlol22:47
*** zhhuabj has joined #openstack-neutron22:47
armaxI always wonder why these happens always in the gate22:48
armaxand never prior to merging!22:48
armaxbah22:48
mtreinisharmax: it's just statistics :)22:48
armaxexistential question that is going to go answered on a Friday afternoon22:48
mriedemit's also friday the 13th22:49
armaxdoh22:49
mriedemso feel free to murder some teenagers at an abandoned summer camp22:49
armaxmriedem: that might be it22:49
mtreinishyou run 4 jobs twice on the commit, while we run a lot in the wider gate22:49
mtreinisharmax: use mysql://query:query@logstash.openstack.org/subunit2sql to find out how many :)22:49
shivharisi also was struggling with the api_workers set to maxcpus as default issue today, this really needs to be a configurable option using devstack22:50
openstackgerritSwaminathan Vasudevan proposed openstack/neutron: Adds DVR functional test for multi-external networks  https://review.openstack.org/16357322:50
shivharisfor now i will be happy if it is retracted22:50
mtreinishshivharis: it is22:50
mtreinishthere is an API_WORKERS flag22:51
mtreinishit should default to nproc/2 for everything though, there might just be a bug and it's not being set for a service22:51
shivharismtreinish: i saw this for other projects - not neutron, maybe i missed?22:51
shivharisi moved forward by add an iniset for default in devstack's neutron file22:52
mtreinishshivharis: if neutron is missing the use of API_WORKERS, that's a simple fix, just push a patch22:54
shivharismtreinish: will do tomorrow22:55
*** pritesh has quit IRC22:55
*** Miouge has joined #openstack-neutron22:55
*** markvoelker has quit IRC22:56
*** jobewan has quit IRC22:59
*** moha_hunt has quit IRC22:59
*** pradip_vedams has quit IRC22:59
*** pradip_vedams has joined #openstack-neutron23:00
kevinbentonarmax: are you working on https://review.openstack.org/#/c/155470/ ?23:01
kevinbentonarmax: py27 failures look related to the rebase23:01
armaxkevinbenton: looking23:03
armaxthanks for the heads-up23:03
*** apuimedo has joined #openstack-neutron23:03
openstackgerritMerged openstack/neutron-vpnaas: Fix functional test breakage from DevStack change  https://review.openstack.org/16416423:08
*** bandarji has quit IRC23:08
openstackgerritMerged openstack/neutron: ML2 cisco_nexus MD: sync config and models with vendor repo  https://review.openstack.org/16304023:08
*** VijayB_ has joined #openstack-neutron23:10
*** tiswanso has quit IRC23:11
openstackgerritMerged openstack/neutron: Remove vendor entry point  https://review.openstack.org/16427023:11
openstackgerritMerged openstack/neutron: Set TEMPEST_CONFIG_DIR in the api tox env  https://review.openstack.org/16429623:11
*** tiswanso has joined #openstack-neutron23:11
*** tiswanso has quit IRC23:15
*** sridhar_ram has quit IRC23:17
*** tsg_ has quit IRC23:19
zui_marmax:hi23:23
*** _cjones_ has quit IRC23:24
*** zui_m has quit IRC23:24
*** sripriya has joined #openstack-neutron23:24
*** Miouge has quit IRC23:24
*** SumitNaiksatam has quit IRC23:24
sripriyaarmax:hi23:25
openstackgerritIsaku Yamahata proposed openstack/neutron: internal callers of create_{network, subnet, port} ignore extension  https://review.openstack.org/12977323:28
*** david-lyle is now known as david-lyle_afk23:29
*** absubram has quit IRC23:29
openstackgerritMerged openstack/neutron: Add rootwrap daemon mode support  https://review.openstack.org/15880523:30
*** rwsu has quit IRC23:35
*** ajmiller_ has quit IRC23:37
*** rwsu has joined #openstack-neutron23:37
*** ChuckC has joined #openstack-neutron23:41
*** csoukup has quit IRC23:41
*** shwetaap has quit IRC23:43
openstackgerritMerged openstack/neutron: Brocade vendor code decomposition from neutron repo.  https://review.openstack.org/15238323:44
openstackgerritArmando Migliaccio proposed openstack/neutron: Break coupling between ML2 and L3 during delete operation  https://review.openstack.org/15547023:48
*** melwitt has joined #openstack-neutron23:49
armaxsripriya: pong23:49
sripriyaarmax: https://review.openstack.org/#/c/152383/ got merged. Can you please review https://review.openstack.org/#/c/159056/23:50
armaxwill do23:51
sripriyaarmax: Thanks much23:51
*** dimsum__ has quit IRC23:52
*** _cjones_ has joined #openstack-neutron23:53
*** vmtrooper has joined #openstack-neutron23:53
*** _cjones_ has quit IRC23:55
openstackgerritAbhishek Raut proposed openstack/neutron: Cisco Nexus1000V ML2 Mechanism Driver  https://review.openstack.org/15724323:55
*** dims_ has joined #openstack-neutron23:55
*** markvoelker has joined #openstack-neutron23:57
*** dims_ has quit IRC23:57
*** dims_ has joined #openstack-neutron23:57
*** vmtrooper has quit IRC23:57
*** thomasem has quit IRC23:59

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