search

voxpupuli / puppet-consul

A Puppet Module to Manage Consul
Apache License 2.0
121 stars 315 forks source link

Add ability to manage consul log directory #589

Open jonesbrennan opened 2 years ago

jonesbrennan commented 2 years ago

I am running this module on RHEL 8 using the package install method and a systemd service. There is a log_file parameter but I don't see where it is used for a systemd service. I attempted to add consul's log_file parameter to this module's config_hash parameter. That works fine but then I get a permissioned denied when the consul service starts up. I'm using /var/log/consul/consul.log as a value. This is a request to manage a log directory via this module. This would include ensuring the log directory is created and permissions set appropriately. For example in my scenario the module would ensure /var/log/consul is created and the consul user and group have permissions.

solarkennedy commented 2 years ago

That would probably be fine, although I recommend this be part of your profile_consul that aggregates your consul + related puppet resources to meet your particular needs.