Repository navigation
Carry the agent manifest inside metadata so it reaches the backend - #12512
gh-worker-dd-mergequeue-cf854d[bot] merged 5 commits into
Conversation
Co-Authored-By: Claude Opus 5 (1M context) <noreply@anthropic.com>
This comment has been minimized.
This comment has been minimized.
🟢 Java Benchmark SLOs — All performance SLOs passed
PR vs. master results
Commit: Load and DaCapo benchmarks can be triggered manually in the GitLab pipeline. Results will appear in the Benchmarking Platform UI after completion. |
Co-Authored-By: Claude Opus 5 (1M context) <noreply@anthropic.com>
Co-Authored-By: Claude Opus 5 (1M context) <noreply@anthropic.com>
There was a problem hiding this comment.
More details
The mapper keeps the agent manifest at meta.metadata._dd.agent_manifest. The updated tests cover storage and serialization at this path.
🤖 Datadog Autotest · Commit ac4e69a · What is Autotest? · @DataDog review to ask questions · Any feedback? Reach out in #autotest
There was a problem hiding this comment.
💡 Codex Review
Here are some automated review suggestions for this pull request.
Reviewed commit: ac4e69a175
ℹ️ About Codex in GitHub
Codex has been enabled to automatically review pull requests in this repo. Reviews are triggered when you
- Open a pull request for review
- Mark a draft as ready
- Comment "@codex review".
If Codex has suggestions, it will comment; otherwise it will react with 👍.
When you sign up for Codex through ChatGPT, Codex can also answer questions or update the PR, like "@codex address that feedback".
|
Is it possible to read it from both spots in the backend to not lose on data coming from this update ? |
@yahya-mouman I would probably suggest against that just because we don't want to add more tech debt to our backend for an update that we have yet to document. It's probably better to patch 1.66 release line instead. |
|
/merge |
|
View all feedbacks in Devflow UI.
It will be processed automatically as soon as GitHub reports it as mergeable. View in MergeQueue UI.
The expected merge time in
|
Co-Authored-By: Claude Opus 5 (1M context) <noreply@anthropic.com>
|
/merge |
|
View all feedbacks in Devflow UI.
PR already in the queue with status waiting |
What Does This Do
annotateAgentManifest()(shipped in v1.66.0) wrote the manifest tometa.agent_manifest, but the backend reads it frommeta.metadata._dd.agent_manifest, so Java manifests never showed up in agent tracking.The manifest now goes into the existing metadata span tag under the reserved
_ddnamespace, so it serializes to the right place. Since it is no longer a span tag of its own,LLMObsSpanMapperloses its dedicated branch and wire key.The public API is unchanged — no caller changes needed.
Manual Testing
Verified that agent manifest annotation gets stored on the span correctly (trace)

Contributor Checklist
./gradlew spotlessApply:dd-trace-core:testand:dd-java-agent:agent-llmobs:testpass locallyClaude session:
1d00ff6a-4bf0-49b6-94c5-3464fb28b08fResume:
claude --resume 1d00ff6a-4bf0-49b6-94c5-3464fb28b08f🤖 Generated with Claude Code