summaryrefslogtreecommitdiff
Commit message (Collapse)AuthorAgeFilesLines
* rearrange dsl mixinslcg/dsl-rearrangementLamont Granquist2016-03-184-67/+136
| | | | | | | | | | | | | | | | | | | | | | | - creates Chef::DSL::MethodMissing which will be nuked in Chef 13 - creates Chef::DSL::Core which is the mixins into core providers - adds build_resource/define_resource to core providers The intent here is to create Chef::DSL::Core which is intended to become the DSL minus method_missing and things that are only intended for recipes. Right now I've only added build_resource/declare_resource to core providers. We should probably push more into the "Core" DSL so that core providers could do things like use the search API. I would have moved everything other than resources and definitions over but the audit mode stuff is really only appropriate for recipes and not provider code and probably should not be moved over. We should probably have an intermediate mixin which is the "Custom Provider" DSL that has Resources and Definitions mixed into it, but which does not mix in the audit stuff that should only go into the Recipe DSL. There may also be a "Resources DSL" which has mixins for use with not_if/only_if and I'm unclear if that should be the "Core DSL" or another fourth mixin of DSL files.
* Merge pull request #4719 from chef/lcg/deprecate-fulldslLamont Granquist2016-03-174-23/+21
|\ | | | | deprecate Chef::DSL::Recipe::FullDSL
| * rub some YARD on deprecation commentLamont Granquist2016-03-171-1/+1
| |
| * deprecate Chef::DSL::Recipe::FullDSLlcg/deprecate-fulldslLamont Granquist2016-03-164-23/+21
|/ | | | | | | | 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.
* Merge pull request #4717 from chef/revert_verMatt Wrock2016-03-163-3/+3
|\ | | | | reverting version to 12.8.1 temporarily to avoid rerelease of ohai
| * reverting version to 12.8.1 temporarily to avoid rerelease of ohaiMatt Wrock2016-03-163-3/+3
|/
* 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
* | Merge pull request #4674 from chef/lcg/use-and-lazy-module-includeLamont Granquist2016-03-157-3/+226
|\ \ | | | | | | implement use DSL method and lazy module inclusion
| * | lazy module inclusion into DSL moduleslcg/use-and-lazy-module-includeLamont Granquist2016-03-157-3/+226
|/ / | | | | | | | | | | 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-159-2/+130
|\ \ | | | | | | Remote file download progress
| * | Enable progress output to be configuredtm/remote_file_download_progressThom May2016-03-049-26/+66
| | | | | | | | | | | | 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-154-2/+7
|\ \ \ \ | | | | | | | | | | Support libraries with sub directories
| * | | | Ensure that chef-zero loads libraries recursivelytm/zero_deep_librariesThom May2016-03-034-2/+7
| |/ / /
* | | | Bump version to 12.9.0John Keiser2016-03-154-3/+5
| | | |
* | | | Merge pull request #4712 from chef/no_winMatt Wrock2016-03-151-6/+8
|\ \ \ \ | | | | | | | | | | temporarily disabling windows and chocolatey to eliminate transient errors on the builders
| * | | | temporarily disabling windows and chocolatey to eliminate transient errors ↵Matt Wrock2016-03-151-6/+8
|/ / / / | | | | | | | | | | | | on the builders
* | | | Merge pull request #4711 from chef/tm/rake_depThom May2016-03-151-0/+3
|\ \ \ \ | | | | | | | | | | ensure we always can rake
| * | | | 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