Note: There are a few cases of duplicates in user autocompletion which are being worked on.

Redirect runner logging output to /var/log/runner.log

RESOLVED FIXED

Status

Release Engineering
Platform Support
RESOLVED FIXED
3 years ago
3 years ago

People

(Reporter: ianconnolly, Assigned: ianconnolly)

Tracking

Firefox Tracking Flags

(Not tracked)

Details

Attachments

(1 attachment, 1 obsolete attachment)

Comment hidden (empty)
we'll need to rotate logs and delete them eventually as well.
(Assignee)

Comment 2

3 years ago
(In reply to Chris AtLee [:catlee] from comment #1)
> we'll need to rotate logs and delete them eventually as well.

Got any suggestions for time-to-rotate and how many we should keep before deleting?
(Assignee)

Updated

3 years ago
Status: NEW → ASSIGNED
No special preferences.

Rotate every boot? Keep 10 previous logs, gzip'ed?
(Assignee)

Comment 4

3 years ago
Created attachment 8447280 [details] [diff] [review]
logging.diff

This should do what we want to do.
Attachment #8447280 - Flags: review?(dustin)
Comment on attachment 8447280 [details] [diff] [review]
logging.diff

Review of attachment 8447280 [details] [diff] [review]:
-----------------------------------------------------------------

r+ with that change

::: modules/runner/templates/runner.initd.erb
@@ +37,2 @@
>      ### XXX -n 1 for testing
> +    if ${CMD} -c ${CONFIG} -n 1 ${TASKDIR} > /var/log/runner.log 2>&1; then

You probably want >> here
Attachment #8447280 - Flags: review?(dustin) → review+
(Assignee)

Comment 6

3 years ago
Hrm, even with the logrotate -f line forcing a log rotation (and thus an empty file) directly above?(In reply to Dustin J. Mitchell [:dustin] from comment #5)
> Comment on attachment 8447280 [details] [diff] [review]
> logging.diff
> 
> Review of attachment 8447280 [details] [diff] [review]:
> -----------------------------------------------------------------
> 
> r+ with that change
> 
> ::: modules/runner/templates/runner.initd.erb
> @@ +37,2 @@
> >      ### XXX -n 1 for testing
> > +    if ${CMD} -c ${CONFIG} -n 1 ${TASKDIR} > /var/log/runner.log 2>&1; then
> 
> You probably want >> here

Hrm, even with the logrotate -f line forcing a log rotation (and thus an empty file) directly above?
(Assignee)

Comment 7

3 years ago
Created attachment 8447293 [details] [diff] [review]
logging.diff

Carrying the r+ over from :dustin.
Attachment #8447280 - Attachment is obsolete: true
Attachment #8447293 - Flags: review+
Attachment #8447293 - Flags: checked-in?
Attachment #8447293 - Flags: checked-in? → checked-in+
(Assignee)

Updated

3 years ago
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.