Skip to content

Conversation

mtoffl01
Copy link
Contributor

@mtoffl01 mtoffl01 commented Aug 19, 2025

Motivation

dd-trace-java stores the default value of telemetry_heartbeat_interval as an int, but resolves any configured values to a float. When DD_TELEMETRY_HEARTBEAT_INTERVAL=2 is set, dd-trace-java will report its value as 2.0 as expected.

This PR is blocking the Enhanced Config Telemetry effort in dd-trace-java (PR).

Changes

Workflow

  1. ⚠️ Create your PR as draft ⚠️
  2. Work on you PR until the CI passes
  3. Mark it as ready for review
    • Test logic is modified? -> Get a review from RFC owner.
    • Framework is modified, or non obvious usage of it -> get a review from R&P team

🚀 Once your PR is reviewed and the CI green, you can merge it!

🛟 #apm-shared-testing 🛟

Reviewer checklist

  • If PR title starts with [<language>], double-check that only <language> is impacted by the change
  • No system-tests internal is modified. Otherwise, I have the approval from R&P team
  • A docker base image is modified?
    • the relevant build-XXX-image label is present
  • A scenario is added (or removed)?

@mtoffl01 mtoffl01 requested review from a team as code owners August 19, 2025 20:07
@mtoffl01 mtoffl01 requested review from paullegranddc and BridgeAR and removed request for a team August 19, 2025 20:07
@mtoffl01 mtoffl01 merged commit f04085f into main Aug 22, 2025
409 checks passed
@mtoffl01 mtoffl01 deleted the mtoff/java-systests-fix branch August 22, 2025 13:35
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