summaryrefslogtreecommitdiff
path: root/lib/chef
Commit message (Collapse)AuthorAgeFilesLines
* Make sure we clean up legacy repo file if it exists.repo_nameNoah Kantrowitz2018-05-131-0/+20
| | | Signed-off-by: Noah Kantrowitz <noah@coderanger.net>
* apt_repository: Use the repo_name name propertyTim Smith2018-05-091-5/+5
| | | | | | We'll continue to use the resource name to name the apt_update resource, but all the other actions will use the name property as the user would expect. Signed-off-by: Tim Smith <tsmith@chef.io>
* Bump version to 14.1.1 by Chef Expeditorv14.1.1Chef Expeditor2018-05-041-1/+1
|
* check the path to --recipe-urlwindows-open-uriThom May2018-05-041-3/+7
| | | | | | 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>
* Bump version to 14.0.218 by Chef Expeditorv14.0.218Chef Expeditor2018-05-021-1/+1
|
* Merge pull request #6815 from ↵Bryan McLellan2018-05-022-511/+590
|\ | | | | | | | | 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-241-7/+1
| | | | | | | | Signed-off-by: vasu1105 <vasundhara.jagdale@msystechnologies.com>
| * [MSYS-752] windows task rewrite using wind32-taskschedulerVasu11052018-04-242-511/+596
| | | | | | | | Signed-off-by: Vasu1105 <vasundhara.jagdale@msystechnologies.com>
* | Bump version to 14.0.217 by Chef Expeditorv14.0.217Chef Expeditor2018-05-021-1/+1
| |
* | Merge pull request #7194 from coderanger/silent-failureThom May2018-05-022-4/+5
|\ \ | | | | | | Allow specifying `ignore_failure :quiet` to disable the error spew
| * | Feed the Rubocop.Noah Kantrowitz2018-04-262-2/+2
| | | | | | | | | | | | | | | | | | 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-262-4/+5
| | | | | | | | | | | | Signed-off-by: Noah Kantrowitz <noah@coderanger.net>
* | | Bump version to 14.0.216 by Chef Expeditorv14.0.216Chef Expeditor2018-05-011-1/+1
| | |
* | | 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>
* | | | Bump version to 14.0.215 by Chef Expeditorv14.0.215Chef Expeditor2018-04-301-1/+1
| | | |
* | | | Merge pull request #7204 from chef/remote_directoryTim Smith2018-04-301-45/+5
|\ \ \ \ | | | | | | | | | | Convert some of remote_directory to use properties
| * | | | Fix source property to use the name_property not nameTim Smith2018-04-301-1/+1
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | 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-271-45/+5
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | 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-301-1/+1
| | | | |
* | | | | 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 #7199 from chef/registry_keyThom May2018-04-301-9/+1
|\ \ \ \ \ \ | | | | | | | | | | | | | | Convert :key property in registry_key to use property
| * | | | | | Convert :key property in registry_key to use propertyregistry_keyTim Smith2018-04-271-9/+1
| |/ / / / / | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | 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>
* | | | | | 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-271-0/+3
|\ \ \ \ \ \ | |_|_|/ / / |/| | | | | 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-271-0/+3
| | |/ / / | |/| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | 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>
* | | | | 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>
| * | | | sysctl: Fix ignore_error being ignoredsysctl_fixTim Smith2018-04-261-1/+4
| |/ / / | | | | | | | | | | | | | | | | | | | | This came into the sysctl cookbook https://github.com/sous-chefs/sysctl/pull/127 Signed-off-by: Tim Smith <tsmith@chef.io>
* | | | Merge pull request #7129 from rmoriz/rmoriz/git-pruneTim Smith2018-04-271-2/+2
|\ \ \ \ | |/ / / |/| | | prune remotes with git fetch, fixes #3929
| * | | prune remotes with git fetch, fixes #3929Roland Moriz2018-04-061-2/+2
| | | | | | | | | | | | | | | | Signed-off-by: Roland Moriz <roland@moriz.de>
* | | | Bump version to 14.0.213 by Chef Expeditorv14.0.213Chef Expeditor2018-04-261-1/+1
| | | |
* | | | Merge pull request #7138 from ↵Tim Smith2018-04-262-12/+32
|\ \ \ \ | |_|/ / |/| | | | | | | | | | | RoboticCheese/roboticcheese/package-locking-nomethoderror Fix NoMethodError when (un)locking single packages in apt and zypper
| * | | Fix NoMethodError when (un)locking single packages in apt and zypperJonathan Hartman2018-04-092-12/+32
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | These resources don't coerce `package_name` to an array so would raise an exception when trying to call `name.all?` when `package_name`, and thus `name`, was a string. Signed-off-by: Jonathan Hartman <j@hartman.io>
* | | | Bump version to 14.0.212 by Chef Expeditorv14.0.212Chef Expeditor2018-04-261-1/+1
| | | |
* | | | Merge pull request #7186 from chef/sp/cli-logging-optionsThom May2018-04-264-3/+10
|\ \ \ \ | | | | | | | | | | CLI help text now includes :trace log level
View Lorry Controller Status