-
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
[Alerts] fix failing executionStatus function test with null deref #79809
Conversation
jenkins retest this please |
flaky test runner passed 42 times - https://kibana-ci.elastic.co/job/kibana+flaky-test-suite-runner/893/ |
resolves elastic#79248 Added some additional checks for potential null/undefined objects before dereferencing them.
b8c6a69
to
a1b572f
Compare
💚 Build SucceededMetrics [docs]
To update your PR or re-run it, just comment with: |
Pinging @elastic/kibana-alerting-services (Team:Alerting Services) |
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
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
…lastic#79809) resolves elastic#79248 Added some additional checks for potential null/undefined objects before dereferencing them.
…lastic#79809) resolves elastic#79248 Added some additional checks for potential null/undefined objects before dereferencing them.
* master: [ML] Fix job selection flyout (elastic#79850) Removed visualization:loadingDelay (elastic#79841) [Alerts] fix failing executionStatus function test with null deref (elastic#79809) Implemented trusted app name truncation. (elastic#79976)
resolves #79248
Summary
Added some additional checks for potentially null/undefined objects before dereferencing them.