summaryrefslogtreecommitdiff
Commit message (Collapse)AuthorAgeFilesLines
* Skip more resources and set others to desired state falseresource_descriptionsTim Smith2018-10-3017-19/+26
| | | | Signed-off-by: Tim Smith <tsmith@chef.io>
* Properly deprecate visudo_path property in sudo resourceTim Smith2018-10-301-1/+1
| | | | Signed-off-by: Tim Smith <tsmith@chef.io>
* Update descriptions and add default_description fieldsTim Smith2018-10-307-10/+10
| | | | Signed-off-by: Tim Smith <tsmith@chef.io>
* Don't document state properties in systemd_unitTim Smith2018-10-301-4/+4
| | | | Signed-off-by: Tim Smith <tsmith@chef.io>
* Bump version to 15.0.16 by Chef Expeditorv15.0.16Chef Expeditor2018-10-305-11/+12
| | | | Obvious fix; these changes are the result of automation not creative thinking.
* Merge pull request #7731 from chef/windows_resourcesTim Smith2018-10-307-0/+676
|\ | | | | Add windows_certificate and windows_share resources
| * Update the introduced field to be 14.7Tim Smith2018-10-302-2/+2
| | | | | | | | | | | | I missed the 14.6 window Signed-off-by: Tim Smith <tsmith@chef.io>
| * Auto mark windows_certificate as sensitive if a password is setTim Smith2018-10-303-3/+14
| | | | | | | | | | | | We're doing this same thing in the execute resource. Signed-off-by: Tim Smith <tsmith@chef.io>
| * Description fixes from reviewTim Smith2018-10-292-6/+6
| | | | | | | | | | | | Thanks @stuartpreston Signed-off-by: Tim Smith <tsmith@chef.io>
| * Add windows_certificate and windows_share resourcesTim Smith2018-10-297-0/+665
| | | | | | | | | | | | Move two additional resources over from the windows cookbook. With these moved we're pretty close to fully deprecating the windows cookbook. Signed-off-by: Tim Smith <tsmith@chef.io>
* | Bump version to 15.0.15 by Chef Expeditorv15.0.15Chef Expeditor2018-10-305-11/+12
| | | | | | | | Obvious fix; these changes are the result of automation not creative thinking.
* | Merge pull request #7240 from chef/route_resourceTim Smith2018-10-303-17/+32
|\ \ | | | | | | Remove unused route resource properties
| * | Fix support for passing route type as a Stringroute_resourceTim Smith2018-10-292-3/+8
| | | | | | | | | | | | | | | | | | Add a test for this as well so we don't break it again later. Signed-off-by: Tim Smith <tsmith@chef.io>
| * | Remove unused route resource propertiesTim Smith2018-10-292-15/+25
| | | | | | | | | | | | | | | | | | These were added, but never actually implemented. In the world of auto generated docs this really breaks things. Signed-off-by: Tim Smith <tsmith@chef.io>
* | | Bump version to 15.0.14 by Chef Expeditorv15.0.14Chef Expeditor2018-10-305-13/+14
| | | | | | | | | | | | Obvious fix; these changes are the result of automation not creative thinking.
* | | Merge pull request #7813 from chef/inspect_symbolsLamont Granquist2018-10-301-1/+1
|\ \ \ | |_|/ |/| | Fix inspector to properly handle defaults that are symbols
| * | Fix inspector to properly handle defaults that are symbolsinspect_symbolsTim Smith2018-10-291-1/+1
|/ / | | | | | | | | | | | | We need to .inspect these so the default value is a symbol not the string version of the symbol. Signed-off-by: Tim Smith <tsmith@chef.io>
* | Bump version to 15.0.13 by Chef Expeditorv15.0.13Chef Expeditor2018-10-295-11/+12
| | | | | | | | Obvious fix; these changes are the result of automation not creative thinking.
* | Merge pull request #7794 from chef/lcg/chef-15-cleanup-cookbook-loaderTim Smith2018-10-292-70/+42
|\ \ | |/ |/| Refactor Cookbook loader logic now that we don't support merging
| * cookbook loader cleanupLamont Granquist2018-10-292-70/+42
|/ | | | | | | | this can be cleaned up by the removal of the multiple loaders per cookbook (since shadowing / merging is gone) and just because the logic was getting really weird. Signed-off-by: Lamont Granquist <lamont@scriptkiddie.org>
* Bump version to 15.0.12 by Chef Expeditorv15.0.12Chef Expeditor2018-10-295-11/+12
| | | | Obvious fix; these changes are the result of automation not creative thinking.
* Bump version to 15.0.11 by Chef Expeditorv15.0.11Chef Expeditor2018-10-295-11/+12
| | | | Obvious fix; these changes are the result of automation not creative thinking.
* Bump version to 15.0.10 by Chef Expeditorv15.0.10Chef Expeditor2018-10-295-11/+12
| | | | Obvious fix; these changes are the result of automation not creative thinking.
* 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