-
Notifications
You must be signed in to change notification settings - Fork 84
chore: update evergreen to filter with install #2614
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
base: main
Are you sure you want to change the base?
Conversation
7d15df4 to
a4c2d77
Compare
785fb8f to
d071833
Compare
There was a problem hiding this 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_WORKSPACEenvironment 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" |
Copilot
AI
Dec 9, 2025
There was a problem hiding this comment.
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.
Stacked on #2613
This should noticably reduce test execution time for some of the smaller packages.