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

[OTel mode] Replace memorylimitprocessor with memorylimitextension #4665

Closed
4 tasks
ycombinator opened this issue May 3, 2024 · 1 comment · Fixed by #4689
Closed
4 tasks

[OTel mode] Replace memorylimitprocessor with memorylimitextension #4665

ycombinator opened this issue May 3, 2024 · 1 comment · Fixed by #4689
Assignees
Labels
enhancement New feature or request Team:Elastic-Agent-Control-Plane Label for the Agent Control Plane team

Comments

@ycombinator
Copy link
Contributor

ycombinator commented May 3, 2024

Describe the enhancement:

As recommended by @michalpristas in #4623 (comment):

maybe not in this PR but we will need to replace memorylimiterprocessor with memorylimiterextension. difference is behavior is that processors drops the events and extensions acts as a pushback mechanism stopping/pausing processing on receiver side

What is the definition of done?

  • ./elastic-agent otel validate -c otel.yml fails if memorylimiterprocessor is used in the otel.yml collector configuration.
  • ./elastic-agent otel validate -c otel.yml succeeds if memorylimiterextension is used in the otel.yml collector configuration.
  • documentation updated
  • example configuration
@ycombinator ycombinator added enhancement New feature or request Team:Elastic-Agent-Control-Plane Label for the Agent Control Plane team labels May 3, 2024
@elasticmachine
Copy link
Contributor

Pinging @elastic/elastic-agent-control-plane (Team:Elastic-Agent-Control-Plane)

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement New feature or request Team:Elastic-Agent-Control-Plane Label for the Agent Control Plane team
Projects
None yet
Development

Successfully merging a pull request may close this issue.

3 participants