summaryrefslogtreecommitdiff
Commit message (Collapse)AuthorAgeFilesLines
* 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>
| * | | 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-272-10/+10
|\ \ \ | |_|/ |/| | prune remotes with git fetch, fixes #3929
| * | prune remotes with git fetch, fixes #3929Roland Moriz2018-04-062-10/+10
| | | | | | | | | | | | Signed-off-by: Roland Moriz <roland@moriz.de>
* | | Merge pull request #7191 from chef/lcg/pin-ffiLamont Granquist2018-04-263-19/+21
|\ \ \ | |_|/ |/| | pin ffi to avoid el6/el7 bugginess
| * | update the Gemfile.lock and remove windows-gemspec ffi pinlcg/pin-ffiLamont Granquist2018-04-262-19/+19
| | | | | | | | | | | | Signed-off-by: Lamont Granquist <lamont@scriptkiddie.org>
| * | pin ffi to avoid el6/el7 bugginessLamont Granquist2018-04-261-0/+2
|/ / | | | | | | | | | | | | absolutely unknown what this bug is, but it seems to fix specs on el6/el7 Signed-off-by: Lamont Granquist <lamont@scriptkiddie.org>
* | Bump version to 14.0.213 by Chef Expeditorv14.0.213Chef Expeditor2018-04-265-11/+12
| |
* | Merge pull request #7138 from ↵Tim Smith2018-04-264-14/+99
|\ \ | | | | | | | | | | | | 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-094-14/+99
| | | | | | | | | | | | | | | | | | | | | | | | 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-265-11/+12
| | |
* | | Merge pull request #7186 from chef/sp/cli-logging-optionsThom May2018-04-265-3/+11
|\ \ \ | | | | | | | | CLI help text now includes :trace log level
| * | | Updating a few logging references to include :tracesp/cli-logging-optionsStuart Preston2018-04-255-3/+11
| | | | | | | | | | | | | | | | Signed-off-by: Stuart Preston <stuart@chef.io>
* | | | Bump version to 14.0.211 by Chef Expeditorv14.0.211Chef Expeditor2018-04-265-14/+15
| | | |
* | | | Merge pull request #7183 from chef/tm/run_all_tests_in_ciLamont Granquist2018-04-251-1/+1
|\ \ \ \ | |/ / / |/| | | Run all tests in CI
| * | | Run all tests in CItm/run_all_tests_in_ciThom May2018-04-251-1/+1
|/ / / | | | | | | | | | | | | | | | | | | | | | | | | | | | Previously, we ran only integration tests. But many of our unit tests are constrained by rootyness and platform, and this means the first time they get run is during the ChefDK verification stage, which uses the released Chef build. So we have no way of fixing test failures short of a new Chef release. Signed-off-by: Thom May <thom@chef.io>
* | | Bump version to 14.0.210 by Expeditorv14.0.210Chef Expeditor2018-04-235-12/+13
| | |
* | | Merge pull request #7179 from chef/lcg/force-expeditor-to-runLamont Granquist2018-04-231-1/+1
|\ \ \ | | | | | | | | whitespace fix to make expeditor run
| * | | whitespace fix to make expeditor runLamont Granquist2018-04-231-1/+1
|/ / / | | | | | | | | | | | | | | | trying to sort out weirdness from this morning Signed-off-by: Lamont Granquist <lamont@scriptkiddie.org>
* | | manually bumping version to 14.0.209Lamont Granquist2018-04-231-1/+1
| | | | | | | | | | | | | | | | | | 14.0.210 should be next and should fix expeditor i guess Signed-off-by: Lamont Granquist <lamont@scriptkiddie.org>
* | | Merge pull request #7167 from chef/lcg/actuall-test-cheffsLamont Granquist2018-04-234-64/+56
|\ \ \ | | | | | | | | actually test against cheffs
| * | | Revert "cookbooks metadata are used instead of directory name"lcg/actuall-test-cheffsLamont Granquist2018-04-231-18/+32
| | | | | | | | | | | | | | | | This reverts commit 85653e8f25c19273fde0d8d512db29ef13c2a579.
View Lorry Controller Status