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

Fixes #36947 - Broken openSUSE Leap 15 provisioning #9927

Open
wants to merge 1 commit into
base: develop
Choose a base branch
from

Conversation

ananace
Copy link
Member

@ananace ananace commented Nov 25, 2023

Currently Puppet uses a mix of keys which are not all retrievable properly using the regular AutoYaST template, so this will directly import them from the RPM repo instead.
Puppet AIO packages also work on openSUSE, so this removes the limitation on SLES to allow proper Puppet installs to work.
Also cleans up the iPXE template and ensures BOOTIF is provided, so that boot works properly on multi-NIC devices.

@tux93
Copy link
Contributor

tux93 commented Feb 27, 2024

Tested the templates as they are in the PR at the moment and they work fine.

@sbernhard
Copy link
Contributor

@ananace can you have a look at the comments from @ekohl, please?

@ananace ananace force-pushed the opensuse-leap-15 branch 2 times, most recently from 279e203 to 490f999 Compare April 14, 2024 13:37
@ananace
Copy link
Member Author

ananace commented Apr 14, 2024

Found some time to do a few test installs, so it should be good in terms of review comments now.
Also redid the repo key handling since doing it properly seems to work again now, and also made sure it results in a repo definition that'll be correctly replaced if/when installing the release RPM.

Also disabled the resolv.conf copy step on Leap 15, since it has them both as symlinks to the same runtime file anyway, which just causes the installer to warn and sleep for five seconds.

@ananace
Copy link
Member Author

ananace commented Apr 19, 2024

Just realized that the OS specifier in the templates has been wrong the entire time, so the template sync wouldn't associate them, that's now fixed as well.

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.

5 participants