Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Full Chef 11 compatibility update. #18

Merged
merged 4 commits into from
Feb 20, 2013
Merged

Conversation

schisamo
Copy link

This PR adds full Chef 11+ compatibility:

  • Vendor missing chef/solr_query/* classes (only added to load path if running in Chef Solo 11+)
  • Install treetop gem if missing (determined by LoadError). This will not occur on Chef 10.x.
  • search and data_bag methods are now part of Chef::DSL:: DataQuery (previously they were part of Chef::Mixin::Language).
  • Fully backward compatibility with Chef 10.x.

See the individual commits for in depth messages/explanation.

These classes have been removed as of Chef 11. We will vendor the files
from Chef 10.x and only add them to the load path if running in Chef
Solo 11+.
…DataQuery`

These methods were previously located in `Chef::Mixin::Language`.

Additionally re-organize library code by creating a "search" subfolder
for override module and parser.
Let's instantiate actual instances of `Chef::Node` and `Chef::Role`
objects using the `json_create` method that all Chef object classes
share.
@hh
Copy link

hh commented Feb 18, 2013

+1

@thekorn
Copy link

thekorn commented Feb 18, 2013

Hi @schisamo,
thank you very much for this patch, I'm going to test it today.

Markus

@webcoyote
Copy link

Thanks @schisamo; I tested your changes on my repository (https://github.com/webcoyote/linux-vm, which builds a Linux virtual machine on Windows) and they worked perfectly with Chef 11. I also learned a few things about Ruby too!

tobami added a commit that referenced this pull request Feb 20, 2013
Full Chef 11 compatibility update.
@tobami tobami merged commit da3cdbd into edelight:master Feb 20, 2013
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

5 participants