Commit message (Collapse) | Author | Age | Files | Lines | |
---|---|---|---|---|---|
* | deprecate Chef::DSL::Recipe::FullDSLlcg/deprecate-fulldsl | Lamont Granquist | 2016-03-16 | 3 | -20/+18 |
| | | | | | | | | this module was created out of a bit of excessive paranoia instead of simply adding mixins that we always use with Chef::DSL::Recipe directly to Chef::DSL::Recipe. lets try and YAGNI this class and just add it directly. these modules are already injected into everyone's recipes and providers, so I can't imagine a huge amount of backcompat breaks. | ||||
* | reverting version to 12.8.1 temporarily to avoid rerelease of ohai | Matt Wrock | 2016-03-16 | 1 | -1/+1 |
| | |||||
* | Merge pull request #4701 from mawinter69/master | Thom May | 2016-03-16 | 1 | -4/+6 |
|\ | | | | | fix for https://github.com/chef/chef/issues/4689 | ||||
| * | remove unnecessary whitespace | Markus Winter | 2016-03-15 | 1 | -2/+2 |
| | | |||||
| * | fix for https://github.com/chef/chef/issues/4689 | Markus Winter | 2016-03-12 | 1 | -4/+6 |
| | | | | | | | | rework the set_members method so duplicates are avoided | ||||
* | | lazy module inclusion into DSL moduleslcg/use-and-lazy-module-include | Lamont Granquist | 2016-03-15 | 3 | -3/+84 |
| | | | | | | | | | | | | Chef::DSL::Recipe::FullDSL.send(:include, MyModule) will now patch all its descendants that it has been included into (works the way actual inheritance works now). | ||||
* | | Merge pull request #4658 from chef/tm/remote_file_download_progress11.9 | Thom May | 2016-03-15 | 5 | -2/+81 |
|\ \ | | | | | | | Remote file download progress | ||||
| * | | Enable progress output to be configuredtm/remote_file_download_progress | Thom May | 2016-03-04 | 5 | -26/+17 |
| | | | | | | | | | | | | Add tests around progress output and tidy up | ||||
| * | | Merge branch 'remote-file-download-progress' of ↵ | Thom May | 2016-03-03 | 5 | -2/+90 |
| |\ \ | | | | | | | | | | | | | https://github.com/brandocorp/chef into tm/remote_file_download_progress | ||||
| | * | | fix broken specs | Brandon Raabe | 2015-03-21 | 1 | -1/+1 |
| | | | | |||||
| | * | | new doc formatter event, remove progress bar | Brandon Raabe | 2015-03-20 | 5 | -82/+69 |
| | | | | |||||
| | * | | pass resource as an option to allow access to resource events | Brandon Raabe | 2015-03-18 | 2 | -10/+21 |
| | | | | |||||
| | * | | add progress interval, add ProgressBar class | Brandon Raabe | 2015-03-15 | 3 | -2/+74 |
| | | | | |||||
| | * | | read content length from the response and use to show overall status | Brandon Raabe | 2015-03-15 | 1 | -1/+4 |
| | | | | |||||
| | * | | add value of show_progress to the http_client_opts | Brandon Raabe | 2015-03-15 | 1 | -0/+8 |
| | | | | |||||
| | * | | add show_progress attribute | Brandon Raabe | 2015-03-15 | 1 | -0/+8 |
| | | | | |||||
* | | | | Merge pull request #4640 from chef/tm/zero_deep_libraries | Thom May | 2016-03-15 | 3 | -2/+3 |
|\ \ \ \ | | | | | | | | | | | Support libraries with sub directories | ||||
| * | | | | Ensure that chef-zero loads libraries recursivelytm/zero_deep_libraries | Thom May | 2016-03-03 | 3 | -2/+3 |
| |/ / / | |||||
* | | | | Bump version to 12.9.0 | John Keiser | 2016-03-15 | 1 | -1/+1 |
| | | | | |||||
* | | | | Merge pull request #4707 from chef/lcg/useless-assignment | Lamont Granquist | 2016-03-15 | 13 | -18/+12 |
|\ \ \ \ | | | | | | | | | | | fix some UselessAssignment cases | ||||
| * | | | | fix some UselessAssignment caseslcg/useless-assignment | Lamont Granquist | 2016-03-14 | 13 | -19/+13 |
| | |_|/ | |/| | | |||||
* | | | | Merge pull request #4706 from chef/lcg/chefstyle-andor | Lamont Granquist | 2016-03-15 | 63 | -134/+133 |
|\ \ \ \ | | | | | | | | | | | chefstyle Style/AndOr fixes | ||||
| * | | | | fix specs | Lamont Granquist | 2016-03-14 | 1 | -3/+3 |
| | | | | | |||||
| * | | | | change to break unless | Lamont Granquist | 2016-03-14 | 1 | -1/+2 |
| | | | | | |||||
| * | | | | remove parens | Lamont Granquist | 2016-03-14 | 1 | -1/+1 |
| | | | | | |||||
| * | | | | manual fixing for raise unless | Lamont Granquist | 2016-03-14 | 5 | -15/+13 |
| | | | | | |||||
| * | | | | fix `or return` to `return unless` | Lamont Granquist | 2016-03-14 | 1 | -1/+1 |
| | | | | | |||||
| * | | | | This commit is why AndOr needs to be banned | Lamont Granquist | 2016-03-14 | 2 | -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 fixes | Lamont Granquist | 2016-03-14 | 62 | -128/+128 |
| |/ / / | | | | | | | | | | | | | | | | | 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-warnings | Chris Doherty | 2016-03-14 | 2 | -5/+6 |
|\ \ \ \ | |/ / / |/| | | | Fix issue 4334: eliminate spurious warnings. | ||||
| * | | | 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 #4693 from nathwill/bugfix/thread-exclusive | Lamont Granquist | 2016-03-11 | 1 | -3/+1 |
|\ \ \ \ | | | | | | | | | | | remove deprecated Thread.exclusive | ||||
| * | | | | rm unnecessary mutex | Nathan Williams | 2016-03-11 | 1 | -4/+1 |
| | | | | | |||||
| * | | | | fix deprecated Thread.exclusive with Mutex | Nathan Williams | 2016-03-10 | 1 | -1/+2 |
| | | | | | |||||
* | | | | | Merge pull request #4692 from chef/salam/mount-fix | Salim Alam | 2016-03-11 | 1 | -0/+6 |
|\ \ \ \ \ | |_|/ / / |/| | | | | Make mount idempotent on Windows | ||||
| * | | | | Make mount idempotent on Windowssalam/mount-fix | Salim Alam | 2016-03-10 | 1 | -0/+6 |
| |/ / / | |||||
* | | | | 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 |