Skip to content

ddtrace/tracer: add JSON logging workaround for tracer logs in WithLogger docs#2406

Merged
darccio merged 1 commit into
mainfrom
dario.castane/AIT-9101/json-logging-workaround-docs
Dec 1, 2023
Merged

ddtrace/tracer: add JSON logging workaround for tracer logs in WithLogger docs#2406
darccio merged 1 commit into
mainfrom
dario.castane/AIT-9101/json-logging-workaround-docs

Conversation

@darccio

@darccio darccio commented Dec 1, 2023

Copy link
Copy Markdown
Member

What does this PR do?

Adds a comment with link to the workaround in the WithLogger, as detailed in #2152 (comment)

Closes #2152

Motivation

Some customers and projects want to have all logs generated as JSON, including the tracer diagnostic and startup logs.

Reviewer's Checklist

  • Changed code has unit tests for its functionality at or near 100% coverage.
  • System-Tests covering this feature have been added and enabled with the va.b.c-dev version tag.
  • There is a benchmark for any new code, or changes to existing code.
  • If this interacts with the agent in a new way, a system test has been added.
  • Add an appropriate team label so this PR gets put in the right place for the release notes.

For Datadog employees:

  • If this PR touches code that handles credentials of any kind, such as Datadog API keys, I've requested a review from @DataDog/security-design-and-guidance.
  • This PR doesn't touch any of that.

Unsure? Have a question? Request a review!

@pr-commenter

pr-commenter Bot commented Dec 1, 2023

Copy link
Copy Markdown

Benchmarks

Benchmark execution time: 2023-12-01 12:43:23

Comparing candidate commit 80fabcd in PR branch dario.castane/AIT-9101/json-logging-workaround-docs with baseline commit d2016f2 in branch main.

Found 0 performance improvements and 0 performance regressions! Performance is the same for 41 metrics, 0 unstable metrics.

@darccio
darccio merged commit 3a9f7f1 into main Dec 1, 2023
@darccio
darccio deleted the dario.castane/AIT-9101/json-logging-workaround-docs branch December 1, 2023 14:58
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

ddtrace/tracer: diagnostic and startup logs should be in JSON format.

2 participants