-
Notifications
You must be signed in to change notification settings - Fork 83
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.
47809c1 to
97c30c1
Compare
c7f0516 to
21d778c
Compare
d2d34c0 to
32c3572
Compare
Stacked on #2613
This should noticably reduce test execution time for some of the smaller packages.
Some examples (around ~30s reduction! compared to base):
test_cli_repl
Duration: 13m 31s
Base commit duration: 14m 56s
test_errors
Duration: 1m 3s
Base commit duration: 1m 35s
test_shell_api
Duration: 5m 12s
Base commit duration: 5m 27s