Skip to content

Conversation

@olapawlus
Copy link
Member

@olapawlus olapawlus commented Nov 27, 2025

Resolves #209228

The link to Flaky Test Runner: https://buildkite.com/elastic/kibana-flaky-test-suite-runner/builds/9932

The test was asserting an exact document count (Found 174 documents.), but the tooltip sometimes reported a partial count (e.g. Found 138 documents.), causing the test to fail intermittently.

Reasoning
Maps fetches documents only for the visible map area, so on smaller or zoomed-in views, fewer documents might appear.
To stabilize the test, I explicitly set the zoom level at the beginning of the test to ensure consistent MVT data loading.

@olapawlus olapawlus self-assigned this Nov 27, 2025
@olapawlus olapawlus added blocker Team:Presentation Presentation Team for Dashboard, Input Controls, and Canvas t// failed-test A test failure on a tracked branch, potentially flaky-test skipped-test release_note:skip Skip the PR/issue when compiling release notes impact:critical This issue should be addressed immediately due to a critical level of impact on the product. backport:version Backport to applied version labels v9.3.0 v9.2.2 labels Nov 27, 2025
@olapawlus olapawlus marked this pull request as ready for review November 27, 2025 13:10
@olapawlus olapawlus requested a review from a team as a code owner November 27, 2025 13:10
@elasticmachine
Copy link
Contributor

Pinging @elastic/kibana-presentation (Team:Presentation)

@kibanamachine
Copy link
Contributor

Flaky Test Runner Stats

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

[✅] x-pack/platform/test/functional/apps/maps/group4/config.ts: 170/170 tests passed.

see run history

@elasticmachine
Copy link
Contributor

💚 Build Succeeded

Metrics [docs]

✅ unchanged

cc @olapawlus

@olapawlus olapawlus merged commit 06d1d51 into elastic:main Nov 28, 2025
13 checks passed
@kibanamachine
Copy link
Contributor

Starting backport for target branches: 9.2

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

kibanamachine pushed a commit to kibanamachine/kibana that referenced this pull request Nov 28, 2025
Resolves elastic#209228

The link to Flaky Test Runner:
https://buildkite.com/elastic/kibana-flaky-test-suite-runner/builds/9932

The test was asserting an exact document count (Found 174 documents.),
but the tooltip sometimes reported a partial count (e.g. Found 138
documents.), causing the test to fail intermittently.

Reasoning
Maps fetches documents only for the visible map area, so on smaller or
zoomed-in views, fewer documents might appear.
To stabilize the test, I explicitly set the zoom level at the beginning
of the test to ensure consistent MVT data loading.

(cherry picked from commit 06d1d51)
@kibanamachine
Copy link
Contributor

💚 All backports created successfully

Status Branch Result
9.2

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 Nov 28, 2025
# Backport

This will backport the following commits from `main` to `9.2`:
- [[Maps] Fix flaky test in maps shape file
(#244484)](#244484)

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

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

<!--BACKPORT [{"author":{"name":"Ola
Pawlus","email":"[email protected]"},"sourceCommit":{"committedDate":"2025-11-28T09:22:47Z","message":"[Maps]
Fix flaky test in maps shape file (#244484)\n\nResolves #209228\n\nThe
link to Flaky Test
Runner:\nhttps://buildkite.com/elastic/kibana-flaky-test-suite-runner/builds/9932\n\nThe
test was asserting an exact document count (Found 174 documents.),\nbut
the tooltip sometimes reported a partial count (e.g. Found
138\ndocuments.), causing the test to fail
intermittently.\n\nReasoning\nMaps fetches documents only for the
visible map area, so on smaller or\nzoomed-in views, fewer documents
might appear.\nTo stabilize the test, I explicitly set the zoom level at
the beginning\nof the test to ensure consistent MVT data
loading.","sha":"06d1d515ac7ad5b2183f67e8f8ff1444f168e71d","branchLabelMapping":{"^v9.3.0$":"main","^v(\\d+).(\\d+).\\d+$":"$1.$2"}},"sourcePullRequest":{"labels":["blocker","Team:Presentation","failed-test","skipped-test","release_note:skip","impact:critical","backport:version","v9.3.0","v9.2.2"],"title":"[Maps]
Fix flaky test in maps shape
file","number":244484,"url":"https://github.com/elastic/kibana/pull/244484","mergeCommit":{"message":"[Maps]
Fix flaky test in maps shape file (#244484)\n\nResolves #209228\n\nThe
link to Flaky Test
Runner:\nhttps://buildkite.com/elastic/kibana-flaky-test-suite-runner/builds/9932\n\nThe
test was asserting an exact document count (Found 174 documents.),\nbut
the tooltip sometimes reported a partial count (e.g. Found
138\ndocuments.), causing the test to fail
intermittently.\n\nReasoning\nMaps fetches documents only for the
visible map area, so on smaller or\nzoomed-in views, fewer documents
might appear.\nTo stabilize the test, I explicitly set the zoom level at
the beginning\nof the test to ensure consistent MVT data
loading.","sha":"06d1d515ac7ad5b2183f67e8f8ff1444f168e71d"}},"sourceBranch":"main","suggestedTargetBranches":["9.2"],"targetPullRequestStates":[{"branch":"main","label":"v9.3.0","branchLabelMappingKey":"^v9.3.0$","isSourceBranch":true,"state":"MERGED","url":"https://github.com/elastic/kibana/pull/244484","number":244484,"mergeCommit":{"message":"[Maps]
Fix flaky test in maps shape file (#244484)\n\nResolves #209228\n\nThe
link to Flaky Test
Runner:\nhttps://buildkite.com/elastic/kibana-flaky-test-suite-runner/builds/9932\n\nThe
test was asserting an exact document count (Found 174 documents.),\nbut
the tooltip sometimes reported a partial count (e.g. Found
138\ndocuments.), causing the test to fail
intermittently.\n\nReasoning\nMaps fetches documents only for the
visible map area, so on smaller or\nzoomed-in views, fewer documents
might appear.\nTo stabilize the test, I explicitly set the zoom level at
the beginning\nof the test to ensure consistent MVT data
loading.","sha":"06d1d515ac7ad5b2183f67e8f8ff1444f168e71d"}},{"branch":"9.2","label":"v9.2.2","branchLabelMappingKey":"^v(\\d+).(\\d+).\\d+$","isSourceBranch":false,"state":"NOT_CREATED"}]}]
BACKPORT-->

Co-authored-by: Ola Pawlus <[email protected]>
tkajtoch pushed a commit to tkajtoch/kibana that referenced this pull request Dec 1, 2025
Resolves elastic#209228

The link to Flaky Test Runner:
https://buildkite.com/elastic/kibana-flaky-test-suite-runner/builds/9932

The test was asserting an exact document count (Found 174 documents.),
but the tooltip sometimes reported a partial count (e.g. Found 138
documents.), causing the test to fail intermittently.

Reasoning
Maps fetches documents only for the visible map area, so on smaller or
zoomed-in views, fewer documents might appear.
To stabilize the test, I explicitly set the zoom level at the beginning
of the test to ensure consistent MVT data loading.
NicholasPeretti pushed a commit to NicholasPeretti/kibana that referenced this pull request Dec 2, 2025
Resolves elastic#209228

The link to Flaky Test Runner:
https://buildkite.com/elastic/kibana-flaky-test-suite-runner/builds/9932

The test was asserting an exact document count (Found 174 documents.),
but the tooltip sometimes reported a partial count (e.g. Found 138
documents.), causing the test to fail intermittently.

Reasoning
Maps fetches documents only for the visible map area, so on smaller or
zoomed-in views, fewer documents might appear.
To stabilize the test, I explicitly set the zoom level at the beginning
of the test to ensure consistent MVT data loading.
@mistic mistic added v9.2.3 and removed v9.2.2 labels Dec 3, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

backport:version Backport to applied version labels blocker failed-test A test failure on a tracked branch, potentially flaky-test impact:critical This issue should be addressed immediately due to a critical level of impact on the product. release_note:skip Skip the PR/issue when compiling release notes skipped-test Team:Presentation Presentation Team for Dashboard, Input Controls, and Canvas t// v9.2.3 v9.3.0

Projects

None yet

5 participants