Skip to content

ci: Reduce test runner startup overhead - #334833

Merged
Dmitriy Vasyura (dmitrivMS) merged 5 commits into
mainfrom
agents/ci-performance-optimization
Sep 8, 2026
Merged

ci: Reduce test runner startup overhead#334833
Dmitriy Vasyura (dmitrivMS) merged 5 commits into
mainfrom
agents/ci-performance-optimization

Conversation

@dmitrivMS

@dmitrivMS Dmitriy Vasyura (dmitrivMS) commented Sep 6, 2026

Copy link
Copy Markdown
Collaborator

Summary

  • skip redundant prelaunch synchronization in already-prepared Linux, macOS, and Windows test jobs
  • avoid compiling the Copilot extension in Browser smoke jobs, which do not register Chat or Agents Window suites
  • skip login-shell environment probing for deterministic Browser integration, Browser smoke, and Remote test servers
  • use Mocha's Dot console reporter for Browser unit tests while preserving full JUnit test results

Expected impact

A focused Browser smoke run took 15.4 seconds with prelaunch synchronization and 12.2 seconds without it. Browser jobs start 17-18 test servers, projecting approximately 50-60 seconds saved per Browser job.

Skipping Browser-only Copilot compilation removes another 5-8 seconds of unnecessary work. The Browser unit reporter benchmark reduced console output from 2,761 to 263 lines and from 177KB to 80KB for the same 2,700-test subset.

The workflow job matrix and test coverage are unchanged.

Run Agent Host E2E beside the remaining Electron integration tests, split Electron smoke coverage into core and chat shards, and isolate the macOS proxy scenarios from the main smoke critical path.

Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com>

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.

Copilot encountered an error and was unable to review this pull request. You can try again by re-requesting a review.

Note

This error may be related to your runner configuration. You can now configure runners for Copilot code review separately from Copilot cloud agent by creating a copilot-code-review.yml file with your setup steps. Read the docs for details.

Skip redundant prelaunch work on prepared CI runners, avoid Browser-only Copilot compilation, disable login-shell probing on test servers, and reduce Browser unit console output while preserving JUnit results.

Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com>

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.

Copilot encountered an error and was unable to review this pull request. You can try again by re-requesting a review.

Note

This error may be related to your runner configuration. You can now configure runners for Copilot code review separately from Copilot cloud agent by creating a copilot-code-review.yml file with your setup steps. Read the docs for details.

@roblourens roblourens left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

[Experimental performance review bot]

Automated experimental performance review.

(Written by Copilot)

Comment thread .github/workflows/pr.yml Outdated
Comment thread .github/workflows/pr.yml Outdated
@dmitrivMS

Copy link
Copy Markdown
Collaborator Author

Going to do some more work on this as feedback is valid, the current refactoring may be putting more pressure on the pool while the wall time win is small.

Run Agent Host E2E concurrently inside the existing Electron integration jobs and recombine core and chat smoke coverage, eliminating repeated checkout, dependency restore, transpilation, and Electron setup. Keep only the macOS network proxy smoke job isolated because it changes global network state.

Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com>
@dmitrivMS Dmitriy Vasyura (dmitrivMS) changed the title ci: Parallelize integration and smoke test shards ci: Reduce test runner startup overhead Sep 8, 2026
@dmitrivMS
Dmitriy Vasyura (dmitrivMS) marked this pull request as ready for review September 8, 2026 03:14
@dmitrivMS
Dmitriy Vasyura (dmitrivMS) merged commit 3c81a51 into main Sep 8, 2026
33 checks passed
@dmitrivMS
Dmitriy Vasyura (dmitrivMS) deleted the agents/ci-performance-optimization branch September 8, 2026 21:23
@vs-code-engineering vs-code-engineering Bot added this to the 1.138.0 milestone Sep 8, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

engineering VS Code - Build / issue tracking / etc. vscode-build VS Code build process issues

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants