openstack
/
puppet-nova
Code Issues Proposed changes

Expose nova config option [workaround]/never_download_image_if_on_rbd 

This exposes the nova workaround to disable downloading images from glance to
rbd (vs a cheap COW clone) when nova-compute and glance are not backed by the
same ceph cluster.

Related nova change: I069b6b1d28eaf1eee5c7fb8d0fdef9c0c229a1bf

Change-Id: I8329810d6c047c0d94e7b123e7cdc1263a7856cd
(cherry picked from commit 9f26045197)
(cherry picked from commit 38c233271a)
(cherry picked from commit f71fa558de)
This commit is contained in:
Oliver Walsh 2020-10-21 16:33:29 +01:00
parent e126ce8145
commit 30684e6469
3 changed files with 17 additions and 1 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

1
manifests/compute.pp
View File

@ -243,6 +243,7 @@ class nova::compute (
include ::nova::deps
include ::nova::params
include ::nova::workarounds
$cpu_shared_set_real = pick(join(any2array($cpu_shared_set), ','), $::os_service_default)
$cpu_dedicated_set_real = pick(join(any2array($cpu_dedicated_set), ','), $::os_service_default)

10
manifests/workarounds.pp
View File

@ -4,17 +4,27 @@
#
# === Parameters:
#
# [*never_download_image_if_on_rbd*]
# (optional) refuse to boot an instance if it would require downloading from
# glance and uploading to ceph instead of a COW clone
# Defaults to $::os_service_default
#
# [*enable_numa_live_migration*]
# (optional) Whether to enable live migration for NUMA topology instances.
# Defaults to false
#
class nova::workarounds (
$enable_numa_live_migration = false,
$never_download_image_if_on_rbd = $::os_service_default,
) {
nova_config {
'workarounds/enable_numa_live_migration': value => $enable_numa_live_migration;
}
nova_config {
'workarounds/never_download_image_if_on_rbd': value => $never_download_image_if_on_rbd;
}
}

7
spec/classes/nova_workarounds_spec.rb
View File

@ -10,14 +10,19 @@ describe 'nova::workarounds' do
context 'with default parameters' do
it { is_expected.to contain_nova_config('workarounds/enable_numa_live_migration').with_value(false) }
it { is_expected.to contain_nova_config('workarounds/never_download_image_if_on_rbd').with_value('<SERVICE DEFAULT>') }
end
context 'with overridden parameters' do
let :params do
{ :enable_numa_live_migration => true,}
{
:enable_numa_live_migration => true,
:never_download_image_if_on_rbd => true
}
end
it { is_expected.to contain_nova_config('workarounds/enable_numa_live_migration').with_value('true') }
it { is_expected.to contain_nova_config('workarounds/never_download_image_if_on_rbd').with_value('true') }
end
end