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 #36126 - handle install-disk in Preseed provisioning template #9627

Closed
wants to merge 1 commit into from

Conversation

olifre
Copy link
Contributor

@olifre olifre commented Feb 17, 2023

No description provided.

@theforeman-bot
Copy link
Member

Can one of the admins verify this patch?

2 similar comments
@theforeman-bot
Copy link
Member

Can one of the admins verify this patch?

@theforeman-bot
Copy link
Member

Can one of the admins verify this patch?

@theforeman-bot
Copy link
Member

Issues: #36126

Copy link
Contributor

@sbernhard sbernhard left a comment

Choose a reason for hiding this comment

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

The install-disk parameter is used here (partition table):

d-i grub-installer/bootdev string <%= host_param('install-disk') %>

can you please clarify if this is really required in the provisioning template?

@olifre
Copy link
Contributor Author

olifre commented Mar 3, 2023

@sbernhard Wow, I completely missed that — I will run further tests, to confirm that it is really not needed anymore when combining install-disk with an expert-recipe and come back to you, likely next week. Thanks for the pointer to the other location(s)!

@olifre
Copy link
Contributor Author

olifre commented Mar 6, 2023

I cross--checked and you are fully correct, the currently used partitioning template is completely sufficient. The need for this addition was caused by an outdated Preseed Partitioning template on our end, which did not yet have this feature included.
So let's close this PR (will also close the issue), sorry for the noise and thans for checking so carefully!

@olifre olifre closed this Mar 6, 2023
This pull request was closed.
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.

3 participants