Opened at 2007-10-09T17:24:01Z
Closed at 2007-10-12T00:32:48Z
#171 closed enhancement (fixed)
new timestamps
Reported by: | zooko | Owned by: | warner |
---|---|---|---|
Priority: | minor | Milestone: | 0.6.1 |
Component: | code | Version: | 0.6.0 |
Keywords: | Cc: | ||
Launchpad Bug: |
Description
As per e-mail discussion, we want to make ISO-8601'ish, UTC, timestamps with "Z" appended to show that they are UTC, with milliseconds precision.
Change History (3)
comment:1 Changed at 2007-10-11T09:14:33Z by warner
- Owner changed from somebody to warner
- Status changed from new to assigned
comment:2 Changed at 2007-10-11T09:25:55Z by warner
oops, so it turns out that the timeFormat= option doesn't let you make the timestamp be UTC. It's always localtime. So I just removed the misleading "Z" from the end of the string. I think we'll have to override FileLogObserver?.formatTime to fix these.
So still to do:
- add milliseconds
- make it UTC instead of localtime
comment:3 Changed at 2007-10-12T00:32:48Z by warner
- Resolution set to fixed
- Status changed from assigned to closed
ok, I figured out how to add milliseconds and the "Z" suffix, and make it emit UTC. I create a new FileLogObserver? and replace any existing ones when the Node (i.e. Client) instance is created. The first few log messages (emitted by twistd before the .tac file is loaded) show up with the default Twisted format ('2007/10/11 17:26 -0700', using localtime), then they switch over to the new format when the .tac gets control.
I've pushed a change to get most of this. It's only lacking the milliseconds, since time.strftime doesn't provide a format character to get these. Adding milliseconds will probably require modifying the logging code some more.. I'll look at it later in the week, maybe for 0.6.1, maybe not.