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

Fixes Failing test: Chrome X-Pack UI Functional Tests.x-pack/test/functional_with_es_ssl/apps/triggers_actions_ui/alerts_list·ts - Actions and Triggers app alerts list should display total alerts by status and error banner only when exists alerts with status error #88119

Merged
merged 3 commits into from
Jan 13, 2021

Conversation

ymao1
Copy link
Contributor

@ymao1 ymao1 commented Jan 13, 2021

Resolves #87105

Summary

Based on the screenshot from the flaky test (seen in the issue), it seems like the alert list wasn't refreshed before counting the number of alerts with each status. Since the error banner can be seen, the Test: Failing alert is (correctly) in an error state but since the alert list wasn't refreshed, it still showed pending. Added a refresh to get an up to date status. Ran it through the flaky test runner a few times and it seems to resolve this flaky test.

@kibanamachine
Copy link
Contributor

💚 Build Succeeded

Metrics [docs]

✅ unchanged

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

@ymao1 ymao1 added Feature:Alerting release_note:skip Skip the PR/issue when compiling release notes Team:ResponseOps Label for the ResponseOps team (formerly the Cases and Alerting teams) labels Jan 13, 2021
@ymao1 ymao1 marked this pull request as ready for review January 13, 2021 03:43
@ymao1 ymao1 requested a review from a team as a code owner January 13, 2021 03:43
@elasticmachine
Copy link
Contributor

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

@ymao1 ymao1 changed the title Flaky test pending alert Fixes Failing test: Chrome X-Pack UI Functional Tests.x-pack/test/functional_with_es_ssl/apps/triggers_actions_ui/alerts_list·ts - Actions and Triggers app alerts list should display total alerts by status and error banner only when exists alerts with status error Jan 13, 2021
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

Copy link
Contributor

@gmmorris gmmorris left a comment

Choose a reason for hiding this comment

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

Worth a shot :)
Good catch if this is in fact the root cause of the flakyness.

@ymao1 ymao1 merged commit 2c8b518 into elastic:master Jan 13, 2021
Copy link
Member

@pmuellr pmuellr left a comment

Choose a reason for hiding this comment

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

LGTM

ymao1 added a commit to ymao1/kibana that referenced this pull request Jan 13, 2021
…ctional_with_es_ssl/apps/triggers_actions_ui/alerts_list·ts - Actions and Triggers app alerts list should display total alerts by status and error banner only when exists alerts with status error (elastic#88119)

* Unskipping test

* Adding refresh before checking status
ymao1 added a commit to ymao1/kibana that referenced this pull request Jan 13, 2021
…ctional_with_es_ssl/apps/triggers_actions_ui/alerts_list·ts - Actions and Triggers app alerts list should display total alerts by status and error banner only when exists alerts with status error (elastic#88119)

* Unskipping test

* Adding refresh before checking status
ymao1 added a commit that referenced this pull request Jan 13, 2021
…ctional_with_es_ssl/apps/triggers_actions_ui/alerts_list·ts - Actions and Triggers app alerts list should display total alerts by status and error banner only when exists alerts with status error (#88119) (#88167)

* Unskipping test

* Adding refresh before checking status
ymao1 added a commit that referenced this pull request Jan 13, 2021
…ctional_with_es_ssl/apps/triggers_actions_ui/alerts_list·ts - Actions and Triggers app alerts list should display total alerts by status and error banner only when exists alerts with status error (#88119) (#88166)

* Unskipping test

* Adding refresh before checking status

Co-authored-by: Kibana Machine <[email protected]>
@ymao1 ymao1 deleted the flaky-test-pending-alert branch February 4, 2021 15:23
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Feature:Alerting release_note:skip Skip the PR/issue when compiling release notes Team:ResponseOps Label for the ResponseOps team (formerly the Cases and Alerting teams) v7.11.0 v7.12.0 v8.0.0
Projects
None yet
6 participants