Skip to content
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

[release/8.0.4xx] Allow reading of OCI Image Indexes to determine RID-specific base image for multi-architecture images #43914

Merged
merged 2 commits into from
Oct 7, 2024

Conversation

github-actions[bot]
Copy link
Contributor

@github-actions github-actions bot commented Oct 4, 2024

Backport of #43631 to release/8.0.4xx

Description

The SDK Container tools didn't know how to read a rapidly-becoming-common format of container images called an OCI Image Index, and this presented to users as a lookup failure - this is very confusing when you can look at the container registry and in fact see that the image you want to use is present! Since for our purposes this structure is very similar to the existing multi-arch lookups that we already understand, this change implements a minimal reading of the OCI Image Index structure to unblock users while we continue to work on our support for authoring these Image Index structures.

Customer Impact

Without this change, users who create custom base images or pull images from feeds that are OCI multi-architecture images would just fail. The only workaround is to manually download and re-publish architecture-specific variants of the images you want to use, which is not always possible and a real roadblock to adoption. While the Microsoft base images are not OCI Image Indexes (yet) this is an area of active discussion and ~15% of all container publishing uses images other than the Microsoft-produced dotnet base images, so that cohort would be exposed to this break happening at any time.

Regression?

No - this is a feature that we hadn't yet implemented but were going to as part of the multi-arch support that we are still working on

Risk

Low - this doesn't impact our other base image format support, only expands the set of images we can support. We also have done manual testing verifying this support, and will add automated testing in main.

@dotnet-issue-labeler dotnet-issue-labeler bot added Area-Infrastructure untriaged Request triage from a team member labels Oct 4, 2024
@baronfel baronfel self-assigned this Oct 4, 2024
@baronfel
Copy link
Member

baronfel commented Oct 7, 2024

Approved over Tactics

@baronfel baronfel merged commit 3d57238 into release/8.0.4xx Oct 7, 2024
17 checks passed
@baronfel baronfel deleted the backport/pr-43631-to-release/8.0.4xx branch October 7, 2024 19:27
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant