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

chef_automate resource needs to set license property as a required #165

Open
wrightp opened this issue May 12, 2017 · 3 comments
Open

chef_automate resource needs to set license property as a required #165

wrightp opened this issue May 12, 2017 · 3 comments
Labels
Triage: Support Indicates an issue that is a support question and will be redirected to other mediums.

Comments

@wrightp
Copy link

wrightp commented May 12, 2017

license needs to be required because of the following line:
https://github.com/chef-cookbooks/chef-ingredient/blob/master/resources/automate.rb#L69

@wrightp
Copy link
Author

wrightp commented May 12, 2017

@ncerny Should license be required or should the files only be created if the property is set? I'm guessing the former.

@ncerny
Copy link
Contributor

ncerny commented May 12, 2017

Well...in the spirit of not dictating how it gets there, I'd lean towards the later. But you're right, that line will fail if you don't specify it. I'm okay either way - because if it's required, we can always give the path to the file in the resource.

Thoughts? anything I'm not thinking of?

@wrightp
Copy link
Author

wrightp commented May 12, 2017

Couple things:
A) This resource won't work as it currently stands if not set
B) This resources is a construct for properly installing Automate so it's not about dictating, it's about follow through.

We need to make these things consistent throughout the resources. We either guide users to configure their environments fully or we don't require configuration properties and code the resources to only configure what's set.

@tas50 tas50 added Triage: Support Indicates an issue that is a support question and will be redirected to other mediums. and removed Type: Question labels Jan 29, 2019
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Triage: Support Indicates an issue that is a support question and will be redirected to other mediums.
Projects
None yet
Development

No branches or pull requests

4 participants