Skip to content

Conversation

@philon-msft
Copy link
Collaborator

@philon-msft philon-msft commented Jul 1, 2021

Prefix each entry with a timestamp that looks like:

18:18:47.9261: Connecting (async) on .NET Core 3.1.16
18:18:47.9263: cachename.redis.cache.windows.net:6380/Interactive: Connecting...
18:18:47.9463: cachename.redis.cache.windows.net:6380: BeginConnectAsync
18:18:47.9483: 1 unique nodes specified

Prefix each entry with a timestamp that looks like:

2021-07-01T18:18:47.9263352Z: cachename.redis.cache.windows.net:6380/Interactive: Connecting...
2021-07-01T18:18:47.9463333Z: cachename.redis.cache.windows.net:6380: BeginConnectAsync
2021-07-01T18:18:47.9483351Z: 1 unique nodes specified
@NickCraver
Copy link
Collaborator

This likely dupes up timing where we do it for tests today - need to adjust there but then good to go 👍 I'll try and take a peek tonight, but away from desk a lot today.

Nick Craver added 2 commits July 1, 2021 22:24
This also shortens the format to HH:mm:ss.ffff, which is all we need for debugging and it's less to grok. Also prevents double timestamp logging in tests.
@NickCraver NickCraver changed the title Add timestamp on default logger entries Add .NET version and timestamp on default logger rows Jul 2, 2021
Copy link
Collaborator

@NickCraver NickCraver left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

👍

@NickCraver NickCraver merged commit ae4d29a into StackExchange:main Jul 2, 2021
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants