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

Fixing the interface names across OS flavors #177

Merged
merged 2 commits into from
Oct 3, 2022
Merged

Conversation

vipsharm
Copy link
Collaborator

@vipsharm vipsharm commented Oct 3, 2022

added net.ifnames=1 to kernel params.

Signed-off-by: vipsharm sharma.vipin@gmail.com

added net.ifnames=1 to kernel params.

Signed-off-by: vipsharm <sharma.vipin@gmail.com>
@netlify
Copy link

netlify bot commented Oct 3, 2022

Deploy Preview for kairos-io canceled.

Name Link
🔨 Latest commit 7af2a32
🔍 Latest deploy log https://app.netlify.com/sites/kairos-io/deploys/633b38d284441800072a5235

@vipsharm vipsharm requested review from mudler and paynejacob October 3, 2022 17:48
paynejacob
paynejacob previously approved these changes Oct 3, 2022
set kernelcmd="console=tty1 root=live:CDLABEL=$recoverylabel rd.live.dir=/ rd.live.squashimg=$img panic=5"
else
# Boot arguments when the image is used as active/passive
set kernelcmd="console=tty1 root=LABEL=$label net.ifnames=1 iso-scan/filename=$img panic=5 security=selinux rd.cos.oemlabel=COS_OEM selinux=1"
Copy link
Contributor

Choose a reason for hiding this comment

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

were these se linux args already there?

Copy link
Collaborator Author

Choose a reason for hiding this comment

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

I copied the default elemental bootargs.cfg from documentation and added net.ifnames=1
Default values copied from here
https://rancher.github.io/elemental-toolkit/docs/customizing/configure_grub/

Signed-off-by: vipsharm <sharma.vipin@gmail.com>
@mudler mudler merged commit 880474e into master Oct 3, 2022
@mudler mudler deleted the NetworkNamesFix branch October 3, 2022 21:46
@mudler mudler mentioned this pull request Oct 5, 2022
28 tasks
mudler added a commit that referenced this pull request Oct 20, 2022
Use cos initrd module by default
mudler added a commit that referenced this pull request Oct 20, 2022
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.

3 participants