Skip to content

Add drop_error_types config option to not retry after certain error types#1228

Merged
estolfo merged 13 commits intologstash-plugins:mainfrom
estolfo:drop-error-types
Oct 6, 2025
Merged

Add drop_error_types config option to not retry after certain error types#1228
estolfo merged 13 commits intologstash-plugins:mainfrom
estolfo:drop-error-types

Conversation

@estolfo
Copy link
Contributor

@estolfo estolfo commented Oct 1, 2025

This PR adds a config option drop_error_types that allows a user to set a list of error types for which a failed bulk request should not be retried and the events should be dropped. The events are not added to the DLQ. A warn message is logged, if the error type is not listed in the config option silence_errors_in_log.

Closes #1225

@jsvd jsvd self-requested a review October 1, 2025 11:45
@estolfo estolfo requested a review from jsvd October 2, 2025 09:24
Copy link
Member

@jsvd jsvd left a comment

Choose a reason for hiding this comment

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

please do a minor version bump to the gemspec and add an entry to the changelog

estolfo and others added 2 commits October 6, 2025 10:15
Co-authored-by: João Duarte <jsvd@users.noreply.github.com>
Co-authored-by: João Duarte <jsvd@users.noreply.github.com>
Copy link
Member

@jsvd jsvd left a comment

Choose a reason for hiding this comment

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

LGTM 🎉 ship it!

@estolfo estolfo merged commit 95a0bcd into logstash-plugins:main Oct 6, 2025
3 checks passed
@lcawl lcawl mentioned this pull request Feb 12, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

2 participants