Template Checking for NilClass

[corey-hammerton]

In https://github.com/pcfens/puppet-filebeat/blob/master/templates/filebeat5.yml.erb#L34 it is currently checking if @filebeat_config['max_procs'] is :undef type, as seen below.

<%- if @filebeat_config['max_procs'] != :undef -%>

In all other if blocks they are checking for a nil value. For consistency's sake this :undef should be changed to nil.

This was uncovered after enabling the Future Parser on a Puppet 3.x environment.

[elconas]

Maybe this is fixed by #65

[corey-hammerton]

@elconas it is not fixed in #65

[pcfens]

What's the specific error message you're getting for this? Originally this was set to nil, but another issue was encountered. It was fixed in 8e5bcf3.

[jarro2783]

I'm seeing this too. I don't know if it actually breaks anything. I suspect filebeat just uses the default value when this is empty. It would be nice not to have nil values set here though.