Loop — Update circus configuration to have timestamp information included

RESOLVED FIXED

Status

Cloud Services
Operations
RESOLVED FIXED
3 years ago
3 years ago

People

(Reporter: alexis, Assigned: deanw)

Tracking

Firefox Tracking Flags

(Not tracked)

Details

(Reporter)

Description

3 years ago
When debugging on stage, it turns out that the circus logs don't contain any time information, so it's a bit tricky to use that information.

We could change the circus configuration so that it includes a timestamp. The configuration values to add are:

> stdout_stream.time_format = [%Y/%m/%d | %H:%M:%S]
> stderr_stream.time_format = [%Y/%m/%d | %H:%M:%S]

Which would bring the configuration to something like:

> stdout_stream.class = FileStream
> stdout_stream.filename = /media/ephemeral0/circus/loop_server/loop_server.out.log
> stdout_stream.max_bytes = 104857600
> stdout_stream.backup_count = 10
> stdout_stream.time_format = [%Y/%m/%d | %H:%M:%S]
> stderr_stream.class = FileStream
> stderr_stream.filename = /media/ephemeral0/circus/loop_server/loop_server.err.log
> stderr_stream.max_bytes = 104857600
> stderr_stream.backup_count = 10
> stderr_stream.time_format = [%Y/%m/%d | %H:%M:%S]

Thanks!
(Assignee)

Comment 1

3 years ago
This requires work to be done on the shared circus module. I'll speak to Daniel about it.
Assignee: nobody → dwilson
Status: NEW → ASSIGNED
(Assignee)

Comment 2

3 years ago
Changes in https://github.com/mozilla-services/puppet-config/pull/992

As this is a heavily used shared module it needs an r+.
(Assignee)

Comment 3

3 years ago
This has been merged and will be picked up on the next full deployment
Status: ASSIGNED → RESOLVED
Last Resolved: 3 years ago
Resolution: --- → FIXED
You need to log in before you can comment on or make changes to this bug.