Skip to content

Conversation

@comandeo-mongo
Copy link
Contributor

No description provided.

Copilot AI review requested due to automatic review settings February 2, 2026 07:26
@comandeo-mongo comandeo-mongo requested a review from a team as a code owner February 2, 2026 07:26
@comandeo-mongo comandeo-mongo requested a review from jamis February 2, 2026 07:26
Copy link
Contributor

Copilot AI left a comment

Choose a reason for hiding this comment

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

Pull request overview

This PR fixes flaky Client-Side Operations Timeout (CSOT) tests by increasing timeout margins and correcting test expectations for transaction behavior.

Changes:

  • Increased timeout values from 50ms to 500ms and block times from 60ms to 600ms, providing a larger margin (100ms vs 10ms) to reduce test flakiness
  • Updated withTransaction test expectations to properly model abort behavior, including both insert and abortTransaction in failpoint configuration
  • Added minPoolSize configuration and upgraded schema version to 1.26 in sessions-inherit-timeoutMS.yml to support awaitMinPoolSizeMS

Reviewed changes

Copilot reviewed 3 out of 3 changed files in this pull request and generated no comments.

File Description
sessions-override-timeoutMS.yml Updated timeout values (50ms→500ms, 60ms→600ms) and corrected withTransaction test to expect abortTransaction events
sessions-override-operation-timeoutMS.yml Updated timeout values (50ms→500ms, 60ms→600ms) and corrected withTransaction test to expect abortTransaction events
sessions-inherit-timeoutMS.yml Updated timeout values (50ms→500ms, 60ms→600ms), upgraded schema to 1.26, added minPoolSize/awaitMinPoolSizeMS configuration, and corrected withTransaction test expectations

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

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.

1 participant