Skip to content

Add AutoYaST and Autoinstall provisioning templates#4892

Open
maximiliankolb wants to merge 1 commit into
theforeman:masterfrom
ATIX-AG:autoyast_and_autoinstall_templates
Open

Add AutoYaST and Autoinstall provisioning templates#4892
maximiliankolb wants to merge 1 commit into
theforeman:masterfrom
ATIX-AG:autoyast_and_autoinstall_templates

Conversation

@maximiliankolb
Copy link
Copy Markdown
Contributor

What changes are you introducing?

This patch adds AutoYaST templates for SLES 15 and below, and Preseed Autoinstall templates for Ubuntu 20.04 and above.

Why are you introducing these changes? (Explanation, links to references, issues, etc.)

  • Enterprise Linux: Always Kickstart
  • Debian: Always Preseed
  • Ubuntu 18.04: Preseed
  • Ubuntu 20.04+: Preseed Autoinstall
  • SLES 12/15: AutoYaST
  • SLES 16+: Agama

Anything else to add? (Considerations, potential downsides, alternative solutions you have explored, etc.)

The "Preseed default" template in foreman for "develop" does not contain the "puppet_setup" snippet:

$ rg --files-with-matches "puppet_setup" | sort -u

The "puppetlabs_repo" snippet is not included in any default provisioning templates for SLES (only cloud init-based templates):

$ rg --files-with-matches "puppetlabs_repo" | sort -u

New custom provisioning snippet for Ubuntu that uses "Preseed Autoinstall" templates compared to "Preseed" for Debian:

$ cp -i guides/common/modules/ref_custom-provisioning-snippet-example-for-debian.adoc guides/common/modules/ref_custom-provisioning-snippet-example-for-ubuntu.adoc

Contributor checklists

  • I am okay with my commits getting squashed when you merge this PR.
  • I am familiar with the contributing guidelines.

Please cherry-pick my commits into:

  • Foreman 3.19/Katello 4.21
  • Foreman 3.18/Katello 4.20 (Satellite 6.19)
  • Foreman 3.17/Katello 4.19
  • Foreman 3.16/Katello 4.18 (Satellite 6.18; orcharhino 7.6, 7.7, and 7.8)
  • Foreman 3.15/Katello 4.17
  • Foreman 3.14/Katello 4.16 (Satellite 6.17; orcharhino 7.4; orcharhino 7.5)
  • Foreman 3.13/Katello 4.15 (EL9 only)
  • Foreman 3.12/Katello 4.14 (Satellite 6.16; orcharhino 7.2 on EL9 only; orcharhino 7.3)
  • We do not accept PRs for Foreman older than 3.12.

@github-actions github-actions Bot added Needs tech review Requires a review from the technical perspective Needs style review Requires a review from docs style/grammar perspective Needs testing Requires functional testing labels Jun 2, 2026
@github-actions
Copy link
Copy Markdown

github-actions Bot commented Jun 2, 2026

The PR preview for 16b1da3 is available at theforeman-foreman-documentation-preview-pr-4892.surge.sh

The following output files are affected by this PR:

show diff

show diff as HTML

@maximiliankolb maximiliankolb force-pushed the autoyast_and_autoinstall_templates branch from 7840143 to cd71b65 Compare June 2, 2026 09:35
Copy link
Copy Markdown
Contributor

@Lennonka Lennonka left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Overall LGTM, just a few nitpicks.

Comment thread guides/common/modules/con_content-types-in-project.adoc
Comment thread guides/common/modules/ref_glossary-of-terms-used-in-project.adoc Outdated
Comment thread guides/common/modules/ref_custom-provisioning-snippet-example-for-ubuntu.adoc Outdated
Comment thread guides/common/modules/con_using-activation-keys-for-host-registration.adoc Outdated
Comment thread guides/common/modules/con_using-activation-keys-for-host-registration.adoc Outdated
@Lennonka Lennonka added the Waiting on contributor Requires an action from the author label Jun 2, 2026
This patch adds AutoYaST templates for SLES 15 and below, and Preseed
Autoinstall templates for Ubuntu 20.04 and above.

The "Preseed default" template in foreman for "develop" does not contain
the "puppet_setup" snippet:

$ rg --files-with-matches "puppet_setup" | sort -u

The "puppetlabs_repo" snippet is not included in any default
provisioning templates for SLES (only cloud init-based templates):

$ rg --files-with-matches "puppetlabs_repo" | sort -u

New custom provisioning snippet for Ubuntu that uses "Preseed
Autoinstall" templates compared to "Preseed" for Debian:

$ cp -i guides/common/modules/ref_custom-provisioning-snippet-example-for-debian.adoc guides/common/modules/ref_custom-provisioning-snippet-example-for-ubuntu.adoc
@maximiliankolb maximiliankolb force-pushed the autoyast_and_autoinstall_templates branch from cd71b65 to 16b1da3 Compare June 3, 2026 06:43
@pr-processor pr-processor Bot added Needs re-review and removed Waiting on contributor Requires an action from the author labels Jun 3, 2026
Copy link
Copy Markdown
Contributor Author

@maximiliankolb maximiliankolb left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I applied some feedback and double checked other suggestions which I did not apply based on my findings in foreman.

Ready for re-review.

Comment thread guides/common/modules/con_content-types-in-project.adoc
Comment thread guides/common/modules/con_using-activation-keys-for-host-registration.adoc Outdated
Comment thread guides/common/modules/con_using-activation-keys-for-host-registration.adoc Outdated
Comment thread guides/common/modules/ref_custom-provisioning-snippet-example-for-ubuntu.adoc Outdated
Comment thread guides/common/modules/ref_glossary-of-terms-used-in-project.adoc Outdated
Copy link
Copy Markdown
Contributor

@Lennonka Lennonka left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM

@Lennonka Lennonka added style review done No issues from docs style/grammar perspective and removed Needs style review Requires a review from docs style/grammar perspective labels Jun 3, 2026
@maximiliankolb
Copy link
Copy Markdown
Contributor Author

@Thorben-D Can you please review my PR?

Copy link
Copy Markdown

@Thorben-D Thorben-D left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Tech review ✅
Looks good to me too. Quite the improvement style wise!

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Needs tech review Requires a review from the technical perspective Needs testing Requires functional testing style review done No issues from docs style/grammar perspective

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants