openstack
/
puppet-ironic
Code Issues Proposed changes

Add uefi_pxe_bootfile_name parameter 

Add uefi_pxe_bootfile_name parameter with the default value from Ironic
master.

Change-Id: Ia7d51e08778c9d1083a40dbc252000689f3bba49
This commit is contained in:
Emilien Macchi 2015-12-17 10:39:35 -05:00
parent 2c83972567
commit 12e15ba21a
2 changed files with 37 additions and 27 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

6
manifests/drivers/pxe.pp
View File

@ -69,6 +69,10 @@
# Should be an valid directory # Should be an valid directory
# Defaults to '/var/lib/ironic/master_images'. # Defaults to '/var/lib/ironic/master_images'.
# #
# [*uefi_pxe_bootfile_name*]
# (optional) Bootfile DHCP parameter for UEFI boot mode.
# Defaults to 'elilo.efi'.
#
class ironic::drivers::pxe ( class ironic::drivers::pxe (
$deploy_kernel = undef, $deploy_kernel = undef,
@ -81,6 +85,7 @@ class ironic::drivers::pxe (
$images_path = '/var/lib/ironic/images/', $images_path = '/var/lib/ironic/images/',
$tftp_master_path = '/tftpboot/master_images', $tftp_master_path = '/tftpboot/master_images',
$instance_master_path = '/var/lib/ironic/master_images', $instance_master_path = '/var/lib/ironic/master_images',
$uefi_pxe_bootfile_name = 'elilo.efi',
) { ) {
# Configure ironic.conf # Configure ironic.conf
@ -93,6 +98,7 @@ class ironic::drivers::pxe (
'pxe/images_path': value => $images_path; 'pxe/images_path': value => $images_path;
'pxe/tftp_master_path': value => $tftp_master_path; 'pxe/tftp_master_path': value => $tftp_master_path;
'pxe/instance_master_path': value => $instance_master_path; 'pxe/instance_master_path': value => $instance_master_path;
'pxe/uefi_pxe_bootfile_name': value => $uefi_pxe_bootfile_name;
} }
if $deploy_kernel { if $deploy_kernel {

8
spec/classes/ironic_drivers_pxe_spec.rb
View File

@ -30,7 +30,8 @@ describe 'ironic::drivers::pxe' do
:tftp_root => '/tftpboot', :tftp_root => '/tftpboot',
:images_path => '/var/lib/ironic/images/', :images_path => '/var/lib/ironic/images/',
:tftp_master_path => '/tftpboot/master_images', :tftp_master_path => '/tftpboot/master_images',
:instance_master_path => '/var/lib/ironic/master_images' } :instance_master_path => '/var/lib/ironic/master_images',
:uefi_pxe_bootfile_name => 'elilo.efi' }
end end
let :params do let :params do
@ -51,6 +52,7 @@ describe 'ironic::drivers::pxe' do
is_expected.to contain_ironic_config('pxe/images_path').with_value(p[:images_path]) is_expected.to contain_ironic_config('pxe/images_path').with_value(p[:images_path])
is_expected.to contain_ironic_config('pxe/tftp_master_path').with_value(p[:tftp_master_path]) is_expected.to contain_ironic_config('pxe/tftp_master_path').with_value(p[:tftp_master_path])
is_expected.to contain_ironic_config('pxe/instance_master_path').with_value(p[:instance_master_path]) is_expected.to contain_ironic_config('pxe/instance_master_path').with_value(p[:instance_master_path])
is_expected.to contain_ironic_config('pxe/uefi_pxe_bootfile_name').with_value(p[:uefi_pxe_bootfile_name])
end end
context 'when overriding parameters' do context 'when overriding parameters' do
@ -65,7 +67,8 @@ describe 'ironic::drivers::pxe' do
:tftp_root => '/mnt/ftp', :tftp_root => '/mnt/ftp',
:images_path => '/mnt/images', :images_path => '/mnt/images',
:tftp_master_path => '/mnt/master_images', :tftp_master_path => '/mnt/master_images',
:instance_master_path => '/mnt/ironic/master_images' :instance_master_path => '/mnt/ironic/master_images',
:uefi_pxe_bootfile_name => 'bootx64.efi'
) )
end end
@ -80,6 +83,7 @@ describe 'ironic::drivers::pxe' do
is_expected.to contain_ironic_config('pxe/images_path').with_value(p[:images_path]) is_expected.to contain_ironic_config('pxe/images_path').with_value(p[:images_path])
is_expected.to contain_ironic_config('pxe/tftp_master_path').with_value(p[:tftp_master_path]) is_expected.to contain_ironic_config('pxe/tftp_master_path').with_value(p[:tftp_master_path])
is_expected.to contain_ironic_config('pxe/instance_master_path').with_value(p[:instance_master_path]) is_expected.to contain_ironic_config('pxe/instance_master_path').with_value(p[:instance_master_path])
is_expected.to contain_ironic_config('pxe/uefi_pxe_bootfile_name').with_value(p[:uefi_pxe_bootfile_name])
end end
end end