Skip to content

[UA] Remove flaky test#227809

Merged
sabarasaba merged 3 commits intoelastic:mainfrom
sabarasaba:ua-remove_flaky_test
Jul 22, 2025
Merged

[UA] Remove flaky test#227809
sabarasaba merged 3 commits intoelastic:mainfrom
sabarasaba:ua-remove_flaky_test

Conversation

@sabarasaba
Copy link
Member

@sabarasaba sabarasaba commented Jul 14, 2025

Closes #217262 (comment)

Summary

Since we migrated the es deprecations logs page into a flyout this a11y test started being flaky and we havent figure out how to fix it yet. Given that we already have functional tests and unit tests for this functionality, we feel confident enough to remove this one and explore a proper solution for it later on.

@sabarasaba sabarasaba self-assigned this Jul 14, 2025
@sabarasaba sabarasaba added Team:Kibana Management Dev Tools, Index Management, Upgrade Assistant, ILM, Ingest Node Pipelines, and more t// release_note:skip Skip the PR/issue when compiling release notes Feature:Upgrade Assistant backport:version Backport to applied version labels v8.18.4 labels Jul 14, 2025
@sabarasaba
Copy link
Member Author

/ci

@sabarasaba
Copy link
Member Author

/ci

@sabarasaba sabarasaba marked this pull request as ready for review July 21, 2025 10:50
@sabarasaba sabarasaba requested a review from a team as a code owner July 21, 2025 10:50
@elasticmachine
Copy link
Contributor

Pinging @elastic/kibana-management (Team:Kibana Management)

@sabarasaba
Copy link
Member Author

@elasticmachine merge upstream

@sabarasaba sabarasaba merged commit 7137d65 into elastic:main Jul 22, 2025
13 checks passed
@kibanamachine
Copy link
Contributor

Starting backport for target branches: 8.18

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

@kibanamachine
Copy link
Contributor

💔 All backports failed

Status Branch Result
8.18 Backport failed because of merge conflicts

Manual backport

To create the backport manually run:

node scripts/backport --pr 227809

Questions ?

Please refer to the Backport tool documentation

sabarasaba added a commit to sabarasaba/kibana that referenced this pull request Jul 22, 2025
(cherry picked from commit 7137d65)

# Conflicts:
#	x-pack/test/accessibility/apps/group3/upgrade_assistant.ts
@sabarasaba
Copy link
Member Author

💚 All backports created successfully

Status Branch Result
8.18

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

Questions ?

Please refer to the Backport tool documentation

maxcold pushed a commit that referenced this pull request Jul 22, 2025
@kibanamachine kibanamachine added the backport missing Added to PRs automatically when the are determined to be missing a backport. label Jul 23, 2025
@kibanamachine
Copy link
Contributor

Looks like this PR has a backport PR but it still hasn't been merged. Please merge it ASAP to keep the branches relatively in sync.
cc: @sabarasaba

kdelemme pushed a commit to kdelemme/kibana that referenced this pull request Jul 23, 2025
@kibanamachine
Copy link
Contributor

Looks like this PR has a backport PR but it still hasn't been merged. Please merge it ASAP to keep the branches relatively in sync.
cc: @sabarasaba

sabarasaba added a commit that referenced this pull request Jul 24, 2025
# Backport

This will backport the following commits from `main` to `8.18`:
- [[UA] Remove flaky test
(#227809)](#227809)

<!--- Backport version: 10.0.1 -->

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

<!--BACKPORT [{"author":{"name":"Ignacio
Rivas","email":"rivasign@gmail.com"},"sourceCommit":{"committedDate":"2025-07-22T10:22:31Z","message":"[UA]
Remove flaky test
(#227809)","sha":"7137d655178034356c255865cba826d92b8e5bdc","branchLabelMapping":{"^v9.2.0$":"main","^v(\\d+).(\\d+).\\d+$":"$1.$2"}},"sourcePullRequest":{"labels":["Team:Kibana
Management","release_note:skip","Feature:Upgrade
Assistant","backport:version","v9.2.0","v8.18.4"],"title":"[UA] Remove
flaky
test","number":227809,"url":"https://github.com/elastic/kibana/pull/227809","mergeCommit":{"message":"[UA]
Remove flaky test
(#227809)","sha":"7137d655178034356c255865cba826d92b8e5bdc"}},"sourceBranch":"main","suggestedTargetBranches":["8.18"],"targetPullRequestStates":[{"branch":"main","label":"v9.2.0","branchLabelMappingKey":"^v9.2.0$","isSourceBranch":true,"state":"MERGED","url":"https://github.com/elastic/kibana/pull/227809","number":227809,"mergeCommit":{"message":"[UA]
Remove flaky test
(#227809)","sha":"7137d655178034356c255865cba826d92b8e5bdc"}},{"branch":"8.18","label":"v8.18.4","branchLabelMappingKey":"^v(\\d+).(\\d+).\\d+$","isSourceBranch":false,"state":"NOT_CREATED"}]}]
BACKPORT-->

Co-authored-by: Elastic Machine <elasticmachine@users.noreply.github.com>
@kibanamachine kibanamachine added v8.18.5 and removed backport missing Added to PRs automatically when the are determined to be missing a backport. labels Jul 24, 2025
kertal pushed a commit to kertal/kibana that referenced this pull request Jul 25, 2025
crespocarlos pushed a commit to crespocarlos/kibana that referenced this pull request Jul 25, 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 Feature:Upgrade Assistant release_note:skip Skip the PR/issue when compiling release notes Team:Kibana Management Dev Tools, Index Management, Upgrade Assistant, ILM, Ingest Node Pipelines, and more t// v8.18.4 v8.18.5 v9.2.0

Projects

None yet

4 participants