Skip to content
This repository was archived by the owner on Apr 13, 2026. It is now read-only.

Added files hash argument to init.pp - #65

Open
jskarpe wants to merge 1 commit into
elastic:masterfrom
jskarpe:files-patch
Open

Added files hash argument to init.pp#65
jskarpe wants to merge 1 commit into
elastic:masterfrom
jskarpe:files-patch

Conversation

@jskarpe

@jskarpe jskarpe commented Sep 11, 2015

Copy link
Copy Markdown
Contributor

Following a pattern to allow for The Foreman to configure all aspects of this module requires all parameters to be available through class definitions.

@ghost

ghost commented Sep 11, 2015

Copy link
Copy Markdown

Jenkins standing by to test this. If you aren't a maintainer, you can ignore this comment. Someone with commit access, please review this and clear it for Jenkins to run; then say 'jenkins, test it'.

@jskarpe

jskarpe commented Sep 11, 2015

Copy link
Copy Markdown
Contributor Author

Seems Travis CI fails due to:

Retrying git clone 'git@github.com:puppetlabs/beaker.git' "/home/travis/.rvm/gems/ruby-1.8.7-p374/cache/bundler/git/beaker-18670cc1b84a185c2066a67869906a4430a4c4a8" --bare --no-hardlinks --quiet due to error (2/4): Bundler::Source::Git::GitCommandError Git error: command git clone 'git@github.com:puppetlabs/beaker.git' "/home/travis/.rvm/gems/ruby-1.8.7-p374/cache/bundler/git/beaker-18670cc1b84a185c2066a67869906a4430a4c4a8" --bare --no-hardlinks --quiet in directory /home/travis/build/elastic/puppet-logstashforwarder has failed.

Permission denied (publickey).

fatal: Could not read from remote repository.

@ozbillwang

Copy link
Copy Markdown

@oloc has reported the problem in #58

@ghost

ghost commented Jan 13, 2016

Copy link
Copy Markdown

Can one of the admins verify this patch?

@jskarpe

jskarpe commented Sep 2, 2016

Copy link
Copy Markdown
Contributor Author

Is there something wrong with this patch since it's not merged?

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

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants