-
Notifications
You must be signed in to change notification settings - Fork 450
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
Make Windows driver plugin opt-in via config #720
Conversation
👍 |
Pretty sure most windows users wanted this 3 years ago |
should the creation of the mashes be moved into the |
The creation of the |
👍 |
@jaym I thought that was how you did it, but maybe I was misremembering or you changed it or something? In any case I guess it's kind of a toss-up. In one case |
1b2a227
to
05e6dfe
Compare
That's the first way I did it, then I decided to change it so that it was like the plugin never ran. |
I'll change to make it consistent with what you did. |
👍 |
Makes the windows
Drivers
plugin opt-in via a config setting, similar to the packages plugin (see: #717 ). This is technically a breaking change, but we decided in this case that the plugin was delivering too much negative value so it's worth it. The plugin can be re-enabled by settingohai.plugins[:drivers][:enabled] = true
in the Chef config file.