Skip to content

Use prebuilt docker images instead of installing dependencies on every run - #9671

Merged
dibarbet merged 1 commit into
mainfrom
dibarbet-reuse-linux-ci-images
Aug 12, 2026
Merged

Use prebuilt docker images instead of installing dependencies on every run#9671
dibarbet merged 1 commit into
mainfrom
dibarbet-reuse-linux-ci-images

Conversation

@dibarbet

Copy link
Copy Markdown
Member

No description provided.

@dibarbet
dibarbet marked this pull request as ready for review August 12, 2026 18:35
@dibarbet
dibarbet requested a review from a team as a code owner August 12, 2026 18:35
Copilot AI lite review requested due to automatic review settings August 12, 2026 18:35

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

This PR updates the Azure Pipelines Linux test stages to rely on prebuilt container images that already contain required runtime prerequisites, removing the previous “install extra Linux dependencies inside the container” flow.

Changes:

  • Removed the installAdditionalLinuxDependencies parameter wiring and the test-linux-docker-prereqs.yml template usage.
  • Switched Linux test stages to use mcr.microsoft.com/dotnet-buildtools/prereqs:* images instead of mcr.microsoft.com/dotnet/sdk:*.
  • Deleted the no-longer-used azure-pipelines/test-linux-docker-prereqs.yml template.
Show a summary per file
File Description
azure-pipelines/test.yml Removes conditional Linux docker prereq install step and related parameter plumbing.
azure-pipelines/test-matrix.yml Removes the now-unused installAdditionalLinuxDependencies parameter passthrough.
azure-pipelines/test-linux-docker-prereqs.yml Deletes the template that installed extra VS Code runtime deps in the container via docker exec.
azure-pipelines.yml Updates Linux stages to use prebuilt prereqs container images for .NET 8–11 test runs.

Review details

💡 Add a code-review agent skill or configure MCP servers for context-aware, tailored reviews. Learn more in the docs.

Suppressed comments (3)

azure-pipelines.yml:109

  • The container image is referenced by a mutable tag. For build reproducibility and supply-chain safety, consider pinning the image to an immutable digest (e.g., ...@sha256:<digest>) so the CI environment can’t change unexpectedly.
      containerName: mcr.microsoft.com/dotnet-buildtools/prereqs:ubuntu-24.04-net9.0-vscode-csharp-amd64

azure-pipelines.yml:124

  • The container image is referenced by a mutable tag. For build reproducibility and supply-chain safety, consider pinning the image to an immutable digest (e.g., ...@sha256:<digest>) so the CI environment can’t change unexpectedly.
      containerName: mcr.microsoft.com/dotnet-buildtools/prereqs:ubuntu-24.04-net10.0-vscode-csharp-amd64

azure-pipelines.yml:139

  • The container image is referenced by a mutable tag. For build reproducibility and supply-chain safety, consider pinning the image to an immutable digest (e.g., ...@sha256:<digest>) so the CI environment can’t change unexpectedly.
      containerName: mcr.microsoft.com/dotnet-buildtools/prereqs:ubuntu-26.04-net11.0-vscode-csharp-amd64
  • Files reviewed: 4/4 changed files
  • Comments generated: 3
  • Review effort level: Lite

Comment thread azure-pipelines/test.yml
Comment thread azure-pipelines.yml
Comment thread azure-pipelines.yml
@dibarbet
dibarbet enabled auto-merge August 12, 2026 18:53
@dibarbet
dibarbet merged commit b0ed40a into main Aug 12, 2026
39 checks passed
@dibarbet
dibarbet deleted the dibarbet-reuse-linux-ci-images branch August 12, 2026 19:28
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.

3 participants