Skip to content
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

[8.x](backport #41911) otelconsumer: remove temporary handling of entity too large #41971

Merged
merged 1 commit into from
Dec 10, 2024

Conversation

mergify[bot]
Copy link
Contributor

@mergify mergify bot commented Dec 10, 2024

Proposed commit message

This code was initially added in #41523 because of a limitation from the elasticsearch exporter.

The elasticsearch exporter has been updated to enforce flush::bytes for the batcher extension and will automatically split the batch if it exceeds the limit.

This error is now fixed in the collector v0.115.0.

See open-telemetry/opentelemetry-collector-contrib#36396.

Checklist

  • My code follows the style guidelines of this project
  • I have commented my code, particularly in hard-to-understand areas
  • I have made corresponding changes to the documentation
  • I have made corresponding change to the default configuration files
  • I have added tests that prove my fix is effective or that my feature works
  • I have added an entry in CHANGELOG.next.asciidoc or CHANGELOG-developer.next.asciidoc.

Author's Checklist

  • [ ]

How to test this PR locally

Related issues


This is an automatic backport of pull request #41911 done by [Mergify](https://mergify.com).

This code was initially added in #41523 because of a limitation from the
elasticsearch exporter.

The elasticsearch exporter has been updated to enforce flush::max_bytes
for the batcher extension and will automatically split the batch if it
exceeds the limit.

This error is now fixed in the collector v0.115.0.

See open-telemetry/opentelemetry-collector-contrib#36396.

(cherry picked from commit dbeb9cd)
@mergify mergify bot requested a review from a team as a code owner December 10, 2024 12:10
@mergify mergify bot added the backport label Dec 10, 2024
@mergify mergify bot requested review from belimawr and leehinman and removed request for a team December 10, 2024 12:10
@botelastic botelastic bot added the needs_team Indicates that the issue/PR needs a Team:* label label Dec 10, 2024
@mauri870 mauri870 added Team:Elastic-Agent-Data-Plane Label for the Agent Data Plane team and removed needs_team Indicates that the issue/PR needs a Team:* label labels Dec 10, 2024
@elasticmachine
Copy link
Collaborator

Pinging @elastic/elastic-agent-data-plane (Team:Elastic-Agent-Data-Plane)

@mauri870 mauri870 enabled auto-merge (squash) December 10, 2024 12:11
@mauri870
Copy link
Member

/test

@mauri870 mauri870 merged commit 795b943 into 8.x Dec 10, 2024
141 checks passed
@mauri870 mauri870 deleted the mergify/bp/8.x/pr-41911 branch December 10, 2024 16:06
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
backport Team:Elastic-Agent-Data-Plane Label for the Agent Data Plane team
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants