summaryrefslogtreecommitdiffstats
path: root/ci
diff options
context:
space:
mode:
authorAlexandru Avadanii <Alexandru.Avadanii@enea.com>2017-11-10 18:01:53 +0100
committerAlexandru Avadanii <Alexandru.Avadanii@enea.com>2017-11-13 15:39:50 +0000
commit8fbe4650d66e18eadd946186584a10a95e2cce4e (patch)
tree4d6ceacd55233c43b4e157d91dc02e8adbe6cbc9 /ci
parenta0a060a131fb8141d0a1210892443caef250b7b5 (diff)
deploy.sh: Check hypervisor after prereq install
On a clean system (i.e. no libvirt installed), the prerequisites should be installed first, before checking for hypervisor access. While at it, add 'libvirt' to the list of prerequisites, as it is not automatically required by the others. Add a note in the documentation about libvirtd service not being automatically started after installation on CentOS. Change-Id: I29d94d4a56cd823362811e8fc9c076c83fb6c04c Signed-off-by: Alexandru Avadanii <Alexandru.Avadanii@enea.com> (cherry picked from commit 8bdc7b442ebd9f162a36f568d9b61ecf85ab23cb)
Diffstat (limited to 'ci')
-rwxr-xr-xci/deploy.sh16
1 files changed, 9 insertions, 7 deletions
diff --git a/ci/deploy.sh b/ci/deploy.sh
index 6a1d6e1e6..06606414e 100755
--- a/ci/deploy.sh
+++ b/ci/deploy.sh
@@ -236,11 +236,6 @@ if [[ "$(sudo whoami)" != 'root' ]]; then
exit 1
fi
-if ! virsh list >/dev/null 2>&1; then
- notify "[ERROR] This script requires hypervisor access\n" 1>&2
- exit 1
-fi
-
# Validate mandatory arguments are set
if [ -z "${TARGET_LAB}" ] || [ -z "${TARGET_POD}" ] || \
[ -z "${DEPLOY_SCENARIO}" ]; then
@@ -265,9 +260,11 @@ pushd "${DEPLOY_DIR}" > /dev/null
# Install required packages
[ -n "$(command -v apt-get)" ] && sudo apt-get install -y \
- git make rsync mkisofs curl virtinst cpu-checker qemu-kvm uuid-runtime
+ git make rsync mkisofs curl virtinst cpu-checker qemu-kvm uuid-runtime \
+ libvirt-bin
[ -n "$(command -v yum)" ] && sudo yum install -y --skip-broken \
- git make rsync genisoimage curl virt-install qemu-kvm util-linux
+ git make rsync genisoimage curl virt-install qemu-kvm util-linux \
+ libvirt
# For baremetal, python is indirectly required for PDF parsing
if [ "${DEPLOY_TYPE}" = 'baremetal' ]; then
@@ -277,6 +274,11 @@ if [ "${DEPLOY_TYPE}" = 'baremetal' ]; then
python python-ipaddress python-jinja2
fi
+if ! virsh list >/dev/null 2>&1; then
+ notify "[ERROR] This script requires hypervisor access\n" 1>&2
+ exit 1
+fi
+
# Clone git submodules and apply our patches
make -C "${REPO_ROOT_PATH}/mcp/patches" deepclean patches-import