Skip to content

tracer: add orchestrion telemetry#2200

Merged
ahmed-mez merged 2 commits into
mainfrom
ahmed/auto-instru-telem
Sep 4, 2023
Merged

tracer: add orchestrion telemetry#2200
ahmed-mez merged 2 commits into
mainfrom
ahmed/auto-instru-telem

Conversation

@ahmed-mez

Copy link
Copy Markdown
Contributor

What does this PR do?

Report orchestrion (auto-instrumentation) telemetry as part of the existing tracer's configuration telemetry.

Orchestrion should start the tracer with the WithOrchestrion option and rely on the metadata argument to report arbitrary key-value strings (e.g version, toolexec..).

Motivation

Orchestrion telemetry.

Reviewer's Checklist

  • Changed code has unit tests for its functionality at or near 100% coverage.
  • 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.

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!

@ahmed-mez
ahmed-mez requested a review from a team September 4, 2023 12:21
@ahmed-mez
ahmed-mez merged commit 3bd2428 into main Sep 4, 2023
@ahmed-mez
ahmed-mez deleted the ahmed/auto-instru-telem branch September 4, 2023 12:37
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