Skip to content

Comments

[APM] Try to fix flaky test#227628

Merged
jennypavlova merged 3 commits intoelastic:mainfrom
jennypavlova:214673-try-to-fix
Jul 15, 2025
Merged

[APM] Try to fix flaky test#227628
jennypavlova merged 3 commits intoelastic:mainfrom
jennypavlova:214673-try-to-fix

Conversation

@jennypavlova
Copy link
Member

@jennypavlova jennypavlova commented Jul 11, 2025

Closes #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.

@jennypavlova jennypavlova self-assigned this Jul 11, 2025
@jennypavlova jennypavlova requested a review from a team July 11, 2025 12:02
@jennypavlova jennypavlova added release_note:skip Skip the PR/issue when compiling release notes backport:all-open Backport to all branches that could still receive a release labels Jul 11, 2025
@botelastic botelastic bot added the Team:obs-ux-infra_services - DEPRECATED DEPRECATED - Use Team:obs-presentation. label Jul 11, 2025
@elasticmachine
Copy link
Contributor

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

@kibanamachine
Copy link
Contributor

Flaky Test Runner Stats

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

[✅] APM - Cypress: 25/25 tests passed.

see run history

@elasticmachine
Copy link
Contributor

💚 Build Succeeded

Metrics [docs]

✅ unchanged

History

cc @jennypavlova

@jennypavlova jennypavlova merged commit 1299379 into elastic:main Jul 15, 2025
12 of 13 checks passed
@kibanamachine
Copy link
Contributor

Starting backport for target branches: 8.17, 8.18, 8.19, 9.0, 9.1

https://github.com/elastic/kibana/actions/runs/16294430550

kibanamachine pushed a commit to kibanamachine/kibana that referenced this pull request Jul 15, 2025
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 pushed a commit to kibanamachine/kibana that referenced this pull request Jul 15, 2025
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 pushed a commit to kibanamachine/kibana that referenced this pull request Jul 15, 2025
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 pushed a commit to kibanamachine/kibana that referenced this pull request Jul 15, 2025
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 pushed a commit to kibanamachine/kibana that referenced this pull request Jul 15, 2025
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
Copy link
Contributor

💚 All backports created successfully

Status Branch Result
8.17
8.18
8.19
9.0
9.1

Note: Successful backport PRs will be merged automatically after passing CI.

Questions ?

Please refer to the Backport tool documentation

kibanamachine added a commit that referenced this pull request Jul 15, 2025
# Backport

This will backport the following commits from `main` to `9.0`:
- [[APM] Try to fix flaky test
(#227628)](#227628)

<!--- Backport version: 9.6.6 -->

### Questions ?
Please refer to the [Backport tool
documentation](https://github.com/sorenlouv/backport)

<!--BACKPORT
[{"author":{"name":"jennypavlova","email":"dzheni.pavlova@elastic.co"},"sourceCommit":{"committedDate":"2025-07-15T13:20:19Z","message":"[APM]
Try to fix flaky test (#227628)\n\nCloses #214673 \n\n## Summary\n\nThis
PR tries to fix a flaky test by intercepting a request and waiting\nfor
it. I couldn't reproduce the flaky behaviour, but I saw that we are\nnot
waiting for the request to finish before checking the value, so
my\nassumption is that the request is slower in the CI compared to the
local\nexecution, and that's why the test is failing.\n\nCo-authored-by:
Sergi Romeu
<sergi.romeu@elastic.co>","sha":"1299379a43f7de109cfccc44525da3c2f6a5663e","branchLabelMapping":{"^v9.2.0$":"main","^v(\\d+).(\\d+).\\d+$":"$1.$2"}},"sourcePullRequest":{"labels":["release_note:skip","backport:all-open","Team:obs-ux-infra_services","v9.2.0"],"title":"[APM]
Try to fix flaky
test","number":227628,"url":"https://github.com/elastic/kibana/pull/227628","mergeCommit":{"message":"[APM]
Try to fix flaky test (#227628)\n\nCloses #214673 \n\n## Summary\n\nThis
PR tries to fix a flaky test by intercepting a request and waiting\nfor
it. I couldn't reproduce the flaky behaviour, but I saw that we are\nnot
waiting for the request to finish before checking the value, so
my\nassumption is that the request is slower in the CI compared to the
local\nexecution, and that's why the test is failing.\n\nCo-authored-by:
Sergi Romeu
<sergi.romeu@elastic.co>","sha":"1299379a43f7de109cfccc44525da3c2f6a5663e"}},"sourceBranch":"main","suggestedTargetBranches":[],"targetPullRequestStates":[{"branch":"main","label":"v9.2.0","branchLabelMappingKey":"^v9.2.0$","isSourceBranch":true,"state":"MERGED","url":"https://github.com/elastic/kibana/pull/227628","number":227628,"mergeCommit":{"message":"[APM]
Try to fix flaky test (#227628)\n\nCloses #214673 \n\n## Summary\n\nThis
PR tries to fix a flaky test by intercepting a request and waiting\nfor
it. I couldn't reproduce the flaky behaviour, but I saw that we are\nnot
waiting for the request to finish before checking the value, so
my\nassumption is that the request is slower in the CI compared to the
local\nexecution, and that's why the test is failing.\n\nCo-authored-by:
Sergi Romeu
<sergi.romeu@elastic.co>","sha":"1299379a43f7de109cfccc44525da3c2f6a5663e"}}]}]
BACKPORT-->

Co-authored-by: jennypavlova <dzheni.pavlova@elastic.co>
Co-authored-by: Sergi Romeu <sergi.romeu@elastic.co>
kibanamachine added a commit that referenced this pull request Jul 15, 2025
# Backport

This will backport the following commits from `main` to `9.1`:
- [[APM] Try to fix flaky test
(#227628)](#227628)

<!--- Backport version: 9.6.6 -->

### Questions ?
Please refer to the [Backport tool
documentation](https://github.com/sorenlouv/backport)

<!--BACKPORT
[{"author":{"name":"jennypavlova","email":"dzheni.pavlova@elastic.co"},"sourceCommit":{"committedDate":"2025-07-15T13:20:19Z","message":"[APM]
Try to fix flaky test (#227628)\n\nCloses #214673 \n\n## Summary\n\nThis
PR tries to fix a flaky test by intercepting a request and waiting\nfor
it. I couldn't reproduce the flaky behaviour, but I saw that we are\nnot
waiting for the request to finish before checking the value, so
my\nassumption is that the request is slower in the CI compared to the
local\nexecution, and that's why the test is failing.\n\nCo-authored-by:
Sergi Romeu
<sergi.romeu@elastic.co>","sha":"1299379a43f7de109cfccc44525da3c2f6a5663e","branchLabelMapping":{"^v9.2.0$":"main","^v(\\d+).(\\d+).\\d+$":"$1.$2"}},"sourcePullRequest":{"labels":["release_note:skip","backport:all-open","Team:obs-ux-infra_services","v9.2.0"],"title":"[APM]
Try to fix flaky
test","number":227628,"url":"https://github.com/elastic/kibana/pull/227628","mergeCommit":{"message":"[APM]
Try to fix flaky test (#227628)\n\nCloses #214673 \n\n## Summary\n\nThis
PR tries to fix a flaky test by intercepting a request and waiting\nfor
it. I couldn't reproduce the flaky behaviour, but I saw that we are\nnot
waiting for the request to finish before checking the value, so
my\nassumption is that the request is slower in the CI compared to the
local\nexecution, and that's why the test is failing.\n\nCo-authored-by:
Sergi Romeu
<sergi.romeu@elastic.co>","sha":"1299379a43f7de109cfccc44525da3c2f6a5663e"}},"sourceBranch":"main","suggestedTargetBranches":[],"targetPullRequestStates":[{"branch":"main","label":"v9.2.0","branchLabelMappingKey":"^v9.2.0$","isSourceBranch":true,"state":"MERGED","url":"https://github.com/elastic/kibana/pull/227628","number":227628,"mergeCommit":{"message":"[APM]
Try to fix flaky test (#227628)\n\nCloses #214673 \n\n## Summary\n\nThis
PR tries to fix a flaky test by intercepting a request and waiting\nfor
it. I couldn't reproduce the flaky behaviour, but I saw that we are\nnot
waiting for the request to finish before checking the value, so
my\nassumption is that the request is slower in the CI compared to the
local\nexecution, and that's why the test is failing.\n\nCo-authored-by:
Sergi Romeu
<sergi.romeu@elastic.co>","sha":"1299379a43f7de109cfccc44525da3c2f6a5663e"}}]}]
BACKPORT-->

Co-authored-by: jennypavlova <dzheni.pavlova@elastic.co>
Co-authored-by: Sergi Romeu <sergi.romeu@elastic.co>
kibanamachine added a commit that referenced this pull request Jul 15, 2025
# Backport

This will backport the following commits from `main` to `8.17`:
- [[APM] Try to fix flaky test
(#227628)](#227628)

<!--- Backport version: 9.6.6 -->

### Questions ?
Please refer to the [Backport tool
documentation](https://github.com/sorenlouv/backport)

<!--BACKPORT
[{"author":{"name":"jennypavlova","email":"dzheni.pavlova@elastic.co"},"sourceCommit":{"committedDate":"2025-07-15T13:20:19Z","message":"[APM]
Try to fix flaky test (#227628)\n\nCloses #214673 \n\n## Summary\n\nThis
PR tries to fix a flaky test by intercepting a request and waiting\nfor
it. I couldn't reproduce the flaky behaviour, but I saw that we are\nnot
waiting for the request to finish before checking the value, so
my\nassumption is that the request is slower in the CI compared to the
local\nexecution, and that's why the test is failing.\n\nCo-authored-by:
Sergi Romeu
<sergi.romeu@elastic.co>","sha":"1299379a43f7de109cfccc44525da3c2f6a5663e","branchLabelMapping":{"^v9.2.0$":"main","^v(\\d+).(\\d+).\\d+$":"$1.$2"}},"sourcePullRequest":{"labels":["release_note:skip","backport:all-open","Team:obs-ux-infra_services","v9.2.0"],"title":"[APM]
Try to fix flaky
test","number":227628,"url":"https://github.com/elastic/kibana/pull/227628","mergeCommit":{"message":"[APM]
Try to fix flaky test (#227628)\n\nCloses #214673 \n\n## Summary\n\nThis
PR tries to fix a flaky test by intercepting a request and waiting\nfor
it. I couldn't reproduce the flaky behaviour, but I saw that we are\nnot
waiting for the request to finish before checking the value, so
my\nassumption is that the request is slower in the CI compared to the
local\nexecution, and that's why the test is failing.\n\nCo-authored-by:
Sergi Romeu
<sergi.romeu@elastic.co>","sha":"1299379a43f7de109cfccc44525da3c2f6a5663e"}},"sourceBranch":"main","suggestedTargetBranches":[],"targetPullRequestStates":[{"branch":"main","label":"v9.2.0","branchLabelMappingKey":"^v9.2.0$","isSourceBranch":true,"state":"MERGED","url":"https://github.com/elastic/kibana/pull/227628","number":227628,"mergeCommit":{"message":"[APM]
Try to fix flaky test (#227628)\n\nCloses #214673 \n\n## Summary\n\nThis
PR tries to fix a flaky test by intercepting a request and waiting\nfor
it. I couldn't reproduce the flaky behaviour, but I saw that we are\nnot
waiting for the request to finish before checking the value, so
my\nassumption is that the request is slower in the CI compared to the
local\nexecution, and that's why the test is failing.\n\nCo-authored-by:
Sergi Romeu
<sergi.romeu@elastic.co>","sha":"1299379a43f7de109cfccc44525da3c2f6a5663e"}}]}]
BACKPORT-->

Co-authored-by: jennypavlova <dzheni.pavlova@elastic.co>
Co-authored-by: Sergi Romeu <sergi.romeu@elastic.co>
kibanamachine added a commit that referenced this pull request Jul 15, 2025
# Backport

This will backport the following commits from `main` to `8.18`:
- [[APM] Try to fix flaky test
(#227628)](#227628)

<!--- Backport version: 9.6.6 -->

### Questions ?
Please refer to the [Backport tool
documentation](https://github.com/sorenlouv/backport)

<!--BACKPORT
[{"author":{"name":"jennypavlova","email":"dzheni.pavlova@elastic.co"},"sourceCommit":{"committedDate":"2025-07-15T13:20:19Z","message":"[APM]
Try to fix flaky test (#227628)\n\nCloses #214673 \n\n## Summary\n\nThis
PR tries to fix a flaky test by intercepting a request and waiting\nfor
it. I couldn't reproduce the flaky behaviour, but I saw that we are\nnot
waiting for the request to finish before checking the value, so
my\nassumption is that the request is slower in the CI compared to the
local\nexecution, and that's why the test is failing.\n\nCo-authored-by:
Sergi Romeu
<sergi.romeu@elastic.co>","sha":"1299379a43f7de109cfccc44525da3c2f6a5663e","branchLabelMapping":{"^v9.2.0$":"main","^v(\\d+).(\\d+).\\d+$":"$1.$2"}},"sourcePullRequest":{"labels":["release_note:skip","backport:all-open","Team:obs-ux-infra_services","v9.2.0"],"title":"[APM]
Try to fix flaky
test","number":227628,"url":"https://github.com/elastic/kibana/pull/227628","mergeCommit":{"message":"[APM]
Try to fix flaky test (#227628)\n\nCloses #214673 \n\n## Summary\n\nThis
PR tries to fix a flaky test by intercepting a request and waiting\nfor
it. I couldn't reproduce the flaky behaviour, but I saw that we are\nnot
waiting for the request to finish before checking the value, so
my\nassumption is that the request is slower in the CI compared to the
local\nexecution, and that's why the test is failing.\n\nCo-authored-by:
Sergi Romeu
<sergi.romeu@elastic.co>","sha":"1299379a43f7de109cfccc44525da3c2f6a5663e"}},"sourceBranch":"main","suggestedTargetBranches":[],"targetPullRequestStates":[{"branch":"main","label":"v9.2.0","branchLabelMappingKey":"^v9.2.0$","isSourceBranch":true,"state":"MERGED","url":"https://github.com/elastic/kibana/pull/227628","number":227628,"mergeCommit":{"message":"[APM]
Try to fix flaky test (#227628)\n\nCloses #214673 \n\n## Summary\n\nThis
PR tries to fix a flaky test by intercepting a request and waiting\nfor
it. I couldn't reproduce the flaky behaviour, but I saw that we are\nnot
waiting for the request to finish before checking the value, so
my\nassumption is that the request is slower in the CI compared to the
local\nexecution, and that's why the test is failing.\n\nCo-authored-by:
Sergi Romeu
<sergi.romeu@elastic.co>","sha":"1299379a43f7de109cfccc44525da3c2f6a5663e"}}]}]
BACKPORT-->

Co-authored-by: jennypavlova <dzheni.pavlova@elastic.co>
Co-authored-by: Sergi Romeu <sergi.romeu@elastic.co>
kibanamachine added a commit that referenced this pull request Jul 16, 2025
# Backport

This will backport the following commits from `main` to `8.19`:
- [[APM] Try to fix flaky test
(#227628)](#227628)

<!--- Backport version: 9.6.6 -->

### Questions ?
Please refer to the [Backport tool
documentation](https://github.com/sorenlouv/backport)

<!--BACKPORT
[{"author":{"name":"jennypavlova","email":"dzheni.pavlova@elastic.co"},"sourceCommit":{"committedDate":"2025-07-15T13:20:19Z","message":"[APM]
Try to fix flaky test (#227628)\n\nCloses #214673 \n\n## Summary\n\nThis
PR tries to fix a flaky test by intercepting a request and waiting\nfor
it. I couldn't reproduce the flaky behaviour, but I saw that we are\nnot
waiting for the request to finish before checking the value, so
my\nassumption is that the request is slower in the CI compared to the
local\nexecution, and that's why the test is failing.\n\nCo-authored-by:
Sergi Romeu
<sergi.romeu@elastic.co>","sha":"1299379a43f7de109cfccc44525da3c2f6a5663e","branchLabelMapping":{"^v9.2.0$":"main","^v(\\d+).(\\d+).\\d+$":"$1.$2"}},"sourcePullRequest":{"labels":["release_note:skip","backport:all-open","Team:obs-ux-infra_services","v9.2.0"],"title":"[APM]
Try to fix flaky
test","number":227628,"url":"https://github.com/elastic/kibana/pull/227628","mergeCommit":{"message":"[APM]
Try to fix flaky test (#227628)\n\nCloses #214673 \n\n## Summary\n\nThis
PR tries to fix a flaky test by intercepting a request and waiting\nfor
it. I couldn't reproduce the flaky behaviour, but I saw that we are\nnot
waiting for the request to finish before checking the value, so
my\nassumption is that the request is slower in the CI compared to the
local\nexecution, and that's why the test is failing.\n\nCo-authored-by:
Sergi Romeu
<sergi.romeu@elastic.co>","sha":"1299379a43f7de109cfccc44525da3c2f6a5663e"}},"sourceBranch":"main","suggestedTargetBranches":[],"targetPullRequestStates":[{"branch":"main","label":"v9.2.0","branchLabelMappingKey":"^v9.2.0$","isSourceBranch":true,"state":"MERGED","url":"https://github.com/elastic/kibana/pull/227628","number":227628,"mergeCommit":{"message":"[APM]
Try to fix flaky test (#227628)\n\nCloses #214673 \n\n## Summary\n\nThis
PR tries to fix a flaky test by intercepting a request and waiting\nfor
it. I couldn't reproduce the flaky behaviour, but I saw that we are\nnot
waiting for the request to finish before checking the value, so
my\nassumption is that the request is slower in the CI compared to the
local\nexecution, and that's why the test is failing.\n\nCo-authored-by:
Sergi Romeu
<sergi.romeu@elastic.co>","sha":"1299379a43f7de109cfccc44525da3c2f6a5663e"}}]}]
BACKPORT-->

Co-authored-by: jennypavlova <dzheni.pavlova@elastic.co>
Co-authored-by: Sergi Romeu <sergi.romeu@elastic.co>
Bluefinger pushed a commit to Bluefinger/kibana that referenced this pull request Jul 22, 2025
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>
kertal pushed a commit to kertal/kibana that referenced this pull request Jul 25, 2025
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>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

backport:all-open Backport to all branches that could still receive a release release_note:skip Skip the PR/issue when compiling release notes Team:obs-ux-infra_services - DEPRECATED DEPRECATED - Use Team:obs-presentation. v8.17.9 v8.18.4 v8.19.0 v9.0.4 v9.1.0 v9.2.0

Projects

None yet

4 participants