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.

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

@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.

@gagik gagik added the no-title-validation Skips validation of PR titles (conventional commit adherence + JIRA ticket inclusion) label Dec 9, 2025
@gagik gagik marked this pull request as draft December 12, 2025 15:36
@gagik gagik force-pushed the gagik/individual-package-deps branch from 47809c1 to 97c30c1 Compare December 22, 2025 11:11
@gagik gagik force-pushed the gagik/individual-package-deps branch from c7f0516 to 21d778c Compare January 7, 2026 12:27
@gagik gagik force-pushed the gagik/individual-package-deps branch from d2d34c0 to 32c3572 Compare January 7, 2026 13:37
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