Skip to content

ddtrace/tracer: fix inaccurate telemetry 'stats_computation_enabled'#2170

Merged
ahmed-mez merged 2 commits into
mainfrom
ahmed/fix-stats-telemetry
Aug 16, 2023
Merged

ddtrace/tracer: fix inaccurate telemetry 'stats_computation_enabled'#2170
ahmed-mez merged 2 commits into
mainfrom
ahmed/fix-stats-telemetry

Conversation

@ahmed-mez

Copy link
Copy Markdown
Contributor

What does this PR do?

Config telemetry was reporting inaccurate metric stats_computation_enabled, it was reporting whether the agent supported client-stats computation (c.agent.Stats) instead of whether the tracer was actually doing stats computation (i.e c.canComputeStats()). This commit fixes it.

Motivation

Bug fix

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.

@ahmed-mez
ahmed-mez requested a review from a team August 10, 2023 07:30
@pr-commenter

pr-commenter Bot commented Aug 10, 2023

Copy link
Copy Markdown

Benchmarks

Benchmark execution time: 2023-08-10 07:40:51

Comparing candidate commit cf2fc2a in PR branch ahmed/fix-stats-telemetry with baseline commit 30bfd52 in branch main.

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

@ahmed-mez
ahmed-mez merged commit 2cddc6a into main Aug 16, 2023
@ahmed-mez
ahmed-mez deleted the ahmed/fix-stats-telemetry branch August 16, 2023 12:23
igoragoli pushed a commit that referenced this pull request Sep 5, 2023
…_enabled' (#2170)

Config telemetry was reporting inaccurate metric stats_computation_enabled, it was reporting whether the agent supported client-stats computation (c.agent.Stats) instead of whether the tracer was actually doing stats computation (i.e c.canComputeStats()). This commit fixes it.
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