summaryrefslogtreecommitdiff
Commit message (Collapse)AuthorAgeFilesLines
* ensure we always can raketm/rake_depThom May2016-03-151-0/+3
|
* Merge pull request #4707 from chef/lcg/useless-assignmentLamont Granquist2016-03-1514-19/+13
|\ | | | | fix some UselessAssignment cases
| * fix some UselessAssignment caseslcg/useless-assignmentLamont Granquist2016-03-1414-21/+15
| |
* | Merge pull request #4706 from chef/lcg/chefstyle-andorLamont Granquist2016-03-1574-151/+150
|\ \ | | | | | | chefstyle Style/AndOr fixes
| * | fix windows specslcg/chefstyle-andorLamont Granquist2016-03-141-2/+2
| | |
| * | fix specsLamont Granquist2016-03-141-3/+3
| | |
| * | change to break unlessLamont Granquist2016-03-141-1/+2
| | |
| * | remove unnecessary variableLamont Granquist2016-03-141-2/+2
| | |
| * | remove parensLamont Granquist2016-03-141-1/+1
| | |
| * | manual fixing for raise unlessLamont Granquist2016-03-145-15/+13
| | |
| * | fix `or return` to `return unless`Lamont Granquist2016-03-141-1/+1
| | |
| * | This commit is why AndOr needs to be bannedLamont Granquist2016-03-142-6/+6
| | | | | | | | | | | | | | | | | | | | | | | | I can barely understand what these intend. I'm pretty certain that the logic in the encrypted data bag item code was wrong (fixed now to match the comment) and I'm believe I converted the yum code to preserve the logic. But holy fuck its hard to read the yum code that I'm replacing.
| * | chefstyle Style/AndOr fixesLamont Granquist2016-03-1473-144/+144
| |/ | | | | | | | | this is part of our informal style guide, lets make it formal since clearly its not getting followed very well.
* | Merge pull request #4684 from chef/cd/fix-spurious-warningsChris Doherty2016-03-143-5/+33
|\ \ | |/ |/| Fix issue 4334: eliminate spurious warnings.
| * Issue 4334: Because set(property, nil) gets turned into get(property), add a ↵cd/fix-spurious-warningsChris Doherty2016-03-113-5/+33
| | | | | | | | 'nil_set' parameter to get() to skip a spurious warning when that happens.
* | Merge pull request #4693 from nathwill/bugfix/thread-exclusiveLamont Granquist2016-03-111-3/+1
|\ \ | | | | | | remove deprecated Thread.exclusive
| * | rm unnecessary mutexNathan Williams2016-03-111-4/+1
| | |
| * | fix deprecated Thread.exclusive with MutexNathan Williams2016-03-101-1/+2
| | |
* | | Merge pull request #4699 from chef/move_pryMatt Wrock2016-03-111-4/+6
|\ \ \ | | | | | | | | moving pry gems to their own very special group
| * | | moving pry gems to their own very special groupMatt Wrock2016-03-111-4/+6
|/ / /
* | | Merge pull request #4696 from chef/lcg/yell-at-usersLamont Granquist2016-03-111-0/+3
|\ \ \ | | | | | | | | Warn users not to tweak Chef::Config[:file_atomic_update] setting
| * | | this is a really, really bad setting to tweaklcg/yell-at-usersLamont Granquist2016-03-111-0/+3
| | |/ | |/| | | | | | | | | | | | | people are actually doing this in order to work around issues with writing /etc/hosts in docker containers. they should seriously stop doing it and fix this problem in the cookbooks.
* | | Merge pull request #4692 from chef/salam/mount-fixSalim Alam2016-03-112-0/+18
|\ \ \ | |/ / |/| | Make mount idempotent on Windows
| * | Update specSalim Alam2016-03-111-0/+12
| | |
| * | Make mount idempotent on Windowssalam/mount-fixSalim Alam2016-03-101-0/+6
| |/
* | Merge pull request #4683 from chef/salam/acceptance-tweakSalim Alam2016-03-117-0/+85
|\ \ | | | | | | Add cookbooks for 64-bit Windows test matrix
| * | Add more cookbooks for Win64 test matrixsalam/acceptance-tweakSalim Alam2016-03-097-0/+85
| | |
* | | Merge pull request #4690 from chef/lcg/fix-escape-globLamont Granquist2016-03-1123-31/+42
|\ \ \ | | | | | | | | s/escape_glob/escape_glob_dir/
| * | | s/escape_glob/escape_glob_dir/lcg/fix-escape-globLamont Granquist2016-03-1023-31/+42
| | |/ | |/| | | | | | | | | | Removes all remaining references to the deprecated escape_glob helper routine.
* | | Merge pull request #4694 from chef/vboxsfNoah Kantrowitz2016-03-111-1/+1
|\ \ \ | |/ / |/| | Vbox's shared filesystem driver uses non-file device names too.
| * | Vbox's shared filesystem driver uses non-file device names too. Noah Kantrowitz2016-03-101-1/+1
|/ /
* | Merge pull request #4686 from chef/cperry/add-cisco-supportNoah Kantrowitz2016-03-101-0/+24
|\ \ | | | | | | Adding myself as LT and Maintainer for Cisco platforms
| * | Fixing typo on iosxr teamcperry/add-cisco-supportCarl Perry2016-03-091-1/+1
| | |
| * | Adding myself as LT and Maintainer for Cisco platformsCarl Perry2016-03-091-0/+24
|/ /
* | Use ruby 2.1.8 on both windows and unixjk/ruby-2.1.8John Keiser2016-03-091-11/+4
| |
* | Merge pull request #4681 from coderanger/params-coerceNoah Kantrowitz2016-03-092-2/+2
|\ \ | |/ |/| Fix property coercion
| * Fix property coercion in both places, because they were different and Noah Kantrowitz2016-03-092-2/+2
|/ | | one in Chef::Mixin::Properties would result in duplicated values.
* Merge pull request #4676 from MichaelPereira/patch-1Thom May2016-03-081-1/+1
|\ | | | | Fix typo in whyrun message
| * Fix typo in whyrun messageMichael Pereira2016-03-081-1/+1
|/
* Merge pull request #4661 from davide125/systemd-userPhil Dibowitz2016-03-085-51/+133
|\ | | | | Extend service resource to support systemd user services
| * rubocop fixesDavide Cavalca2016-03-071-4/+4
| |
| * add a test for user servicesDavide Cavalca2016-03-041-8/+34
| |
| * error out if provider doesn't support user servicesDavide Cavalca2016-03-043-0/+18
| |
| * double quotes to make rubocop happyDavide Cavalca2016-03-031-6/+6
| |
| * Extend service resource to support systemd user servicesDavide Cavalca2016-03-033-47/+85
| |
* | Merge pull request #4673 from dduponchel/GH-2561Thom May2016-03-082-4/+33
|\ \ | | | | | | preferred_manifest_record: fix pretty print.
| * | preferred_manifest_record: fix pretty print.David Duponchel2016-03-072-4/+33
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | The `preferred_manifest_record` function pretty prints the list of existing files when the asked file can't be found. To do that, the root path of the cookbook is removed from each existing file but the update is done on the `existing_files` variable, not on a copy. When called several times, this variable will eventually be equals to `[nil]`, leading to a NoMethodError. Without the fix, the new unit test fails with > expected Chef::Exceptions::FileNotFound, got #<NoMethodError: undefined method `[]' for nil:NilClass> with backtrace: > # ./lib/chef/cookbook_version.rb:321:in `block in preferred_manifest_record' > # ./lib/chef/cookbook_version.rb:319:in `map!' > # ./lib/chef/cookbook_version.rb:319:in `preferred_manifest_record' > # ./spec/unit/cookbook_version_file_specificity_spec.rb:318:in `block (3 levels) in <top (required)>' > # ./spec/unit/cookbook_version_file_specificity_spec.rb:317:in `block (2 levels) in <top (required)>' Fixes #2561.
* | | Bump version to 12.8.112.8.1jk/bump-versionJohn Keiser2016-03-074-6/+11
|/ /
* | Merge pull request #4667 from chef/accept_dockerMatt Wrock2016-03-043-0/+16
|\ \ | | | | | | adding docker to top_cookbooks
| * | adding docker to top_cookbooksMatt Wrock2016-03-043-0/+16
| | |
View Lorry Controller Status