Skip to content

A test to check if creating snapshots blocks other operations#7510

Merged
tellet-q merged 3 commits intodevfrom
test-for-long-snapshot-lock
Feb 11, 2026
Merged

A test to check if creating snapshots blocks other operations#7510
tellet-q merged 3 commits intodevfrom
test-for-long-snapshot-lock

Conversation

@tellet-q
Copy link
Contributor

Fails right now as it requires a fix in Core (see #7489)

All Submissions:

  • Contributions should target the dev branch. Did you create your branch from dev?
  • Have you followed the guidelines in our Contributing document?
  • Have you checked to ensure there aren't other open Pull Requests for the same update/change?

coderabbitai[bot]

This comment was marked as resolved.

coderabbitai[bot]

This comment was marked as resolved.

Copy link
Member

@timvisee timvisee left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Looks good. Though of course we must fix the actual problem before we can merge this one.

@qdrant qdrant deleted a comment from coderabbitai bot Feb 9, 2026
@tellet-q tellet-q force-pushed the test-for-long-snapshot-lock branch from 9a9b539 to 0c4319f Compare February 11, 2026 08:40
coderabbitai[bot]

This comment was marked as resolved.

Copy link
Contributor

@coderabbitai coderabbitai bot left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Actionable comments posted: 1

🤖 Fix all issues with AI agents
In `@tests/e2e_tests/test_bugs.py`:
- Around line 6-27: The test
TestSnapshotsInterferenceWithConsensus.test_snapshot_does_not_block_other_operations
is expected to fail until the Core fix for issue `#7489` lands, so mark it with
`@pytest.mark.xfail`(reason="Depends on Core fix for `#7489`", strict=False) to
avoid breaking CI, and add a short inline comment before the
client.delete_collection("small", timeout=2) call (e.g., "# If snapshots block
consensus, this will time out and raise") to document that the timeout/raise is
the implicit assertion; ensure pytest is imported if not already.

@qdrant qdrant deleted a comment from coderabbitai bot Feb 11, 2026
@tellet-q tellet-q merged commit 3ba62e3 into dev Feb 11, 2026
15 checks passed
@tellet-q tellet-q deleted the test-for-long-snapshot-lock branch February 11, 2026 11:20
timvisee pushed a commit that referenced this pull request Feb 13, 2026
)

* Add a test to check if creating snapshots blocks other operations
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants