Skip to content

Conversation

@pmuellr
Copy link
Member

@pmuellr pmuellr commented Oct 7, 2020

resolves #79248

Summary

Added some additional checks for potentially null/undefined objects before dereferencing them.

@pmuellr pmuellr requested a review from a team as a code owner October 7, 2020 02:46
@pmuellr pmuellr marked this pull request as draft October 7, 2020 02:47
@pmuellr pmuellr added release_note:skip Skip the PR/issue when compiling release notes v7.10.0 v8.0.0 labels Oct 7, 2020
@pmuellr
Copy link
Member Author

pmuellr commented Oct 7, 2020

jenkins retest this please

@pmuellr
Copy link
Member Author

pmuellr commented Oct 7, 2020

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.
@pmuellr pmuellr force-pushed the alerts/fix-broken-test-79248 branch from b8c6a69 to a1b572f Compare October 7, 2020 03:01
@kibanamachine
Copy link
Contributor

💚 Build Succeeded

Metrics [docs]

✅ unchanged

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

@pmuellr pmuellr added Feature:Alerting Team:ResponseOps Platform ResponseOps team (formerly the Cases and Alerting teams) t// labels Oct 7, 2020
@pmuellr pmuellr marked this pull request as ready for review October 7, 2020 11:46
@elasticmachine
Copy link
Contributor

Pinging @elastic/kibana-alerting-services (Team:Alerting Services)

Copy link
Contributor

@ymao1 ymao1 left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM

Copy link
Contributor

@YulNaumenko YulNaumenko left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM

@pmuellr pmuellr merged commit 3bbd679 into elastic:master Oct 8, 2020
pmuellr added a commit to pmuellr/kibana that referenced this pull request Oct 8, 2020
…lastic#79809)

resolves elastic#79248

Added some additional checks for potential null/undefined objects before
dereferencing them.
pmuellr added a commit to pmuellr/kibana that referenced this pull request Oct 8, 2020
…lastic#79809)

resolves elastic#79248

Added some additional checks for potential null/undefined objects before
dereferencing them.
gmmorris added a commit to gmmorris/kibana that referenced this pull request Oct 8, 2020
* 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)
pmuellr added a commit that referenced this pull request Oct 8, 2020
…79809) (#79991)

resolves #79248

Added some additional checks for potential null/undefined objects before
dereferencing them.
pmuellr added a commit that referenced this pull request Oct 8, 2020
…79809) (#79992)

resolves #79248

Added some additional checks for potential null/undefined objects before
dereferencing them.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

backported Feature:Alerting release_note:skip Skip the PR/issue when compiling release notes Team:ResponseOps Platform ResponseOps team (formerly the Cases and Alerting teams) t// v7.10.0 v8.0.0

Projects

None yet

5 participants