Merge pull request #4 from matthewbarr/autoslave

Autoslave
Merge pull request #5 from rtyler/master

Keeping up to date
fix for slave not running under puppet
Merge branch 'master' of github.com:matthewbarr/puppet-jenkins
Merge of autoslave
Merge branch 'master' of github.com:rtyler/puppet-jenkins into rtyler-master

Conflicts:
	manifests/slave.pp
fixed slave.pp
* new class param config_hash to configure Jenkins
* new class config to pass config options
* new define to set the actual options
Add posibility to install multiple plugins in one call or via Hiera config.
Ordering
Update firewall.pp

fix for [issue#52](https://github.com/rtyler/puppet-jenkins/issues/52)
Merge pull request #56 from leonmax/patch-1

Update firewall.pp
Merge pull request #55 from LarsFronius/fixuplts

Fix LTS issue + testing RT #51
Use the proper rubygems.org URL
Update the README to be a bit better organized, and contain a bit of info about librarian
For local development, pin to ~2.7, not 3. Might need to change this real soon now
Merge pull request #54 from zipkid/configuration

Optional config param to set sysconfig settings
Fix a number of puppet-lint warnings/errors, cleaned up a good chunk of code
Fixing two issues in jenkins::config

1) For the Debian family (Checked Debian Wheezy and Ubuntu Precise) the defaults folder is in /etc/default instead of /etc/defaults
2) Removed prefixing of JENKINS_ on each variable as it removes flexibilty. If a user wants to add that - they can. This now enables me to set things like
HTTP_PORT correctly. Also - as a word of caution, file_line adds new resources to the end of the file - This may cause issues (For example, if HTTP_PORT is set after
JENKINS_ARGS then HTTP_PORT will not actually be read in, as JENKINS_ARGS calls it.

Tested on Ubuntu Precise 12.04 (Only checked the default part on Wheezy, didn't test - Can't imagine it being any different though)

  class { 'jenkins':
    config_hash => {
      'HTTP_PORT' => { 'value' => 2222 }
    }
  }

Was applied to my test node successfully.
Fixing Test (They were all due to linter erroring out on trailing whitespace)

All "errors" were caused by the linter giving an exit code on trailing white space. Went ahead and removed it
all - Also cleaned up as many warning as I could for lines > 80 - Left a few but as they're strings I'm not really sure
if forcing it under 80 would make it look any cleaner.