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

[logstash] add missing queue.capacity mappings #9339

Merged
merged 7 commits into from Mar 14, 2024

Conversation

klacabane
Copy link
Contributor

Summary

Part of elastic/beats#38164

@klacabane
Copy link
Contributor Author

/test

3 similar comments
@klacabane
Copy link
Contributor Author

/test

@klacabane
Copy link
Contributor Author

/test

@klacabane
Copy link
Contributor Author

/test

packages/logstash/changelog.yml Outdated Show resolved Hide resolved
@klacabane
Copy link
Contributor Author

Agent is apparently unable to reach logstash service during system tests for the node and node_stats data streams. I can't reproduce locally. I know elastic-package introduced improvements on failure detection during tests recently and wondering if this could be a false positive triggered by that new mechanism (ie if there are any erroneous documents collected during tests we fail)

one or more errors found in documents stored in metrics-logstash.stack_monitoring.node-ep data stream: [0] found error.message in event: error making http request: Get "http://elastic-package-service-logstash-1:9600/": dial tcp 172.18.0.5:9600: connect: connection refused

@klacabane
Copy link
Contributor Author

/test

@klacabane
Copy link
Contributor Author

I think the previous errors about service being unavailable were fixed by 9e23173 but now getting errors about log data stream. Unfortunately I can't reproduce locally

one or more errors found in documents stored in logs-logstash.log-ep data stream: [0] found error.message in event: Duplicate field 'message'\n at [Source: (org.elasticsearch.common.io.stream.ByteBufferStreamInput); line: 1, column: 206]
[1] found error.message in event: Duplicate field 'message'\n at [Source: (org.elasticsearch.common.io.stream.ByteBufferStreamInput); line: 1, column: 209]
[2] found error.message in event: Duplicate field 'message'\n at [Source: (org.elasticsearch.common.io.stream.ByteBufferStreamInput); line: 1, column: 331]

@elasticmachine
Copy link

🚀 Benchmarks report

To see the full report comment with /test benchmark fullreport

@elasticmachine
Copy link

💚 Build Succeeded

History

Copy link

Quality Gate passed Quality Gate passed

Kudos, no new issues were introduced!

0 New issues
0 Security Hotspots
No Coverage information No data about Coverage
No Duplication information No data about Duplication

See analysis details on SonarQube

@klacabane
Copy link
Contributor Author

Lot of flakiness involved in the package system tests. Merging the green build but I'll log an issue to investigate the errors surfaced in the previous builds

@klacabane klacabane merged commit 78e918d into elastic:main Mar 14, 2024
5 checks passed
@elasticmachine
Copy link

Package logstash - 2.4.3 containing this change is available at https://epr.elastic.co/search?package=logstash

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

3 participants