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

[POC] Use a single network dracut package #1900

Closed
wants to merge 1 commit into from

Conversation

Itxaka
Copy link
Member

@Itxaka Itxaka commented Oct 9, 2023

POC of using a single network package that loads and installs the needed module in dracut so we dont have to choose between network and network-legacy and can keep it down to just one package.

This checks the dracut module to check if it exists and if it does, it sets it as dependency of the kairos-network dracut module, so it includes the proper one on each.

This is also extensible, so we can include connman, NetworkManager and systemd-networkd at any moment.

This can be provided via luet packages and the other 2 network packages dropped altogether, so we can have just one package for all flavors, no need to manually choose and/or keep it.

Could even be included as part of the immucore dracut package so immucore deals with this directly and selects the proper network stuff, but that makes it dependent on immucore releases.

Tested with:

  • fedora
  • ubuntu
  • debian
  • opensuse-leap

All of them booted with rd.neednet=1 and rd.break=pre-pivot to test inside initramfs. In all cases, the eth0 had an ip and external network connectivity.

@Itxaka Itxaka requested a review from a team October 9, 2023 14:10
@Itxaka Itxaka added the blocked label Oct 9, 2023
@jimmykarily
Copy link
Contributor

Nice! I guess this has to be move to the packages repo to be properly used right?

@Itxaka
Copy link
Member Author

Itxaka commented Oct 10, 2023

Nice! I guess this has to be move to the packages repo to be properly used right?

Yep!

Mainly here to test if it worked as expected!

Copy link
Member

@mudler mudler left a comment

Choose a reason for hiding this comment

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

nais!

@Itxaka
Copy link
Member Author

Itxaka commented Oct 10, 2023

ok, seems that this had a good reception, will create a new package with it and move kairos to use it.

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

Successfully merging this pull request may close these issues.

3 participants