From bebe21ae1369e42bf3d4664facaaeac4a5b16ab6 Mon Sep 17 00:00:00 2001 From: rabi Date: Sat, 27 Aug 2016 08:30:32 +0530 Subject: [PATCH] Revert "Fix devstack plugin" workaround This reverts the workaround for the devstack plugin, as now the gate jobs pass(with the plugin and no devstack tree heat code) and make it only a devstack plugin. Change-Id: Ia8d88eca2dec301b692bbef3e1d8988f00e1936a Depends-On: I5b60422bf1f5fa78aa8f3383f7a222e0356d9e42 --- devstack/lib/heat | 21 +++------------------ devstack/plugin.sh | 6 +++--- devstack/upgrade/upgrade.sh | 2 +- 3 files changed, 7 insertions(+), 22 deletions(-) diff --git a/devstack/lib/heat b/devstack/lib/heat index 8682428a5f..08c9ec0b32 100644 --- a/devstack/lib/heat +++ b/devstack/lib/heat @@ -240,13 +240,8 @@ function configure_heat { [ $err_count -eq 0 ] || die $LINENO "$err_count of the requested Heat plugins could not be installed." } -# dummy function called by stack.sh, remove it once moved to plugin -function init_heat { - : -} - # init_heat() - Initialize database -function init_heat_with_plugin { +function init_heat { # (re)create heat database recreate_database heat @@ -284,13 +279,8 @@ function install_heat_other { : } -# dummy function called by stack.sh, remove it once moved to plugin -function start_heat { - : -} - # start_heat() - Start running processes, including screen -function start_heat_with_plugin { +function start_heat { run_process h-eng "$HEAT_BIN_DIR/heat-engine --config-file=$HEAT_CONF" # If the site is not enabled then we are in a grenade scenario @@ -392,13 +382,8 @@ function _config_heat_apache_wsgi { } -# dummy function called by stack.sh, remove it once moved to plugin -function create_heat_accounts { - : -} - # create_heat_accounts() - Set up common required heat accounts -function create_heat_accounts_with_plugin { +function create_heat_accounts { if [[ "$HEAT_STANDALONE" != "True" ]]; then create_service_user "heat" "admin" diff --git a/devstack/plugin.sh b/devstack/plugin.sh index 3abe4f0a65..900a5b0c57 100644 --- a/devstack/plugin.sh +++ b/devstack/plugin.sh @@ -23,16 +23,16 @@ if is_heat_enabled; then configure_heat if is_service_enabled key; then - create_heat_accounts_with_plugin + create_heat_accounts fi elif [[ "$1" == "stack" && "$2" == "extra" ]]; then # Initialize heat - init_heat_with_plugin + init_heat # Start the heat API and heat taskmgr components echo_summary "Starting heat" - start_heat_with_plugin + start_heat fi if [[ "$1" == "unstack" ]]; then diff --git a/devstack/upgrade/upgrade.sh b/devstack/upgrade/upgrade.sh index 5dc7362be2..594f95f81e 100755 --- a/devstack/upgrade/upgrade.sh +++ b/devstack/upgrade/upgrade.sh @@ -82,7 +82,7 @@ HEAT_BIN_DIR=$(dirname $(which heat-manage)) $HEAT_BIN_DIR/heat-manage --config-file $HEAT_CONF db_sync || die $LINENO "DB sync error" # Start Heat -start_heat_with_plugin +start_heat # Don't succeed unless the services come up ensure_services_started heat-api heat-engine heat-api-cloudwatch heat-api-cfn