summaryrefslogtreecommitdiff
Commit message (Collapse)AuthorAgeFilesLines
* feed chefstyletm+nk/verify-commandThom May2017-12-182-2/+2
| | | | Signed-off-by: Thom May <thom@chef.io>
* Slight improvements to validation failures. Shows the failing command if ↵Noah Kantrowitz2017-12-184-6/+38
| | | | | 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.
* Bump version to 13.6.42 by Expeditorv13.6.42Chef Expeditor2017-12-185-11/+12
|
* Merge pull request #6660 from chef/tm/credentialsThom May2017-12-1812-100/+287
|\ | | | | implement credential management
| * ensure that the Config object is reset before each testtm/credentialsThom May2017-12-181-0/+1
| | | | | | | | Signed-off-by: Thom May <thom@chef.io>
| * respond to review requestsThom May2017-12-182-1/+21
| | | | | | | | Signed-off-by: Thom May <thom@chef.io>
| * Specify a profile on the command lineThom May2017-12-185-4/+18
| | | | | | | | Signed-off-by: Thom May <thom@chef.io>
| * fixup! implement credential managementThom May2017-12-151-1/+1
| |
| * ensure we set up the raw key correctlyThom May2017-12-151-0/+1
| | | | | | | | Signed-off-by: Thom May <thom@chef.io>
| * Generate a credentials fileThom May2017-12-142-96/+22
| | | | | | | | Signed-off-by: Thom May <thom@chef.io>
| * implement credential managementThom May2017-12-136-1/+226
| | | | | | | | Signed-off-by: Thom May <thom@chef.io>
* | Bump version to 13.6.41 by Expeditorv13.6.41Chef Expeditor2017-12-185-11/+12
| |
* | Merge pull request #6683 from chef/reboot_resourceThom May2017-12-183-20/+78
|\ \ | | | | | | Modernize reboot resource and add spec
| * | Modernize reboot resource and add specTim Smith2017-12-153-20/+78
| | | | | | | | | | | | Signed-off-by: Tim Smith <tsmith@chef.io>
* | | Bump version to 13.6.40 by Expeditorv13.6.40Chef Expeditor2017-12-185-11/+12
| | |
* | | Merge pull request #6686 from chef/msu_and_cab_defaultsThom May2017-12-184-12/+37
|\ \ \ | |/ / |/| | Fix bugs in handling 'source' in msu_package and cab_package
| * | These specs can't be written in a cross platform waymsu_and_cab_defaultsTim Smith2017-12-162-10/+0
| | | | | | | | | | | | | | | | | | The path helper does exactly what we expect it to and that makes this only work on *nix Signed-off-by: Tim Smith <tsmith@chef.io>
| * | Fix bugs in handling 'source' in msu_package and cab_packageTim Smith2017-12-154-11/+46
|/ / | | | | | | | | | | | | | | | | | | | | | | | | 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>
* | Bump version to 13.6.39 by Expeditorv13.6.39Chef Expeditor2017-12-155-11/+12
| |
* | Merge pull request #6673 from chef/shain/move_git_and_docker_cookbook_testsLamont Granquist2017-12-156-30/+10
|\ \ | | | | | | Move docker and git top cookbook tests to travis
| * | Move git and docker setup top cookbook tests to travis.shain/move_git_and_docker_cookbook_testsScott Hain2017-12-156-30/+10
| | | | | | | | | | | | Signed-off-by: Scott Hain <shain@chef.io>
* | | Bump version to 13.6.38 by Expeditorv13.6.38Chef Expeditor2017-12-155-11/+12
| | |
* | | Merge pull request #6676 from chef/modernize_log_resourceTim Smith2017-12-152-59/+19
|\ \ \ | | | | | | | | Modernize the log resource
| * | | Remove author yard commentsmodernize_log_resourceTim Smith2017-12-152-6/+0
| | | | | | | | | | | | | | | | Signed-off-by: Tim Smith <tsmith@chef.io>
| * | | Add back the provides to the providerTim Smith2017-12-151-0/+2
| | | | | | | | | | | | | | | | | | | | | | | | Magic doesn't work here since the class name isn't magical Signed-off-by: Tim Smith <tsmith@chef.io>
| * | | Modernize the log resourceTim Smith2017-12-152-61/+25
| | | | | | | | | | | | | | | | | | | | | | | | General cleanup Signed-off-by: Tim Smith <tsmith@chef.io>
* | | | Bump version to 13.6.37 by Expeditorv13.6.37Chef Expeditor2017-12-155-11/+12
| | | |
* | | | Merge pull request #6671 from chef/resource_fixesTim Smith2017-12-158-32/+16
|\ \ \ \ | |/ / / |/| | | Avoid a few initializers in resources by using the DSL we have
| * | | Fix failing msu_package resource specsTim Smith2017-12-152-5/+5
| | | | | | | | | | | | | | | | | | | | | | | | | | | | Actions are arrays now Since we properly set source as a name property name gets coerced into a source path as expected. This was a bug before. This also uncovered #6674 which I'll fix in another PR. 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 #6677 from chef/fedora_travisThom May2017-12-152-3/+3
|\ \ \ | | | | | | | | Remove the ciphers that openSSH 7.6 doesn't support
| * | | Remove the ciphers that openSSH 7.6 doesn't supportfedora_travisTim Smith2017-12-142-3/+3
|/ / / | | | | | | | | | | | | | | | fedora-latest just went to Fedora 27 and blowfish / cast have been removed with the inclusion of openSSH 7.6. The cookbook is fine, but we set this here. Signed-off-by: Tim Smith <tsmith@chef.io>
* | | Merge pull request #6665 from chef/yard2Tim Smith2017-12-1417-43/+76
|\ \ \ | | | | | | | | Add a few more YARD comments to resources
| * | | Cleanup some YARD warningsyard2Tim Smith2017-12-1314-41/+49
| | | | | | | | | | | | | | | | 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>
* | | | Bump version to 13.6.36 by Expeditorv13.6.36Chef Expeditor2017-12-155-11/+12
| | | |
* | | | Merge pull request #6675 from chef/windows_task_ruby23Lamont Granquist2017-12-141-2/+4
|\ \ \ \ | |_|/ / |/| | | Don't use .match? which is Ruby 2.4+ only in windows_task
| * | | 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>
* | | Merge pull request #6669 from chef/schisamo/update-codeownersThom May2017-12-141-1/+1
|\ \ \ | | | | | | | | Update CODEOWNERS file
| * | | Update CODEOWNERS fileschisamo/update-codeownersSeth Chisamore2017-12-141-1/+1
|/ / / | | | | | | | | | Signed-off-by: Seth Chisamore <schisamo@chef.io>
* | | Bump version to 13.6.35 by Expeditorv13.6.35Chef Expeditor2017-12-135-11/+12
| | |
* | | Merge pull request #6658 from chef/better_issuesThom May2017-12-131-23/+24
|\ \ \ | |_|/ |/| | Use comments in the github issue template
| * | Use comments in the github issue templatebetter_issuesTim Smith2017-12-111-23/+24
|/ / | | | | This way we don't see the end users notes when they submit it. This should make it easier to read the issues.
* | Bump version to 13.6.34 by Expeditorv13.6.34Chef Expeditor2017-12-115-11/+12
| |
* | Merge pull request #6654 from ↵Tim Smith2017-12-112-7/+19
|\ \ | | | | | | | | | | | | MsysTechnologiesllc/vasundhara/chef_windows_task_resource_not_idempotent_for_any_command windows_task: Fix resource isn't fully idempotent due to command property
| * | Removed pry statementVasu11052017-12-111-1/+0
| | | | | | | | | | | | Signed-off-by: Vasu1105 <vasundhara.jagdale@msystechnologies.com>
| * | [MSYS-731] Fix for [chef] windows_task resource isn't idempotent for any commandVasu11052017-12-112-7/+20
| | | | | | | | | | | | Signed-off-by: Vasu1105 <vasundhara.jagdale@msystechnologies.com>
* | | Bump version to 13.6.33 by Expeditorv13.6.33Chef Expeditor2017-12-115-11/+12
| | |
* | | Merge pull request #6618 from MsysTechnologiesllc/nim/on_logon_issueTim Smith2017-12-112-10/+24
|\ \ \ | | | | | | | | Invalid date error on windows_task with frequency :on_logon