-
Notifications
You must be signed in to change notification settings - Fork 392
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
checkpoint: enhance processor error reporting #5295
Conversation
e754d1b
to
552d443
Compare
🌐 Coverage report
|
Pinging @elastic/security-external-integrations (Team:Security-External Integrations) |
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.
Minor change, rest LGTM 👍🏼
@@ -1025,5 +1039,8 @@ processors: | |||
ignore_missing: true | |||
on_failure: | |||
- set: | |||
field: event.kind | |||
value: pipeline_failure |
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.
The value is pipeline_error
as per the doc
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.
Thanks
552d443
to
5eb6215
Compare
Package checkpoint - 1.15.0 containing this change is available at https://epr.elastic.co/search?package=checkpoint |
What does this PR do?
Errors are made more informative and provide tags to allow programmatic repair.
Checklist
changelog.yml
file.Author's Checklist
How to test this PR locally
Related issues
Screenshots