Skip to content

Conversation

@gagik
Copy link
Contributor

@gagik gagik commented Dec 5, 2025

Stacked on #2613

This should noticably reduce test execution time for some of the smaller packages.

@gagik gagik changed the base branch from main to gagik/adopt-knip December 5, 2025 15:29
@gagik gagik force-pushed the gagik/adopt-knip branch 4 times, most recently from 7d15df4 to a4c2d77 Compare December 5, 2025 16:20
Base automatically changed from gagik/adopt-knip to main December 8, 2025 10:23
@gagik gagik force-pushed the gagik/individual-package-deps branch from 785fb8f to d071833 Compare December 9, 2025 11:04
@gagik gagik marked this pull request as ready for review December 9, 2025 11:10
@gagik gagik requested a review from a team as a code owner December 9, 2025 11:10
Copilot AI review requested due to automatic review settings December 9, 2025 11:10
Copy link

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 optimizes Evergreen CI builds by enabling selective workspace installation through a new MONGOSH_INSTALL_WORKSPACE environment variable. When set, only the specified workspace and its dependencies are installed, reducing installation time for smaller packages.

Key changes:

  • Added conditional workspace-specific installation logic to the npm dependency installation script
  • Introduced MONGOSH_INSTALL_WORKSPACE environment variable in Evergreen configuration
  • Applied workspace-specific installation settings to all 26 package test tasks

Reviewed changes

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

File Description
.evergreen/install-npm-deps.sh Added conditional logic to install only specified workspace when MONGOSH_INSTALL_WORKSPACE is set, including special handling for mongodb-client-encryption optional dependency
.evergreen/evergreen.yml.in Added MONGOSH_INSTALL_WORKSPACE variable to environment and templated task configuration to pass workspace-specific values
.evergreen.yml Applied mongosh_install_workspace setting to all 26 package-specific test tasks with appropriate package names

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

else
npm ci --verbose
fi
echo "MONOGDB_DRIVER_VERSION_OVERRIDE:$MONOGDB_DRIVER_VERSION_OVERRIDE"
Copy link

Copilot AI Dec 9, 2025

Choose a reason for hiding this comment

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

Corrected spelling of 'MONOGDB' to 'MONGODB' in variable name.

Copilot uses AI. Check for mistakes.
@gagik gagik added the no-title-validation Skips validation of PR titles (conventional commit adherence + JIRA ticket inclusion) label Dec 9, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

no-title-validation Skips validation of PR titles (conventional commit adherence + JIRA ticket inclusion)

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants