openstack/devstack
Code Issues Proposed changes

Merge "Add function is_plugin_enabled"

Browse Source
This commit is contained in:
Jenkins 2017-09-06 01:15:32 +00:00 committed by Gerrit Code Review
commit bd054c58de
1 changed files with 14 additions and 1 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

15
functions-common
View File

@ -1882,7 +1882,7 @@ function enable_plugin {
local name=$1 local name=$1
local url=$2 local url=$2
local branch=${3:-master} local branch=${3:-master}
if [[ ",${DEVSTACK_PLUGINS}," =~ ,${name}, ]]; then if is_plugin_enabled $name; then
die $LINENO "Plugin attempted to be enabled twice: ${name} ${url} ${branch}" die $LINENO "Plugin attempted to be enabled twice: ${name} ${url} ${branch}"
fi fi
DEVSTACK_PLUGINS+=",$name" DEVSTACK_PLUGINS+=",$name"
@ -1891,6 +1891,19 @@ function enable_plugin {
GITBRANCH[$name]=$branch GITBRANCH[$name]=$branch
} }
# is_plugin_enabled <name>
#
# Check if the plugin was enabled, e.g. using enable_plugin
#
# ``name`` The name with which the plugin was enabled
function is_plugin_enabled {
local name=$1
if [[ ",${DEVSTACK_PLUGINS}," =~ ",${name}," ]]; then
return 0
fi
return 1
}
# fetch_plugins # fetch_plugins
# #
# clones all plugins # clones all plugins