From ec3299c72eae195106092a646409564b5b3eb560 Mon Sep 17 00:00:00 2001 From: Stanislaw Kardach Date: Wed, 24 Feb 2016 20:08:24 +0100 Subject: [PATCH 10/20] Make TestVM creation architecture aware This depends on cirros-testvm package to contain both amd64 and arm64 images. --- deployment/puppet/openstack/manifests/img/cirros.pp | 8 ++++++-- .../osnailyfacter/manifests/generate_vms/vm_config.pp | 11 +++++++++++ deployment/puppet/osnailyfacter/templates/vm_libvirt.erb | 15 ++++++++++++++- files/fuel-migrate/fuel-migrate | 5 ++++- 4 files changed, 35 insertions(+), 4 deletions(-) diff --git a/deployment/puppet/openstack/manifests/img/cirros.pp b/deployment/puppet/openstack/manifests/img/cirros.pp index 84e202e..cff06e0 100644 --- a/deployment/puppet/openstack/manifests/img/cirros.pp +++ b/deployment/puppet/openstack/manifests/img/cirros.pp @@ -9,6 +9,10 @@ class openstack::img::cirros ( $img_name = 'cirros', $os_name = 'cirros', ) { + $arch = $::architecture ? { + /(arm64|aarch64)/ => 'aarch64', + default => 'x86_64', + } package { 'cirros-testvm': ensure => "present" @@ -17,14 +21,14 @@ class openstack::img::cirros ( case $::osfamily { 'RedHat': { exec { 'upload-img': - command => "/usr/bin/glance -N ${os_auth_url} -T ${os_tenant_name} -I ${os_username} -K ${os_password} add name=${img_name} is_public=${public} container_format=${container_format} disk_format=${disk_format} distro=${os_name} < /opt/vm/cirros-x86_64-disk.img", + command => "/usr/bin/glance -N ${os_auth_url} -T ${os_tenant_name} -I ${os_username} -K ${os_password} add name=${img_name} is_public=${public} container_format=${container_format} disk_format=${disk_format} distro=${os_name} < /opt/vm/cirros-${arch}-disk.img", unless => "/usr/bin/glance -N ${os_auth_url} -T ${os_tenant_name} -I ${os_username} -K ${os_password} index && (/usr/bin/glance -N ${os_auth_url} -T ${os_tenant_name} -I ${os_username} -K ${os_password} index | grep ${img_name})", } } 'Debian': { exec { 'upload-img': - command => "/usr/bin/glance -N ${os_auth_url} -T ${os_tenant_name} -I ${os_username} -K ${os_password} add name=${img_name} is_public=${public} container_format=${container_format} disk_format=${disk_format} distro=${os_name} < /usr/share/cirros-testvm/cirros-x86_64-disk.img", + command => "/usr/bin/glance -N ${os_auth_url} -T ${os_tenant_name} -I ${os_username} -K ${os_password} add name=${img_name} is_public=${public} container_format=${container_format} disk_format=${disk_format} distro=${os_name} < /usr/share/cirros-testvm/cirros-${arch}-disk.img", unless => "/usr/bin/glance -N ${os_auth_url} -T ${os_tenant_name} -I ${os_username} -K ${os_password} index && (/usr/bin/glance -N ${os_auth_url} -T ${os_tenant_name} -I ${os_username} -K ${os_password} index | grep ${img_name})", } diff --git a/deployment/puppet/osnailyfacter/manifests/generate_vms/vm_config.pp b/deployment/puppet/osnailyfacter/manifests/generate_vms/vm_config.pp index 70fef32..dfefcbe 100644 --- a/deployment/puppet/osnailyfacter/manifests/generate_vms/vm_config.pp +++ b/deployment/puppet/osnailyfacter/manifests/generate_vms/vm_config.pp @@ -4,6 +4,17 @@ define osnailyfacter::generate_vms::vm_config( $details = $name $id = $details['id'] + case $::architecture { + 'arm64', 'aarch64': { + $machine_arch = 'aarch64' + $machine_type = 'virt' + } + default: { + $machine_arch = 'x86_64' + $machine_type = 'pc-i440fx-trusty' + } + } + file { "${template_dir}/template_${id}_vm.xml": owner => 'root', group => 'root', diff --git a/deployment/puppet/osnailyfacter/templates/vm_libvirt.erb b/deployment/puppet/osnailyfacter/templates/vm_libvirt.erb index ae2ba5b..64e716d 100644 --- a/deployment/puppet/osnailyfacter/templates/vm_libvirt.erb +++ b/deployment/puppet/osnailyfacter/templates/vm_libvirt.erb @@ -3,14 +3,23 @@ <%= @details['mem'] %> <%= @details['cpu'] %> - hvm + hvm + <% if == 'aarch64' %> + /var/lib/nova/<%= @details['id'] %>_vm.kernel + /var/lib/nova/<%= @details['id'] %>_vm.initramfs + root=/dev/vda1 rw rootwait console=tty0 console=ttyS0 console=ttyAMA0 + <% else %> + <% end %> + <% if $machine_arch == 'aarch64' %> + + <% end %> destroy @@ -57,7 +66,11 @@ diff --git a/files/fuel-migrate/fuel-migrate b/files/fuel-migrate/fuel-migrate index a215e0a..7d5b6cd 100755 --- a/files/fuel-migrate/fuel-migrate +++ b/files/fuel-migrate/fuel-migrate @@ -176,6 +176,8 @@ create_vm(){ exit 1 fi fi + # figure out atchitecture of the destination + arch="$(virsh -c "${dkvm_uri}" nodeinfo | awk '/CPU model:/ { print $3}')" # create disk ssh "${dkvm_ip}" qemu-img create -f raw "${dkvm_folder}/${fvm_name}.img" "${fvm_disk_size}" # make template for virsh @@ -185,13 +187,14 @@ create_vm(){ ${fvm_ram} ${fvm_cpu} - hvm + hvm + ${arch/aarch64/} destroy