-
Notifications
You must be signed in to change notification settings - Fork 8.3k
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
[Security Solution] Incorrect count is displayed when export the prebuilt + custom rules #138354
Comments
Pinging @elastic/security-solution (Team: SecuritySolution) |
Pinging @elastic/security-detections-response (Team:Detections and Resp) |
@vitaliidm In the environment access to which @deepikakeshav-qasource kindly shared with me, I was able to reproduce the bug. This is what I noticed. The bulk action request body was: {"action":"export","ids":["adf8c7a0-13ee-11ed-9617-b1cad46473e2","066e8050-13ef-11ed-9617-b1cad46473e2","20b90840-1480-11ed-9617-b1cad46473e2","513bbd80-148c-11ed-9617-b1cad46473e2","e2182400-170d-11ed-bf82-2f6ce8fb01fa","ae8e77f0-13ee-11ed-9617-b1cad46473e2","ae8e50e0-13ee-11ed-9617-b1cad46473e2","ae8ddbb0-13ee-11ed-9617-b1cad46473e2","ae8db4a0-13ee-11ed-9617-b1cad46473e2","ae8d6680-13ee-11ed-9617-b1cad46473e2","ae8d3f70-13ee-11ed-9617-b1cad46473e2","ae8cca40-13ee-11ed-9617-b1cad46473e2","ae8c06f0-13ee-11ed-9617-b1cad46473e2","ae8c5510-13ee-11ed-9617-b1cad46473e2","ae8b1c90-13ee-11ed-9617-b1cad46473e2","ae8bb8d0-13ee-11ed-9617-b1cad46473e2","ae8a8050-13ee-11ed-9617-b1cad46473e2","ae8b6ab0-13ee-11ed-9617-b1cad46473e2","ae8c7c20-13ee-11ed-9617-b1cad46473e2","ae8af580-13ee-11ed-9617-b1cad46473e2"]} The response body was:
See The endpoint exported 12 objects, only 4 of which were rules, there were exception lists there as well. We should fix and cover it with e2e tests. If possible, let's try to squeeze the fix into |
Pending to be validated on BC5. @deepikakeshav-qasource can you please validate this fix in latest 8.4 branch while BC5 is not available? Thanks!! |
…rules with exceptions (elastic#138598) ## Summary - addresses elastic#138354 - adds e2e test ### Checklist Delete any items that are not applicable to this PR. - [x] [Unit or functional tests](https://www.elastic.co/guide/en/kibana/master/development-tests.html) were updated or added to match the most common scenarios (cherry picked from commit c0363b0)
…rules with exceptions (elastic#138598) ## Summary - addresses elastic#138354 - adds e2e test ### Checklist Delete any items that are not applicable to this PR. - [x] [Unit or functional tests](https://www.elastic.co/guide/en/kibana/master/development-tests.html) were updated or added to match the most common scenarios
fixed in #138598 |
Hi Team, We have validated this issue on 8.4.0 Branch and observed that issue is Fixed. Please find the below Testing Details: Build info
Screen-cast export.mp4We will going to retest this issue on BC5 so keeping the ticket open till then. cc: @MadameSheema Thanks!! |
Hi @vitaliidm, We have Validated this issue on 8.4.0 BC5 Build and Observed that issue is Fixed. 🟢 Please find the below Testing Details: Build info
Screen-cast export.rules.mp4Hence, We are closing this issue and marking as QA Validated!! Thanks!! |
…rules with exceptions (elastic#138598) ## Summary - addresses elastic#138354 - adds e2e test ### Checklist Delete any items that are not applicable to this PR. - [x] [Unit or functional tests](https://www.elastic.co/guide/en/kibana/master/development-tests.html) were updated or added to match the most common scenarios
Describe the bug
Incorrect count is displayed when export the prebuilt + custom rules
Build info
Preconditions
Steps to Reproduce
Actual Result
Incorrect count is displayed when export the prebuilt + custom rules
Expected Result
Correct count should be displayed when export the prebuilt + custom rules
Screen-cast
prebuild.rule.mp4
The text was updated successfully, but these errors were encountered: