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

refs #37792 - :template_name in correct scope #169

Merged
merged 1 commit into from
Oct 9, 2024

Conversation

stejskalleos
Copy link
Collaborator

The template_name method must be under the Bootdisk scope; otherwise,
it overrides the method from Foreman and all templates (like Grub2 or Kickstart PXE)
have name ForemanBootdisk.

How to reproduce:

  • Do not apply this patch
  • Pull the latest bootdisk master
  • Provision host (with kickstart PXE or Grub)
  • See the rendered template

The template_name method must be under the Bootdisk
scope, otherwise it overrides the method from Foreman
and all templates (like Grub2 or Kickstart PXE)
have name ForemanBootdisk.
Copy link
Contributor

@nofaralfasi nofaralfasi left a comment

Choose a reason for hiding this comment

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

Works well for me.

@nofaralfasi nofaralfasi merged commit 2fc50d4 into theforeman:master Oct 9, 2024
22 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants