Project

General

Profile

action #101379

Reduce amount of unhelpful log messages at debug level

Added by kraih about 1 month ago. Updated 1 day ago.

Status:
Resolved
Priority:
Low
Assignee:
Category:
Feature requests
Target version:
Start date:
2021-10-22
Due date:
% Done:

0%

Estimated time:
Difficulty:
easy

Description

Observation

Even though we keep a lot of logs on OSD, most of it is unhelpful for day to day debugging tasks. Especially the token and API key messages
take up a lot of disk space and provide little value. Mojolicious recently introduced a new log level below debug called trace,
that could be used for these and similar log messages.

[2021-10-21T10:00:12.976949+02:00] [debug] [DhPF1FwhQ81e] 200 OK (0s, ??/s)
[2021-10-21T10:00:12.977305+02:00] [debug] [MknhbAKzf44C] 200 OK (0s, ??/s)
[2021-10-21T10:00:12.977339+02:00] [debug] Received JobToken: A1234567890
[2021-10-21T10:00:12.977642+02:00] [debug] API key from client: *B1234567890*
[2021-10-21T10:00:12.978024+02:00] [debug] API key from client: *C1234567890*
[2021-10-21T10:00:12.978031+02:00] [debug] API key from client: *D1234567890*

Acceptance criteria

  • AC1: Review OSD/O3 logs to identify unhelpful log messages
  • AC2: Add necessary APIs and tests to support trace log level
  • AC3: Move unhelpful log messages to trace level (or remove them completely if they serve no real purpose)

Suggestion

  • Ignore [DhPF1FwhQ81e] 200 OK (0s, ??/s) log messages, since they are a Mojolicious bug that will be gone with the next update

Related issues

Related to openQA Infrastructure - action #96551: Persistent records of systemd journal size:SFeedback2021-10-22

History

#1 Updated by kraih about 1 month ago

  • Parent task deleted (#96551)

#2 Updated by kraih about 1 month ago

  • Related to action #96551: Persistent records of systemd journal size:S added

#3 Updated by kraih about 1 month ago

  • Assignee set to kraih

#4 Updated by kraih 29 days ago

  • Status changed from New to In Progress

#5 Updated by cdywan 27 days ago

kraih wrote:

PR opened. https://github.com/os-autoinst/openQA/pull/4350

Since Mergify doesn't seem to care about open comments, I'm mentioning it here: This PR bumps the minimum Mojo version to 9.20

#6 Updated by kraih 26 days ago

  • Status changed from In Progress to Feedback

#7 Updated by kraih 23 days ago

  • Status changed from Feedback to Resolved

#8 Updated by tinita 23 days ago

  • Target version changed from future to Ready

#9 Updated by okurz 1 day ago

  • Category set to Feature requests

Also available in: Atom PDF