Add logrotate to docker container

[DjSni]

Hello,

I have mounted the logs of nginx via a volume, because crouwdsec has to read them and I can also have a look from time to time even if the docker container is off.
I noticed that my logs have become almost 1GB in size in the meantime.
I think it would be good if the logrotate service is installed in the docker container.
Since nginx already provides the corresponding configs, nothing else needs to be set up.

Greetings

[jearton]

The log rotation is already configured in the /etc/logrotate.d/nginx file within docker container.

Reference tutorial: https://www.digitalocean.com/community/tutorials/how-to-configure-logging-and-log-rotation-in-nginx-on-an-ubuntu-vps#log-rotation-with-logrotate

[DjSni]

The log rotation is already configured in the /etc/logrotate.d/nginx file within docker container.

Reference tutorial: https://www.digitalocean.com/community/tutorials/how-to-configure-logging-and-log-rotation-in-nginx-on-an-ubuntu-vps#log-rotation-with-logrotate

But not installed.

[jearton]

I got it. So now we can only enter the container and install it.

apt update && apt install logrotate

[DjSni]

Ohh stupid, unfortunately it's not that easy since logrotate requires cron, this would also have to be installed.

This would only make the image unnecessarily larger, but 0xJacky could trigger it via the internal cron of nginx-ui, I just don't know how time-consuming that would be.

PS: it can be executed manually via logrotate /etc/logrotate.d/nginx.

[0xJacky]

Ok, I plan to add this command into gocron, maybe later this week.

[DjSni]

Ok, I plan to add this command into gocron, maybe later this week.

cool :) but don't forget you still have to install logrotate in the docker image.

[jearton]

Recommended configuration about postrotate:

    postrotate
        /bin/kill -USR1 `cat /var/run/nginx.pid 2>/dev/null` 2>/dev/null || true
    endscript

[0xJacky]

Hi,
This feature has been added in v2.0.0-beta.18, please note that you have to pull the lastest nginx-ui image from dockerhub, or you can maually install logrotate via apt in current container before upgrading nginx-ui.

If you find any problem about this feature, please reply to this issue, thanks.