Update otel deps from v0.125.0 to v0.126.0 #3781

mar4uk · 2025-06-05T12:23:30Z

PR Description

The aim of this PR is upgrading otel dependencies version step by step. The renovate upgrade to v0.127.0 fails. The issue there is not straight forward to fix. It was caused by the change open-telemetry/opentelemetry-collector#12996 in v0.127.0.
Upgrading to v0.126.0 doesn't contain that change, and upgrading to it is pretty easy. Upgrading to v0.126.0 will make the next upgrading to v0.127.0 a bit easier as well

For Reviewers

open-telemetry/opentelemetry-collector-contrib#39767 - change in contrib related to the datadog exporter metrics config instrumentation_scope_metadata_as_tags changed default value to true

PR Checklist

Tests updated
Config converters updated

… default

clayton-cornell · 2025-06-09T16:53:52Z

If we update the OTel version, the main _index.md and _index.md.tfiles in the docs needs a variable update as well (update the*.tfile and runmake generate-versioned-files`).

There's a PR in-progress that updates these files to v0.125.1 #3790 I'm guessing that since this is marked as WIP, the other PR will probably merge first and there won't be any collisions.

clayton-cornell · 2025-06-09T21:58:40Z

docs/sources/reference/components/otelcol/otelcol.auth.oauth2.md

@@ -81,7 +83,13 @@ You can use the following blocks with `otelcol.auth.oauth2`:
 The `tls` block configures TLS settings used for connecting to the token client.
 If the `tls` block isn't provided, TLS won't be used for communication.

-{{< docs/shared lookup="reference/components/otelcol-tls-client-block.md" source="alloy" version="<ALLOY_VERSION>" >}}
+\{\{< docs/shared lookup="reference/components/otelcol-tls-client-block.md" source="alloy" version="<ALLOY_VERSION>" >\}\}


Something broke here? We don't need to escape the curly brackets... Same throughout the topics here... copy pasted the error?

dehaansa · 2025-06-10T04:16:36Z

docs/sources/reference/components/otelcol/otelcol.connector.spanmetrics.md

+| Block                                      | Description                                                                                                                                                | Required |
+|--------------------------------------------|------------------------------------------------------------------------------------------------------------------------------------------------------------|----------|
+| [`histogram`][histogram]                   | Configures the histogram derived from spans durations.                                                                                                     | yes      |
+| `histogram` > [`dimension`][dimension]     | Span event attributes to add as dimensions to the duration metric, _on top of_ the default ones and the ones configured in the top-level `dimension` block | no       |


CC @clayton-cornell - as this block is nested, I wanted it by its parent, however the parent is required and it is not. Thoughts?

All good. There are cases like that scattered throughout the component docs. I clustered them exactly like this. This makes sense.

dehaansa · 2025-06-10T04:23:22Z

The only relevant thing I didn't pull in from the otel changelog was the datadogreceiver's 128bit traceID support. This seems like a great thing to bring in, but I was just triaging an issue with it earlier this week and want to be sure it's ready for production use before we enable it by default, as we don't have a pattern right now for supporting otelcol feature flags being configured by the user.

- `datadogreceiver`: Support 128 bits TraceIDs coming from Datadog (#36926)
  This fix is released behind a featuregate so as to not be a breaking change for users. To enable it, enable 
  `receiver.datadogreceiver.Enable128BitTraceID`. The feature uses an LRU cache that will keep the last 100 trace ids by
  default. This can be configured via `receivers.datadog.trace_id_cache_size`.

thampiotr

Nothing major, so giving it LGTM % comments

CHANGELOG.md

docs/sources/reference/components/otelcol/otelcol.exporter.kafka.md

docs/sources/shared/reference/components/otelcol-tls-tpm-block.md

Co-authored-by: Piotr <[email protected]>

clayton-cornell

Minor comments, otherwise good to go.

clayton-cornell · 2025-06-10T16:50:54Z

docs/sources/reference/components/otelcol/otelcol.connector.spanmetrics.md

+| Block                                      | Description                                                                                                                                                | Required |
+|--------------------------------------------|------------------------------------------------------------------------------------------------------------------------------------------------------------|----------|
+| [`histogram`][histogram]                   | Configures the histogram derived from spans durations.                                                                                                     | yes      |
+| `histogram` > [`dimension`][dimension]     | Span event attributes to add as dimensions to the duration metric, _on top of_ the default ones and the ones configured in the top-level `dimension` block | no       |


All good. There are cases like that scattered throughout the component docs. I clustered them exactly like this. This makes sense.

clayton-cornell · 2025-06-10T16:51:59Z

docs/sources/reference/components/otelcol/otelcol.connector.spanmetrics.md

+| `histogram` > [`explicit`][explicit]       | Configuration for a histogram with explicit buckets.                                                                                                       | no       |
+| `histogram` > [`exponential`][exponential] | Configuration for a histogram with exponential buckets.                                                                                                    | no       |


I'd move these two up with the parent histogram block as well.

update otel deps from v0.125.0 to v0.126.0

4e3a6f3

mar4uk marked this pull request as ready for review June 5, 2025 13:43

mar4uk requested review from clayton-cornell and a team as code owners June 5, 2025 13:43

fix datadogexporter: instrumentation_scope_metadata_as_tags = true by…

1650186

… default

mar4uk force-pushed the update-otel-deps branch from 62c262a to 1650186 Compare June 5, 2025 14:34

mar4uk changed the title ~~update otel deps from v0.125.0 to v0.126.0~~ [WIP] update otel deps from v0.125.0 to v0.126.0 Jun 5, 2025

Add TLS tpm config

d55f5e2

clayton-cornell reviewed Jun 9, 2025

View reviewed changes

dehaansa added 2 commits June 9, 2025 21:32

Fix find-replace error

b74e287

Update docs and components to match otel 126 updates

4d8a308

dehaansa reviewed Jun 10, 2025

View reviewed changes

dehaansa changed the title ~~[WIP] update otel deps from v0.125.0 to v0.126.0~~ Update otel deps from v0.125.0 to v0.126.0 Jun 10, 2025

dehaansa added 2 commits June 10, 2025 00:32

Fix tests

231c0a6

another test fix

dbcc1e1

thampiotr approved these changes Jun 10, 2025

View reviewed changes

Apply suggestions from code review

395d309

Co-authored-by: Piotr <[email protected]>

clayton-cornell approved these changes Jun 10, 2025

View reviewed changes

dehaansa added 2 commits June 10, 2025 13:07

Merge branch 'main' into update-otel-deps

2914b47

Update docs from PR feedback

0cc4e10

dehaansa enabled auto-merge (squash) June 10, 2025 17:09

dehaansa merged commit 529eed0 into grafana:main Jun 10, 2025
37 checks passed

Provide feedback

Saved searches

Use saved searches to filter your results more quickly

Uh oh!

Update otel deps from v0.125.0 to v0.126.0 #3781

Update otel deps from v0.125.0 to v0.126.0 #3781

mar4uk commented Jun 5, 2025 •

edited

Loading

Uh oh!

clayton-cornell commented Jun 9, 2025

Uh oh!

clayton-cornell Jun 9, 2025 •

edited

Loading

Uh oh!

dehaansa Jun 10, 2025

Uh oh!

clayton-cornell Jun 10, 2025

Uh oh!

dehaansa commented Jun 10, 2025

Uh oh!

thampiotr left a comment

Uh oh!

Uh oh!

Uh oh!

Uh oh!

Uh oh!

Uh oh!

clayton-cornell left a comment

Uh oh!

clayton-cornell Jun 10, 2025

Uh oh!

clayton-cornell Jun 10, 2025

Uh oh!

Uh oh!

Uh oh!

		\| `histogram` > [`explicit`][explicit] \| Configuration for a histogram with explicit buckets. \| no \|
		\| `histogram` > [`exponential`][exponential] \| Configuration for a histogram with exponential buckets. \| no \|

Update otel deps from v0.125.0 to v0.126.0 #3781

Update otel deps from v0.125.0 to v0.126.0 #3781

Conversation

mar4uk commented Jun 5, 2025 • edited Loading Uh oh! There was an error while loading. Please reload this page.

Uh oh!

PR Description

For Reviewers

PR Checklist

Uh oh!

clayton-cornell commented Jun 9, 2025

Uh oh!

clayton-cornell Jun 9, 2025 • edited Loading Uh oh! There was an error while loading. Please reload this page.

Uh oh!

Choose a reason for hiding this comment

Uh oh!

dehaansa Jun 10, 2025

Choose a reason for hiding this comment

Uh oh!

clayton-cornell Jun 10, 2025

Choose a reason for hiding this comment

Uh oh!

dehaansa commented Jun 10, 2025

Uh oh!

thampiotr left a comment

Choose a reason for hiding this comment

Uh oh!

Uh oh!

Uh oh!

Uh oh!

Uh oh!

Uh oh!

clayton-cornell left a comment

Choose a reason for hiding this comment

Uh oh!

clayton-cornell Jun 10, 2025

Choose a reason for hiding this comment

Uh oh!

clayton-cornell Jun 10, 2025

Choose a reason for hiding this comment

Uh oh!

Uh oh!

Uh oh!

mar4uk commented Jun 5, 2025 •

edited

Loading

clayton-cornell Jun 9, 2025 •

edited

Loading