summaryrefslogtreecommitdiff
path: root/lib/chef/resource
Commit message (Collapse)AuthorAgeFilesLines
* Remove erl_call and deploy resourcesdeprecated_resourcesTim Smith2018-01-204-596/+0
| | | | | | We deprecated these for removal in Chef 14. Signed-off-by: Tim Smith <tsmith@chef.io>
* fixed functional testdheerajd-msys2018-01-161-1/+1
| | | | Signed-off-by: dheerajd-msys <dheeraj.dubey@msystechnologies.com>
* Fix style errordheerajd-msys2018-01-161-1/+0
| | | | Signed-off-by: dheerajd-msys <dheeraj.dubey@msystechnologies.com>
* Fix rspecsdheerajd-msys2018-01-161-1/+2
| | | | Signed-off-by: dheerajd-msys <dheeraj.dubey@msystechnologies.com>
* fix validation for idle_timedheerajd-msys2018-01-161-2/+4
| | | | Signed-off-by: dheerajd-msys <dheeraj.dubey@msystechnologies.com>
* Merge pull request #6688 from ↵Bryan McLellan2018-01-081-31/+6
|\ | | | | | | | | MsysTechnologiesllc/vasundhara/chef#6638_windows_task_not_fully_idempotent Fix windows_task resource not being idempotent for random_delay and execution_time_limit
| * Fixed review comments.Vasu11052018-01-051-4/+5
| | | | | | | | Signed-off-by: Vasu1105 <vasundhara.jagdale@msystechnologies.com>
| * Added new specs and updated code to handle more scenariosVasu11052018-01-051-4/+5
| | | | | | | | Signed-off-by: Vasu1105 <vasundhara.jagdale@msystechnologies.com>
| * chef#6638 fixed random_delay and execution_time_limit property not ↵Vasu11052018-01-051-28/+1
| | | | | | | | | | | | idempotent for windows_task resource Signed-off-by: Vasu1105 <vasundhara.jagdale@msystechnologies.com>
* | Merge pull request #6627 from ↵Thom May2018-01-051-1/+2
|\ \ | |/ |/| | | | | MsysTechnologiesllc/vasundhara/issue_6073_allow_forcing_sensitive_to_false_on_execute_or_batch_resource [MSYS-725] [chef#6073] allow forcing sensitive to false on execute and batch resource if password is set.
| * [MSYS-725] [chef#6073] allow forcing senistive to false on execute and batch ↵Vasu11052017-12-261-1/+2
| | | | | | | | | | | | resource Signed-off-by: Vasu1105 <vasundhara.jagdale@msystechnologies.com>
* | Merge pull request #6699 from chef/windows_pathThom May2018-01-031-10/+2
|\ \ | | | | | | Modernize windows_path resource
| * | Don't restrict the provides to os: windowswindows_pathTim Smith2018-01-021-1/+1
| | | | | | | | | | | | Signed-off-by: Tim Smith <tsmith@chef.io>
| * | Modernize windows_path resourceTim Smith2017-12-211-9/+1
| |/ | | | | | | Signed-off-by: Tim Smith <tsmith@chef.io>
* | Merge pull request #6710 from chef/useless_regexTim Smith2018-01-021-1/+1
|\ \ | | | | | | Remove a useless regex in zypper_repository resource
| * | Remove a useless regex in zypper_repositoryuseless_regexTim Smith2017-12-211-1/+1
| |/ | | | | | | | | | | Pretty sure I mistakenly copied this from yum_repository. There's no real point and it's just going to slow things down. Signed-off-by: Tim Smith <tsmith@chef.io>
* | Deprecate erl_calldeprecate_erl_callTim Smith2018-01-011-0/+5
|/ | | | | | This implements RFC 98 from https://github.com/chef/chef-rfc Signed-off-by: Tim Smith <tsmith@chef.io>
* Merge pull request #6698 from chef/launchdThom May2017-12-201-4/+2
|\ | | | | Modernize launchd resource
| * Modernize launchd resourcelaunchdTim Smith2017-12-191-4/+2
| | | | | | | | Signed-off-by: Tim Smith <tsmith@chef.io>
* | Merge pull request #6697 from chef/yum_repo_cleanupTim Smith2017-12-191-22/+22
|\ \ | | | | | | Fix yum_repository allowing priority of 0 and remove string regexes
| * | Fix priority to accept 1-99 not 0-99yum_repo_cleanupTim Smith2017-12-191-1/+1
| | | | | | | | | | | | Signed-off-by: Tim Smith <tsmith@chef.io>
| * | Remove the regex from every string propertyTim Smith2017-12-191-21/+21
| |/ | | | | | | | | | | We copied this in from the cookbook and I assume at one point there was a reason for it, but I can't think of one now. Signed-off-by: Tim Smith <tsmith@chef.io>
* | Add descriptions and @since comments to all resourcesTim Smith2017-12-1969-3/+227
|/ | | | | | From the docs site Signed-off-by: Tim Smith <tsmith@chef.io>
* Merge pull request #6693 from chef/cleanupTim Smith2017-12-191-26/+8
|\ | | | | Modernize directory resource
| * Chefstyle fixTim Smith2017-12-181-1/+1
| | | | | | | | Signed-off-by: Tim Smith <tsmith@chef.io>
| * Modernize directory resourceTim Smith2017-12-181-26/+8
| | | | | | | | Signed-off-by: Tim Smith <tsmith@chef.io>
* | Merge pull request #6684 from chef/ifconfig_resourceThom May2017-12-191-150/+23
|\ \ | | | | | | Modernize the ifconfig resource
| * | Fix identity on device propertyifconfig_resourceTim Smith2017-12-181-2/+1
| | | | | | | | | | | | Signed-off-by: Tim Smith <tsmith@chef.io>
| * | Modernize the ifconfig resourceTim Smith2017-12-181-149/+23
| |/ | | | | | | Signed-off-by: Tim Smith <tsmith@chef.io>
* | Merge pull request #6690 from chef/tm+nk/verify-commandThom May2017-12-191-0/+10
|\ \ | |/ |/| Slight improvements to validation failures
| * feed chefstyletm+nk/verify-commandThom May2017-12-181-1/+1
| | | | | | | | Signed-off-by: Thom May <thom@chef.io>
| * Slight improvements to validation failures. Shows the failing command if ↵Noah Kantrowitz2017-12-181-0/+10
| | | | | | | | | | possible and doesn't show anything for sensitive resources to be safe. Refs https://github.com/chef/chef/issues/4793 and ping @chef/client-core for review.
* | Merge pull request #6685 from chef/osx_profileTim Smith2017-12-181-40/+10
|\ \ | | | | | | Modernize osx_profile resource
| * | Modernize osx_profile resourceTim Smith2017-12-151-40/+10
| | | | | | | | | | | | Signed-off-by: Tim Smith <tsmith@chef.io>
* | | Update lazy usage per code reviewresource_provider_magicTim Smith2017-12-181-1/+1
| | | | | | | | | | | | Signed-off-by: Tim Smith <tsmith@chef.io>
* | | Modernize cookbook_file and expand specsTim Smith2017-12-181-15/+4
| |/ |/| | | | | | | | | | | | | Don't hardcode the provider since the provider already has a provides line Test the source logic default properly by creating the resource name as a absolute path, which is how the user would use it Signed-off-by: Tim Smith <tsmith@chef.io>
* | Merge pull request #6683 from chef/reboot_resourceThom May2017-12-181-20/+14
|\ \ | | | | | | Modernize reboot resource and add spec
| * | Modernize reboot resource and add specTim Smith2017-12-151-20/+14
| |/ | | | | | | Signed-off-by: Tim Smith <tsmith@chef.io>
* | Fix bugs in handling 'source' in msu_package and cab_packageTim Smith2017-12-152-3/+4
|/ | | | | | | | | | | | | The two issues fixed here: 1) cab_package did not require source, but didn't set it from the resource name so cab_package 'xyz.cab' would just fail since source was nil 2) msu_package would coerce the resource name and not package_name so if you overwrote the package name you would also have to set the source, although you wouldn't realize that until it failed oddly So the solution here is to set the source value to package_name by default and that gets coerced into a path as was previously done. This adds tests for this behavior and the coerce behavior itself to both resources. This fixes issue #6674 Signed-off-by: Tim Smith <tsmith@chef.io>
* Merge pull request #6676 from chef/modernize_log_resourceTim Smith2017-12-151-46/+16
|\ | | | | Modernize the log resource
| * Remove author yard commentsmodernize_log_resourceTim Smith2017-12-151-3/+0
| | | | | | | | Signed-off-by: Tim Smith <tsmith@chef.io>
| * Modernize the log resourceTim Smith2017-12-151-46/+19
| | | | | | | | | | | | General cleanup Signed-off-by: Tim Smith <tsmith@chef.io>
* | Fix for readability and a failing testTim Smith2017-12-152-2/+3
| | | | | | | | Signed-off-by: Tim Smith <tsmith@chef.io>
* | Avoid a few initializers in resources by using the DSL we haveTim Smith2017-12-156-28/+11
|/ | | | | | Tip of the iceberg here, but it's the low hanging fruit Signed-off-by: Tim Smith <tsmith@chef.io>
* Merge pull request #6665 from chef/yard2Tim Smith2017-12-144-3/+28
|\ | | | | Add a few more YARD comments to resources
| * Cleanup some YARD warningsyard2Tim Smith2017-12-131-1/+1
| | | | | | | | Signed-off-by: Tim Smith <tsmith@chef.io>
| * Add a few more comments to resourcesTim Smith2017-12-133-2/+27
| | | | | | | | | | | | I try to add them as I read through resources Signed-off-by: Tim Smith <tsmith@chef.io>
* | Don't use .match? which is Ruby 2.4+ onlyTim Smith2017-12-141-2/+4
|/ | | | | | | Added a todo to fix this when Ruby 2.3 support goes away since it's 3x faster Signed-off-by: Tim Smith <tsmith@chef.io>
* Fix the time regex and improve the error messageTim Smith2017-12-081-1/+1
| | | | Signed-off-by: Tim Smith <tsmith@chef.io>
* These are all input validation errors so they should raise ArgumentError not ↵more_taskTim Smith2017-12-081-7/+5
| | | | | | RuntimeError Signed-off-by: Tim Smith <tsmith@chef.io>