Skip to content

Refactor Linux Dockerfile templates to make changing shas easier #6500

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Merged
merged 12 commits into from
Jun 26, 2025

Conversation

lbussell
Copy link
Contributor

@lbussell lbussell commented Jun 24, 2025

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 benjaminpetit, a team, 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
114 checks passed
@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