-
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] Updates mappings and re-enables tests #69872
Conversation
Pinging @elastic/siem (Team:SIEM) |
@elasticmachine merge upstream |
Signed-off-by: Tyler Smalley <tyler.smalley@elastic.co>
Signed-off-by: Tyler Smalley <tyler.smalley@elastic.co>
1bb0e92
to
63ec50c
Compare
Since #69871 was merged - I am re-enabling the tests here to see if they can be restored. |
@stephmilovic, looks like we no longer have the field count error - but appears there are now some actual failing tests. Could you or someone from your team take over from here? |
💔 Build Failed
Failed CI StepsTest FailuresKibana Pipeline / kibana-xpack-agent / Host stats render with correct values.Overview Page Host stats render with correct valuesStandard Out
Stack Trace
Kibana Pipeline / kibana-xpack-agent / Host stats render with correct values.Overview Page Host stats render with correct valuesStandard Out
Stack Trace
Kibana Pipeline / kibana-xpack-agent / Network stats render with correct values.Overview Page Network stats render with correct valuesStandard Out
Stack Trace
and 4 more failures, only showing the first 3. Build metrics
History
To update your PR or re-run it, just comment with: |
@elastic/siem & @stephmilovic, want to raise this again to see if someone on your team can take this over. There are failures not related to the field count which would need addressed from your team. |
Pinging @elastic/security-solution (Team: SecuritySolution) |
The provided mappings were resulting in > 1k mapping fields after a migration. This updates them so there aren't any dangling fields so the tests can be re-enabled.
We probably want to wait until #69871 is merged and update the mappings here again.
Closes #69848
Blocked on #69871