Skip to content

Conversation

KarstenSchnitter
Copy link
Contributor

Dynatrace only accepts "delta" aggregation temporality for counters. The "deltaPreferred" option still uses cumulative temporality for up-and-down-counters. This lets Dynatrace drop the data altogether. This commit will send all counters with delta aggregation temporality.

Dynatrace only accepts "delta" aggregation temporality for counters.
The "deltaPreferred" option still uses cumulative temporality for
up-and-down-counters. This lets Dynatrace drop the data altogether.
This commit will send all counters with delta aggregation temporality.

Signed-off-by: Karsten Schnitter <[email protected]>
Copy link
Contributor

@Anika-Sol Anika-Sol 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, I tested it with an LongUpDownCounter and was able to the the metrics show up in Dynatrace.
Thank you for implementing this fix.

@nicklas-dohrn nicklas-dohrn self-requested a review July 15, 2024 11:18
Copy link
Contributor

@nicklas-dohrn nicklas-dohrn left a comment

Choose a reason for hiding this comment

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

LGTM

@KarstenSchnitter KarstenSchnitter merged commit 9732fe3 into main Jul 15, 2024
@KarstenSchnitter KarstenSchnitter deleted the fix-dt-otel-temporality branch July 15, 2024 11:43
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