opendev/system-config
Code Issues Proposed changes

Export prepare_node variables rather than pass them around

Browse Source 
The prepare_node scripts are primarily called from within nodepool,
which only passes in a single argument: the hostname of the node.

The rest of the values that we want to pass in to the underlying
prepare_node script can be shared through the environment, making order
no longer important and alleviating the need to duplicate setting
defaults in order to pad positional arguments to a newly created
variable.

Change-Id: I0a13e27524b75a50a482870414ebe2fd6ac8315e
This commit is contained in:
K Jonathan Harker 2014-07-02 16:45:56 -07:00
parent 0bd5abba0f
commit 8bd0056443
6 changed files with 36 additions and 30 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

13
modules/openstack_project/files/nodepool/scripts/prepare_node.sh
View File

@ -17,12 +17,13 @@
# limitations under the License.
HOSTNAME=$1
SUDO=$2
THIN=$3
PYTHON3=${4:-false}
PYPY=${5:-false}
ALL_MYSQL_PRIVS=${6:-false}
GIT_BASE=${7:-git://git.openstack.org}
SUDO=${SUDO:-true}
THIN=${THIN:-true}
PYTHON3=${PYTHON3:-false}
PYPY=${PYPY:-false}
ALL_MYSQL_PRIVS=${ALL_MYSQL_PRIVS:-false}
GIT_BASE=${GIT_BASE:-git://git.openstack.org}
# Save the nameservers configured by our provider.
cat >/tmp/forwarding.conf <<EOF

13
modules/openstack_project/files/nodepool/scripts/prepare_node_bare.sh
View File

@ -17,12 +17,13 @@
# limitations under the License.
HOSTNAME=$1
SUDO='true'
THIN='false'
PYTHON3='false'
PYPY='false'
ALL_MYSQL_PRIVS='true'
./prepare_node.sh "$HOSTNAME" "$SUDO" "$THIN" "$PYTHON3" "$PYPY" "$ALL_MYSQL_PRIVS"
export SUDO='true'
export THIN='false'
export PYTHON='false'
export PYPY='false'
export ALL_MYSQL_PRIVS='true'
./prepare_node.sh "$HOSTNAME"
./restrict_memory.sh

9
modules/openstack_project/files/nodepool/scripts/prepare_node_devstack.sh
View File

@ -17,10 +17,11 @@
# limitations under the License.
HOSTNAME=$1
SUDO='true'
THIN='true'
./prepare_node.sh "$HOSTNAME" "$SUDO" "$THIN"
sudo -u jenkins -i /opt/nodepool-scripts/prepare_devstack.sh $HOSTNAME
export SUDO='true'
export THIN='true'
./prepare_node.sh "$HOSTNAME"
sudo -u jenkins -i /opt/nodepool-scripts/prepare_devstack.sh "$HOSTNAME"
./restrict_memory.sh

9
modules/openstack_project/files/nodepool/scripts/prepare_node_devstack_new_kernel.sh
View File

@ -17,10 +17,11 @@
# limitations under the License.
HOSTNAME=$1
SUDO='true'
THIN='true'
./prepare_node.sh "$HOSTNAME" "$SUDO" "$THIN"
sudo -u jenkins -i /opt/nodepool-scripts/prepare_devstack_new_kernel.sh $HOSTNAME
export SUDO='true'
export THIN='true'
./prepare_node.sh "$HOSTNAME"
sudo -u jenkins -i /opt/nodepool-scripts/prepare_devstack_new_kernel.sh "$HOSTNAME"
./restrict_memory.sh

13
modules/openstack_project/files/nodepool/scripts/prepare_node_py3k.sh
View File

@ -17,12 +17,13 @@
# limitations under the License.
HOSTNAME=$1
SUDO='false'
THIN='false'
PYTHON3='true'
PYPY='true'
ALL_MYSQL_PRIVS='true'
./prepare_node.sh "$HOSTNAME" "$SUDO" "$THIN" "$PYTHON3" "$PYPY" "$ALL_MYSQL_PRIVS"
export SUDO='false'
export THIN='false'
export PYTHON3='true'
export PYPY='true'
export ALL_MYSQL_PRIVS='true'
./prepare_node.sh "$HOSTNAME"
./fix_pip.sh
./restrict_memory.sh

9
modules/openstack_project/files/nodepool/scripts/prepare_node_tripleo.sh
View File

@ -17,13 +17,14 @@
# limitations under the License.
HOSTNAME=$1
SUDO='true'
THIN='true'
export SUDO='true'
export THIN='true'
# Workaround bug 1270646 during node bootstrapping.
sudo ip link set mtu 1458 dev eth0
./prepare_node.sh "$HOSTNAME" "$SUDO" "$THIN"
sudo -u jenkins -i /opt/nodepool-scripts/prepare_tripleo.sh $HOSTNAME
./prepare_node.sh "$HOSTNAME"
sudo -u jenkins -i /opt/nodepool-scripts/prepare_tripleo.sh "$HOSTNAME"
sync
sleep 5