Skip to content

Conversation

@himanshusinghs
Copy link
Collaborator

Proposed changes

This temporarily pins the Atlas image to a tag with which our tests were working fine.

Checklist

@himanshusinghs himanshusinghs requested a review from a team as a code owner November 27, 2025 15:01
Copilot AI review requested due to automatic review settings November 27, 2025 15:01
@himanshusinghs himanshusinghs added the no-title-validation Add this label to disable the title check for this PR. label Nov 27, 2025
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 temporarily pins the MongoDB Atlas Local Docker image to a specific working version (8.2.2-20251125T154829Z) to resolve test failures, while also updating test expectations to handle the race condition between PENDING and BUILDING states during vector index creation.

  • Pins the Atlas Local image to a specific version tag instead of the generic 8 tag
  • Updates vector index status test to accept both PENDING and BUILDING states

Reviewed changes

Copilot reviewed 2 out of 2 changed files in this pull request and generated 1 comment.

File Description
tests/integration/tools/mongodb/mongodbClusterProcess.ts Pins Atlas Local image to specific version tag with TODO comment for reversion
tests/integration/tools/mongodb/create/createIndex.test.ts Updates test expectation to handle race condition in vector index status

Copy link
Collaborator

@fmenezes fmenezes left a comment

Choose a reason for hiding this comment

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

LGTM

@himanshusinghs himanshusinghs merged commit 9963a2a into main Nov 27, 2025
26 of 28 checks passed
@himanshusinghs himanshusinghs deleted the fix/tests branch November 27, 2025 15:14
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

no-title-validation Add this label to disable the title check for this PR.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants