Skip to content

[9.1] [ResponseOps] Fix embeddable alerts panel flaky tests (#227999)#241154

Merged
cnasikas merged 4 commits intoelastic:9.1from
cnasikas:backport/9.1/pr-227999
Nov 3, 2025
Merged

[9.1] [ResponseOps] Fix embeddable alerts panel flaky tests (#227999)#241154
cnasikas merged 4 commits intoelastic:9.1from
cnasikas:backport/9.1/pr-227999

Conversation

@cnasikas
Copy link
Copy Markdown
Member

Backport

This will backport the following commits from main to 9.1:

Questions ?

Please refer to the Backport tool documentation

## Summary

Another attempt to fix the flaky tests with the embeddable alerts panel.
The issue is that alerts are not being generated. I made the rules more
sensitive so they will always produce alerts and use the event log to
check the rules' execution status. I run the flaky test runner 3 x 200
without any issues.

Fixes: elastic#227748
Fixes: elastic#220807

### Checklist

Check the PR satisfies following conditions.

Reviewers should verify this PR satisfies this list as well.

- [x] [Unit or functional
tests](https://www.elastic.co/guide/en/kibana/master/development-tests.html)
were updated or added to match the most common scenarios

---------

Co-authored-by: kibanamachine <42973632+kibanamachine@users.noreply.github.com>
(cherry picked from commit 3cb9979)

# Conflicts:
#	x-pack/platform/test/functional_with_es_ssl/apps/embeddable_alerts_table/embeddable_alerts_table.ts
#	x-pack/platform/test/functional_with_es_ssl/plugins/alerts/tsconfig.json
@cnasikas cnasikas added the backport This PR is a backport of another PR label Oct 29, 2025
@cnasikas cnasikas enabled auto-merge (squash) October 29, 2025 14:57
@cnasikas
Copy link
Copy Markdown
Member Author

cnasikas commented Nov 3, 2025

@elasticmachine merge upstream

@cnasikas cnasikas disabled auto-merge November 3, 2025 10:33
@elasticmachine
Copy link
Copy Markdown
Contributor

💚 Build Succeeded

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
embeddableAlertsTable 978.5KB 978.5KB +11.0B

History

@kibanamachine
Copy link
Copy Markdown
Contributor

Flaky Test Runner Stats

🎉 All tests passed! - kibana-flaky-test-suite-runner#9707

[✅] x-pack/platform/test/functional_with_es_ssl/apps/embeddable_alerts_table/config.ts: 200/200 tests passed.

see run history

@cnasikas cnasikas merged commit b8851ca into elastic:9.1 Nov 3, 2025
12 checks passed
@cnasikas cnasikas deleted the backport/9.1/pr-227999 branch November 3, 2025 12:27
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

backport This PR is a backport of another PR

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants