Closed Bug 414958 Opened 15 years ago Closed 15 years ago

Move talos utils output to python logger.


(Release Engineering :: General, defect, P2)



(Not tracked)



(Reporter: mikeal, Assigned: mikeal)




(1 file, 4 obsolete files)

User-Agent:       Mozilla/5.0 (Macintosh; U; Intel Mac OS X 10.5; en-US; rv:1.9b3pre) Gecko/2008012804 Minefield/3.0b3pre
Build Identifier: 

The talos utils module does a bunch of output masking and timestamping that could all be replaced using the standard python logging module and would make it more efficient/standardized.

Reproducible: Always

Steps to Reproduce:
Attached file logging patch 1 (obsolete) —
Patch to replace utils output functionality with python logging module.
Attachment #300487 - Flags: review?(anodelman)
Attached patch Logging patch 2 (obsolete) — Splinter Review
Attached patch Logging patch 3 (obsolete) — Splinter Review
Attachment #300487 - Attachment is obsolete: true
Attachment #301307 - Attachment is obsolete: true
Attachment #300487 - Flags: review?(anodelman)
Attached patch Logging patch 4 (obsolete) — Splinter Review
Attachment #301318 - Attachment is obsolete: true
Comment on attachment 301326 [details] [diff] [review]
Logging patch 4

     #set up the counters for this test
     cm = CounterManager("firefox", counters)
+    cm.setDaemon(1)
     counter_results = {}

setDaemon breaks talos on windows machines, raises an AttributeError.
Attachment #301326 - Flags: review-
Attached patch Logging patch 5Splinter Review
Fixing windows failure.

Sets better spacing minimums on log output for better alignment.
Attachment #301326 - Attachment is obsolete: true
Attachment #301608 - Flags: review?(anodelman)
Comment on attachment 301608 [details] [diff] [review]
Logging patch 5

Raised the following error after a run of tsvg (with --debug --noisy).

Traceback (most recent call last):
  File "c:\Python25\lib\logging\", line 744, in emit
    msg = self.format(record)
  File "c:\Python25\lib\logging\", line 630, in format
    return fmt.format(record)
  File "c:\Python25\lib\logging\", line 418, in format
    record.message = record.getMessage()
  File "c:\Python25\lib\logging\", line 288, in getMessage
    msg = msg % self.args
TypeError: not all arguments converted during string formatting

This didn't crash anything but I'd rather it not show up with any frequency in the logs.
Attachment #301608 - Flags: review?(anodelman) → review-
Assignee: nobody → mrogers
Ever confirmed: true
OS: Mac OS X → All
This is currently on the back burner.
Priority: -- → P3
Alice, do we plan on taking this at any time?

Since any change, regardless of how minor, seem to screw the numbers a bit is this worth the risk?

If not we should just resolve as invalid.
This is blocked on comment #7.  I'm not worried about hurting the numbers, more that we don't currently have a patch posted here that works cleanly.
Component: Testing → Release Engineering: Talos
Product: Core →
Version: unspecified → other
Assignee: mrogers → nobody
Priority: P3 → P2
QA Contact: testing → release
Assignee: nobody → mrogers
Wow, a whole lot has changed since that last patch was generated.

Not only does the patch fail to apply in any way, but my local copy of these changes couldn't be brought up to date via cvs up.

At this point, this would have to be completely re-written from scratch and all we would really get out of it is a slightly different logging format than we have now.
Sounds like this is at an impasse.  I'm going to mark wontfix since we do currently have a logging strategy that works - even if it doesn't happen to be the most pretty solution.

We can re-open if a patch becomes workable.
Closed: 15 years ago
Resolution: --- → WONTFIX
Component: Release Engineering: Talos → Release Engineering
Product: → Release Engineering
You need to log in before you can comment on or make changes to this bug.