-
-
Notifications
You must be signed in to change notification settings - Fork 37
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Improve syslinux::system class #76
base: master
Are you sure you want to change the base?
Improve syslinux::system class #76
Conversation
manifests/syslinux/system.pp
Outdated
file { "${tftp_root}/syslinux/${file}": | ||
source => "${syslinux_dir}/${file}", | ||
require => Package['syslinux'], | ||
} |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Maybe we can manage the directory itself. If source
is a directory and recurse
is set to true, it should copy all files and not need an update if the files change at some point.
file { "${tftp_root}/syslinux/${file}": | |
source => "${syslinux_dir}/${file}", | |
require => Package['syslinux'], | |
} | |
file { "${tftp_root}/syslinux": | |
source => $syslinux_dir, | |
recurse => true, | |
require => Package['syslinux'], | |
} |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Made the changes I thought would work, but I'm getting CI errors on BSD. An extra set of eyes would be appreciated.
Co-authored-by: Romain Tartière <romain@blogreen.org>
Pull Request (PR) description
Improve syslinux::system class
This Pull Request (PR) fixes the following issues
No issues submitted so far, but it looked like most of the development focus was on deploying syslinux from upstream sources rather than OS packages. Out of the box, the module didn't include all the files needed to present a working PXE menu.