action #6660

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

Added by lnussel about 5 years ago. Updated almost 2 years ago.

Status:ResolvedStart date:06/09/2017
Priority:NormalDue date:
Assignee:dasantiago% Done:

100%

Category:Feature requests
Target version:QA - future
Difficulty:
Duration:

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

action #23944: Use Mojo::Log in isotovideo to improve logging. Resolved

action #25294: [tools] Changes to --verbose flag and logging levels on t...Resolveddasantiago

action #25296: [tools] Bookmarking of worker log: Have the worker log sh...Rejecteddasantiago

History

#1 Updated by lnussel about 5 years ago

  • Estimated time set to 2.00

#2 Updated by okurz about 4 years ago

  • Category set to Feature requests
  • Priority changed from Normal to Low

#3 Updated by okurz almost 4 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 3 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 3 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 2 years ago

  • Assignee set to dasantiago

#7 Updated by szarate over 2 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 2 years ago

  • Description updated (diff)

#9 Updated by dasantiago over 2 years ago

  • Status changed from New to In Progress

#10 Updated by dasantiago over 2 years ago

  • Status changed from In Progress to Resolved

#11 Updated by szarate over 2 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 2 years ago

  • Target version changed from future to future

Also available in: Atom PDF