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

udev-rules: make sure the hwdb with ifname renaming rules is around #1681

Closed
wants to merge 1 commit into from

Conversation

lkundrak
Copy link
Contributor

@lkundrak lkundrak commented Jan 5, 2022

Changes

The stable network interface naming relies on hwdb. In one particular
case actually: the Dell "idrac" interface.

The systemd-udevd module actually already includes the hwdb for hostonly
images, but we need it to work for non-hostonly images as well as those
that don't utilize the systemd-udevd module as well.

Let's make udev-rules also pull in the binary hwdb for hostonly builds.
On Fedora 35 this adds 2M compressed, 11M uncompressed. That is quite
some extra fat, but it's consistent with what systemd-udevd does.

For non-hostonly ones, pull in only what we know we need in binary form.
The extra space consumed is negligible (300B uncompressed).

Checklist

  • [ x ] I have tested it locally
  • [ x ] I have reviewed and updated any documentation if relevant
  • [ :( ] I am providing new code and test(s) for it

@github-actions github-actions bot added modules Issue tracker for all modules udev-rules Issues related to the udev-rules module labels Jan 5, 2022
The stable network interface naming relies on hwdb. In one particular
case actually: the Dell "idrac" interface.

The systemd-udevd module actually already includes the hwdb for hostonly
images, but we need it to work for non-hostonly images as well as those
that don't utilize the systemd-udevd module as well.

Let's make udev-rules also pull in the binary hwdb for hostonly builds.
On Fedora 35 this adds 2M compressed, 11M uncompressed. That is quite
some extra fat, but it's consistent with what systemd-udevd does.

For non-hostonly ones, pull in only what we know we need in binary form.
The extra space consumed is negligible (300B uncompressed).
Copy link
Collaborator

@johannbg johannbg left a comment

Choose a reason for hiding this comment

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

Eventually support for systemd in udev-rules will be dropped and it exclusively be used for non systemd based systems while systemd based system will use systemd-udevd instead that said it makes no sense to only include 20-net-ifname.hwdb and include all hwdb's only when the case is hostonly ( we dont add vendor paths to hostonly but I can see why you chose to do that there ) it's all or nothing situation so this PR wont work as is.

Now that does not mean that we cant solve what you are trying to achieve, it just means we need to be smarter how we approach it, which probably will require a bit larger scope to be able to achieve that.

@stale
Copy link

stale bot commented Feb 7, 2022

This issue is being marked as stale because it has not had any recent activity. It will be closed if no further activity occurs. If this is still an issue in the latest release of Dracut and you would like to keep it open please comment on this issue within the next 7 days. Thank you for your contributions.

@stale stale bot added the stale communication is stuck label Feb 7, 2022
@stale stale bot closed this Feb 15, 2022
@johannbg johannbg reopened this Feb 15, 2022
@stale stale bot removed the stale communication is stuck label Feb 15, 2022
@johannbg
Copy link
Collaborator

Keeping this one open until it get's cleaned up as a part of a larger scope.

@stale
Copy link

stale bot commented Mar 17, 2022

This issue is being marked as stale because it has not had any recent activity. It will be closed if no further activity occurs. If this is still an issue in the latest release of Dracut and you would like to keep it open please comment on this issue within the next 7 days. Thank you for your contributions.

@stale stale bot added the stale communication is stuck label Mar 17, 2022
@johannbg johannbg removed the stale communication is stuck label Mar 17, 2022
@stale
Copy link

stale bot commented Apr 17, 2022

This issue is being marked as stale because it has not had any recent activity. It will be closed if no further activity occurs. If this is still an issue in the latest release of Dracut and you would like to keep it open please comment on this issue within the next 7 days. Thank you for your contributions.

@stale stale bot added the stale communication is stuck label Apr 17, 2022
@johannbg johannbg removed the stale communication is stuck label Apr 18, 2022
@johannbg johannbg self-assigned this Apr 29, 2022
@stale
Copy link

stale bot commented May 30, 2022

This issue is being marked as stale because it has not had any recent activity. It will be closed if no further activity occurs. If this is still an issue in the latest release of Dracut and you would like to keep it open please comment on this issue within the next 7 days. Thank you for your contributions.

@stale stale bot added the stale communication is stuck label May 30, 2022
@johannbg johannbg removed the stale communication is stuck label May 30, 2022
@stale
Copy link

stale bot commented Jul 1, 2022

This issue is being marked as stale because it has not had any recent activity. It will be closed if no further activity occurs. If this is still an issue in the latest release of Dracut and you would like to keep it open please comment on this issue within the next 7 days. Thank you for your contributions.

@stale stale bot added the stale communication is stuck label Jul 1, 2022
@stale stale bot closed this Jul 10, 2022
pvalena added a commit to pvalena/dracut that referenced this pull request Jul 23, 2023
Adding a module to install hwdb. Further extensions might make only selected
part of hwdb installable, to save space. The module is not included by default.

Including the module adds 2MB of compressed data (on Fedora, the file has 12MB).

hwdb is needed in case of custom HW, like a keyboard/mouse or various interfaces.

Original PR: dracutdevs#1681
pvalena added a commit to pvalena/dracut that referenced this pull request Jul 23, 2023
Adding a module to install hwdb. Further extensions might make only selected
part of hwdb installable, to save space. The module is not included by default.

Including the module adds 2MB of compressed data (on Fedora, the file has 12MB).

hwdb is needed in case of custom HW, like a keyboard/mouse or various interfaces.

Original PR: dracutdevs#1681
pvalena added a commit to pvalena/dracut-rhel8 that referenced this pull request Jul 23, 2023
Adding a module to install hwdb. Further extensions might make only selected
part of hwdb installable, to save space. The module is not included by default.

Including the module adds 2MB of compressed data (on Fedora, the file has 12MB).

hwdb is needed in case of custom HW, like a keyboard/mouse or various interfaces.

Original PR: dracutdevs/dracut#1681

(Cherry-picked commit: ae2ab8c8a65c4ea4f4ad4e8c8f99d493a8f686eb)

Resolves: #1968118
pvalena added a commit to pvalena/dracut that referenced this pull request Jul 24, 2023
Adding a module to install hwdb. Further extensions might make only selected
part of hwdb installable, to save space. The module is not included by default.

Including the module adds 2MB of compressed data (on Fedora, the file has 12MB).

hwdb is needed in case of custom HW, like a keyboard/mouse or various interfaces.

Original PR: dracutdevs#1681
pvalena added a commit to pvalena/dracut-rhel8 that referenced this pull request Jul 24, 2023
Adding a module to install hwdb. Further extensions might make only selected
part of hwdb installable, to save space. The module is not included by default.

Including the module adds 2MB of compressed data (on Fedora, the file has 12MB).

hwdb is needed in case of custom HW, like a keyboard/mouse or various interfaces.

Original PR: dracutdevs/dracut#1681

(Cherry-picked commit: 062e739d89543a38d4b3e2cab155912bc4bf9e56)

Resolves: #1968118
pvalena added a commit to pvalena/dracut that referenced this pull request Jul 25, 2023
Adding a module to install hwdb. Further extensions might make only selected
part of hwdb installable, to save space. The module is not included by default.

Including the module adds 2MB of compressed data (on Fedora, the file has 12MB).

hwdb is needed in case of custom HW, like a keyboard/mouse or various interfaces.

Original PR: dracutdevs#1681
pvalena added a commit to redhat-plumbers/dracut-rhel8 that referenced this pull request Jul 25, 2023
Adding a module to install hwdb. Further extensions might make only selected
part of hwdb installable, to save space. The module is not included by default.

Including the module adds 2MB of compressed data (on Fedora, the file has 12MB).

hwdb is needed in case of custom HW, like a keyboard/mouse or various interfaces.

Original PR: dracutdevs/dracut#1681

(Cherry-picked commit: 062e739d89543a38d4b3e2cab155912bc4bf9e56)

Resolves: #1968118
pvalena added a commit to pvalena/dracut-rhel9 that referenced this pull request Aug 22, 2023
Adding a module to install hwdb. Further extensions might make only selected
part of hwdb installable, to save space. The module is not included by default.

Including the module adds 2MB of compressed data (on Fedora, the file has 12MB).

hwdb is needed in case of custom HW, like a keyboard/mouse or various interfaces.

Original PR: dracutdevs/dracut#1681

(Cherry-picked commit: 062e739d89543a38d4b3e2cab155912bc4bf9e56)

Resolves: #2233597
pvalena added a commit to redhat-plumbers/dracut-rhel9 that referenced this pull request Aug 22, 2023
Adding a module to install hwdb. Further extensions might make only selected
part of hwdb installable, to save space. The module is not included by default.

Including the module adds 2MB of compressed data (on Fedora, the file has 12MB).

hwdb is needed in case of custom HW, like a keyboard/mouse or various interfaces.

Original PR: dracutdevs/dracut#1681

(Cherry-picked commit: 062e739d89543a38d4b3e2cab155912bc4bf9e56)

Resolves: #2233597
pvalena added a commit to pvalena/dracut that referenced this pull request Sep 20, 2023
Adding a module to install hwdb. Further extensions might make only selected
part of hwdb installable, to save space. The module is not included by default.

Including the module adds 2MB of compressed data (on Fedora, the file has 12MB).

hwdb is needed in case of custom HW, like a keyboard/mouse or various interfaces.

Original PR: dracutdevs#1681
dracutng pushed a commit to dracutng/dracut-ng that referenced this pull request Apr 8, 2024
Adding a module to install hwdb. Further extensions might make only selected
part of hwdb installable, to save space. The module is not included by default.

Including the module adds 2MB of compressed data (on Fedora, the file has 12MB).

hwdb is needed in case of custom HW, like a keyboard/mouse or various interfaces.

Original PR: dracutdevs/dracut#1681
pvalena added a commit to pvalena/dracut-ng that referenced this pull request Apr 22, 2024
Module to install hwdb.bin. Further extensions might make only selected
part of hwdb installable to save space. The module is not included by default.

Including the module adds 2MB of compressed data (on Fedora, the file has 12MB).

Installing hwdb.bin is needed in case of custom HW like a keyboard/mouse, or various interfaces.

Original PR: dracutdevs/dracut#1681
pvalena added a commit to pvalena/dracut-ng that referenced this pull request Apr 22, 2024
Module to install hwdb.bin. Further extensions might make only selected
part of hwdb installable to save space. The module is not included by default.

Including the module adds 2MB of compressed data (on Fedora, the file has 12MB).

Installing hwdb.bin is needed in case of custom HW like a keyboard/mouse, or various interfaces.

Original PR: dracutdevs/dracut#1681
pvalena added a commit to pvalena/dracut-ng that referenced this pull request May 15, 2024
Module to install hwdb.bin. Further extensions might make only selected
part of hwdb installable to save space. The module is not included by default.

Including the module adds 2MB of compressed data (on Fedora, the file has 12MB).

Installing hwdb.bin is needed in case of custom HW like a keyboard/mouse, or various interfaces.

Original PR: dracutdevs/dracut#1681
pvalena added a commit to pvalena/dracut-ng that referenced this pull request May 15, 2024
Module to install hwdb.bin. Further extensions might make only selected
part of hwdb installable to save space. The module is not included by default.

Including the module adds 2MB of compressed data (on Fedora, the file has 12MB).

Installing hwdb.bin is needed in case of custom HW like a keyboard/mouse, or various interfaces.

Original PR: dracutdevs/dracut#1681
pvalena added a commit to pvalena/dracut-fedora that referenced this pull request May 15, 2024
Module to install hwdb.bin. Further extensions might make only selected
part of hwdb installable to save space. The module is not included by default.

Including the module adds 2MB of compressed data (on Fedora, the file has 12MB).

Installing hwdb.bin is needed in case of custom HW like a keyboard/mouse, or various interfaces.

Original PR: dracutdevs/dracut#1681
pvalena added a commit to pvalena/dracut-fedora that referenced this pull request May 15, 2024
Module to install hwdb.bin. Further extensions might make only selected
part of hwdb installable to save space. The module is not included by default.

Including the module adds 2MB of compressed data (on Fedora, the file has 12MB).

Installing hwdb.bin is needed in case of custom HW like a keyboard/mouse, or various interfaces.

Original PR: dracutdevs/dracut#1681
pvalena added a commit to redhat-plumbers/dracut-fedora that referenced this pull request May 16, 2024
Module to install hwdb.bin. Further extensions might make only selected
part of hwdb installable to save space. The module is not included by default.

Including the module adds 2MB of compressed data (on Fedora, the file has 12MB).

Installing hwdb.bin is needed in case of custom HW like a keyboard/mouse, or various interfaces.

Original PR: dracutdevs/dracut#1681
pvalena added a commit to pvalena/dracut-fedora that referenced this pull request May 16, 2024
Module to install hwdb.bin. Further extensions might make only selected
part of hwdb installable to save space. The module is not included by default.

Including the module adds 2MB of compressed data (on Fedora, the file has 12MB).

Installing hwdb.bin is needed in case of custom HW like a keyboard/mouse, or various interfaces.

Original PR: dracutdevs/dracut#1681
pvalena added a commit to pvalena/dracut-fedora that referenced this pull request Jun 3, 2024
Module to install hwdb.bin. Further extensions might make only selected
part of hwdb installable to save space. The module is not included by default.

Including the module adds 2MB of compressed data (on Fedora, the file has 12MB).

Installing hwdb.bin is needed in case of custom HW like a keyboard/mouse, or various interfaces.

Original PR: dracutdevs/dracut#1681
pvalena added a commit to pvalena/dracut-fedora that referenced this pull request Jun 28, 2024
Module to install hwdb.bin. Further extensions might make only selected
part of hwdb installable to save space. The module is not included by default.

Including the module adds 2MB of compressed data (on Fedora, the file has 12MB).

Installing hwdb.bin is needed in case of custom HW like a keyboard/mouse, or various interfaces.

Original PR: dracutdevs/dracut#1681
pvalena added a commit to redhat-plumbers/dracut-fedora that referenced this pull request Sep 16, 2024
Module to install hwdb.bin. Further extensions might make only selected
part of hwdb installable to save space. The module is not included by default.

Including the module adds 2MB of compressed data (on Fedora, the file has 12MB).

Installing hwdb.bin is needed in case of custom HW like a keyboard/mouse, or various interfaces.

Original PR: dracutdevs/dracut#1681
pvalena added a commit to redhat-plumbers/dracut-fedora that referenced this pull request Sep 18, 2024
Module to install hwdb.bin. Further extensions might make only selected
part of hwdb installable to save space. The module is not included by default.

Including the module adds 2MB of compressed data (on Fedora, the file has 12MB).

Installing hwdb.bin is needed in case of custom HW like a keyboard/mouse, or various interfaces.

Original PR: dracutdevs/dracut#1681
pvalena added a commit to redhat-plumbers/dracut-rhel10 that referenced this pull request Oct 10, 2024
Module to install hwdb.bin. Further extensions might make only selected
part of hwdb installable to save space. The module is not included by default.

Including the module adds 2MB of compressed data (on Fedora, the file has 12MB).

Installing hwdb.bin is needed in case of custom HW like a keyboard/mouse, or various interfaces.

Original PR: dracutdevs/dracut#1681
pvalena added a commit to redhat-plumbers/dracut-rhel10 that referenced this pull request Oct 27, 2024
Module to install hwdb.bin. Further extensions might make only selected
part of hwdb installable to save space. The module is not included by default.

Including the module adds 2MB of compressed data (on Fedora, the file has 12MB).

Installing hwdb.bin is needed in case of custom HW like a keyboard/mouse, or various interfaces.

Original PR: dracutdevs/dracut#1681
pvalena added a commit to redhat-plumbers/dracut-rhel10 that referenced this pull request Oct 27, 2024
Module to install hwdb.bin. Further extensions might make only selected
part of hwdb installable to save space. The module is not included by default.

Including the module adds 2MB of compressed data (on Fedora, the file has 12MB).

Installing hwdb.bin is needed in case of custom HW like a keyboard/mouse, or various interfaces.

Original PR: dracutdevs/dracut#1681
pvalena added a commit to pvalena/dracut-fedora that referenced this pull request Jan 20, 2025
Module to install hwdb.bin. Further extensions might make only selected
part of hwdb installable to save space. The module is not included by default.

Including the module adds 2MB of compressed data (on Fedora, the file has 12MB).

Installing hwdb.bin is needed in case of custom HW like a keyboard/mouse, or various interfaces.

Original PR: dracutdevs/dracut#1681
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
modules Issue tracker for all modules stale communication is stuck udev-rules Issues related to the udev-rules module
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants