Skip to content

Add release note and documentation for OpenTelemetry HTTP export support #25682

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Merged
merged 1 commit into from
Apr 28, 2025

Conversation

OmerRaifler
Copy link
Contributor

@OmerRaifler OmerRaifler commented Apr 27, 2025

Description

  • Add a release note in 475 mentioning support for exporting OpenTelemetry traces using the HTTP (OTLP/HTTP) protocol.
  • Document the new tracing.exporter.protocol configuration property.
  • Note that the feature was implemented in the Airlift repository.

Additional context and related issues

#25338

Release notes

(x) This is not user-visible or is docs only, and no release notes are required.

@mosabua mosabua requested a review from wendigo April 28, 2025 04:43
@wendigo wendigo force-pushed the release-notes-otlp-http branch from e60c644 to e727070 Compare April 28, 2025 09:30
Copy link
Contributor

@wendigo wendigo left a comment

Choose a reason for hiding this comment

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

I've applied your review comments @mosabua

@wendigo wendigo dismissed mosabua’s stale review April 28, 2025 09:33

Review comments applied

@@ -43,6 +43,10 @@ is accessible from the coordinator and all workers of the cluster. The preceding
example uses a observability platform deployment available by
HTTP at the host `observe.example.com`, port `4317`.

Use the `tracing.exporter.protocol` property to configure the protocol for exporting traces.
Copy link
Member

Choose a reason for hiding this comment

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

It would be nice to use a table format to document these config properties in follow-up.

@wendigo wendigo merged commit fc5e17c into trinodb:master Apr 28, 2025
4 checks passed
@github-actions github-actions bot added this to the 476 milestone Apr 28, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Development

Successfully merging this pull request may close these issues.

4 participants