2 fixes

[mfournier]

No description provided.

[electrical]

Hi,

The reason for splitting up per distro is done so i can have even specific settings for each.
The service isn't restarted because its not notified in the config class, that is something i still need to implement.

The main problem is that i haven't spent much time on this module lately but planning to do allot of work on it this weekend to get it up to spec like the Elasticsearch and Logstash modules.

I hope you don't mind i won't merge this PR because of the work im planning to do?

[mfournier]

Sure, @electrical :-) I'm looking forward to test & use your updated module !

Could you take in account the fact that it doesn't work on RHEL currently (where $::operatingsystem == "RedHat") ? Thanks !

[mfournier]

Oh @electrical, BTW it would be great if there was a way to just install the package and SSL cert, without requiring all the configuration & initscript. My use case is that I'm running some instances of lumberjack on the right-hand side of a unix pipe, not as a standalone daemon. With the module as it is now, I had to to something like:

class { '::lumberjack':
  ssl_ca_path => 'puppet:///path/to/lumberjack.crt',
  status      => 'unmanaged',
  host        => 'unspecified',
  port        => '1234',
  files       => [],
}

... which is a bit hackish and not very nice...

Reopening this PR just to be sure you read this msg. Let me know if you prefer I open a "feature-request" bug !

Thanks :-)

[electrical]

Ahh okay. let me take a look at that to see how we can implement that in a decent way :-)

[electrical]

Sorry i haven't done anything to the module.
My development server required a disk replacement and its finally up and running decently again.
I will try to get this stuff done in the next few days.

[electrical]

Finally managed to get some stuff done. still need to update the documentation for it.

Now you can simply install the package by calling the class:

 class { 'lumberjack':; }

and for every instance you want to create:

 lumberjack::instance { 'instance_name':
   some_settings_stuff
 }

if you want to setup the instance but not the service you can set:

 run_as_service => false