diff options
author | John L. Villalovos <john.l.villalovos@intel.com> | 2017-03-07 14:39:49 -0800 |
---|---|---|
committer | John L. Villalovos <john.l.villalovos@intel.com> | 2017-03-08 08:19:40 -0800 |
commit | 1e560b182f3d6380d6c2a10d01361f00b5f2b73f (patch) | |
tree | eca47494e11363d5f9b5b418a505191f97c3c00a /ironic_tempest_plugin | |
parent | e2e53409815da2eca637bb3c427b2dd456356ec4 (diff) | |
download | ironic-1e560b182f3d6380d6c2a10d01361f00b5f2b73f.tar.gz |
devstack: Make sentry _IRONIC_DEVSTACK_LIB a global variable
Make the sentry variable _IRONIC_DEVSTACK_LIB a global variable so
that it will work as expected.
When variables use the 'declare' directive, it is by default a local
variable. While other variables have global scope.
For example:
declare -A AN_ARRAY # local in scope
foo=1 # global in scope
declare -A -g SPAM # global in scope because of -g
This causes errors to occur as some of the variables will be local only
and others will be global.
The issue that can occur is that when sourcing devstack/lib/ironic a
second time the sentry _IRONIC_DEVSTACK_LIB will not be set but all
the other variables from devstack/lib/ironic will already have been
set as they are by default global variables. And the purpose of
_IRONIC_DEVSTACK_LIB is to prevent the file from being sourced twice.
Change-Id: I24b6ebb89f3d950d39b0665346b4c17bf05d0a0b
Diffstat (limited to 'ironic_tempest_plugin')
0 files changed, 0 insertions, 0 deletions