- 
                Notifications
    
You must be signed in to change notification settings  - Fork 5k
 
[OTel] Add logstash exporter logic #46705
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] Add logstash exporter logic #46705
Conversation
          🤖 GitHub commentsExpand to view the GitHub comments
 Just comment with: 
  | 
    
| 
           This pull request does not have a backport label. 
 To fixup this pull request, you need to add the backport labels for the needed 
  | 
    
| 
           Pinging @elastic/elastic-agent-data-plane (Team:Elastic-Agent-Data-Plane)  | 
    
| 
           @elastic/beats-tech-leads could you please take a look at it so we can merge it?  | 
    
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Approving go.mod changes
Proposed commit message
This PR implements the logstash exporter logic, which includes:
A few considerations:
ConsumeLogsresponse is sync, and it returns to the actual batch processing results, so theotelconsumeroutcome is correct.outputs.Clientcreated by theotelconsumer, which given the syncConsumeLogsnature, should be enough. A few adjusts might be necessary after benchmarking it, but it's not an issue at the moment.consumererror.NewPermanent, which leads theotelconsumerto also drop the batch on the beats side. Besides that, all other flow errors are being wrapped intoconsumererror.NewLogs.Checklist
CHANGELOG.next.asciidocorCHANGELOG-developer.next.asciidoc.Disruptive User Impact
How to test this PR locally
Run am agent
filebeatin OTel mode, adding on the pipeline alogstashoutput configuration.Related issues
Use cases
Screenshots
Logs