Ticket #196 (closed enhancement: fixed)

Opened 4 years ago

Last modified 2 years ago

Use standard logging facilities

Reported by: kparal Owned by: mkrizek
Priority: minor Milestone: 0.7.0
Component: core Keywords:
Cc: Blocked By:
Blocking:

Description (last modified by kparal) (diff)

Currently our watchers and the main autoqa script print all debug/error messages to standard (error) output. If it is run via cron, error output is sent to root. But there's no log of it. We should change that. We should use syslog and print all our messages to /var/log/autoqa.log or similar file (we can put there a lot of debug messages which can help resolve possible problems in the future).

So let's examine standard python logging capabilities and use them if possible.

We don't probably need to adjust tests, because tests are executed on clients and everything is perfectly logged and stored by autotest. Therefore this concerns just watchers/autoqa script I think.

Change History

comment:1 Changed 3 years ago by kparal

  • Description modified (diff)
  • Summary changed from Log failed job scheduling to Use standard logging facilities
  • Milestone set to 0.4.4
  • Priority changed from major to minor

comment:2 Changed 3 years ago by kparal

  • Owner kparal deleted

comment:3 Changed 3 years ago by kparal

mkrizek tried to implement this, his code is in the 'logging' branch

comment:4 Changed 3 years ago by kparal

  • Milestone changed from 0.4.4 to 0.5.0

comment:5 Changed 3 years ago by kparal

  • Milestone changed from 0.5.0 to Nice to have soon

comment:6 Changed 3 years ago by tflink

  • Milestone changed from Nice to have soon to 0.7.0

comment:7 Changed 3 years ago by mkrizek

  • Owner set to mkrizek

comment:9 Changed 3 years ago by kparal

Heavy comments addition warning!

comment:10 Changed 2 years ago by mkrizek

I added comments.

comment:11 Changed 2 years ago by mkrizek

The patch is updated.

comment:12 Changed 2 years ago by kparal

Reviewed.

Btw, please update the 'logging' branch.

comment:13 Changed 2 years ago by mkrizek

The patch is updated as well as the 'logging' branch.

comment:14 Changed 2 years ago by kparal

Reviewed.

comment:15 Changed 2 years ago by mkrizek

  • Resolution set to fixed
  • Status changed from new to closed
Note: See TracTickets for help on using tickets.