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

cannot change modprobe_file name #11

Open
bcg62 opened this issue Jun 7, 2017 · 3 comments
Open

cannot change modprobe_file name #11

bcg62 opened this issue Jun 7, 2017 · 3 comments

Comments

@bcg62
Copy link

bcg62 commented Jun 7, 2017

I have a situation where an rpm drops its own modprobe.d conf file which i would like to respect and as per its documentation it recommends adding a second file, something like:

modprobe.d/${module}_options.conf

As far as i can tell this currently cannot be done

@jmauro
Copy link
Contributor

jmauro commented Jun 7, 2017

Hello @bcg62,
I understand why the documentation wrote this, when you don't use any configuration management, when you will update the RPM, your personal settings will stay since there is an another file. But when using chef, each time the file will be overwritten by the rpm, chef will make sure the settings will be re-applied.

You would personally recommend putting everything in one single file by module, less confusing, wouldn't you?

Regards,
JM

@bcg62
Copy link
Author

bcg62 commented Jun 8, 2017

This is the file that ships with the rpm. Its installed as /etc/modprobe.d/onload.conf and the module name is onload

https://github.com/majek/openonload/blob/master/scripts/onload_misc/onload_modprobe.conf

It's not possible to encapsulate all of this with this cookbook/chef nor is it worth the effort. Also i would have to check every rpm release and keep chef in sync.

I have a handful of module parameters i drop in a different filename which keeps this simple and easy.

I could always move /etc/modprobe.d/onload.conf to something else so this cookbook doesnt overwrite it but now i have to go though the effort of subscribing or notifying an action anytime an rpm is installed or updated and also as you have stated chef will cobber the file the rpm provides.

@jmauro
Copy link
Contributor

jmauro commented Jun 11, 2017

Hello @bcg62,

Why can't you define a module called onload_settings with the onboot option to false and the check_availability to false? Wouldn't this do exactly what you want?

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

No branches or pull requests

2 participants