Skip to content

Support sending OpenTelemetry logs over OTLP (http/protobuf) - #11253

Merged
gh-worker-dd-mergequeue-cf854d[bot] merged 7 commits into
masterfrom
mcculls/otlp-logs-proto
May 2, 2026
Merged

Support sending OpenTelemetry logs over OTLP (http/protobuf)#11253
gh-worker-dd-mergequeue-cf854d[bot] merged 7 commits into
masterfrom
mcculls/otlp-logs-proto

Conversation

@mcculls

Copy link
Copy Markdown
Contributor

Additional Notes

Builds on top of #11250

Contributor Checklist

Jira ticket: [PROJ-IDENT]

Note:Once your PR is ready to merge, add it to the merge queue by commenting /merge./merge -c cancels the queue request. /merge -f --reason "reason" skips all merge queue checks; please use this judiciously, as some checks do not run at the PR-level. For more information, see this doc.

@mcculls
mcculls requested review from a team as code ownersMay 1, 2026 08:57
@mcculls
mcculls requested review from dougqh and removed request for a teamMay 1, 2026 08:57
@mccullsmcculls added the inst: opentelemetry OpenTelemetry instrumentation label May 1, 2026
@mcculls
mcculls requested review from mhlidd and mtoffl01 and removed request for a teamMay 1, 2026 08:57

@chatgpt-codex-connectorchatgpt-codex-connectorBot left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

💡 Codex Review

Here are some automated review suggestions for this pull request.

Reviewed commit:48a6076f65

ℹ️ About Codex in GitHub

Codex has been enabled to automatically review pull requests in this repo. Reviews are triggered when you

  • Open a pull request for review
  • Mark a draft as ready
  • Comment "@codex review".

If Codex has suggestions, it will comment; otherwise it will react with 👍.

When you sign up for Codex through ChatGPT, Codex can also answer questions or update the PR, like "@codex address that feedback".

Comment threaddd-trace-core/src/main/java/datadog/trace/core/otlp/logs/OtlpLogsProto.java Outdated
Comment threadinternal-api/src/main/java/datadog/trace/api/Config.java Outdated
Comment threaddd-trace-core/src/main/java/datadog/trace/core/otlp/logs/OtlpLogsProto.java Outdated
Comment threaddd-trace-core/src/main/java/datadog/trace/core/otlp/logs/OtlpLogsProto.java Outdated

@dougqhdougqh left a comment

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.

Overall, it looks good to me.

I've added a few performance suggestions both my own and from Claude.
I don't feel strongly that any of them must be addressed before this is merged.

@mcculls
mccullsforce-pushed the mcculls/otlp-logs-proto branch from 890ebfa to a28ffd6CompareMay 2, 2026 11:41
@mcculls
mcculls enabled auto-merge May 2, 2026 11:42
@mcculls
mcculls disabled auto-merge May 2, 2026 12:48
@mcculls
mccullsforce-pushed the mcculls/otlp-logs-proto branch from a28ffd6 to 8cacef8CompareMay 2, 2026 12:53
@mcculls
mcculls enabled auto-merge May 2, 2026 13:23
@mcculls
mcculls added this pull request to the merge queueMay 2, 2026
@dd-octo-sts

Copy link
Copy Markdown
Contributor

/merge

@gh-worker-devflow-routing-ef8351

gh-worker-devflow-routing-ef8351Bot commented May 2, 2026

Copy link
Copy Markdown

View all feedbacks in Devflow UI.

2026-05-02 13:48:04 UTC ℹ️ Start processing command /merge


2026-05-02 13:48:08 UTC ℹ️ MergeQueue: pull request added to the queue

The expected merge time in master is approximately 1h (p90).


2026-05-02 15:48:39 UTCMergeQueue: The build pipeline has timeout

The merge request has been interrupted because the build 0 took longer than expected. The current limit for the base branch 'master' is 120 minutes.

@github-merge-queue
github-merge-queueBot removed this pull request from the merge queue due to failed status checks May 2, 2026
@mcculls
mcculls added this pull request to the merge queueMay 2, 2026
@dd-octo-sts

Copy link
Copy Markdown
Contributor

/merge

@gh-worker-devflow-routing-ef8351

gh-worker-devflow-routing-ef8351Bot commented May 2, 2026

Copy link
Copy Markdown

View all feedbacks in Devflow UI.

2026-05-02 15:49:11 UTC ℹ️ Start processing command /merge


2026-05-02 15:49:16 UTC ℹ️ MergeQueue: pull request added to the queue

The expected merge time in master is approximately 1h (p90).


2026-05-02 17:02:28 UTC ℹ️ MergeQueue: This merge request was merged

@github-merge-queue
github-merge-queueBot removed this pull request from the merge queue due to failed status checks May 2, 2026
@gh-worker-dd-mergequeue-cf854d
gh-worker-dd-mergequeue-cf854dBot merged commit 0422673 into masterMay 2, 2026
570 checks passed
@gh-worker-dd-mergequeue-cf854d
gh-worker-dd-mergequeue-cf854dBot deleted the mcculls/otlp-logs-proto branch May 2, 2026 17:02
@github-actionsgithub-actionsBot added this to the 1.62.0 milestone May 2, 2026
Sign up for freeto join this conversation on GitHub. Already have an account? Sign in to comment

Labels

inst: opentelemetryOpenTelemetry instrumentationtype: feature request

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants

@mcculls@dougqh