-
Notifications
You must be signed in to change notification settings - Fork 401
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
Multiple source for dracut configuration directory #2210
base: master
Are you sure you want to change the base?
Conversation
Out of curiosity: what's the use case for this? |
Let's say we have a config file And it is possible to have more than one flavor of kernel, like realtime, generic, secure etc. While creating initrd I need a way to include kernel's own custom config from modules directory and fips.conf from default location. Meaning, I want fips configs to be included in all flavors along with kernel's own custom configs. With this change I can do:
Now dracut consumes configs from both locations, fips.conf is a common config for all kernel flavors in this example, so I want to maintain a single copy of it. Same applies for config files. So, thought of making this change and keeping it generic, users can specify any number of config files and locations to consume configs from. Hope this is a valid case. If there is any other way to do it, please let me know. |
d6b2009
to
8a9c2f6
Compare
I dont see a valid usecase to support this |
I will try to explain it wrt to our system (Photon OS 3.0) We have a This creates a conf file under And each kernel provides it's own drivers list and the list is unique. Currently we are using a work around to get this done. Now I'm trying to create a conf file for each kernel under If I keep everything under So, it should be possible to use multiple config dirs while creating initrd. Please let me know if there is any other way to achieve this. |
Strictly speaking, it seems to me that your use case does not need changes to |
Yes |
I am open to consider this change, but only for directory argument and not for conf files. I think conf file argument is somewhat confusing and we should not improve it or add complexity to it. See also #1974 We already have 10+ arguments that take a list instead of a single entry. This is a niche use case but it is also a low risk, low maintenance and small change. I think having support for conf dir under |
I don't see the need for this change either. If a package wants to add its own custom dracut configuration, it can create the .conf files under |
Agreed. Having support for multiple conf dirs & files is redundant. Dropped the multiple conf file change. Thanks. |
Configuration can come from many places, users should not be restricted to keep all configuration files in one directory. Signed-off-by: Shreenidhi Shedi <[email protected]>
I have not tried this, but can you just symlink /usr/lib/dracut/dracut.conf.d to /lib/modules//dracut.conf.d for your use case (for the current kernel) ? Before considering this PR, we should understand if there is a possibility to work around it (even if not convenient) . |
What if user already has some conf files under |
This pull request changes...
Changes
Checklist
Fixes #