Skip to content

Use online checksums for Linux .NET images #6508

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 14 commits into from
Jul 2, 2025

Conversation

lbussell
Copy link
Contributor

Part of #6495. This PR is based on #6506, so this is in draft mode for now.

This replaces hardcoded checksums for images matching all of the following criteria

  • Linux
  • .NET 8, 9, and 10
  • runtime, aspnet, and sdk

Work still remaining:

@lbussell
Copy link
Contributor Author

The reason some images are failing to build is because of non-normalized line endings in the aggregate sha files (.NET 8/9).

@richlander
Copy link
Member

richlander commented Jun 28, 2025

Can you file an issue on that?

I wrote a document that mentioned this topic area (not specific to checksums). I made an issue for the top-level concern.

@lbussell
Copy link
Contributor Author

Can you file an issue on that?

@richlander I filed dotnet/core#9958

@lbussell lbussell force-pushed the online-checksums-2 branch from 13e3cc7 to 5f55811 Compare June 30, 2025 17:00
@lbussell
Copy link
Contributor Author

(rebased onto nightly since #6506 was merged)

@lbussell lbussell marked this pull request as ready for review July 1, 2025 04:06
@lbussell lbussell requested a review from a team as a code owner July 1, 2025 04:06
@lbussell lbussell requested a review from mthalman July 1, 2025 19:38
@lbussell lbussell requested review from mthalman and richlander July 1, 2025 20:56
@lbussell lbussell enabled auto-merge (squash) July 2, 2025 14:54
@lbussell
Copy link
Contributor Author

lbussell commented Jul 2, 2025

archive.ubuntu.com had an outage this morning (https://status.canonical.com/). Looks like it's still being very slow (with the one job left on this PR...). I may merge around this since the build got past installing .NET successfully.

@lbussell lbussell disabled auto-merge July 2, 2025 17:22
@lbussell lbussell merged commit eeedad0 into dotnet:nightly Jul 2, 2025
110 of 112 checks passed
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.

3 participants