Commit message (Collapse) | Author | Age | Files | Lines | |
---|---|---|---|---|---|
* | Issue 4334: Because set(property, nil) gets turned into get(property), add a ↵cd/fix-spurious-warnings | Chris Doherty | 2016-03-11 | 2 | -5/+6 |
| | | | | 'nil_set' parameter to get() to skip a spurious warning when that happens. | ||||
* | Merge pull request #4690 from chef/lcg/fix-escape-glob | Lamont Granquist | 2016-03-11 | 21 | -30/+30 |
|\ | | | | | s/escape_glob/escape_glob_dir/ | ||||
| * | s/escape_glob/escape_glob_dir/lcg/fix-escape-glob | Lamont Granquist | 2016-03-10 | 21 | -30/+30 |
| | | | | | | | | | | Removes all remaining references to the deprecated escape_glob helper routine. | ||||
* | | Vbox's shared filesystem driver uses non-file device names too. | Noah Kantrowitz | 2016-03-10 | 1 | -1/+1 |
|/ | |||||
* | Fix property coercion in both places, because they were different and | Noah Kantrowitz | 2016-03-09 | 2 | -2/+2 |
| | | | one in Chef::Mixin::Properties would result in duplicated values. | ||||
* | Fix typo in whyrun message | Michael Pereira | 2016-03-08 | 1 | -1/+1 |
| | |||||
* | Merge pull request #4661 from davide125/systemd-user | Phil Dibowitz | 2016-03-08 | 4 | -11/+67 |
|\ | | | | | Extend service resource to support systemd user services | ||||
| * | error out if provider doesn't support user services | Davide Cavalca | 2016-03-04 | 3 | -0/+18 |
| | | |||||
| * | double quotes to make rubocop happy | Davide Cavalca | 2016-03-03 | 1 | -6/+6 |
| | | |||||
| * | Extend service resource to support systemd user services | Davide Cavalca | 2016-03-03 | 2 | -11/+49 |
| | | |||||
* | | Merge pull request #4673 from dduponchel/GH-2561 | Thom May | 2016-03-08 | 1 | -3/+9 |
|\ \ | | | | | | | preferred_manifest_record: fix pretty print. | ||||
| * | | preferred_manifest_record: fix pretty print. | David Duponchel | 2016-03-07 | 1 | -3/+9 |
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | 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-version | John Keiser | 2016-03-07 | 1 | -1/+1 |
|/ / | |||||
* | | Merge pull request #4617 from chef/lcg/chefstyle-perf | Lamont Granquist | 2016-03-04 | 54 | -83/+133 |
|\ \ | | | | | | | Autofixing new Perf cops in 0.37.2 | ||||
| * | | add some @yield docs | Lamont Granquist | 2016-03-04 | 16 | -5/+53 |
| | | | | | | | | | | | | added some FIXMEs because trying to sort out types for random undocumented code was making me completely aggro | ||||
| * | | remove unused block argumentslcg/chefstyle-perf | Lamont Granquist | 2016-02-23 | 18 | -22/+22 |
| | | | |||||
| * | | Autofixing new Perf cops in 0.37.2 | Lamont Granquist | 2016-02-23 | 49 | -59/+61 |
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | 6 Performance/Casecmp 18 Performance/Detect 1 Performance/RangeInclude 27 Performance/RedundantBlockCall 6 Performance/RedundantMatch 5 Performance/RedundantMerge 18 Performance/StringReplacement | ||||
* | | | Adding specs for rehash update | tyler-ball | 2016-03-04 | 2 | -3/+4 |
| | | | |||||
* | | | Changing the knife behavior when using the cached plugins to only display 1 ↵ | tyler-ball | 2016-03-04 | 1 | -3/+21 |
| | | | | | | | | | | | | error message when plugin files are missing from the cache | ||||
* | | | First part of fixing the rehash command, it now always loads gems from disc ↵ | tyler-ball | 2016-03-04 | 3 | -1/+15 |
| | | | | | | | | | | | | instead of using the hash file | ||||
* | | | Merge pull request #4664 from chef/salam/databag-fix | Thom May | 2016-03-04 | 1 | -1/+0 |
|\ \ \ | | | | | | | | | Data bag item hash can have name key | ||||
| * | | | Data bag item hash can have name keysalam/databag-fix | Salim Alam | 2016-03-03 | 1 | -1/+0 |
| | | | | |||||
* | | | | Merge pull request #4478 from chef/lcg/rfc-060-gem-metadata | Lamont Granquist | 2016-03-03 | 8 | -5/+198 |
|\ \ \ \ | |/ / / |/| | | | RFC-060 gem metadata MVP | ||||
| * | | | need to require bundlerlcg/rfc-060-gem-metadata | Lamont Granquist | 2016-02-23 | 1 | -0/+1 |
| | | | | | | | | | | | | | | | | for Bundler::UI | ||||
| * | | | remove now-unnecessary require | Lamont Granquist | 2016-02-09 | 1 | -1/+0 |
| | | | | |||||
| * | | | rubocopisms | Lamont Granquist | 2016-02-09 | 1 | -2/+2 |
| | | | | |||||
| * | | | add comments and cleanup | Lamont Granquist | 2016-02-09 | 3 | -7/+14 |
| | | | | |||||
| * | | | remove unnecessary paranoia | Lamont Granquist | 2016-02-09 | 1 | -5/+1 |
| | | | | |||||
| * | | | make rubygems.org configurable | Lamont Granquist | 2016-02-09 | 2 | -2/+2 |
| | | | | | | | | | | | | | | | | for bonus points gem_package will pick this up as well. | ||||
| * | | | get the formatting for cookbook gems correct | Lamont Granquist | 2016-02-09 | 6 | -18/+119 |
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | this leverages the PR: https://github.com/bundler/bundler/pull/4245 to redirect output from bundler into chef's logger/formatter to get the UI right. if you don't have that in the external bundler you'll just get output on STDOUT. fix is to upgrade bundler. | ||||
| * | | | fixed my .vimrc auto-copyright header | Lamont Granquist | 2016-02-09 | 1 | -1/+1 |
| | | | | |||||
| * | | | RFC-060 gem metadata MVP | Lamont Granquist | 2016-02-09 | 5 | -1/+90 |
| | | | | |||||
* | | | | Merge pull request #4655 from coderanger/solobag | Noah Kantrowitz | 2016-03-03 | 1 | -0/+1 |
|\ \ \ \ | |_|_|/ |/| | | | Clearer exception for loading non-existent data bag items in solo mode. | ||||
| * | | | Clearer exception for loading non-existent data bag items in solo mode. | Noah Kantrowitz | 2016-03-02 | 1 | -0/+1 |
| | | | | |||||
* | | | | Merge pull request #4307 from davide125/systemd-mask | Phil Dibowitz | 2016-03-03 | 3 | -2/+67 |
|\ \ \ \ | |/ / / |/| | | | Extend service resource to support masking | ||||
| * | | | add tests for is_masked? | Davide Cavalca | 2016-02-22 | 1 | -1/+1 |
| | | | | |||||
| * | | | is_masked? fix | Davide Cavalca | 2016-02-22 | 1 | -1/+1 |
| | | | | |||||
| * | | | rubocop doesn't like single quotes | Davide Cavalca | 2016-02-22 | 1 | -1/+1 |
| | | | | |||||
| * | | | rework to use service resource directly instead of subclassing | Davide Cavalca | 2016-02-22 | 4 | -84/+52 |
| | | | | |||||
| * | | | rework to use a subclassed SystemdService resource | Davide Cavalca | 2016-02-22 | 4 | -50/+84 |
| | | | | |||||
| * | | | raise UnsupportedAction for mask / unmask in the base class | Davide Cavalca | 2016-02-22 | 1 | -0/+8 |
| | | | | |||||
| * | | | Extend service resource to support masking | Davide Cavalca | 2016-02-22 | 3 | -2/+57 |
| | | | | |||||
* | | | | Revert "Merge pull request #4465 from ↵tm/revert_4465 | Thom May | 2016-03-02 | 1 | -2/+2 |
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | artursitarski/change-shell-for-locked-user" This reverts commit 94494e761763b877c3ef3b52eee9ac9df2186340, reversing changes made to a95a87d50c9e5af2fbd583afcf972880656d1a4b. | ||||
* | | | | Merge pull request #4646 from coderanger/negative-length | Noah Kantrowitz | 2016-03-01 | 1 | -0/+5 |
|\ \ \ \ | | | | | | | | | | | Handle negative content length headers too. | ||||
| * | | | | Split the negative check to its own log message. | Noah Kantrowitz | 2016-02-29 | 1 | -1/+6 |
| | | | | | |||||
| * | | | | Handle negative content length headers too. | Noah Kantrowitz | 2016-02-29 | 1 | -1/+1 |
| | | | | | | | | | | | | | | | This is equivalent to not passing one, we can't check for bad downloads. | ||||
* | | | | | Merge pull request #4645 from chef/msi | Matt Wrock | 2016-02-29 | 1 | -1/+1 |
|\ \ \ \ \ | | | | | | | | | | | | | fixes #4639 and returns true from define_resource_requirements for MSIs | ||||
| * | | | | | fixes #4639 and returns true from define_resource_requirements for MSIsmsi | Matt Wrock | 2016-02-29 | 1 | -1/+1 |
| |/ / / / | |||||
* | | | | | if no module name is found for a valid dsc resource default to ↵dsc | Matt Wrock | 2016-02-29 | 1 | -5/+2 |
|/ / / / | | | | | | | | | | | | | PSDesiredStateConfiguration | ||||
* | | | | use Pathname#cleanpath instead of Chef::Util::PathHelper.cleanpath to ↵ | Matt Wrock | 2016-02-29 | 1 | -2/+2 |
| | | | | | | | | | | | | | | | | compare globed files with forward slashes |