summaryrefslogtreecommitdiff
Commit message (Collapse)AuthorAgeFilesLines
* Move zypper_repo tests out of linux only and cleanupresource_specsTim Smith2017-12-191-27/+17
| | | | | | | | The alias tests I added don't actually work, but everything was in linux only so I didn't notice. Move the things out of linux only that can run on any platform and remove the alias tests for now. Signed-off-by: Tim Smith <tsmith@chef.io>
* Add a test for the name property in subversion resourceTim Smith2017-12-181-0/+4
| | | | Signed-off-by: Tim Smith <tsmith@chef.io>
* Improve the windows_path specsTim Smith2017-12-181-4/+6
| | | | Signed-off-by: Tim Smith <tsmith@chef.io>
* Improve the yum_repository resource specsTim Smith2017-12-182-9/+61
| | | | Signed-off-by: Tim Smith <tsmith@chef.io>
* Improve the zypper_repository resource specsTim Smith2017-12-181-5/+14
| | | | Signed-off-by: Tim Smith <tsmith@chef.io>
* Cleanup for resource specsTim Smith2017-12-1858-1377/+1281
| | | | | | | | | Convert everything to let and update the tense Nothing real magical here, but gets us one step closer to uniformity in the specs Signed-off-by: Tim Smith <tsmith@chef.io>
* Merge pull request #6689 from chef/resource_provider_magicTim Smith2017-12-182-45/+28
|\ | | | | Modernize cookbook_file resource and expand specs
| * 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-182-45/+28
| | | | | | | | | | | | | | | | 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>
* | 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
View Lorry Controller Status