summaryrefslogtreecommitdiff
Commit message (Collapse)AuthorAgeFilesLines
* 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>
| * | Use integer for numeric evaluationTim Smith2017-12-081-2/+3
| | | | | | | | | | | | | | | | | | Why is this not just in the duration calculator? Well we use it in two places and I would have had to add the same logic in both of those places. It's nice and easy to read this way Signed-off-by: Tim Smith <tsmith@chef.io>
| * | Properly handle an ISO8601 duration of 0 secondsTim Smith2017-12-082-2/+4
| | | | | | | | | | | | Signed-off-by: Tim Smith <tsmith@chef.io>
| * | Add tests for invalid datesTim Smith2017-12-071-2/+21
| | | | | | | | | | | | Signed-off-by: Tim Smith <tsmith@chef.io>
| * | Improve the integer/string warning messageTim Smith2017-12-073-7/+7
| | | | | | | | | | | | Signed-off-by: Tim Smith <tsmith@chef.io>
| * | Validate the start_date is correctly passedTim Smith2017-12-072-2/+5
| | | | | | | | | | | | Signed-off-by: Tim Smith <tsmith@chef.io>
| * | Explain what PT72H isTim Smith2017-12-071-1/+1
| | | | | | | | | | | | Signed-off-by: Tim Smith <tsmith@chef.io>
| * | allow random_delay and execution_time_limit to be integers and properly validateTim Smith2017-12-072-33/+61
| | | | | | | | | | | | | | | | | | Also set the default the right way with execution_time_limit Signed-off-by: Tim Smith <tsmith@chef.io>
| * | Validate that starttime is actually in HH:mm formatTim Smith2017-12-072-4/+15
| | | | | | | | | | | | Signed-off-by: Tim Smith <tsmith@chef.io>
| * | Test lowercase system users not requiring passwordsTim Smith2017-12-071-0/+5
| | | | | | | | | | | | Signed-off-by: Tim Smith <tsmith@chef.io>
| * | Add testing of sec_to_durTim Smith2017-12-071-0/+16
| | | | | | | | | | | | Signed-off-by: Tim Smith <tsmith@chef.io>
* | | Bump version to 13.6.29 by Expeditorv13.6.29Chef Expeditor2017-12-085-11/+12
| | |
* | | Merge pull request #6631 from chef/jsinha/fix_svcadmThom May2017-12-082-5/+11
|\ \ \ | | | | | | | | Solaris: Fix svcadm clear to only run in maintenance state
| * | | Fix svcadm clear to only run in maintenance statejsinha/fix_svcadmJaymala Sinha2017-12-072-5/+11
| | | | | | | | | | | | | | | | | | | | Signed-off-by: Jaymala Sinha <jsinha@chef.io> Signed-off-by: Patrick Wright <patrick@chef.io>
* | | | Bump version to 13.6.28 by Expeditorv13.6.28Chef Expeditor2017-12-085-11/+12
| | | |
* | | | Merge pull request #6632 from chef/lcg/node-map-speedupThom May2017-12-082-28/+61
|\ \ \ \ | |_|_|/ |/| | | speedup node_map get and set operations
| * | | fix alphabetic sorting of classeslcg/node-map-speedupLamont Granquist2017-12-072-8/+22
| | | | | | | | | | | | | | | | | | | | | | | | but we really shouldn't do this... Signed-off-by: Lamont Granquist <lamont@scriptkiddie.org>
| * | | remove yields and handle common cases earlyLamont Granquist2017-12-071-20/+15
| | | | | | | | | | | | | | | | Signed-off-by: Lamont Granquist <lamont@scriptkiddie.org>
| * | | a typoLamont Granquist2017-12-071-1/+1
| | | | | | | | | | | | | | | | Signed-off-by: Lamont Granquist <lamont@scriptkiddie.org>