Skip to content

Refactor Linux Dockerfile templates to make changing shas easier - #6500

Merged
lbussell merged 12 commits into
dotnet:nightlyfrom
lbussell:online-checksums
Jun 26, 2025
Merged

lbussell merged 12 commits into
dotnet:nightlyfrom
lbussell:online-checksums

Conversation

@lbussell

@lbussell lbussell commented Jun 24, 2025 •

Copy link
Copy Markdown
Member

The goal of this PR is to refactor Dockerfile templates to make the following two tasks easier:

Summary:

  • Simplified/re-implemented templates for downloading and extracting .NET archives
    • Added support for "sha-url" and "sha-url-aggregate" arguments to the new download-dotnet template. So when we want to switch to online shas, we only need to change the arguments to download-dotnet.
  • Removed lots of unused code and conditions
  • Inlined some templates in some of the simpler appliance Dockerfiles

There should be no functional changes here. There are a few minor stylistic changes:

  • ASP.NET Core Composite images are now always multi-stage, just like every other .NET Dockerfile
  • .NET Monitor extension files use a combined curl command to download both extension binaries
  • Internal Dockerfiles lost the quotes around the download URL 🤷‍♂️

@lbussell
lbussell requested review from a team, benjaminpetit, joperezr and wiktork as code owners June 24, 2025 21:39
@lbussell
lbussell enabled auto-merge (squash) June 26, 2025 15:16
@lbussell
lbussell merged commit 53f2ce6 into dotnet:nightly Jun 26, 2025
@lbussell
lbussell deleted the online-checksums branch June 26, 2025 17:34
lbussell added a commit to lbussell/dotnet-docker that referenced this pull request Jul 7, 2025
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