Skip to content

Make Windows driver plugin opt-in via config#720

Merged
danielsdeleo merged 2 commits into
masterfrom
make-pnp-opt-in
Feb 3, 2016
Merged

Make Windows driver plugin opt-in via config#720
danielsdeleo merged 2 commits into
masterfrom
make-pnp-opt-in

Conversation

@danielsdeleo
Copy link
Copy Markdown
Contributor

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 setting ohai.plugins[:drivers][:enabled] = true in the Chef config file.

@chefsalim
Copy link
Copy Markdown
Contributor

👍

@tas50
Copy link
Copy Markdown
Contributor

tas50 commented Feb 2, 2016

Pretty sure most windows users wanted this 3 years ago

@jaym
Copy link
Copy Markdown
Contributor

jaym commented Feb 2, 2016

should the creation of the mashes be moved into the if enabled? If not, I can change the packages plugin to behave the same way

@jaym
Copy link
Copy Markdown
Contributor

jaym commented Feb 2, 2016

The creation of the wmi object also does not need to happen unless enabled

@mwrock
Copy link
Copy Markdown
Member

mwrock commented Feb 2, 2016

👍

@danielsdeleo
Copy link
Copy Markdown
Contributor Author

should the creation of the mashes be moved into the if enabled? If not, I can change the packages plugin to behave the same way

@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 node["attribute-from-disabled-plugin"].each will blow up with NoMethodError on nil, in the other, the same code will iterate over zero items.

@jaym
Copy link
Copy Markdown
Contributor

jaym commented Feb 2, 2016

That's the first way I did it, then I decided to change it so that it was like the plugin never ran.

@danielsdeleo
Copy link
Copy Markdown
Contributor Author

I'll change to make it consistent with what you did.

@jaym
Copy link
Copy Markdown
Contributor

jaym commented Feb 3, 2016

👍

@danielsdeleo danielsdeleo merged commit 5171849 into master Feb 3, 2016
@danielsdeleo danielsdeleo deleted the make-pnp-opt-in branch February 3, 2016 01:04
@tas50 tas50 changed the title Make pnp plugin opt-in via config Make Windows driver plugin opt-in via config Feb 3, 2016
@thommay thommay added Type: Enhancement Adds new functionality. and removed Enhancement labels Jan 24, 2017
@chef chef locked and limited conversation to collaborators Nov 16, 2017
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

Type: Enhancement Adds new functionality.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

7 participants