-
Notifications
You must be signed in to change notification settings - Fork 10.5k
Closed
Labels
DoneThis issue has been fixedThis issue has been fixedarea-healthchecksIncludes: Healthchecks (some bugs also in Extensions repo)Includes: Healthchecks (some bugs also in Extensions repo)help wantedUp for grabs. We would accept a PR to help resolve this issueUp for grabs. We would accept a PR to help resolve this issue
Milestone
Description
Disclamer: I think logging the duration of health checks is great.
I also think it should be logged as debug only. The current code logs the duration in warnings and errors too - those who are important to monitor in production. In Splunk we group log messages before creating a daily report. When duration is the only differing factor our reports look like this:
Proposal: only include ElapsedMilliseconds in debug messages
And the link to the code
Metadata
Metadata
Assignees
Labels
DoneThis issue has been fixedThis issue has been fixedarea-healthchecksIncludes: Healthchecks (some bugs also in Extensions repo)Includes: Healthchecks (some bugs also in Extensions repo)help wantedUp for grabs. We would accept a PR to help resolve this issueUp for grabs. We would accept a PR to help resolve this issue
