action #6660
[tools][tools][sprint 201709.1][easy hack] use mojo::log in the worker
100%
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
History
#1
Updated by lnussel about 6 years ago
- Estimated time set to 2.00 h
#2
Updated by okurz about 5 years ago
- Category set to Feature requests
- Priority changed from Normal to Low
#3
Updated by okurz almost 5 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.
#4
Updated by okurz over 4 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
#5
Updated by okurz over 4 years ago
- Subject changed from use mojo::log in the worker to [easy hack] use mojo::log in the worker
#6
Updated by dasantiago over 3 years ago
- Assignee set to dasantiago
#7
Updated by szarate over 3 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
#8
Updated by szarate over 3 years ago
- Description updated (diff)
#9
Updated by dasantiago over 3 years ago
- Status changed from New to In Progress
#10
Updated by dasantiago over 3 years ago
- Status changed from In Progress to Resolved
#11
Updated by szarate over 3 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
#12
Updated by okurz almost 3 years ago
- Target version changed from future to future