Skip to content
This repository has been archived by the owner on Feb 10, 2022. It is now read-only.

Add logrotate property for containers #153

Open
wants to merge 1 commit into
base: main
Choose a base branch
from

Conversation

Nibooor
Copy link

@Nibooor Nibooor commented Dec 1, 2020

Standard docker container logs are only available for as long as
the container is running, since a monit stop also removes the
container (including logs) using docker rm.

To properly persist logs even after a container is stopped or
restarted it seems that you have to write your own application logs
and bind-mount the log directory to a persistent disk.

However, logs are not rotated this way.

To avoid the necessity of doing manual configurations to the bosh
release or on a VM, make custom logrotation settings possible via
a container property.

Standard docker container logs are only available for as long as
the container is running, since a monit stop also removes the
container (including logs) using docker rm.

To properly persist logs even after a container is stopped or
restarted it seems that you have to write your own application logs
and bind-mount the log directory to a persistent disk.

However, logs are not rotated this way.

To avoid the necessity of doing manual configurations to the bosh
release or on a VM, make custom logrotation settings possible via
a container property.
@linux-foundation-easycla
Copy link

linux-foundation-easycla bot commented Dec 1, 2020

CLA Signed

The committers are authorized under a signed CLA.

@github-actions
Copy link

github-actions bot commented Aug 5, 2021

Stale pull request message

@thoHeinze
Copy link

@neil-hickey Any chance somebody could do a code review for this change? I see you recently merged some PRs, maybe you can help to identify somebody to look into this.

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants