Commit message (Collapse) | Author | Age | Files | Lines | |
---|---|---|---|---|---|
* | rearrange dsl mixinslcg/dsl-rearrangement | Lamont Granquist | 2016-03-18 | 4 | -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-fulldsl | Lamont Granquist | 2016-03-17 | 4 | -23/+21 |
|\ | | | | | deprecate Chef::DSL::Recipe::FullDSL | ||||
| * | rub some YARD on deprecation comment | Lamont Granquist | 2016-03-17 | 1 | -1/+1 |
| | | |||||
| * | deprecate Chef::DSL::Recipe::FullDSLlcg/deprecate-fulldsl | Lamont Granquist | 2016-03-16 | 4 | -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_ver | Matt Wrock | 2016-03-16 | 3 | -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 ohai | Matt Wrock | 2016-03-16 | 3 | -3/+3 |
|/ | |||||
* | 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 | ||||
* | | Merge pull request #4674 from chef/lcg/use-and-lazy-module-include | Lamont Granquist | 2016-03-15 | 7 | -3/+226 |
|\ \ | | | | | | | implement use DSL method and lazy module inclusion | ||||
| * | | lazy module inclusion into DSL moduleslcg/use-and-lazy-module-include | Lamont Granquist | 2016-03-15 | 7 | -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.9 | Thom May | 2016-03-15 | 9 | -2/+130 |
|\ \ | | | | | | | Remote file download progress | ||||
| * | | Enable progress output to be configuredtm/remote_file_download_progress | Thom May | 2016-03-04 | 9 | -26/+66 |
| | | | | | | | | | | | | 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 | 4 | -2/+7 |
|\ \ \ \ | | | | | | | | | | | Support libraries with sub directories | ||||
| * | | | | Ensure that chef-zero loads libraries recursivelytm/zero_deep_libraries | Thom May | 2016-03-03 | 4 | -2/+7 |
| |/ / / | |||||
* | | | | Bump version to 12.9.0 | John Keiser | 2016-03-15 | 4 | -3/+5 |
| | | | | |||||
* | | | | Merge pull request #4712 from chef/no_win | Matt Wrock | 2016-03-15 | 1 | -6/+8 |
|\ \ \ \ | | | | | | | | | | | temporarily disabling windows and chocolatey to eliminate transient errors on the builders | ||||
| * | | | | temporarily disabling windows and chocolatey to eliminate transient errors ↵ | Matt Wrock | 2016-03-15 | 1 | -6/+8 |
|/ / / / | | | | | | | | | | | | | on the builders | ||||
* | | | | Merge pull request #4711 from chef/tm/rake_dep | Thom May | 2016-03-15 | 1 | -0/+3 |
|\ \ \ \ | | | | | | | | | | | ensure we always can rake | ||||
| * | | | | ensure we always can raketm/rake_dep | Thom May | 2016-03-15 | 1 | -0/+3 |
|/ / / / | |||||
* | | | | Merge pull request #4707 from chef/lcg/useless-assignment | Lamont Granquist | 2016-03-15 | 14 | -19/+13 |
|\ \ \ \ | | | | | | | | | | | fix some UselessAssignment cases | ||||
| * | | | | fix some UselessAssignment caseslcg/useless-assignment | Lamont Granquist | 2016-03-14 | 14 | -21/+15 |
| | |_|/ | |/| | | |||||
* | | | | Merge pull request #4706 from chef/lcg/chefstyle-andor | Lamont Granquist | 2016-03-15 | 74 | -151/+150 |
|\ \ \ \ | | | | | | | | | | | chefstyle Style/AndOr fixes | ||||
| * | | | | fix windows specslcg/chefstyle-andor | Lamont Granquist | 2016-03-14 | 1 | -2/+2 |
| | | | | | |||||
| * | | | | fix specs | Lamont Granquist | 2016-03-14 | 1 | -3/+3 |
| | | | | | |||||
| * | | | | change to break unless | Lamont Granquist | 2016-03-14 | 1 | -1/+2 |
| | | | | | |||||
| * | | | | remove unnecessary variable | Lamont Granquist | 2016-03-14 | 1 | -2/+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 | 73 | -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-warnings | Chris Doherty | 2016-03-14 | 3 | -5/+33 |
|\ \ \ \ | |/ / / |/| | | | 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 | 3 | -5/+33 |
| | | | | | | | | | | | | | | | | '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 #4699 from chef/move_pry | Matt Wrock | 2016-03-11 | 1 | -4/+6 |
|\ \ \ \ \ | | | | | | | | | | | | | moving pry gems to their own very special group | ||||
| * | | | | | moving pry gems to their own very special group | Matt Wrock | 2016-03-11 | 1 | -4/+6 |
|/ / / / / | |||||
* | | | | | Merge pull request #4696 from chef/lcg/yell-at-users | Lamont Granquist | 2016-03-11 | 1 | -0/+3 |
|\ \ \ \ \ | | | | | | | | | | | | | Warn users not to tweak Chef::Config[:file_atomic_update] setting | ||||
| * | | | | | this is a really, really bad setting to tweaklcg/yell-at-users | Lamont Granquist | 2016-03-11 | 1 | -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-fix | Salim Alam | 2016-03-11 | 2 | -0/+18 |
|\ \ \ \ \ | |/ / / / |/| | | | | Make mount idempotent on Windows |