openstack
/
fuel-main
Code Issues Proposed changes

Update dependency checks 

VirtualBox scripts rely on various utilities to function. To validate
the environment, virtualbox/prepare-environment.sh checks if all these
utilities are available to the shell. However, some of the checks
(namely xxd and ssh) were missing. This change should fix this by
aborting the prepare step in case if these utilities are not found.

Change-Id: Ic49c11054345964c9785e9c4c791ea33dba4791c
Partial-Bug: 1525967
This commit is contained in:
Dmitry Bilunov 2015-12-29 12:19:32 +03:00
parent 7e95e11fe7
commit fc9f3c11f9
1 changed files with 20 additions and 0 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

20
virtualbox/actions/prepare-environment.sh
View File

@ -50,6 +50,26 @@ else
echo "OK"
fi
# Check for ssh
echo -n "Checking for 'ssh'... "
execute type ssh >/dev/null 2>&1
if [ $? -eq 1 ]; then
echo "\"ssh\" is not available in the path, but it's required. Please install the \"openssh\" package. Aborting."
exit 1
else
echo "OK"
fi
# Check for xxd
echo -n "Checking for 'xxd'... "
execute type xxd >/dev/null 2>&1
if [ $? -eq 1 ]; then
echo "\"xxd\" is not available in the path, but it's required. Please install the \"xxd\" package. Aborting."
exit 1
else
echo "OK"
fi
# Check for VirtualBox
echo "If you run this script under Cygwin, you may have to add path to VirtualBox directory to your PATH. "
echo "Usually it is enough to run \"export PATH=\$PATH:\"/cygdrive/c/Program Files/Oracle/VirtualBox\" "