Skip to content

[8.x] Retry release to fix flaky tests (#216781)#216798

Merged
kibanamachine merged 1 commit intoelastic:8.xfrom
kibanamachine:backport/8.x/pr-216781
Apr 2, 2025
Merged

[8.x] Retry release to fix flaky tests (#216781)#216798
kibanamachine merged 1 commit intoelastic:8.xfrom
kibanamachine:backport/8.x/pr-216781

Conversation

@kibanamachine
Copy link
Contributor

Backport

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

Questions ?

Please refer to the Backport tool documentation

Related to elastic#216397
Closes elastic#216763

This change ensures that we do not send the `release` request and
`extendTtl` request simultaneously in `withLock`. This caused a conflict
causing tests to fail:

```
           └-> "before all" hook for "should return the result of the callback"
             │ERROR Failed to release lock "my_lock_with_ttl_extension": version_conflict_engine_exception
             │      	Root causes:
             │      		version_conflict_engine_exception: [my_lock_with_ttl_extension]: version conflict, required seqNo [43], primary term [1]. current document has seqNo [44] and primary term [1]
```

Flaky tests:
https://buildkite.com/elastic/kibana-flaky-test-suite-runner/builds/8142

(cherry picked from commit 7275d2e)
@kibanamachine kibanamachine added the backport This PR is a backport of another PR label Apr 2, 2025
@kibanamachine kibanamachine enabled auto-merge (squash) April 2, 2025 11:27
@botelastic botelastic bot added the Team:Obs AI Assistant Observability AI Assistant label Apr 2, 2025
@elasticmachine
Copy link
Contributor

Pinging @elastic/obs-ai-assistant (Team:Obs AI Assistant)

@kibanamachine kibanamachine merged commit b421fe7 into elastic:8.x Apr 2, 2025
13 checks passed
@elasticmachine
Copy link
Contributor

💛 Build succeeded, but was flaky

Failed CI Steps

Test Failures

  • [job] [logs] FTR Configs #60 / telemetry Telemetry service Screenshot mode detects that telemetry cannot be sent in screenshot mode

Metrics [docs]

✅ unchanged

cc @sorenlouv

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 Team:Obs AI Assistant Observability AI Assistant

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants