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

dracut: depend on net-lib not ifcfg #6155

Merged
merged 1 commit into from
Feb 13, 2025

Conversation

M4rtinK
Copy link
Contributor

@M4rtinK M4rtinK commented Feb 10, 2025

ifcfg is gone in dracut-ng 204. Looking at the module and its history, I'm pretty sure we don't actually use anything from the ifcfg module any more. anaconda-ifcfg.sh only uses save_netinfo, which is in net-lib. So let's turn the ifcfg dep into a net-lib dep instead.

If I missed anything and we really are still relying on the ifcfg module then we need to fix that, since it's gone now.

Related: rhbz#2343125
Resolves: RHEL-78746

RHEL 10 port of #6125

TODO

  • add a RHEL 10 Jira reference to the commit message
  • sync with upstream PR before merging if necessary

@M4rtinK
Copy link
Contributor Author

M4rtinK commented Feb 10, 2025

/kickstart-test --testtype smoke

@pvalena
Copy link

pvalena commented Feb 11, 2025

LGTM, makes sense; the module is here: https://github.com/redhat-plumbers/dracut-rhel10/tree/main/modules.d/45net-lib

ifcfg is gone in dracut-ng 204. Looking at the module and its
history, I'm pretty sure we don't actually use anything from the
ifcfg module any more. anaconda-ifcfg.sh only uses save_netinfo,
which is in net-lib. So let's turn the ifcfg dep into a net-lib
dep instead.

If I missed anything and we really are still relying on the ifcfg
module then we need to fix that, since it's gone now.

Related: rhbz#2343125
Resolves: RHEL-78746
@M4rtinK M4rtinK force-pushed the rhel-10-dracut-no-ifcfg branch from e8d6761 to 7aa63cf Compare February 12, 2025 13:10
@M4rtinK
Copy link
Contributor Author

M4rtinK commented Feb 12, 2025

/kickstart-test --testtype smoke

@M4rtinK M4rtinK added the ready to merge The PR can be merged. It should have all BZ flags required for releasing set (usually release+). label Feb 12, 2025
@rvykydal rvykydal merged commit 3fe81ba into rhinstaller:rhel-10 Feb 13, 2025
15 checks passed
@KKoukiou KKoukiou deleted the rhel-10-dracut-no-ifcfg branch February 13, 2025 13:06
achilleas-k added a commit to achilleas-k/images that referenced this pull request Feb 19, 2025
In RHEL 10 and Fedora 42, the ifcfg module was replaced by net-lib.
Anaconda changes:
- rhinstaller/anaconda#6125
- rhinstaller/anaconda#6155

Remove ifcfg from common anaconda dracut stage modules and add either
ifcfg or net-lib to each installer based on distro version.
achilleas-k added a commit to achilleas-k/images that referenced this pull request Feb 19, 2025
In RHEL 10 and Fedora 42, the ifcfg module was replaced by net-lib.
Anaconda changes:
- rhinstaller/anaconda#6125
- rhinstaller/anaconda#6155

Remove ifcfg from common anaconda dracut stage modules and add either
ifcfg or net-lib to each installer based on distro version.
achilleas-k added a commit to achilleas-k/images that referenced this pull request Feb 19, 2025
In RHEL 10 and Fedora 42, the ifcfg module was replaced by net-lib.
Anaconda changes:
- rhinstaller/anaconda#6125
- rhinstaller/anaconda#6155

Remove ifcfg from common anaconda dracut stage modules and add either
ifcfg or net-lib to each installer based on distro version.
github-merge-queue bot pushed a commit to osbuild/images that referenced this pull request Feb 20, 2025
In RHEL 10 and Fedora 42, the ifcfg module was replaced by net-lib.
Anaconda changes:
- rhinstaller/anaconda#6125
- rhinstaller/anaconda#6155

Remove ifcfg from common anaconda dracut stage modules and add either
ifcfg or net-lib to each installer based on distro version.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
ready to merge The PR can be merged. It should have all BZ flags required for releasing set (usually release+). rhel-10
Development

Successfully merging this pull request may close these issues.

5 participants