[Date Prev][Date Next] [Thread Prev][Thread Next]
[Thread Index]
[Date Index]
[Author Index]
Re: [atomic-devel] cron job for logrotate on containers
- From: Clayton Coleman <ccoleman redhat com>
- To: Daniel J Walsh <dwalsh redhat com>
- Cc: "walters redhat com" <walters redhat com>, "atomic-devel projectatomic io" <atomic-devel projectatomic io>
- Subject: Re: [atomic-devel] cron job for logrotate on containers
- Date: Tue, 16 Dec 2014 13:06:57 -0500 (EST)
We can't get docker to log to the journal? Oh, because then "docker logs" won't work...
> On Dec 16, 2014, at 12:42 PM, Daniel J Walsh <dwalsh redhat com> wrote:
>
> Working on it, although it requires running systemd/journal within the
> container.
>
>> On 12/16/2014 11:46 AM, Clayton Coleman wrote:
>> I really wish we could just fix Docker logging to not go to disk... and instead go to the journal........
>>
>> :)
>>
>> ----- Original Message -----
>>>> On 12/16/2014 11:10 AM, Lokesh Mandvekar wrote:
>>>> I have a script which would run logrotate on all running docker containers.
>>>> Plan is to have this installed via the docker rpm into /etc/cron.daily.
>>>> Posting here
>>>> for further discussion as per Colin's suggestion before I include it in any
>>>> rpms.
>>>>
>>>> ---
>>>> #!/bin/sh
>>>>
>>>> for id in $(docker ps -q); do
>>>> docker exec $id logrotate -s /var/log/logstatus /etc/logrotate.conf
>>>>> /dev/null 2>&1
>>>> done
>>>> exit 0
>>>> ---
>>>>
>>>> Of course this assumes that logrotate is installed on the containers and
>>>> will ignore any failures
>>>>
>>>> Comments/questions?
>>> This script would run at approximately the same time as the system
>>> logrotate script runs.
>>>
>>> The only problem I see with this is there is no way for the admin to
>>> stop the script from running, if
>>> he does not want logrotate to happen within the containers. We could
>>> fix this by adding a field
>>> to /etc/sysconfig/docker (logrote=true) or something like that.
>>>
>>> We want this script for containers that have logs written within the
>>> container, while we don't recommend doing this, we still want to support
>>> the use case.
>>>
>>> We want to add this to the docker-1.4 release, but would like to get
>>> other peoples comments.
>
[Date Prev][Date Next] [Thread Prev][Thread Next]
[Thread Index]
[Date Index]
[Author Index]