Project

General

Profile

Actions

action #6660

closed

[tools][tools][sprint 201709.1][easy hack] use mojo::log in the worker

Added by lnussel over 9 years ago. Updated over 6 years ago.

Status:
Resolved
Priority:
Normal
Assignee:
Category:
Feature requests
Target version:
Start date:
2017-09-06
Due date:
% Done:

100%

Estimated time:
(Total: 0.00 h)

Description

The worker app should use Mojo::Log by setting up a mojo fake app, to ensure that the logging facility is the same across all parts of openQA.

mudler | log_* calls are checking if there is an app and route the messages to Mojo::Log 
mudler | https://github.com/os-autoinst/openQA/blob/master/lib/OpenQA/Utils.pm#L101 &&
mudler | https://github.com/os-autoinst/openQA/blob/master/lib/OpenQA/Utils.pm#L212

Using these different calls, leaves us with a bit of inconsistency when reading log output from different parts.


Subtasks 3 (0 open3 closed)

action #23944: Use Mojo::Log in isotovideo to improve logging. Resolved2017-09-06

Actions
action #25294: [tools] Changes to --verbose flag and logging levels on the workerResolveddasantiago2017-09-14

Actions
action #25296: [tools] Bookmarking of worker log: Have the worker log show what job was it working in the logs.Rejecteddasantiago2017-09-15

Actions
Actions #1

Updated by lnussel over 9 years ago

  • Estimated time set to 2.00 h
Actions #2

Updated by okurz over 8 years ago

  • Category set to Feature requests
  • Priority changed from Normal to Low
Actions #3

Updated by okurz over 8 years ago

  • Priority changed from Low to Normal

I don't know why I lowered priority and I should not without giving a reason ;-)

I don't really understand the benefit of the ticket as described but coolo said it would help us to gather logs of the worker which we could store with the test, e.g. store "ps axf" output every two minutes to see if SUT is under load when wait_serial or assert_screen times out.

Actions #4

Updated by okurz almost 8 years ago

  • Target version changed from Easy hacks to future

as mentioned on irc os-fctry: [Wednesday, 30 November 2016] [15:34:56 CET] anyone minds if I replace the milestone "easy hacks" with a keyword in subject line? IMHO it's not a proper milestone and not at all useful for the roadmap

Actions #5

Updated by okurz almost 8 years ago

  • Subject changed from use mojo::log in the worker to [easy hack] use mojo::log in the worker
Actions #6

Updated by dasantiago about 7 years ago

  • Assignee set to dasantiago
Actions #7

Updated by szarate about 7 years ago

A while ago, I was working on a similar thing but using log4perl instead of Mojo::Log. While is not the best approach, here was part of the idea... https://github.com/os-autoinst/os-autoinst/pull/691

Actions #8

Updated by szarate about 7 years ago

  • Description updated (diff)
Actions #9

Updated by dasantiago about 7 years ago

  • Status changed from New to In Progress
Actions #10

Updated by dasantiago about 7 years ago

  • Status changed from In Progress to Resolved
Actions #11

Updated by szarate about 7 years ago

  • Subject changed from [easy hack] use mojo::log in the worker to [tools][tools][sprint 201709.1][easy hack] use mojo::log in the worker
Actions #12

Updated by okurz over 6 years ago

  • Target version changed from future to future
Actions

Also available in: Atom PDF