Opened 19 months ago

Closed 5 months ago

Last modified 5 months ago

#2449 closed enhancement (fixed)

Add option to clear screen logs on /logs

Reported by: Reportage Owned by: zzz
Priority: minor Milestone: 0.9.46
Component: apps/console Version: 0.9.38
Keywords: logging, UX Cc:
Parent Tickets: Sensitive: no

Description

As discussed on IRC, a button in the header of the critical, router and wrapper logs to clear the existing screen logs would be useful in cases where the logs are stale or consuming too much screen real estate. A button to purge logs from disc might also be worth considering.

Subtickets

Change History (4)

comment:1 Changed 7 months ago by zzz

Milestone: undecided0.9.46
Sensitive: unset
Status: newaccepted

We could do the critical one, it's in-memory. Router and wrapper, maybe, would have to store a line number to skip to, if we're not just zeroing out the file.

comment:2 Changed 5 months ago by zzz

Both crit and router are in memory, so same solution worked for both.
In fcbf54daf71337dbc3775222804f924b70988dd5 to be 0.9.45-16.
Leaving open to decide if doing something for wrapper log would be worth it, where we have the issue described above.

comment:3 Changed 5 months ago by zzz

Resolution: fixed
Status: acceptedclosed

It was as painful as I thought it would be, but didn't want to leave it half done. Wrapper log button in 5d8e914ab051e0e70232360220d5ebc7db600fdd to be 0.9.45-16. Only supported with real wrapper. Runplain works on timestamp only, not worth fixing that one.

comment:4 Changed 5 months ago by zzz

Fixed the no-wrapper case in 3a4b8c3f65ad560ac429293d40868fca3b023e16 to be 0.9.45-16

Note: See TracTickets for help on using tickets.