Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

[8.17] [Index Management FTR] Improve delete indices test (#200941) #201318

Merged
merged 1 commit into from
Nov 22, 2024

Conversation

kibanamachine
Copy link
Contributor

Backport

This will backport the following commits from main to 8.17:

Questions ?

Please refer to the Backport tool documentation

Closes elastic#192351
Closes elastic#200660
Closes elastic#193599
Closes elastic#192532

## Summary

This PR improves the IM functional test for indices table. In
elastic#192351, the failure most likely
occurred because the webdriver tried to interact with elements that are
no longer in the DOM - in this case the index rows and table. To avoid
this, we are adding a try-catch for `StaleElementReferenceError` in the
`expectIndexIsDeleted` test helper. In
elastic#192351,
elastic#20066, and
elastic#193599, the error most likely
was caused because the check if an index row exists was performed before
the row element was removed from the DOM, therefore it was incorrectly
failing. To fix this, we are adding a retry so that the check is retried
in case it fails.

(cherry picked from commit e48ad46)
@kibanamachine kibanamachine merged commit 233d3fb into elastic:8.17 Nov 22, 2024
26 checks passed
@elasticmachine
Copy link
Contributor

💚 Build Succeeded

Metrics [docs]

✅ unchanged

cc @ElenaStoeva

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants