Skip to content

[8.19] [APM] Try to fix flaky test (#227628)#228017

Merged
kibanamachine merged 4 commits intoelastic:8.19from
kibanamachine:backport/8.19/pr-227628
Jul 16, 2025
Merged

[8.19] [APM] Try to fix flaky test (#227628)#228017
kibanamachine merged 4 commits intoelastic:8.19from
kibanamachine:backport/8.19/pr-227628

Conversation

@kibanamachine
Copy link
Contributor

Backport

This will backport the following commits from main to 8.19:

Questions ?

Please refer to the Backport tool documentation

Closes elastic#214673

## Summary

This PR tries to fix a flaky test by intercepting a request and waiting
for it. I couldn't reproduce the flaky behaviour, but I saw that we are
not waiting for the request to finish before checking the value, so my
assumption is that the request is slower in the CI compared to the local
execution, and that's why the test is failing.

Co-authored-by: Sergi Romeu <sergi.romeu@elastic.co>
(cherry picked from commit 1299379)
@kibanamachine kibanamachine added the backport This PR is a backport of another PR label Jul 15, 2025
@kibanamachine kibanamachine enabled auto-merge (squash) July 15, 2025 13:27
@botelastic botelastic bot added the Team:obs-ux-infra_services - DEPRECATED DEPRECATED - Use Team:obs-presentation. label Jul 15, 2025
@elasticmachine
Copy link
Contributor

Pinging @elastic/obs-ux-infra_services-team (Team:obs-ux-infra_services)

@kibanamachine kibanamachine merged commit 9928c8e into elastic:8.19 Jul 16, 2025
8 checks passed
@elasticmachine
Copy link
Contributor

💛 Build succeeded, but was flaky

Failed CI Steps

Test Failures

  • [job] [logs] FTR Configs #29 / saved objects tagging - functional tests feature controls tag management write privileges can bulk delete tags

Metrics [docs]

✅ unchanged

History

cc @jennypavlova

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 Team:obs-ux-infra_services - DEPRECATED DEPRECATED - Use Team:obs-presentation.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants