Skip to content
This repository has been archived by the owner on Jun 10, 2019. It is now read-only.

Add PNIN as a system manifest parameter. #485

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

Conversation

john-pierce
Copy link
Contributor

@john-pierce john-pierce commented Jul 16, 2018

Fixes: #349

Adds a new boolean parameter indicating weather to use PNIN or not to
the system section of the manifest.

The default, False enables tasks for extlinux and grub to pass
net.ifnames=0 biosdevnames=0 parameters to the kernel.

Previously for grub PNIN was disabled for stretch and later. This
remains the default but setting system.pnin True will allow the behavior
to be overridden. Extlinux had no prior special behavior,
it now mirrors grub.

Fixes: andsens#349

Adds a new boolean parameter indicating weather to use PNIN or not to
the system section of the manifest.

The default, False enables tasks for extlinux and grub to pass
net.ifnames=0 biosdevnames=0 parameters to the kernel.

Previously for grub PNIN was disabled for stretch and later. This
remains the default but setting system.pnin True will allow the behavior
to be overridden. The additional parameters have no effect on earlier
releases are passed regardless. Extlinux had no prior special behavior,
it now mirrors grub.
@john-pierce
Copy link
Contributor Author

PR description edited to more accurately reflect the implementation, I changed the behavior before submitting without changing the commit message.

Edit: The additional parameters have no effect on earlier
releases are passed regardless.

The parameters are only passed to extlinux if pnin is False or unset and release is stretch or greater.

Feel free to edit the commit message while merging.

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

DisablePNIN hardwired to debian version > stretch
1 participant