Skip to content
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

Support Ubuntu 20.04 #669

Merged
merged 2 commits into from
Apr 29, 2021
Merged

Support Ubuntu 20.04 #669

merged 2 commits into from
Apr 29, 2021

Conversation

ekohl
Copy link
Member

@ekohl ekohl commented Apr 29, 2021

The most notable change is that Ubuntu 20.04 changes the tftp root from /var/lib/tftpboot to /srv/tftp. This matches Debian again.

The other change is that it ships a new enough Ansible to use the callback from collections.

Replaces #666 & #667.

@ekohl
Copy link
Member Author

ekohl commented Apr 29, 2021

Test failure is expected since there is no Puppet 5 package on Ubuntu 20.04.

ekohl added 2 commits April 29, 2021 15:52
This uses the fact that the tftp module uses Hiera data in modules. This
allows lookup to work even if the class isn't included. It saves having
to duplicate the logic.

To do this, the minimum version is raised to 3.0.0 which made that
change.
The most notable change is that Ubuntu 20.04 changes the tftp root from
/var/lib/tftpboot to /srv/tftp. This matches Debian again.

The other change is that it ships a new enough Ansible to use the
callback from collections.
@ekohl ekohl force-pushed the support-ubuntu-20.04 branch from 0f87f40 to 9686d75 Compare April 29, 2021 13:52
@ekohl
Copy link
Member Author

ekohl commented Apr 29, 2021

Updated by raising the minimum tftp version to actually use data in modules.

@ekohl ekohl merged commit 9686d75 into theforeman:master Apr 29, 2021
@ekohl ekohl deleted the support-ubuntu-20.04 branch April 29, 2021 15:15
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants