openstack
/
puppet-nova
Code Issues Proposed changes

Merge "mdev: Simplify handling of device_addresses"

This commit is contained in:
Zuul 2024-03-19 04:17:53 +00:00 committed by Gerrit Code Review
parent 69a538fe9e 1fbdef7ac6
commit d18827eca2
2 changed files with 4 additions and 13 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

15
manifests/compute/mdev_type.pp
View File

@ -29,18 +29,9 @@ define nova::compute::mdev_type (
$max_instances = $facts['os_service_default'],
) {
if empty($device_addresses) {
nova_config {
"mdev_${mdev_type}/device_addresses": value => $facts['os_service_default']
}
} else {
nova_config {
"mdev_${mdev_type}/device_addresses": value => join(any2array($device_addresses), ',');
}
}
nova_config {
"mdev_${mdev_type}/mdev_class": value => $mdev_class;
"mdev_${mdev_type}/max_instances": value => $max_instances;
"mdev_${mdev_type}/device_addresses": value => join(any2array($device_addresses), ',');
"mdev_${mdev_type}/mdev_class": value => $mdev_class;
"mdev_${mdev_type}/max_instances": value => $max_instances;
}
}

2
spec/classes/nova_compute_mdev_spec.rb
View File

@ -33,7 +33,7 @@ describe 'nova::compute::mdev' do
is_expected.to contain_nova_config('mdev_nvidia-35/device_addresses').with_value('0000:84:00.0,0000:85:00.0')
is_expected.to contain_nova_config('mdev_nvidia-35/mdev_class').with_value('<SERVICE DEFAULT>')
is_expected.to contain_nova_config('mdev_nvidia-35/max_instances').with_value('<SERVICE DEFAULT>')
is_expected.to contain_nova_config('mdev_nvidia-36/device_addresses').with_value('<SERVICE DEFAULT>')
is_expected.to contain_nova_config('mdev_nvidia-36/device_addresses').with_value('')
is_expected.to contain_nova_config('mdev_nvidia-36/mdev_class').with_value('CUSTOM_MDEV1')
is_expected.to contain_nova_config('mdev_nvidia-36/max_instances').with_value('<SERVICE DEFAULT>')
is_expected.to contain_nova_config('mdev_nvidia-37/device_addresses').with_value('<SERVICE DEFAULT>')