search

saltstack-formulas / logrotate-formula

http://docs.saltstack.com/en/latest/topics/development/conventions/formulas.html
Other
23 stars 71 forks source link

Added support for FreeBSD #21

Closed msiebeneicher closed 5 years ago

msiebeneicher commented 8 years ago

hi,

i added the basic support for FreeBSD. Worked fine in general (tested on FreeBSD 10.2) - The necessary cronjob will not added to the crontab automatically - that is correct, right?

aboe76 commented 8 years ago

@msiebeneicher I don't know how freebsd packaged logrotate, if they automatically include a cronjob

If I look for logrotate and freebsd I get something about newsyslog.conf? see: http://www.claudiokuenzler.com/blog/474/looking-for-logrotate-in-freebsd-newsyslog.conf

msiebeneicher commented 8 years ago

@aboe76 : newsyslog is the default BSD program to handle log file rotation which is preinstalled on BSD based systems. So normally you don't need logrotate for basic log rotation: https://www.freebsd.org/cgi/man.cgi?query=newsyslog&sektion=8&apropos=0&manpath=FreeBSD+10.2-RELEASE

Did the debian installation automatically add the logrotate cronjob or do you also have to add it after the installation manually?

aboe76 commented 8 years ago

@msiebeneicher debian automatically installs a cron.d entry for logrotate. so that's why it isn't in the formula, besides not all Os-es use cron for logrotate, some use systemd-timers...

msiebeneicher commented 8 years ago

@aboe76: alright - i will check the automatic cron addition again and add a state in case that no cron will added automatically.

javierbertoli commented 5 years ago

@msiebeneicher, @aboe76 closing this as #35 was merged and added support for FreeBSD. Thanks you both!