Skip to content

add process tags to runtime metrics#5210

Merged
wantsui merged 7 commits intomasterfrom
add-process-tags-to-runtime-metrics
Jan 14, 2026
Merged

add process tags to runtime metrics#5210
wantsui merged 7 commits intomasterfrom
add-process-tags-to-runtime-metrics

Conversation

@wantsui
Copy link
Copy Markdown
Collaborator

@wantsui wantsui commented Jan 7, 2026

What does this PR do?

The purpose is to add process tags to the runtime metrics when enabled.

Motivation:

Follows the Python implementation:

Addresses AIDM-263
Change log entry

Yes. Add process tags to runtime metrics when DD_EXPERIMENTAL_PROPAGATE_PROCESS_TAGS_ENABLED is enabled.

Additional Notes:

How to test the change?

Locally:

bundle exec rspec spec/datadog/core/runtime/metrics_spec.rb
bundle exec rspec spec/datadog/core/configuration/components_spec.rb
bundle exec rspec spec/datadog/core/environment/process_spec.rb

I updated one of the Rails tests locally to send the metrics, and I can see them correctly get attached when things are enabled:

image

@github-actions github-actions Bot added the core Involves Datadog core libraries label Jan 7, 2026
@wantsui wantsui added the AI Generated Largely based on code generated by an AI or LLM. This label is the same across all dd-trace-* repos label Jan 7, 2026
Comment thread lib/datadog/core/runtime/metrics.rb Outdated
Copy link
Copy Markdown
Member

@marcotc marcotc left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Looks good!

@pr-commenter
Copy link
Copy Markdown

pr-commenter Bot commented Jan 7, 2026

Benchmarks

Benchmark execution time: 2026-01-08 20:11:20

Comparing candidate commit 82ff245 in PR branch add-process-tags-to-runtime-metrics with baseline commit 5ec9115 in branch master.

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

scenario:profiling - stack collector (ruby frames - native filenames enabled)

  • 🟩 throughput [+161.060op/s; +162.902op/s] or [+5.210%; +5.270%]

@wantsui wantsui marked this pull request as ready for review January 8, 2026 14:59
@wantsui wantsui requested a review from a team as a code owner January 8, 2026 14:59
Comment thread spec/datadog/core/runtime/metrics_spec.rb Outdated
@datadog-official
Copy link
Copy Markdown

datadog-official Bot commented Jan 9, 2026

✅ Tests

🎉 All green!

❄️ No new flaky tests detected
🧪 All tests passed

🎯 Code Coverage
Patch Coverage: 100.00%
Overall Coverage: 95.21% (-0.00%)

View detailed report

This comment will be updated automatically if new data arrives.
🔗 Commit SHA: aa8142a | Docs | Datadog PR Page | Was this helpful? Give us feedback!

@wantsui wantsui merged commit 8e17135 into master Jan 14, 2026
640 checks passed
@wantsui wantsui deleted the add-process-tags-to-runtime-metrics branch January 14, 2026 16:14
@github-actions github-actions Bot added this to the 2.26.0 milestone Jan 14, 2026
@lloeki lloeki mentioned this pull request Jan 16, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

AI Generated Largely based on code generated by an AI or LLM. This label is the same across all dd-trace-* repos core Involves Datadog core libraries

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants