Use ark LWRP instead remote_file+execute+link #17

Closed
wants to merge 17 commits into
from

Conversation

Projects
None yet
2 participants
@bryanwb

bryanwb commented Jun 27, 2012

Prior to making this change, I wasn't able to use this cookbook on RHEL 6.2. after switching to ark, it works fine

Thanks for the great cookbook!

karmi added a commit that referenced this pull request Jul 6, 2012

[#17] Use the `ark` [https://github.com/bryanwb/chef-ark] LWRP to dow…
…nload and install elasticsearch

With this change, a specialized external Chef LWRP resource, [`ark`](https://github.com/bryanwb/chef-ark)
is used for downloading, installing, linking, etc of elasticsearch,
instead of using a "remote_file + execute + symlink" combination of resources

* "ark" is added as a dependency
* The cookbook version is increased to reflect the change.
* An `elasticsearch[:checksum]` node attribute is added.
* The installation procedure is declared with the `ark` resource

karmi added a commit that referenced this pull request Jul 6, 2012

karmi added a commit that referenced this pull request Jul 6, 2012

karmi added a commit that referenced this pull request Jul 6, 2012

[#17] Added a Berksfile
[Berkshelf](http://berkshelf.com) is "Bundler for Chef cookbooks",
making it easy to declare and install cookbook dependencies.

karmi added a commit that referenced this pull request Jul 6, 2012

[#17] Used "community" versions of the Java and Ark cookbooks
The GitHub version of the "ark" cookbook fails due to <bryanwb/chef-ark#24>

karmi added a commit that referenced this pull request Jul 6, 2012

karmi added a commit that referenced this pull request Jul 6, 2012

[#17] Refactored and cleaned up the Vagrantfile structure
* Use a Hash with configs for separate distros
* Avoid duplication in the config
* Clean up the core Vagrant configuration block
* `deep_merge` distro specific configuration into node attributes

karmi added a commit that referenced this pull request Jul 6, 2012

[#17] Cleaned up the Minitest specs
* Use `wait_for_status` parameter of Cluster Health API for waiting until elasticsearch is up
* DRY up the URLs
* Silence `curl` transfer stats output
* Add `tests` symlink

karmi added a commit that referenced this pull request Jul 6, 2012

@karmi

This comment has been minimized.

Show comment Hide comment
@karmi

karmi Jul 7, 2012

Owner

Hi, commits were combined, manipulated, reordered, commit messages rewritten, but the gist of your changes should be applied! Thanks!!!

Owner

karmi commented Jul 7, 2012

Hi, commits were combined, manipulated, reordered, commit messages rewritten, but the gist of your changes should be applied! Thanks!!!

@karmi karmi closed this Jul 7, 2012

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment