Skip to content

Changed otlphttp to otlp_http in otel-config.yaml due to deprecation of the old name#1253

Merged
martincostello merged 1 commit into
grafana:mainfrom
gfoidl:patch-1
Apr 9, 2026
Merged

Changed otlphttp to otlp_http in otel-config.yaml due to deprecation of the old name#1253
martincostello merged 1 commit into
grafana:mainfrom
gfoidl:patch-1

Conversation

@gfoidl
Copy link
Copy Markdown
Contributor

@gfoidl gfoidl commented Apr 9, 2026

Cf. open-telemetry/opentelemetry-collector#14397 and it's documented in the README.md of the HTTP exporter.

Note: with the old name it still works, but some warnings get logged.

Signed-off-by: Günther Foidl <gue@korporal.at>
@gfoidl gfoidl requested a review from a team as a code owner April 9, 2026 10:51
@cla-assistant
Copy link
Copy Markdown

cla-assistant Bot commented Apr 9, 2026

CLA assistant check
All committers have signed the CLA.

@cla-assistant

This comment was marked as duplicate.

@martincostello martincostello added the enhancement New feature or request label Apr 9, 2026
@martincostello
Copy link
Copy Markdown
Member

Thanks for the PR.

Note: with the old name it still works, but some warnings get logged.

Do you have an example of where these are logged? I wonder if it's something we could detect in CI in the future to avoid this sort of thing (i.e. break CI when something changes).

@gfoidl
Copy link
Copy Markdown
Contributor Author

gfoidl commented Apr 9, 2026

Do you have an example of where these are logged?

Runnint with ENABLE_LOGS_OTELCOL=true the logs showed

...
foo-grafana  | 2026-04-09T10:38:36.418Z        warn    builders/builders.go:40 "otlphttp" alias is deprecated; use "otlp_http" instead {"resource": {"service.instance.id": "ef8332d6-c72d-4e1b-973b-713021f0ddd2", "service.name": "otelcol-contrib", "service.version": "0.147.0"}, "otelcol.component.id": "otlphttp/metrics", "otelcol.component.kind": "exporter", "otelcol.signal": "metrics"}
foo-grafana  | 2026-04-09T10:38:36.432Z        warn    builders/builders.go:40 "otlphttp" alias is deprecated; use "otlp_http" instead {"resource": {"service.instance.id": "ef8332d6-c72d-4e1b-973b-713021f0ddd2", "service.name": "otelcol-contrib", "service.version": "0.147.0"}, "otelcol.component.id": "otlphttp/logs", "otelcol.component.kind": "exporter", "otelcol.signal": "logs"}
foo-grafana  | 2026-04-09T10:38:36.433Z        warn    builders/builders.go:40 "otlphttp" alias is deprecated; use "otlp_http" instead {"resource": {"service.instance.id": "ef8332d6-c72d-4e1b-973b-713021f0ddd2", "service.name": "otelcol-contrib", "service.version": "0.147.0"}, "otelcol.component.id": "otlphttp/traces", "otelcol.component.kind": "exporter", "otelcol.signal": "traces"}
...

So the warn could be grepped quite easily.
Unfortunately I'm not familiar with the CI here, so I would be happy if this could be addressed by someone else (I know your comment was not an intention that I should do it anyway).

@martincostello martincostello merged commit 60a13ff into grafana:main Apr 9, 2026
15 checks passed
@gfoidl gfoidl deleted the patch-1 branch April 14, 2026 10:11
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

enhancement New feature or request

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants