not able to set logstashforwarder::file parameters through yaml

[ltutar]

I am able to define the servers through yaml as
logstashforwarder::servers: ["localhost:5043"]

but this was not possible for the logstashforwarder::file

I created the following wrapper class to enable setting the logstashforwarder::file through yaml as
logstashforwarder_wrapper::files:
'messages':
'paths': ["/var/log/messages"]
'fields':
'type': 'syslog'

class logstashforwarder_wrapper (
$files = $logstashforwarder_wrappper::params::files,
) inherits xlrelease_template::params {
create_resources('logstashforwarder::file', $files)
}

maybe you would like add this functionality to the master. I would not need an extra wrapper class and the parameters can be set through yaml

[electrical]

Hi,

There are several plans for this module.
One is to update it to make it a bit more usable, but Logstash Forwarder is also being replaced by filebeat ( https://www.elastic.co/blog/weekly-beats-first-filebeat-release )

I'll update the repo once the plans are a bit more concrete.

[ltutar]

Hi Richard,
I wanted to use logstash-forwarder because it is stable. I saw some performance issues/bugs in heartbeat puppet module. I wanted to wait before using it.

[electrical]

which heartbeat module?

[ltutar]

my apologies. I meant filebeat. https://github.com/elastic/filebeat/issues

[electrical]

Ah okay. yeah its still in beta at the moment. When GA comes out it should be more stable.
Building the puppet module for the different beats is still in the planning.
I'll discuss internally, but i think we won't be doing much work on this module further except bug fixes.

[ltutar]

ok. no problem. I am able to achieve the functionality with the extra class. so no impediments on my side.