Skip to content

[8.x] [Obs AI Assistant] Attempt to fix flaky KB migration test (#206324)#206543

Merged
sorenlouv merged 1 commit intoelastic:8.xfrom
sorenlouv:backport/8.x/pr-206324
Jan 14, 2025
Merged

[8.x] [Obs AI Assistant] Attempt to fix flaky KB migration test (#206324)#206543
sorenlouv merged 1 commit intoelastic:8.xfrom
sorenlouv:backport/8.x/pr-206324

Conversation

@sorenlouv
Copy link
Copy Markdown
Contributor

Backport

This will backport the following commits from main to 8.x:

Questions ?

Please refer to the Backport tool documentation

…6324)

## Summary

Closes elastic#202926

### Problem
The KB migration test appears to be flaky sometimes, because the
migration takes a few seconds to run. Therefore, when the test expects
to have semantic_text for all entries, it fails.

### Solution
Adding a retry statement to avoid the failure in cases where the
migration takes a little while to run.

### Checklist

- [x] [Flaky Test
Runner](https://ci-stats.kibana.dev/trigger_flaky_test_runner/1) was
used on any tests changed
- [x] The PR description includes the appropriate Release Notes section,
and the correct `release_note:*` label is applied per the
[guidelines](https://www.elastic.co/guide/en/kibana/master/contributing.html#kibana-release-notes-process)

(cherry picked from commit 95b76dc)
@sorenlouv sorenlouv added the backport This PR is a backport of another PR label Jan 14, 2025
@sorenlouv sorenlouv enabled auto-merge (squash) January 14, 2025 08:09
@elasticmachine
Copy link
Copy Markdown
Contributor

💛 Build succeeded, but was flaky

Failed CI Steps

Test Failures

  • [job] [logs] Jest Tests #11 / useMonitorStatusData handles resize events and requests based on new data

Metrics [docs]

✅ unchanged

@sorenlouv sorenlouv merged commit cb7cd55 into elastic:8.x Jan 14, 2025
@sorenlouv sorenlouv deleted the backport/8.x/pr-206324 branch January 14, 2025 13:27
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

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants