openstack/puppet-tripleo
Code Issues Proposed changes

Repair immediate VF configuration for PCI SR-IOV

Browse Source 
Change I71edc135432ab2193741c37ce977dd11172401e6 broke the host VF
configuration that set VF counts when the puppet is applied. This patch
re-adds the ensure => present property required for proper behavior.

Change-Id: Ibd18c4f3b7f0d8cee330f94f87e7ad4ea7ceeffb
Closes-Bug: #1712903
This commit is contained in:
Brent Eagles 2017-08-24 17:13:01 -02:30
parent 7e57d3aac4
commit 1e133f3146
2 changed files with 3 additions and 3 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

2
manifests/host/sriov.pp
View File

@ -16,7 +16,7 @@ class tripleo::host::sriov (
) { ) {
if !empty($number_of_vfs) { if !empty($number_of_vfs) {
sriov_vf_config { $number_of_vfs: } sriov_vf_config { $number_of_vfs: ensure => present }
# the numvfs configuration needs to be persisted for every boot # the numvfs configuration needs to be persisted for every boot
tripleo::host::sriov::numvfs_persistence {'persistent_numvfs': tripleo::host::sriov::numvfs_persistence {'persistent_numvfs':

4
spec/classes/tripleo_host_sriov_spec.rb
View File

@ -17,8 +17,8 @@ describe 'tripleo::host::sriov' do
end end
it 'configures numvfs' do it 'configures numvfs' do
is_expected.to contain_sriov_vf_config('eth0:4') is_expected.to contain_sriov_vf_config('eth0:4').with( :ensure => 'present' )
is_expected.to contain_sriov_vf_config('eth1:5') is_expected.to contain_sriov_vf_config('eth1:5').with( :ensure => 'present' )
is_expected.to contain_tripleo__host__sriov__numvfs_persistence('persistent_numvfs').with( is_expected.to contain_tripleo__host__sriov__numvfs_persistence('persistent_numvfs').with(
:vf_defs => ['eth0:4','eth1:5'], :vf_defs => ['eth0:4','eth1:5'],
:content_string => "#!/bin/bash\n" :content_string => "#!/bin/bash\n"