summaryrefslogtreecommitdiff
Commit message (Collapse)AuthorAgeFilesLines
* Remove set_or_returns in macosx_serviceresource_fixesTim Smith2017-12-151-20/+2
| | | | Signed-off-by: Tim Smith <tsmith@chef.io>
* Just use platform_family to restrict apt_package to debianTim Smith2017-12-151-1/+1
| | | | | | This aligns with how we do it on other platforms Signed-off-by: Tim Smith <tsmith@chef.io>
* Don't restrict ips_package to running on just solaris2Tim Smith2017-12-151-1/+0
| | | | | | We don't do this on our other package resources. If someone explicitly calls this we shouldn't get in the way Signed-off-by: Tim Smith <tsmith@chef.io>
* Remove redundant providesTim Smith2017-12-153-3/+0
| | | | | | We're already defining the resource_name on these and we're not trying to limit where it runs Signed-off-by: Tim Smith <tsmith@chef.io>
* Remove provider requires we don't needTim Smith2017-12-154-4/+0
| | | | Signed-off-by: Tim Smith <tsmith@chef.io>
* Add resource names to resourcesTim Smith2017-12-1520-17/+23
| | | | Signed-off-by: Tim Smith <tsmith@chef.io>
* Remove extra provides from apt_preference / apt_repository / apt_updateTim Smith2017-12-153-3/+0
| | | | | | We already name the resources. Signed-off-by: Tim Smith <tsmith@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
| * | | invalid date error on windows_task with frequency :on_logonnimisha2017-12-082-10/+24
| | | | | | | | | | | | | | | | Signed-off-by: nimisha <nimisha.sharad@msystechnologies.com>
* | | | Bump version to 13.6.32 by Expeditorv13.6.32Chef Expeditor2017-12-115-11/+12
| | | |
* | | | Merge pull request #6655 from chef/tm/fix_chefstyleThom May2017-12-111-3/+3
|\ \ \ \ | |_|/ / |/| | | Fix sneaky chefstyle violations
| * | | Fix sneaky chefstyle violationstm/fix_chefstyleThom May2017-12-111-3/+3
|/ / / | | | | | | | | | Signed-off-by: Thom May <thom@chef.io>
* | | Bump version to 13.6.31 by Expeditorv13.6.31Chef Expeditor2017-12-105-12/+13
| | |
* | | Merge pull request #6636 from EmFl/patch-1Thom May2017-12-102-2/+23
|\ \ \ | |_|/ |/| | Ensure data bags names can contain reserved words
| * | added test for data bag name containing reserved wordEmilien2017-12-091-1/+22
| | |
| * | Fix related to #3058EmFl2017-12-081-1/+1
| | | | | | | | | I think there was a bug introduced by #3058 where any data bag *containing* reserved words are seen as invalid
* | | Bump version to 13.6.30 by Expeditorv13.6.30Chef Expeditor2017-12-085-11/+12
| | |
* | | Merge pull request #6628 from chef/more_taskTim Smith2017-12-083-53/+136
|\ \ \ | |/ / |/| | windows_task: Add additional input validation to properties
| * | Update the start_time spec for the new messageTim Smith2017-12-081-1/+1
| | | | | | | | | | | | 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-082-16/+14
| | | | | | | | | | | | | | | | | | RuntimeError Signed-off-by: Tim Smith <tsmith@chef.io>
| * | Expect the right exceptionsTim Smith2017-12-081-23/+23
| | | | | | | | | | | | Signed-off-by: Tim Smith <tsmith@chef.io>
View Lorry Controller Status