From: Tim Rozet <trozet@redhat.com>
Date: Thu, 5 May 2016 19:18:04 +0000 (-0400)
Subject: Allows for specifying CPUs and RAM for VMs in virtual deployments
X-Git-Tag: colorado.1.0~236^2
X-Git-Url: https://gerrit.opnfv.org/gerrit/gitweb?a=commitdiff_plain;h=refs%2Fchanges%2F89%2F13689%2F5;p=apex.git

Allows for specifying CPUs and RAM for VMs in virtual deployments

Changes Include:
 - All VMs (undercloud and overcloud) now default to 4 vCPUs
 - Allow specifying number of vCPUs per overcloud VM with --virtual-cpus <number>
 - Allow specifying amount of RAM per overcloud VM with --virtual-ram <size in GB>
 - Undercloud now uses 10GiB of RAM to avoid OOM

JIRA: APEX-111

Change-Id: I541c669b869a90134bc8af25786fe3b1a557376b
Signed-off-by: Tim Rozet <trozet@redhat.com>
---

diff --git a/ci/build.sh b/ci/build.sh
index 26cdf7a0..7e080bed 100755
--- a/ci/build.sh
+++ b/ci/build.sh
@@ -96,7 +96,7 @@ if [ -n "$CACHE_DEST" ]; then
     fi
 fi
 
-#create build_output for legecy functionality compatibiltiy in jenkins
+#create build_output for legacy functionality compatibility in jenkins
 if [[ ! -d ../build_output  ]]; then
     rm -f ../build_output
     ln -s build/noarch/ ../build_output
diff --git a/ci/deploy.sh b/ci/deploy.sh
index cd76558c..996ca071 100755
--- a/ci/deploy.sh
+++ b/ci/deploy.sh
@@ -41,6 +41,8 @@ DEPLOY_OPTIONS=""
 RESOURCES=${RESOURCES:-'/var/opt/opnfv/images'}
 CONFIG=${CONFIG:-'/var/opt/opnfv'}
 OPNFV_NETWORK_TYPES="admin_network private_network public_network storage_network"
+VM_CPUS=4
+VM_RAM=8
 # Netmap used to map networks to OVS bridge names
 NET_MAP['admin_network']="br-admin"
 NET_MAP['private_network']="br-private"
@@ -369,7 +371,7 @@ function setup_undercloud_vm {
       if [[ $enabled_network_list =~ "public_network" ]]; then
         undercloud_nets+=" public_network"
       fi
-      define_vm undercloud hd 30 "$undercloud_nets"
+      define_vm undercloud hd 30 "$undercloud_nets" 4 10240
 
       ### this doesn't work for some reason I was getting hangup events so using cp instead
       #virsh vol-upload --pool default --vol undercloud.qcow2 --file $CONFIG/stack/undercloud.qcow2
@@ -468,8 +470,19 @@ function setup_undercloud_vm {
 }
 
 ##Create virtual nodes in virsh
-##params: none
+##params: vcpus, ramsize
 function setup_virtual_baremetal {
+  local vcpus ramsize
+  if [ -z "$1" ]; then
+    vcpus=4
+    ramsize=8192
+  elif [ -z "$2" ]; then
+    vcpus=$1
+    ramsize=8192
+  else
+    vcpus=$1
+    ramsize=$(($2*1024))
+  fi
   #start by generating the opening json for instackenv.json
   cat > $CONFIG/instackenv-virt.json << EOF
 {
@@ -479,7 +492,7 @@ EOF
   # next create the virtual machines and add their definitions to the file
   for i in $(seq 0 $vm_index); do
     if ! virsh list --all | grep baremetal${i} > /dev/null; then
-      define_vm baremetal${i} network 41 'admin_network'
+      define_vm baremetal${i} network 41 'admin_network' $vcpus $ramsize
       for n in private_network public_network storage_network; do
         if [[ $enabled_network_list =~ $n ]]; then
           echo -n "$n "
@@ -501,8 +514,8 @@ EOF
       "mac": [
         "$mac"
       ],
-      "cpu": "2",
-      "memory": "8192",
+      "cpu": "$vcpus",
+      "memory": "$ramsize",
       "disk": "41",
       "arch": "x86_64"
     },
@@ -531,7 +544,22 @@ EOF
 ##        bootdev - String: boot device for the VM
 ##        disksize - Number: size of the disk in GB
 ##        ovs_bridges: - List: list of ovs bridges
+##        vcpus - Number of VCPUs to use (defaults to 4)
+##        ramsize - Size of RAM for VM in MB (defaults to 8192)
 function define_vm () {
+  local vcpus ramsize
+
+  if [ -z "$5" ]; then
+    vcpus=4
+    ramsize=8388608
+  elif [ -z "$6" ]; then
+    vcpus=$5
+    ramsize=8388608
+  else
+    vcpus=$5
+    ramsize=$(($6*1024))
+  fi
+
   # Create the libvirt storage volume
   if virsh vol-list default | grep ${1}.qcow2 2>&1> /dev/null; then
     volume_path=$(virsh vol-path --pool default ${1}.qcow2 || echo "/var/lib/libvirt/images/${1}.qcow2")
@@ -553,8 +581,8 @@ function define_vm () {
                                               --image "$volume_path" \
                                               --diskbus sata \
                                               --arch x86_64 \
-                                              --cpus 2 \
-                                              --memory 8388608 \
+                                              --cpus $vcpus \
+                                              --memory $ramsize \
                                               --libvirt-nic-driver virtio \
                                               --baremetal-interface $4
 }
@@ -994,6 +1022,8 @@ display_usage() {
   echo -e "   --no-post-config : disable Post Install configuration."
   echo -e "   --debug : enable debug output."
   echo -e "   --interactive : enable interactive deployment mode which requires user to confirm steps of deployment."
+  echo -e "   --virtual-cpus : Number of CPUs to use per Overcloud VM in a virtual deployment (defaults to 4)."
+  echo -e "   --virtual-ram : Amount of RAM to use per Overcloud VM in GB (defaults to 8)."
 }
 
 ##translates the command line parameters into variables
@@ -1071,6 +1101,16 @@ parse_cmdline() {
                 echo "Interactive mode enabled"
                 shift 1
             ;;
+        --virtual-cpus )
+                VM_CPUS=$2
+                echo "Number of CPUs per VM set to $VM_CPUS"
+                shift 2
+            ;;
+        --virtual-ram )
+                VM_RAM=$2
+                echo "Amount of RAM per VM set to $VM_RAM"
+                shift 2
+            ;;
         *)
                 display_usage
                 exit 1
@@ -1138,7 +1178,7 @@ main() {
   fi
   setup_undercloud_vm
   if [ "$virtual" == "TRUE" ]; then
-    setup_virtual_baremetal
+    setup_virtual_baremetal $VM_CPUS $VM_RAM
   elif [ -n "$INVENTORY_FILE" ]; then
     parse_inventory_file
   fi