summaryrefslogtreecommitdiff
path: root/lib/chef
Commit message (Collapse)AuthorAgeFilesLines
* deprecate Chef::DSL::Recipe::FullDSLlcg/deprecate-fulldslLamont Granquist2016-03-163-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 ohaiMatt Wrock2016-03-161-1/+1
|
* Merge pull request #4701 from mawinter69/masterThom May2016-03-161-4/+6
|\ | | | | fix for https://github.com/chef/chef/issues/4689
| * remove unnecessary whitespaceMarkus Winter2016-03-151-2/+2
| |
| * fix for https://github.com/chef/chef/issues/4689Markus Winter2016-03-121-4/+6
| | | | | | | | rework the set_members method so duplicates are avoided
* | lazy module inclusion into DSL moduleslcg/use-and-lazy-module-includeLamont Granquist2016-03-153-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.9Thom May2016-03-155-2/+81
|\ \ | | | | | | Remote file download progress
| * | Enable progress output to be configuredtm/remote_file_download_progressThom May2016-03-045-26/+17
| | | | | | | | | | | | Add tests around progress output and tidy up
| * | Merge branch 'remote-file-download-progress' of ↵Thom May2016-03-035-2/+90
| |\ \ | | | | | | | | | | | | https://github.com/brandocorp/chef into tm/remote_file_download_progress
| | * | fix broken specsBrandon Raabe2015-03-211-1/+1
| | | |
| | * | new doc formatter event, remove progress barBrandon Raabe2015-03-205-82/+69
| | | |
| | * | pass resource as an option to allow access to resource eventsBrandon Raabe2015-03-182-10/+21
| | | |
| | * | add progress interval, add ProgressBar classBrandon Raabe2015-03-153-2/+74
| | | |
| | * | read content length from the response and use to show overall statusBrandon Raabe2015-03-151-1/+4
| | | |
| | * | add value of show_progress to the http_client_optsBrandon Raabe2015-03-151-0/+8
| | | |
| | * | add show_progress attributeBrandon Raabe2015-03-151-0/+8
| | | |
* | | | Merge pull request #4640 from chef/tm/zero_deep_librariesThom May2016-03-153-2/+3
|\ \ \ \ | | | | | | | | | | Support libraries with sub directories
| * | | | Ensure that chef-zero loads libraries recursivelytm/zero_deep_librariesThom May2016-03-033-2/+3
| |/ / /
* | | | Bump version to 12.9.0John Keiser2016-03-151-1/+1
| | | |
* | | | Merge pull request #4707 from chef/lcg/useless-assignmentLamont Granquist2016-03-1513-18/+12
|\ \ \ \ | | | | | | | | | | fix some UselessAssignment cases
| * | | | fix some UselessAssignment caseslcg/useless-assignmentLamont Granquist2016-03-1413-19/+13
| | |_|/ | |/| |
* | | | Merge pull request #4706 from chef/lcg/chefstyle-andorLamont Granquist2016-03-1563-134/+133
|\ \ \ \ | | | | | | | | | | chefstyle Style/AndOr fixes
| * | | | fix specsLamont Granquist2016-03-141-3/+3
| | | | |
| * | | | change to break unlessLamont Granquist2016-03-141-1/+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-1462-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-warningsChris Doherty2016-03-142-5/+6
|\ \ \ \ | |/ / / |/| | | 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-112-5/+6
| | | | | | | | | | | | | | | | '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 #4692 from chef/salam/mount-fixSalim Alam2016-03-111-0/+6
|\ \ \ \ \ | |_|/ / / |/| | | | Make mount idempotent on Windows
| * | | | Make mount idempotent on Windowssalam/mount-fixSalim Alam2016-03-101-0/+6
| |/ / /
* | | | Merge pull request #4690 from chef/lcg/fix-escape-globLamont Granquist2016-03-1121-30/+30
|\ \ \ \ | | | | | | | | | | s/escape_glob/escape_glob_dir/
| * | | | s/escape_glob/escape_glob_dir/lcg/fix-escape-globLamont Granquist2016-03-1021-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 Kantrowitz2016-03-101-1/+1
|/ / /
* | | 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.
* | | Fix typo in whyrun messageMichael Pereira2016-03-081-1/+1
| | |
* | | Merge pull request #4661 from davide125/systemd-userPhil Dibowitz2016-03-084-11/+67
|\ \ \ | | | | | | | | Extend service resource to support systemd user services
| * | | 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-032-11/+49
| | | |
* | | | Merge pull request #4673 from dduponchel/GH-2561Thom May2016-03-081-3/+9
|\ \ \ \ | | | | | | | | | | preferred_manifest_record: fix pretty print.
| * | | | preferred_manifest_record: fix pretty print.David Duponchel2016-03-071-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-versionJohn Keiser2016-03-071-1/+1
|/ / / /
* | | | Merge pull request #4617 from chef/lcg/chefstyle-perfLamont Granquist2016-03-0454-83/+133
|\ \ \ \ | | | | | | | | | | Autofixing new Perf cops in 0.37.2
| * | | | add some @yield docsLamont Granquist2016-03-0416-5/+53
| | | | | | | | | | | | | | | | | | | | added some FIXMEs because trying to sort out types for random undocumented code was making me completely aggro