Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

ddtrace/tracer: add propagation style info to the startup logs #2748

Merged
merged 4 commits into from
Jun 25, 2024

Conversation

khanayan123
Copy link
Contributor

@khanayan123 khanayan123 commented Jun 17, 2024

What does this PR do?

exposes additional settings in startup info logs

Motivation

to enable golang system tests for otel env var mapping, expose additional configuration settings in startup logs

@pr-commenter
Copy link

pr-commenter bot commented Jun 17, 2024

Benchmarks

Benchmark execution time: 2024-06-24 15:41:26

Comparing candidate commit a235f46 in PR branch khanayan123/add-additional-info-to-startup-logs with baseline commit 0905a11 in branch main.

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

@khanayan123 khanayan123 marked this pull request as ready for review June 17, 2024 22:01
@khanayan123 khanayan123 requested a review from a team as a code owner June 17, 2024 22:01
@khanayan123 khanayan123 force-pushed the khanayan123/add-additional-info-to-startup-logs branch from 70de900 to e70e08f Compare June 17, 2024 22:08
@dianashevchenko dianashevchenko changed the title add propagation style info to the startup logs ddtrace/tracer: add propagation style info to the startup logs Jun 25, 2024
@dianashevchenko dianashevchenko enabled auto-merge (squash) June 25, 2024 21:04
@dianashevchenko dianashevchenko merged commit b70405a into main Jun 25, 2024
162 of 163 checks passed
@dianashevchenko dianashevchenko deleted the khanayan123/add-additional-info-to-startup-logs branch June 25, 2024 21:12
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.

3 participants