Skip to content

[DOCS-14571] Update Metrics on Config doc#37620

Merged
maycmlee merged 7 commits into
masterfrom
may/op-metrics-ga
Jun 25, 2026
Merged

[DOCS-14571] Update Metrics on Config doc#37620
maycmlee merged 7 commits into
masterfrom
may/op-metrics-ga

Conversation

@maycmlee

Copy link
Copy Markdown
Contributor

What does this PR do? What is the motivation?

Merge readiness

  • Ready for merge

For Datadog employees:

  • ⚠️ Your branch name MUST follow the <name>/<description> convention and include the forward slash (/). If you've already created your PR with an incorrect branch name, please rename your branch and open a fresh PR.
  • 🤖 New: Comment with /review to run an automated check that catches common issues before a Documentation team member reviews your PR.

AI assistance

Additional notes

@maycmlee maycmlee added the WORK IN PROGRESS No review needed, it's a wip ;) label Jun 18, 2026
@maycmlee maycmlee requested a review from a team as a code owner June 18, 2026 16:23
@github-actions

Copy link
Copy Markdown
Contributor

Preview links (active after the build_preview check completes)

Modified Files

Comment thread content/en/observability_pipelines/configuration/_index.md Outdated
maycmlee and others added 2 commits June 23, 2026 17:28
Co-authored-by: ArunPiduguDD <arun.pidugu@datadoghq.com>
@maycmlee maycmlee removed the WORK IN PROGRESS No review needed, it's a wip ;) label Jun 25, 2026
Comment thread content/en/observability_pipelines/configuration/_index.md Outdated
The `counter` metric type is the only `incremental` metric. `gauge`, `distribution`, and `histogram` metric types are `absolute` metrics.
Whether a received metric is `incremental` or `absolute` depends on the source. For example, metrics from [OpenTelemetry][4] can either be incremental or absolute based on their temporality. The following table is an example of an OTel counter metric sent with delta versus cumulative temporality.

| Metric Type | Incremental | Absolute |

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Suggested change
| Metric Type | Incremental | Absolute |
| Metric Type | Delta | Cumulative |

Also wondering if these should use the same terminology as above?

Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

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

Actually, they are different things. Metric kinds are incremental or absolute and for OTel that is determined by temporality (deltas or cumulative sum).

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Oh, gotcha, thanks for clarifying! I wonder if users will make the same mistake; might be worth clarifying if you think they might?

Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

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

We do mention:

OpenTelemetry can either be incremental or absolute based on their [temporality][4].

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Yeah, that's true, I think I'm just not familiar with the product area so I got confused 😅 If you think readers will get it then let's leave it!

Co-authored-by: Janine Chan <64388808+janine-c@users.noreply.github.com>
@maycmlee maycmlee merged commit 619d736 into master Jun 25, 2026
28 checks passed
@maycmlee maycmlee deleted the may/op-metrics-ga branch June 25, 2026 18:59
cswatt pushed a commit that referenced this pull request Jun 29, 2026
* update metrics config

* Apply suggestion from @ArunPiduguDD

Co-authored-by: ArunPiduguDD <arun.pidugu@datadoghq.com>

* fix columns

* fix link

* Apply suggestion from @janine-c

Co-authored-by: Janine Chan <64388808+janine-c@users.noreply.github.com>

---------

Co-authored-by: ArunPiduguDD <arun.pidugu@datadoghq.com>
Co-authored-by: Janine Chan <64388808+janine-c@users.noreply.github.com>
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.

3 participants