choria-legacy / puppet-nats

NATS.io module for the Choria Orchestrator
http://choria.io
Apache License 2.0
5 stars 16 forks source link

(#33) Add option to run NATS as a user #32

Closed jeefberkey closed 7 years ago

jeefberkey commented 7 years ago

It's best practice to run daemons as their own user, so I have added an option to facilitate that.

When testing, I found that the PIDFile option was not only unneeded, but causing issues when running NATS as another user. A systemd unit file of type simple (default) doesn't need a pid file.

trevor-vaughan commented 7 years ago

Code looks good to me. @ripienaar Any chance of getting this in, or is there more that needs to be done?

ripienaar commented 7 years ago

Thanks @jeefberkey !