summaryrefslogtreecommitdiff
Commit message (Collapse)AuthorAgeFilesLines
* check the path to --recipe-urlwindows-open-uriThom May2018-05-042-3/+13
| | | | | | Provide the user with feedback if it's invalid or missing. Signed-off-by: Thom May <thom@chef.io>
* Trying to use --recipe-url on Windows with local file failstyler-ball2018-05-031-3/+7
| | | | | | | I'm guessing open-uri doesn't handle local file paths well with `open`? Either way this fixes the issue. Signed-off-by: tyler-ball <tyleraball@gmail.com>
* Merge pull request #7214 from chef/tm/fix_unix_testsThom May2018-05-035-5/+7
|\ | | | | Ensure we don't run tests on unsupported platforms
| * Ensure we don't run tests on unsupported platformstm/fix_unix_testsThom May2018-05-025-5/+7
| | | | | | | | | | | | Our more esoteric unices need some more careful hand holding Signed-off-by: Thom May <thom@chef.io>
* | Bump version to 14.0.218 by Chef Expeditorv14.0.218Chef Expeditor2018-05-025-12/+13
| |
* | Merge pull request #6815 from ↵Bryan McLellan2018-05-028-1307/+1784
|\ \ | | | | | | | | | | | | MsysTechnologiesllc/vasundhara/MSYS_752_windows_task_rewrite [MSYS-752] windows task rewrite using win32-taskscheduler
| * | Fix for why runvasu11052018-05-021-13/+13
| | | | | | | | | | | | Signed-off-by: vasu1105 <vasundhara.jagdale@msystechnologies.com>
| * | Fix for delete task with task_namevasu11052018-05-021-1/+1
| | | | | | | | | | | | Signed-off-by: vasu1105 <vasundhara.jagdale@msystechnologies.com>
| * | Fix for failing functional spec on adhoc buildbtm/MSYS-752_buildvasu11052018-04-242-8/+2
| | | | | | | | | | | | Signed-off-by: vasu1105 <vasundhara.jagdale@msystechnologies.com>
| * | Bump omnibus Gemfile.lock to pull in license_scout 1.0.2Bryan McLellan2018-04-241-4/+4
| | | | | | | | | | | | Signed-off-by: Bryan McLellan <btm@loftninjas.org>
| * | [MSYS-752] windows task rewrite using wind32-taskschedulerVasu11052018-04-247-1303/+1786
| | | | | | | | | | | | Signed-off-by: Vasu1105 <vasundhara.jagdale@msystechnologies.com>
* | | Bump version to 14.0.217 by Chef Expeditorv14.0.217Chef Expeditor2018-05-025-11/+12
| | |
* | | Merge pull request #7194 from coderanger/silent-failureThom May2018-05-024-9/+49
|\ \ \ | | | | | | | | Allow specifying `ignore_failure :quiet` to disable the error spew
| * | | Feed the Rubocop.Noah Kantrowitz2018-04-264-10/+10
| | | | | | | | | | | | | | | | | | | | | | | | Some day I will remember which is which on string styles. Signed-off-by: Noah Kantrowitz <noah@coderanger.net>
| * | | Allow specifying `ignore_failure :quiet` to disable the error spew.Noah Kantrowitz2018-04-264-9/+49
| | | | | | | | | | | | | | | | Signed-off-by: Noah Kantrowitz <noah@coderanger.net>
* | | | Merge pull request #7211 from chef/revert-7183-tm/run_all_tests_in_ciThom May2018-05-011-1/+1
|\ \ \ \ | | | | | | | | | | Revert "Run all tests in CI"
| * | | | Revert "Run all tests in CI"Thom May2018-05-011-1/+1
|/ / / /
* | | | Bump version to 14.0.216 by Chef Expeditorv14.0.216Chef Expeditor2018-05-015-11/+12
| | | |
* | | | Merge pull request #7208 from chef/tm/lazy_is_too_eagerThom May2018-05-011-1/+1
|\ \ \ \ | | | | | | | | | | Don't always request lazy files
| * | | | Don't always request lazy filestm/lazy_is_too_eagerThom May2018-04-301-1/+1
| | | | | | | | | | | | | | | | | | | | Signed-off-by: Thom May <thom@chef.io>
* | | | | Merge pull request #7210 from chef/update_changelogTim Smith2018-04-301-2/+9
|\ \ \ \ \ | | | | | | | | | | | | Update changelog due to expeditor missing PRs
| * | | | | Update changelog due to expeditor missing PRsupdate_changelogTim Smith2018-04-301-2/+9
|/ / / / / | | | | | | | | | | | | | | | | | | | | | | | | | I nuked out 2 items that didn't need to be in there while I was in the file Signed-off-by: Tim Smith <tsmith@chef.io>
* | | | | Bump version to 14.0.215 by Chef Expeditorv14.0.215Chef Expeditor2018-04-305-11/+12
| | | | |
* | | | | Merge pull request #7204 from chef/remote_directoryTim Smith2018-04-302-48/+9
|\ \ \ \ \ | | | | | | | | | | | | Convert some of remote_directory to use properties
| * | | | | Fix source property to use the name_property not nameTim Smith2018-04-302-2/+3
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Source was building off name not path, which means the name _property never actually worked. I modified an existing test to check this. Signed-off-by: Tim Smith <tsmith@chef.io>
| * | | | | Convert some of remote_directory to use propertiesremote_directoryTim Smith2018-04-272-47/+7
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | This isn't all of it, but it's a start Signed-off-by: Tim Smith <tsmith@chef.io>
* | | | | | Bump version to 14.0.214 by Chef Expeditorv14.0.214Chef Expeditor2018-04-305-11/+12
| | | | | |
* | | | | | Merge pull request #7200 from chef/subversionTim Smith2018-04-301-26/+13
|\ \ \ \ \ \ | |_|_|_|_|/ |/| | | | | Convert properties in subversion to use property
| * | | | | Convert properties in subversion to use propertysubversionTim Smith2018-04-271-26/+13
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Retained the False = nil behavior which is actually tested in the specs already. Signed-off-by: Tim Smith <tsmith@chef.io>
* | | | | | Merge pull request #7207 from chef/tm/fixipsThom May2018-04-303-0/+3
|\ \ \ \ \ \ | | | | | | | | | | | | | | fix application tests on linux
| * | | | | | fix application tests on linuxThom May2018-04-303-0/+3
|/ / / / / / | | | | | | | | | | | | | | | | | | Signed-off-by: Thom May <thom@chef.io>
* | | | | | Merge pull request #7199 from chef/registry_keyThom May2018-04-302-10/+2
|\ \ \ \ \ \ | | | | | | | | | | | | | | Convert :key property in registry_key to use property
| * | | | | | Convert :key property in registry_key to use propertyregistry_keyTim Smith2018-04-272-10/+2
| |/ / / / / | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Convert this to a true property. It required updating the specs a tiny bit since key gets returned as a state value now (as it should). Signed-off-by: Tim Smith <tsmith@chef.io>
* | | | | | Merge pull request #7201 from chef/desired_stateThom May2018-04-305-7/+7
|\ \ \ \ \ \ | | | | | | | | | | | | | | Set desired_state: false on several properties
| * | | | | | Set desired_state: false on several propertiesdesired_stateTim Smith2018-04-275-7/+7
| |/ / / / / | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | These are not state attributes, but we're treating them like state attributes. Signed-off-by: Tim Smith <tsmith@chef.io>
* | | | | | Merge pull request #7206 from chef/fix_link_ownerLamont Granquist2018-04-281-5/+19
|\ \ \ \ \ \ | | | | | | | | | | | | | | Fix regression where owner was removed from link resource
| * | | | | | Fix regression where owner was removed from link resourcefix_link_ownerTim Smith2018-04-281-5/+19
|/ / / / / / | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | This was accidentally renamed from owner to user. While I'm here add all the descriptions. Signed-off-by: Tim Smith <tsmith@chef.io>
* | | | | | Merge pull request #7195 from chef/cleanupTim Smith2018-04-2712-68/+75
|\ \ \ \ \ \ | | | | | | | | | | | | | | Add descriptions to more resources
| * | | | | | Add missing periods and rely on word wrapcleanupTim Smith2018-04-279-34/+13
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Just stick the descriptions on a single line so that we utilize large monitors. Signed-off-by: Tim Smith <tsmith@chef.io>
| * | | | | | Add descriptions to more resourcesTim Smith2018-04-266-40/+68
| | |_|_|/ / | |/| | | | | | | | | | | | | | | | Signed-off-by: Tim Smith <tsmith@chef.io>
* | | | | | Merge pull request #7197 from chef/build_nameTim Smith2018-04-273-1/+12
|\ \ \ \ \ \ | |_|_|/ / / |/| | | | | Allow build_essential resource to be used without setting a name
| * | | | | Allow build_essential resource to be used without setting a namebuild_nameTim Smith2018-04-273-1/+12
| | |/ / / | |/| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | This is the same as apt_update. It seems odd to use this resource with a name like this: ```ruby build_essential "build-essential" ``` Signed-off-by: Tim Smith <tsmith@chef.io>
* | | | | Merge pull request #7193 from chef/rhsm_fixLamont Granquist2018-04-271-1/+1
|\ \ \ \ \ | | | | | | | | | | | | rhsm_register: skip the registration check if force is specified
| * | | | | rhsm_register: skip the registration check if force is specifiedrhsm_fixTim Smith2018-04-261-1/+1
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Thanks @donwlewis for pointing this out here: https://github.com/chef-partners/redhat-subscription-manager-cookbook/pull/43 Signed-off-by: Tim Smith <tsmith@chef.io>
* | | | | | Merge pull request #7198 from chef/lcg/whitelist-node-methodsLamont Granquist2018-04-272-0/+3
|\ \ \ \ \ \ | |_|/ / / / |/| | | | | Whitelist some additional Hash/Array methods
| * | | | | Whitelist some additional Hash/Array methodslcg/whitelist-node-methodsLamont Granquist2018-04-272-0/+3
|/ / / / / | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | these get monkeypatched in by gems which are included in ChefDK and cause those tests to break. kind of unpleasant workflow, but this is doing its job correctly. Signed-off-by: Lamont Granquist <lamont@scriptkiddie.org>
* | | | | Merge pull request #7192 from chef/sysctl_fixTim Smith2018-04-271-10/+14
|\ \ \ \ \ | | | | | | | | | | | | sysctl: Fix ignore_error being ignored
| * | | | | Set ignore_error to desired_state falseTim Smith2018-04-271-1/+1
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | This avoids Chef reporting that the value has changed from false to true. It's not a state property so we don't want that. Signed-off-by: Tim Smith <tsmith@chef.io>
| * | | | | Use more friendly resource names when reloading valuesTim Smith2018-04-271-2/+5
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | What we're doing here is loading values. Just state that. Signed-off-by: Tim Smith <tsmith@chef.io>
| * | | | | Simplify how we fetch systctl current valuesTim Smith2018-04-271-7/+5
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Remove the raise that wasn't needed since we pass -e Remove the .to_s on the shell_out value since stdout is already a string Fail hard if we can't shellout for some reason by using shell_out! not shell_out Don't create an object out of the shellout result since we don't need that Kill the method entirely since it's now only a single line Signed-off-by: Tim Smith <tsmith@chef.io>
View Lorry Controller Status