--- a/dev/provisioning/modules/elasticsearch/CONTRIBUTING.md Tue Nov 08 18:23:01 2016 +0100
+++ b/dev/provisioning/modules/elasticsearch/CONTRIBUTING.md Wed Nov 09 15:05:41 2016 +0100
@@ -1,46 +1,70 @@
-If you have a bugfix or new feature that you would like to contribute to this puppet module, please find or open an issue about it first. Talk about what you would like to do. It may be that somebody is already working on it, or that there are particular issues that you should know about before implementing the change.
+# Contributing
-We enjoy working with contributors to get their code accepted. There are many approaches to fixing a problem and it is important to find the best approach before writing too much code.
+If you have a bugfix or new feature that you would like to contribute to this puppet module, please find or open an issue about it first.
+Talk about what you would like to do.
+It may be that somebody is already working on it, or that there are particular issues that you should know about before implementing the change.
+
+**Note**: If you have support-oriented questions that aren't a bugfix or feature request, please post your questions on the [discussion forums](https://discuss.elastic.co/c/elasticsearch).
+
+We enjoy working with contributors to get their code accepted.
+There are many approaches to fixing a problem and it is important to find the best approach before writing too much code.
The process for contributing to any of the Elastic repositories is similar.
-1. Sign the contributor license agreement
-Please make sure you have signed the [Contributor License Agreement](http://www.elastic.co/contributor-agreement/). We are not asking you to assign copyright to us, but to give us the right to distribute your code without restriction. We ask this of all contributors in order to assure our users of the origin and continuing existence of the code. You only need to sign the CLA once.
+## The Contributor License Agreement
+
+Please make sure you have signed the [Contributor License Agreement](http://www.elastic.co/contributor-agreement/).
+We are not asking you to assign copyright to us, but to give us the right to distribute your code without restriction.
+We ask this of all contributors in order to assure our users of the origin and continuing existence of the code.
+You only need to sign the CLA once.
+
+## Development Setup
+
+There are a few testing prerequisites to meet:
+
+* Ruby.
+ As long as you have a recent version with `bundler` available, `bundler` will install development dependencies.
-2. Run the rspec tests and ensure it completes without errors with your changes.
+You can then install the necessary gems with:
+
+ make
+
+This will install the requisite rubygems for testing into `.vendor`.
+Note that you can purge all testing fixtures/artifacts/gems with `make clean`.
-3. Run the acceptance tests
+* Docker.
+ Note that Docker is used to run tests that require a Linux container/VM - if you only need to run simple rspec/doc tests, this shouldn't be necessary.
+ If you are developing on a Linux machine with a working Docker instance, this should be sufficient.
+ On OS X, just use the official [Docker installation method](https://docs.docker.com/engine/installation/mac/) to get a working `docker` setup.
+ Confirm that you can communicate with the Docker hypervisor with `docker version`.
-These instructions are for Ubuntu 14.04
+## Testing
-* install docker 0.11.1
- * wget https://get.docker.io/ubuntu/pool/main/l/lxc-docker/lxc-docker_0.11.1_amd64.deb
- * wget https://get.docker.io/ubuntu/pool/main/l/lxc-docker-0.11.1/lxc-docker-0.11.1_0.11.1_amd64.deb
- * sudo dpkg -i lxc-docker_0.11.1_amd64.deb lxc-docker-0.11.1_0.11.1_amd64.deb
- * sudo usermod -a -G docker $USER
-* export RS_SET='ubuntu-server-1404-x64' # see spec/acceptance/nodesets for more
-* export VM_PUPPET_VERSION='3.7.0'
-* wget https://download.elastic.co/elasticsearch/elasticsearch/elasticsearch-1.1.0.deb
-* wget https://forgeapi.puppetlabs.com/v3/files/puppetlabs-stdlib-3.2.0.tar.gz
-* wget https://forgeapi.puppetlabs.com/v3/files/puppetlabs-apt-1.4.2.tar.gz
-* export files_dir=$(pwd)
-* bundle install
-* bundle exec rspec --require ci/reporter/rspec --format CI::Reporter::RSpecFormatter spec/acceptance/*_spec.rb
+Running through the tests on your own machine can get ahead of any problems others (or Jenkins) may run into.
+
+First, run the rspec tests and ensure it completes without errors with your changes. These are lightweight tests.
+
+ make test-rspec
+
+Next, run the more thorough acceptance tests.
+By default, the test will run against a Debian 8 Docker image - other available hosts can be found in `spec/acceptance/nodesets`.
+For example, to run the acceptance tests against CentOS 6, run the following:
+
+ DISTRO=centos-6-x64 make test-acceptance
+
+The final output line will tell you which, if any, tests failed.
-```
- Hypervisor for ubuntu-14-04 is docker
- Beaker::Hypervisor, found some docker boxes to create
- Provisioning docker
- provisioning ubuntu-14-04
- ...
- Finished in 18 minutes 6 seconds
- 224 examples, 0 failures, 3 pending
-```
+## Opening Pull Requests
+
+In summary, to open a new PR:
-4. Rebase your changes
-Update your local repository with the most recent code from the main this puppet module repository, and rebase your branch on top of the latest master branch. We prefer your changes to be squashed into a single commit.
+* Sign the Contributor License Agreement
+* Run the tests to confirm everything works as expected
+* Rebase your changes.
+ Update your local repository with the most recent code from this puppet module repository, and rebase your branch on top of the latest master branch.
+* Submit a pull request
+ Push your local changes to your forked copy of the repository and submit a pull request.
+ In the pull request, describe what your changes do and mention the number of the issue where discussion has taken place, eg "Closes #123".
-5. Submit a pull request
-Push your local changes to your forked copy of the repository and submit a pull request. In the pull request, describe what your changes do and mention the number of the issue where discussion has taken place, eg “Closes #123″.
-
-Then sit back and wait. There will probably be discussion about the pull request and, if any changes are needed, we would love to work with you to get your pull request merged into this puppet module.
+Then sit back and wait!
+There will probably be discussion about the pull request and, if any changes are needed, we would love to work with you to get your pull request merged into this puppet module.