I didn't spot the bug report 839702 before opening this one:

https://bugs.debian.org/cgi-bin/bugreport.cgi?bug=839702

Looking closer to the security issue mentioned in 839702, a fix came out
recently and the configuration suggested previously should be dropped.

A log-rotate file is present in the gitlab repo here:
https://gitlab.com/gitlab-org/gitlab/-/blob/master/lib/support/logrotate/gitlab

That being said, the current user use by gitlab in debian package is
gitlab. So I think "su git git" should be replace by "su gitlab gitlab".

# GitLab logrotate settings
# based on: http://stackoverflow.com/a/4883967

/home/git/gitlab/log/*.log {
su git git
daily
missingok
rotate 90
compress
notifempty
copytruncate
}

/home/git/gitlab-shell/gitlab-shell.log {
su git git
daily
missingok
rotate 90
compress
notifempty
copytruncate
}

Sorry for the noise,

-- 
**Fabrice MEYER*
Software and System Engineer*

EDF Store & Forecast
13 Avenue Albert Einstein
69100 Villeurbanne
France

*fabrice.me...@edf-sf.co*
*www.edf-sf.com*

Reply via email to