Skip to content
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

[7.x] [Security Solution][RAC] - Add reason field (#107532) #108298

Closed

Conversation

michaelolo24
Copy link
Contributor

Backports the following commits to 7.x:

# Conflicts:
#	x-pack/plugins/security_solution/cypress/integration/detection_alerts/alerts_details.spec.ts
@michaelolo24 michaelolo24 enabled auto-merge (squash) August 11, 2021 22:33
@kibanamachine
Copy link
Contributor

kibanamachine commented Aug 12, 2021

💔 Build Failed

Failed CI Steps


Test Failures

Kibana Pipeline / general / Displays the unmapped field on the table.Alert details with unmapped fields Displays the unmapped field on the table

Link to Jenkins

Stack Trace

Failed Tests Reporter:
  - Test has not failed recently on tracked branches

AssertionError: Timed out retrying after 60000ms: expected '<div.euiText.euiText--extraSmall>' to have text 'unmapped', but the text was 'signal.status'
    at Context.eval (http://localhost:6121/__cypress/tests?p=cypress/integration/detection_alerts/alerts_details.spec.ts:20812:54)

Kibana Pipeline / general / Displays the unmapped field on the table.Alert details with unmapped fields Displays the unmapped field on the table

Link to Jenkins

Stack Trace

Failed Tests Reporter:
  - Test has failed 1 times on tracked branches: https://dryrun

AssertionError: Timed out retrying after 60000ms: expected '<div.euiText.euiText--extraSmall>' to have text 'unmapped', but the text was 'signal.status'
    at Context.eval (http://localhost:6121/__cypress/tests?p=cypress/integration/detection_alerts/alerts_details.spec.ts:20812:54)

Metrics [docs]

Async chunks

Total size of all lazy-loaded chunks that will be downloaded as the user navigates the app

id before after diff
securitySolution 6.5MB 6.5MB -574.0B
timelines 267.6KB 267.6KB +16.0B
total -558.0B

To update your PR or re-run it, just comment with:
@elasticmachine merge upstream

auto-merge was automatically disabled August 12, 2021 03:28

Pull request was closed

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants