[RFE] neutron_dynamic_routing project is bound to rpc driver closely

Bug #1611632 reported by Yang Yu
This bug affects 1 person
Affects Status Importance Assigned to Milestone

Bug Description

The interface https://github.com/openstack/neutron-dynamic-routing/blob/master/neutron_dynamic_routing/services/bgp/bgp_plugin.py is bound to rpc driver so closely, it is not suitable to extend the BGP driver which is agentless.

So we need to do following steps:

1> Refactor the code to support agentless driver
2> Using service_providers method to load the different driver
3> Deliver an overall agent driver interface so that some agent driver such as quagga and ryu are easy to be load.

For this refactor, I added followings in my patch https://review.openstack.org/#/c/349921/.

1> Add a new configuration file named neutron_dynamic_routing.conf to provider the service_providers configuration.
2> The neutron-bgp-dragent binary will invoke the neutron_dynamic_routing.conf file.
3> The default service_providers will be DYNAMIC_ROUTING:RPCDriver:neutron_dynamic_routing.services.bgp.service_drivers.rpc_bgp.BgpRpcPluginDriver:default
4> Now the bgp_plugin.py will be a unified the interface to invoking the dynamic routing, the rpc_bgp.py will be a agent driver and be responsible for interact with lower layer interface such as ryu/quagga.

Tags: rfe
Yang Yu (yuyangbj)
Changed in neutron:
assignee: nobody → Yang Yu (yuyangbj)
tags: added: l3-bgp
Changed in neutron:
importance: Undecided → Wishlist
Changed in neutron:
status: New → In Progress
Revision history for this message
Yang Yu (yuyangbj) wrote :
Yang Yu (yuyangbj)
description: updated
tags: added: rfe
removed: l3-bgp
Revision history for this message
OpenStack Infra (hudson-openstack) wrote : Related fix proposed to neutron-specs (master)

Related fix proposed to branch: master
Review: https://review.openstack.org/373029

Boden R (boden)
summary: - neutron_dynamic_routing project is bound to rpc driver closely
+ [RFE] neutron_dynamic_routing project is bound to rpc driver closely
Revision history for this message
vikram.choudhary (vikschw) wrote :

Request looks reasonable to me.. Hence, +1 from my end.

Revision history for this message
YAMAMOTO Takashi (yamamoto) wrote :

midonet want to use this if possible.
it currently provides the whole bgp plugin. [1]

[1] https://github.com/openstack/networking-midonet/blob/37b632f3bf3211512f5fda45623c2d302eef6a51/midonet/neutron/services/bgp/plugin.py

Revision history for this message
OpenStack Infra (hudson-openstack) wrote : Related fix merged to neutron-specs (master)

Reviewed: https://review.openstack.org/373029
Committed: https://git.openstack.org/cgit/openstack/neutron-specs/commit/?id=57f92340c9ba8a72431b4e57989d4831df385700
Submitter: Jenkins
Branch: master

commit 57f92340c9ba8a72431b4e57989d4831df385700
Author: yuyangbj <email address hidden>
Date: Tue Sep 20 16:11:55 2016 +0800

    Support agentless driver in neutron-dynamic-routing

    Some BGP driver is agentless, but the neutron-dynamic-routing is
    bound to RPC driver so closely. So the codes need to be refactored.

    Change-Id: Id134faa2d08369a7ad7aab1d353b83b1fbc4d48f
    Related-Bug: #1611632

Changed in neutron:
milestone: none → ocata-3
Changed in neutron:
milestone: ocata-3 → ocata-rc1
milestone: ocata-rc1 → pike-1
Changed in neutron:
milestone: pike-1 → pike-2
Revision history for this message
OpenStack Infra (hudson-openstack) wrote : Change abandoned on neutron-dynamic-routing (master)

Change abandoned by boden (<email address hidden>) on branch: master
Review: https://review.openstack.org/349921
Reason: we no longer need this support

Changed in neutron:
milestone: pike-2 → none
assignee: Yang Yu (yuyangbj) → nobody
status: In Progress → Incomplete
Revision history for this message
Launchpad Janitor (janitor) wrote :

[Expired for neutron because there has been no activity for 60 days.]

Changed in neutron:
status: Incomplete → Expired
To post a comment you must log in.
This report contains Public information  
Everyone can see this information.

Other bug subscribers

Remote bug watches

Bug watches keep track of this bug in other bug trackers.