Merged
Conversation
We need to migrate to the new Linux image in CI so we can run tests on Linux again. dotnet#2557 This PR initially had the changes but it should really be an isolated fix - was intending to put it there to improve PR merge velocity
Contributor
There was a problem hiding this comment.
Pull request overview
This PR migrates the Linux CI environment from Ubuntu 20.04 to Ubuntu 22.04 to restore functional Linux testing. The changes include updating the pipeline configuration, installing necessary dependencies for running VS Code tests on the new image, and adjusting test configurations to match the CI environment.
Changes:
- Updated CI pipeline to use Ubuntu 22.04 image instead of 20.04
- Added VS Code runtime dependencies installation for Linux builds
- Fixed test utility to remove invalid Ubuntu version check and align .NET version selection with actual Ubuntu releases
- Updated test configuration to use Ubuntu 22.04 matching the new CI environment
- Added missing
descriptionproperty to mock object to match current VS Code API
Reviewed changes
Copilot reviewed 5 out of 5 changed files in this pull request and generated no comments.
Show a summary per file
| File | Description |
|---|---|
| 1pr-azure-pipeline.yml | Updated Linux pool image from 1es-ubuntu-2004-open to build.ubuntu.2204.amd64.open |
| pipeline-templates/build-test.yaml | Added installation of VS Code dependencies (libgbm, gtk, nss, etc.) required for headless testing on Ubuntu 22.04 |
| vscode-dotnet-runtime-library/src/test/unit/TestUtility.ts | Removed invalid Ubuntu 22.06 condition (no such release exists) and corrected .NET SDK version logic for Ubuntu < 22.04 to return 7.0.100 instead of 9.0.100 |
| vscode-dotnet-runtime-library/src/test/unit/LinuxDistroTests.test.ts | Changed test Ubuntu version from 24.04 to 22.04 to align with the new CI environment |
| vscode-dotnet-runtime-library/src/test/mocks/MockEnvironmentVariableCollection.ts | Added description property to match the current VS Code EnvironmentVariableCollection API |
JoeRobich
approved these changes
Feb 23, 2026
Member
Author
|
Thank you 🥳 |
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
We need to migrate to the new Linux image in CI so we can run tests on Linux again. #2557 This PR initially had the changes but it should really be an isolated fix - was intending to put it there to improve PR merge velocity.
The new linux image doesn't support some linux versions and is missing some vscode dependencies, which is also resolved in this PR.