Skip to content

[8.19] [ResponseOps] Fix embeddable alerts panel flaky tests (#227999)#241155

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

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

Conversation

@cnasikas
Copy link
Copy Markdown
Member

Backport

This will backport the following commits from main to 8.19:

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:59
@cnasikas
Copy link
Copy Markdown
Member Author

cnasikas commented Nov 3, 2025

@elasticmachine merge upstream

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

Flaky Test Runner Stats

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

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

see run history

@cnasikas cnasikas enabled auto-merge (squash) November 3, 2025 12:42
@cnasikas cnasikas merged commit 5d93b81 into elastic:8.19 Nov 3, 2025
8 checks passed
@cnasikas cnasikas deleted the backport/8.19/pr-227999 branch November 3, 2025 12:50
@elasticmachine
Copy link
Copy Markdown
Contributor

💛 Build succeeded, but was flaky

Failed CI Steps

Test Failures

  • [job] [logs] FTR Configs #78 / core plugins rendering service renders "core" application without user settings
  • [job] [logs] Jest Tests #3 / getFileType getFileAttachmentViewObject getActions renders a delete button

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

History

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