-
Notifications
You must be signed in to change notification settings - Fork 42
fix: "success" was removed from response #278
fix: "success" was removed from response #278
Conversation
💔 Tests FailedExpand to view the summary
Build stats
Test stats 🧪
Test errorsExpand to view the tests failures
Steps errorsExpand to view the steps failures
Log outputExpand to view the last 100 lines of log output
|
Failed test are not related to this change, and we are aware of the stand-alone ones here: elastic/beats#20992 Merging |
We do not need backport to 7.9.x because of elastic/kibana#73223 (comment) |
What does this PR do?
It removes the only one check for the
success
response field, which was removed here: elastic/kibana#73223Why is it important?
Code will fail with panic because the used conversion was not captured
Checklist
make notice
in the proper directory)How to test this PR locally
The scenario using that field is tagged with the
@endpoint-policy-check
annotationRelated issues