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

Chocolatey missing in dependencies #691

Closed
zilchms opened this issue Nov 15, 2021 · 1 comment
Closed

Chocolatey missing in dependencies #691

zilchms opened this issue Nov 15, 2021 · 1 comment

Comments

@zilchms
Copy link
Contributor

zilchms commented Nov 15, 2021

When applied on a windows agent, chocolatey as a package provider is used. Chocolatey however is not in the dependencies of the module.

Expected Behavior

The module should tell the user that it has a dependency on chocolatey if it is used in the code.

Current Behavior

Without the chocolatey module it fails with:

Failed to apply catalog: Parameter provider failed on Package[icinga2]: Invalid package provider 'chocolatey'

Possible Solution

Either a hard dependency on puppetlabs-chocolatey in the metadata.json is used. Or since this is not necessary for all OS systems, a "soft" dependency like icinga/icinga in the README.md would be nice to have.

Context

Minor improvement, nothing serious.

Edit: According to the puppet style guide https://puppet.com/docs/puppet/7/style_guide.html#dependencies, the chocolatey dependency should be in the README.md and definitely not in the metadata.json

@lbetz
Copy link
Contributor

lbetz commented Nov 17, 2021

thx

@lbetz lbetz closed this as completed Nov 17, 2021
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