openstack
/
neutron-dynamic-routing
Code Issues Proposed changes

BGP: remove unnecessary configuration setting 

By default oslo CFG sets the default value as None. There is no
need to specifically do this.

The patch also moves the RYU_BGP_SPEAKER_DRIVER to the devstack
settings file.

TrivialFix

Change-Id: Ic33871a0a42fa9a5e5af3158532ecdad9817e5ab
This commit is contained in:
Gary Kotton 2016-02-28 01:29:21 -08:00
parent a8270d82a9
commit 93307b2f27
3 changed files with 3 additions and 4 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

2
devstack/lib/bgp
View File

@ -1,5 +1,3 @@
RYU_BGP_SPEAKER_DRIVER="neutron.services.bgp.driver.ryu.driver.RyuBgpDriver"
function configure_bgp_service_plugin {
_neutron_service_plugin_class_add "bgp"
}

4
devstack/settings
View File

@ -3,4 +3,6 @@ L2_AGENT_EXTENSIONS=${L2_AGENT_EXTENSIONS:-}
#BGP binary and config information
AGENT_BGP_BINARY=${AGENT_BGP_BINARY:-"$NEUTRON_BIN_DIR/neutron-bgp-dragent"}
Q_BGP_DRAGENT_CONF_FILE=${Q_BGP_DRAGENT_CONF_FILE:-"$NEUTRON_CONF_DIR/bgp_dragent.ini"}
BGP_ROUTER_ID=${BGP_ROUTER_ID:-}
BGP_ROUTER_ID=${BGP_ROUTER_ID:-}
RYU_BGP_SPEAKER_DRIVER="neutron.services.bgp.driver.ryu.driver.RyuBgpDriver"

1
neutron/services/bgp/agent/config.py
View File

@ -19,7 +19,6 @@ from neutron._i18n import _
BGP_DRIVER_OPTS = [
cfg.StrOpt('bgp_speaker_driver',
default=None,
help=_("BGP speaker driver class to be instantiated."))
]