PYTHON-6056 - Search index helper prose tests are flaky - #3012
Conversation
There was a problem hiding this comment.
Pull request overview
This PR addresses flakiness in the “search index helper prose” integration tests by marking the affected prose test cases as flaky on CPython/Linux, allowing limited automatic retries in CI where the flakiness is observed.
Changes:
- Import the shared
flakytest decorator in the synchronous and asynchronous index management test modules. - Apply
@flaky(reason="PYTHON-6056", affects_cpython_linux=True)to theTestSearchIndexProseprose test cases (1–7) in both sync and async suites.
Reviewed changes
Copilot reviewed 2 out of 2 changed files in this pull request and generated no comments.
| File | Description |
|---|---|
| test/test_index_management.py | Marks the synchronous SearchIndex prose test cases as flaky on CPython/Linux to reduce CI noise from intermittent failures. |
| test/asynchronous/test_index_management.py | Marks the asynchronous SearchIndex prose test cases as flaky on CPython/Linux to reduce CI noise from intermittent failures. |
💡 Add a code-review agent skill or configure MCP servers for context-aware, tailored reviews. Learn more in the docs.
Codecov Report✅ All modified and coverable lines are covered by tests. 📢 Thoughts on this report? Let us know! |
| class TestSearchIndexProse(SearchIndexIntegrationBase): | ||
| db_name = "test_search_index_prose" | ||
|
|
||
| @flaky(reason="PYTHON-6056", affects_cpython_linux=True) |
There was a problem hiding this comment.
Do we need reset_func for these?
There was a problem hiding this comment.
Good catch, these all need to undo whatever successful parts of their execution ran between attempts.
PYTHON-6056
Changes in this PR
See ticket.
Test Plan
Checklist
Checklist for Author
Checklist for Reviewer