Skip to content

[11.x] Improves a visibility the case events routed to DLQ. (#1253)#1258

Merged
mashhurs merged 2 commits into
logstash-plugins:11.xfrom
mashhurs:dlq-improvements-11.x
Apr 17, 2026
Merged

[11.x] Improves a visibility the case events routed to DLQ. (#1253)#1258
mashhurs merged 2 commits into
logstash-plugins:11.xfrom
mashhurs:dlq-improvements-11.x

Conversation

@mashhurs
Copy link
Copy Markdown
Contributor

@mashhurs mashhurs commented Apr 16, 2026

Backport of #1253 PR

  • Improves a visibility about how many failed events went to DLQ. If error happens during the DLQ persist, shows the status, action and response for better visibility.

  • Capture a sample event for each status to show what is happening with ES.

(cherry picked from commit c09c4ab)

@mashhurs mashhurs requested a review from andsel April 16, 2026 19:11
Comment thread CHANGELOG.md Outdated
@mashhurs
Copy link
Copy Markdown
Contributor Author

#1260 PR addresses the CI failures

Copy link
Copy Markdown
Contributor

@andsel andsel left a comment

Choose a reason for hiding this comment

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

LGTM once you rebase on top of #1260 and CI is 🟢

…s#1253)

* Improves a visibility about how many failed events went to DLQ. If error happens during the DLQ persist, shows the status, action and response for better visibility.

* Capture a sample event for each status to show what is happening with ES.

(cherry picked from commit c09c4ab)
@mashhurs mashhurs force-pushed the dlq-improvements-11.x branch from d922667 to ff070f4 Compare April 17, 2026 16:35
@mashhurs
Copy link
Copy Markdown
Contributor Author

Rebased after merging #1260 into upstream

@mashhurs mashhurs merged commit 60342c0 into logstash-plugins:11.x Apr 17, 2026
3 checks passed
@mashhurs mashhurs deleted the dlq-improvements-11.x branch April 17, 2026 17:36
@mashhurs mashhurs changed the title Improves a visibility the case events routed to DLQ. (#1253) [11.x] Improves a visibility the case events routed to DLQ. (#1253) Apr 17, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants