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: fix running with pre-v103 Dracut #501

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

Conversation

oldium
Copy link
Contributor

@oldium oldium commented Nov 9, 2024

Dracut v103+ moved SystemD cryptsetup code into a separate module, so detect it and when found, use it.

Clevis units do not depend on cryptsetup.target, so do not install it. The target cryptsetup.target is installed by both 90systemd-cryptsetup in v103+ and 01systemd-cryptsetup in v105+, so there is no need to install it by Clevis.

Fixes: #493

Note: Tested with Dracut v102 (Fedora 40) and v103 (Fedora Rawhide).

@oldium oldium force-pushed the fix/pre_v103_dracut branch 2 times, most recently from 00a3403 to d81d61f Compare November 9, 2024 13:23
Dracut v103+ moved SystemD cryptsetup code into a separate module, so
detect it and when found, use it.

Clevis units do not depend on `cryptsetup.target`, so do not install it.
The target `cryptsetup.target` is installed by both
[90systemd-cryptsetup][1] in v103+ and [01systemd-cryptsetup][2] in v105+,
so there is no need to install it by Clevis.

[1]: https://github.com/dracut-ng/dracut-ng/blob/3fce598fb45aa5618cdf885eb48cf327104ffcb8/modules.d/90systemd-cryptsetup/module-setup.sh#L53
[2]: https://github.com/dracut-ng/dracut-ng/blob/a2669c447bd5e0406f55efcc8c9c58be89458b08/modules.d/01systemd-cryptsetup/module-setup.sh#L54

Signed-off-by: Oldřich Jedlička <[email protected]>
@oldium oldium force-pushed the fix/pre_v103_dracut branch from d81d61f to 28e3d66 Compare December 3, 2024 21:58
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.

1 participant