Update _index.md on the runtime_id_enabled #29782
Merged
+4
−11
Conversation
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
✅ Documentation Team ReviewThe documentation team has approved this pull request. Thank you for your contribution! |
Preview links (active after the
|
duncanhewett
reviewed
Jun 6, 2025
There was a problem hiding this comment.
Looks good. I suggested a couple of minor formatting changes, plus:
- In the Compatibility table above, I think we should remove "Generates runtime-id granularity" from each tab now, since it's addressed in the new configuration description?
- Not strictly related but in that table we could also remove the "Support Level", just keeping it for Python since it's in Preview? (I think users should expect everything in docs to be GA unless otherwise stated)
bm1549
reviewed
Jun 6, 2025
| @@ -163,6 +163,10 @@ Use the following environment variables to configure runtime metrics in your app | |||
| : **Default**: `true` for Java, `false` for all other languages <br> | |||
| **Description**: Enables the collection of runtime metrics. Metrics are sent to the Datadog agent, as configured for the instrumented application. | |||
|
|
|||
| `DD_RUNTIME_ID_ENABLED` | |||
| : **Default**: `false` for Node.js, Ruby and Python languages <br> | |||
There was a problem hiding this comment.
[optional] consider inverting the definition here to make true appear more as the intended behavior
Suggested change
| : **Default**: `false` for Node.js, Ruby and Python languages <br> | |
| : **Default**: `true` for Java, .NET, and Go; `false` for all other languages <br> |
anna-git
force-pushed
the
anna/runtime-id-documentation
branch
from
8db5169 to
75fabf4
Compare
Co-authored-by: Duncan Hewett <[email protected]>
Co-authored-by: Duncan Hewett <[email protected]>
Co-authored-by: Brian Marks <[email protected]>
Signed-off-by: Anna <[email protected]>
anna-git
force-pushed
the
anna/runtime-id-documentation
branch
from
75fabf4 to
c74f05a
Compare
anna-git
commented
Jun 24, 2025
| @@ -163,6 +157,10 @@ Use the following environment variables to configure runtime metrics in your app | |||
| : **Default**: `true` for Java, `false` for all other languages <br> | |||
| **Description**: Enables the collection of runtime metrics. Metrics are sent to the Datadog agent, as configured for the instrumented application. | |||
|
|
|||
| `DD_RUNTIME_METRICS_RUNTIME_ID_ENABLED` | |||
| : **Default**: `true` for Java, `false` for Node.js, Ruby and Python. Does not exist for .NET and Go, runtime-id is always reported <br> | |||
There was a problem hiding this comment.
@bm1549 changed to this as .net and go just don't have the env var and it's not configurable, so itwont work if someone tries to deactivate it for these languages. Let me know if you think it's too complicated and we should just say instead:
true for Java, .NET and Go, false for all other languages.
Done both thanks, left Preview for Python as there is still the overshadowing issue that needs the |
Co-authored-by: Brian Marks <[email protected]>
Co-authored-by: Brett Blue <[email protected]>
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
What does this PR do? What is the motivation?
It adds information about the runtime-id granularity possibility for nodejs/python/ruby
Merge instructions
Merge readiness: