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

Add support for Logstash 7.x #390

Closed
Yggdrasil opened this issue Aug 1, 2019 · 2 comments
Closed

Add support for Logstash 7.x #390

Yggdrasil opened this issue Aug 1, 2019 · 2 comments

Comments

@Yggdrasil
Copy link

Please add support for Logstash 7.x to this module.

Given that the pcfens-filebeat module supports the 7.x branch now, and that Logstash version 7.0 has been release almost four months ago, it would be very nice to be able to upgrade our entire ELK pipeline to 7 to take advantage of the new features of that release.

@ninaspitfire
Copy link
Contributor

Logstash 7 is actually already supported. You can just do this:

class { 'elastic_stack::repo':
  version => 7,
}

class { 'logstash':
  manage_repo => true,
}

You are absolutely right, though, that it should be the default now and should be clear in the documentation.

This PR should have been merged a long time ago, and I can only apologize for the fact that it wasn't.

@ninaspitfire ninaspitfire self-assigned this Aug 5, 2019
@ninaspitfire ninaspitfire removed their assignment Apr 17, 2020
@ninaspitfire
Copy link
Contributor

Fixed in voxpupuli/puppet-elastic_stack#26.

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

No branches or pull requests

3 participants