From 18d5c833d47e41c8c8dcd73f35268d6e2b43df5b Mon Sep 17 00:00:00 2001 From: Ryu Ishimoto Date: Wed, 19 Feb 2014 00:33:46 +0900 Subject: [PATCH] Remove provider router configuration To be compatible with the Icehouse release of MidoNet, the provider router configuration is removed from devstack since it is no longer necessary to configure it. Change-Id: I4be2d9bbf2c82fd375702cbb1d60c3277086134f Implements: blueprint remove-provider-router-config-for-midonet --- lib/neutron_plugins/midonet | 11 ++++++----- lib/neutron_thirdparty/midonet | 19 ++----------------- 2 files changed, 8 insertions(+), 22 deletions(-) diff --git a/lib/neutron_plugins/midonet b/lib/neutron_plugins/midonet index f95fcb75b9..dd3b2baeca 100644 --- a/lib/neutron_plugins/midonet +++ b/lib/neutron_plugins/midonet @@ -1,6 +1,10 @@ # Neutron MidoNet plugin # ---------------------- +MIDONET_DIR=${MIDONET_DIR:-$DEST/midonet} +MIDONET_API_PORT=${MIDONET_API_PORT:-8080} +MIDONET_API_URL=${MIDONET_API_URL:-http://localhost:$MIDONET_API_PORT/midonet-api} + # Save trace setting MY_XTRACE=$(set +o | grep xtrace) set +o xtrace @@ -47,8 +51,8 @@ function neutron_plugin_configure_plugin_agent() { } function neutron_plugin_configure_service() { - if [[ "$MIDONET_API_URI" != "" ]]; then - iniset /$Q_PLUGIN_CONF_FILE MIDONET midonet_uri $MIDONET_API_URI + if [[ "$MIDONET_API_URL" != "" ]]; then + iniset /$Q_PLUGIN_CONF_FILE MIDONET midonet_uri $MIDONET_API_URL fi if [[ "$MIDONET_USERNAME" != "" ]]; then iniset /$Q_PLUGIN_CONF_FILE MIDONET username $MIDONET_USERNAME @@ -59,9 +63,6 @@ function neutron_plugin_configure_service() { if [[ "$MIDONET_PROJECT_ID" != "" ]]; then iniset /$Q_PLUGIN_CONF_FILE MIDONET project_id $MIDONET_PROJECT_ID fi - if [[ "$MIDONET_PROVIDER_ROUTER_ID" != "" ]]; then - iniset /$Q_PLUGIN_CONF_FILE MIDONET provider_router_id $MIDONET_PROVIDER_ROUTER_ID - fi Q_L3_ENABLED=True Q_L3_ROUTER_PER_TENANT=True diff --git a/lib/neutron_thirdparty/midonet b/lib/neutron_thirdparty/midonet index e672528a2d..98be4254fc 100644 --- a/lib/neutron_thirdparty/midonet +++ b/lib/neutron_thirdparty/midonet @@ -10,20 +10,12 @@ # MidoNet devstack destination dir MIDONET_DIR=${MIDONET_DIR:-$DEST/midonet} -MIDONET_API_PORT=${MIDONET_API_PORT:-8080} -MIDONET_API_URL=${MIDONET_API_URL:-http://localhost:$MIDONET_API_PORT/midonet-api} # MidoNet client repo MIDONET_CLIENT_REPO=${MIDONET_CLIENT_REPO:-https://github.com/midokura/python-midonetclient.git} MIDONET_CLIENT_BRANCH=${MIDONET_CLIENT_BRANCH:-master} MIDONET_CLIENT_DIR=${MIDONET_CLIENT_DIR:-$MIDONET_DIR/python-midonetclient} -# MidoNet OpenStack repo -MIDONET_OS_REPO=${MIDONET_OS_REPO:-https://github.com/midokura/midonet-openstack.git} -MIDONET_OS_BRANCH=${MIDONET_OS_BRANCH:-master} -MIDONET_OS_DIR=${MIDONET_OS_DIR:-$MIDONET_DIR/midonet-openstack} -MIDONET_SETUP_SCRIPT=${MIDONET_SETUP_SCRIPT:-$MIDONET_OS_DIR/bin/setup_midonet_topology.py} - # Save trace setting MY_XTRACE=$(set +o | grep xtrace) set +o xtrace @@ -33,19 +25,12 @@ function configure_midonet() { } function init_midonet() { - - # Initialize DB. Evaluate the output of setup_midonet_topology.py to set - # env variables for provider router ID. - eval `python $MIDONET_SETUP_SCRIPT $MIDONET_API_URL admin $ADMIN_PASSWORD admin provider_devices` - die_if_not_set $LINENO provider_router_id "Error running midonet setup script, provider_router_id was not set." - - iniset /$Q_PLUGIN_CONF_FILE MIDONET provider_router_id $provider_router_id + : } function install_midonet() { git_clone $MIDONET_CLIENT_REPO $MIDONET_CLIENT_DIR $MIDONET_CLIENT_BRANCH - git_clone $MIDONET_OS_REPO $MIDONET_OS_DIR $MIDONET_OS_BRANCH - export PYTHONPATH=$MIDONET_CLIENT_DIR/src:$MIDONET_OS_DIR/src:$PYTHONPATH + export PYTHONPATH=$MIDONET_CLIENT_DIR/src:$PYTHONPATH } function start_midonet() {