summaryrefslogtreecommitdiff
Commit message (Collapse)AuthorAgeFilesLines
* Remove the deprecated ohai_name property from the ohai resourceohai_nameTim Smith2018-10-292-8/+0
| | | | | | This did nothing and we deprecated it already. Signed-off-by: Tim Smith <tsmith@chef.io>
* Merge pull request #7789 from chef/freebsdTim Smith2018-10-298-455/+5
|\ | | | | Remove deprecated support for FreeBSD pkg provider
| * Remove deprecated support for FreeBSD pkg providerfreebsdTim Smith2018-10-288-455/+5
| | | | | | | | | | | | We continue to support pkgng which shipped in FreeBSD 10. Signed-off-by: Tim Smith <tsmith@chef.io>
* | Merge pull request #7802 from chef/dmg_packageLamont Granquist2018-10-291-39/+51
|\ \ | | | | | | Multiple fixes to dmg_package
| * | Fix accepting the EULA not workingdmg_packageTim Smith2018-10-281-2/+5
| | | | | | | | | | | | | | | | | | Turns out it was breaking due to the quiet flag. Sadly there was a PR to turn off this flag from 3 years ago in the old cookbook repo :( Signed-off-by: Tim Smith <tsmith@chef.io>
| * | Better detect currently installed packagesTim Smith2018-10-281-3/+3
| | | | | | | | | | | | | | | | | | Pulls in https://github.com/chef-boneyard/dmg/pull/24 which avoids the current regex like method and instead requires an exact match. Signed-off-by: Tim Smith <tsmith@chef.io>
| * | memoize passphrase_cmdTim Smith2018-10-281-1/+1
| | | | | | | | | | | | | | | | | | We access this multiple times in this resource Signed-off-by: Tim Smith <tsmith@chef.io>
| * | Move dmg attached check out to a helper methodTim Smith2018-10-281-1/+7
| | | | | | | | | | | | Signed-off-by: Tim Smith <tsmith@chef.io>
| * | header property: Don't default to nil or accept nilTim Smith2018-10-281-2/+2
| | | | | | | | | | | | | | | | | | This is a old legacy leftover from LWRP-land Signed-off-by: Tim Smith <tsmith@chef.io>
| * | Move more functionality to helper methods and propertiesTim Smith2018-10-281-13/+20
| | | | | | | | | | | | | | | | | | | | | | | | More passphrase_cmd and software_license_agreement over to helper methods that we can potentially test later on. Improve the requires EULA error message Use a default value for volumes_dir that sets it to the value of the app property instead of creating a variable later. Signed-off-by: Tim Smith <tsmith@chef.io>
| * | Use full path to hdiutil and prevent finder opening the mounted dmg windowTim Smith2018-10-281-4/+4
| | | | | | | | | | | | Signed-off-by: Tim Smith <tsmith@chef.io>
| * | Don't rely on shelling out to grep to determine if EULA is necessaryTim Smith2018-10-281-3/+3
| | | | | | | | | | | | | | | | | | Do this natively in Ruby. Signed-off-by: Tim Smith <tsmith@chef.io>
| * | Remove declare_resource usageTim Smith2018-10-271-4/+5
| | | | | | | | | | | | | | | | | | This isn't necessary since this is a custom resource Signed-off-by: Tim Smith <tsmith@chef.io>
| * | Fix dmg_name to actually work the way it was meant toTim Smith2018-10-271-2/+3
| | | | | | | | | | | | | | | | | | We created a variable with the same name as the property and this probably did the right thing in Chef 11, but what we really want is dmg_name to take on the app name by default. Now it does. Signed-off-by: Tim Smith <tsmith@chef.io>
| * | Remove logic that uniquely named the remote_file resourceTim Smith2018-10-271-2/+1
| | | | | | | | | | | | | | | | | | This was probably added before use_inline_resources was a thing Signed-off-by: Tim Smith <tsmith@chef.io>
| * | Set desired state on several propertiesTim Smith2018-10-271-5/+6
| | | | | | | | | | | | Signed-off-by: Tim Smith <tsmith@chef.io>
| * | Add missing periods to the descriptionsTim Smith2018-10-271-2/+2
| | | | | | | | | | | | Signed-off-by: Tim Smith <tsmith@chef.io>
| * | Use current_value instead of the installed propertyTim Smith2018-10-271-8/+2
| |/ | | | | | | | | | | Remove the need for tracking state in the installed property Signed-off-by: Tim Smith <tsmith@chef.io>
* | Merge pull request #7803 from chef/mount_arrayLamont Granquist2018-10-291-2/+2
|\ \ | | | | | | Allow passing array to supports in mount again
| * | Allow passing array to supports in mount againmount_arrayTim Smith2018-10-271-2/+2
| |/ | | | | | | | | | | | | We try to coerce the array into a Hash, but we don't allow the Array value. This makes this actually work as we documented it. Signed-off-by: Tim Smith <tsmith@chef.io>
* | Merge pull request #7804 from chef/documentation_enhancementsLamont Granquist2018-10-295-8/+48
|\ \ | | | | | | Add skip_docs and default_description to resource properties
| * | Add skip_docs and default_description to resource propertiesdocumentation_enhancementsTim Smith2018-10-285-8/+48
| |/ | | | | | | | | | | | | | | | | skip_docs is used to specify a property that we don't want to include in the documentation. We have several of these. default_description is used to describe the default in a way we'd want to put on the docs site. This is particularly useful for describing all our lazy / computed values. I've used both of these in resources to show how I think we'll use them. Signed-off-by: Tim Smith <tsmith@chef.io>
* | Merge pull request #7797 from chef/macos_timezoneTim Smith2018-10-291-32/+60
|\ \ | | | | | | Add macOS support to the timezone resource
| * | Add macOS support to the timezone resourcemacos_timezoneTim Smith2018-10-271-32/+60
| |/ | | | | | | | | | | Moving towards a world where we properly load the current state. Adds a method that grabs the current timezone and properly fails with a warning message if we don't have the privs we need. Using converge_by and shell_out to set the timezone so we get a nice clean converge messaging vs. execute resources. Also improves the description to give a link to where users can find all the TZ options. Signed-off-by: Tim Smith <tsmith@chef.io>
* | Merge pull request #7800 from chef/appveyorLamont Granquist2018-10-291-4/+27
|\ \ | | | | | | Break out appveyor tests
| * | Break out appveyorTim Smith2018-10-271-4/+27
| |/ | | | | | | Signed-off-by: Tim Smith <tsmith@chef.io>
* | Bump version to 15.0.9 by Chef Expeditorv15.0.9Chef Expeditor2018-10-295-11/+12
| | | | | | | | Obvious fix; these changes are the result of automation not creative thinking.
* | Merge pull request #7798 from chef/require_shelloutLamont Granquist2018-10-291-0/+1
|\ \ | |/ |/| Require mixin::shellout where we use it
| * Require mixin::shellout where we use itrequire_shelloutTim Smith2018-10-271-0/+1
|/ | | | | | We're not requiring it here even though we extend it below. Signed-off-by: Tim Smith <tsmith@chef.io>
* Bump version to 15.0.8 by Chef Expeditorv15.0.8Chef Expeditor2018-10-275-11/+12
| | | | Obvious fix; these changes are the result of automation not creative thinking.
* Merge pull request #7796 from chef/lcg/fix-external-testsTim Smith2018-10-261-1/+1
|\ | | | | add GEMFILE_MOD to pin ohai to github master
| * add GEMFILE_MOD to pin ohai to github masterlcg/fix-external-testsLamont Granquist2018-10-261-1/+1
| | | | | | | | Signed-off-by: Lamont Granquist <lamont@scriptkiddie.org>
* | Bump version to 15.0.7 by Chef Expeditorv15.0.7Chef Expeditor2018-10-275-11/+12
| | | | | | | | Obvious fix; these changes are the result of automation not creative thinking.
* | Merge pull request #7793 from chef/lcg/chef-15-shell-out-edge-conditionLamont Granquist2018-10-261-1/+3
|\ \ | |/ |/| shell_out auto-timeout still needs to be restricted to only providers
| * this still needs to be restricted to only providersLamont Granquist2018-10-261-1/+3
|/ | | | Signed-off-by: Lamont Granquist <lamont@scriptkiddie.org>
* Bump version to 15.0.6 by Chef Expeditorv15.0.6Chef Expeditor2018-10-275-11/+12
| | | | Obvious fix; these changes are the result of automation not creative thinking.
* Merge pull request #7792 from chef/lcg/chef-15-cookbook-loaderLamont Granquist2018-10-265-142/+27
|\ | | | | Remove cookbook merging/shadowing from the cookbooker loader
| * bit of cleanuplcg/chef-15-cookbook-loaderLamont Granquist2018-10-262-3/+4
| | | | | | | | Signed-off-by: Lamont Granquist <lamont@scriptkiddie.org>
| * Remove cookbook merging/shadowing from the cookbooker loaderLamont Granquist2018-10-265-140/+24
|/ | | | | | This has been deprecated since forever. Signed-off-by: Lamont Granquist <lamont@scriptkiddie.org>
* Bump version to 15.0.5 by Chef Expeditorv15.0.5Chef Expeditor2018-10-265-11/+12
| | | | Obvious fix; these changes are the result of automation not creative thinking.
* Merge pull request #7790 from chef/require_recipeTim Smith2018-10-261-6/+0
|\ | | | | Remove legacy require_recipe DSL method
| * Remove legacy require_recipe DSL methodTim Smith2018-10-261-6/+0
| | | | | | | | | | | | | | We've been warning on this one for YEARS and we have a Foodcritic rule for it as well, which has been around since before I was at Chef. Signed-off-by: Tim Smith <tsmith@chef.io>
* | Bump version to 15.0.4 by Chef Expeditorv15.0.4Chef Expeditor2018-10-265-11/+12
| | | | | | | | Obvious fix; these changes are the result of automation not creative thinking.
* | Merge pull request #7788 from chef/lcg/chef-15-shell-out-removalLamont Granquist2018-10-265-302/+3
|\ \ | |/ |/| Do the shell_out deprecations for Chef-15.
| * remove more deprecated specsLamont Granquist2018-10-261-30/+0
| | | | | | | | Signed-off-by: Lamont Granquist <lamont@scriptkiddie.org>
| * more spec removallcg/chef-15-shell-out-removalLamont Granquist2018-10-261-56/+0
| | | | | | | | Signed-off-by: Lamont Granquist <lamont@scriptkiddie.org>
| * delete the argument that will go away in chef 15Lamont Granquist2018-10-261-1/+0
| | | | | | | | | | | | almost forgot to delete it. Signed-off-by: Lamont Granquist <lamont@scriptkiddie.org>
| * Do the shell_out deprecations for Chef-15.Lamont Granquist2018-10-263-215/+3
| | | | | | | | | | | | | | | | Whoever wrote the what-to-do-in-Chef-15 notes did a pretty decent job of it, but the specs for the timeouts could have used a pointer into the code where the TODOs were located, I got lost there for a few minutes... Signed-off-by: Lamont Granquist <lamont@scriptkiddie.org>
* | Bump version to 15.0.3 by Chef Expeditorv15.0.3Chef Expeditor2018-10-265-13/+14
| | | | | | | | Obvious fix; these changes are the result of automation not creative thinking.
* | Merge pull request #7784 from chef/descriptionsTim Smith2018-10-268-17/+17
|\ \ | | | | | | Update description fields from the docs site