-
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
Unskipped and added additional filters to test #187869
Unskipped and added additional filters to test #187869
Conversation
…writing saved objects
/ci |
/ci |
Flaky Test Runner Stats🟠 Some tests failed. - kibana-flaky-test-suite-runner#6501[❌] x-pack/test/security_api_integration/audit.config.ts: 30/50 tests passed. |
/ci |
Flaky Test Runner Stats🎉 All tests passed! - kibana-flaky-test-suite-runner#6503[✅] x-pack/test/security_api_integration/audit.config.ts: 50/50 tests passed. |
Flaky Test Runner Stats🎉 All tests passed! - kibana-flaky-test-suite-runner#6504[✅] x-pack/test/security_api_integration/audit.config.ts: 100/100 tests passed. |
Flaky Test Runner Stats🎉 All tests passed! - kibana-flaky-test-suite-runner#6505[✅] x-pack/test/security_api_integration/audit.config.ts: 100/100 tests passed. |
Pinging @elastic/kibana-security (Team:Security) |
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.
LGTM!
@elasticmachine merge upstream |
💛 Build succeeded, but was flaky
Failed CI StepsTest Failures
Metrics [docs]
History
|
Summary
Unskipped and added additional filter for testing audit events when reading and writing saved objects.
We were filtering event by action only (
http_request
andsaved_object_create
), which probably sometimes returned audit events logged for background tasks, that lack user information.Checklist
Fixes: #119267