Skip to content

Commit

Permalink
Do not install whitebox twice
Browse files Browse the repository at this point in the history
Setting TEMPEST_PLUGINS is enough for devstack to install whitebox for
us. There is no need to install it ourselves in our devstack plugin.

Change-Id: I8446b54dd1e8ca1551e3064006576190564deeac
  • Loading branch information
notartom committed May 28, 2020
1 parent 36e4ca5 commit 1a55c01
Showing 1 changed file with 1 addition and 8 deletions.
9 changes: 1 addition & 8 deletions devstack/plugin.sh
Original file line number Diff line number Diff line change
@@ -1,10 +1,5 @@
#!/bin/sh

function install {
echo_summary "Installing whitebox-tempest-plugin"
setup_dev_lib "whitebox-tempest-plugin"
}

function configure {
echo_summary "Configuring whitebox-tempest-plugin options"
iniset $TEMPEST_CONFIG whitebox ctlplane_ssh_username $STACK_USER
Expand All @@ -28,9 +23,7 @@ function configure {

if [[ "$1" == "stack" ]]; then
if is_service_enabled tempest; then
if [[ "$2" == "install" ]]; then
install
elif [[ "$2" == "test-config" ]]; then
if [[ "$2" == "test-config" ]]; then
configure
fi
fi
Expand Down

0 comments on commit 1a55c01

Please sign in to comment.