Skip to content

chore(ci): give the VS Code extension test container a larger /dev/shm MONGOSH-3410#2737

Open
nbbeeken wants to merge 1 commit into
mainfrom
vscode-test-shm-size
Open

chore(ci): give the VS Code extension test container a larger /dev/shm MONGOSH-3410#2737
nbbeeken wants to merge 1 commit into
mainfrom
vscode-test-shm-size

Conversation

@nbbeeken

@nbbeeken nbbeeken commented Jun 9, 2026

Copy link
Copy Markdown
Contributor

What

Run the VS Code extension test container with --shm-size=2g.

Why

The VS Code extension tests drive a headless Chromium inside the ubuntu24.04-xvfb container. Docker's default /dev/shm is only 64MB, which is too small for the Chromium renderer — it intermittently crashes (renderer process exits with code 133) partway through the suite, failing test_vscode even though the mongosh tests themselves were passing.

Giving the container a 2g /dev/shm removes that crash.

Notes

  • Edited .evergreen/evergreen.yml.in and regenerated .evergreen.yml (npm run update-evergreen-config); the two changes are identical.
  • This is the first focused piece extracted from chore(ci): various mongosh CI fixes MONGOSH-2855  #2736, which is being split into smaller, independently-reviewable PRs.

The VS Code extension tests run a headless Chromium inside the
ubuntu24.04-xvfb container. Docker's default 64MB /dev/shm is too small
for the renderer, which intermittently crashes (renderer process exits
with code 133) partway through the suite. Run the container with
--shm-size=2g.

Extracted from #2736 as part of splitting that branch into focused PRs.
Copilot AI review requested due to automatic review settings June 9, 2026 20:55
@nbbeeken nbbeeken requested a review from a team as a code owner June 9, 2026 20:55
@nbbeeken nbbeeken requested a review from dudaschar June 9, 2026 20:55

Copilot AI left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Pull request overview

Updates Evergreen CI configuration to run the VS Code extension integration tests’ Docker container with a larger shared memory segment to prevent intermittent headless Chromium renderer crashes.

Changes:

  • Add --shm-size=2g to the docker run invocation used by the test_vscode function.
  • Add an inline comment documenting the Chromium /dev/shm crash rationale.
  • Apply the same update to both the Evergreen template and the generated Evergreen config.

Reviewed changes

Copilot reviewed 2 out of 2 changed files in this pull request and generated no comments.

File Description
.evergreen/evergreen.yml.in Adds --shm-size=2g to the VS Code test container run command and documents the reason.
.evergreen.yml Regenerated config mirroring the same --shm-size=2g update for CI execution.

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

@nbbeeken nbbeeken requested review from addaleax and removed request for dudaschar June 9, 2026 20:59
@nbbeeken nbbeeken changed the title fix(ci): give the VS Code extension test container a larger /dev/shm chore(ci): give the VS Code extension test container a larger /dev/shm Jun 9, 2026
@nbbeeken nbbeeken changed the title chore(ci): give the VS Code extension test container a larger /dev/shm fix(ci): give the VS Code extension test container a larger /dev/shm MONGOSH-3410 Jun 9, 2026
@nbbeeken nbbeeken changed the title fix(ci): give the VS Code extension test container a larger /dev/shm MONGOSH-3410 chore(ci): give the VS Code extension test container a larger /dev/shm MONGOSH-3410 Jun 9, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants