Skip to content

Commit

Permalink
Containers: add mappings for s390x and ppc64le platform architectures…
Browse files Browse the repository at this point in the history
… to corresponding rids. (#33280)
  • Loading branch information
baronfel authored Jun 21, 2023
2 parents 819d998 + 79e4d91 commit e36fc30
Showing 1 changed file with 2 additions and 2 deletions.
4 changes: 2 additions & 2 deletions src/Containers/Microsoft.NET.Build.Containers/Registry.cs
Original file line number Diff line number Diff line change
Expand Up @@ -265,8 +265,6 @@ private async Task<HttpResponseMessage> GetBlobAsync(string repositoryName, stri
private static string? CreateRidForPlatform(PlatformInformation platform)
{
// we only support linux and windows containers explicitly, so anything else we should skip past.
// there are theoretically other platforms/architectures that Docker supports (s390x?), but we are
// deliberately ignoring them without clear user signal.
var osPart = platform.os switch
{
"linux" => "linux",
Expand All @@ -286,6 +284,8 @@ private async Task<HttpResponseMessage> GetBlobAsync(string repositoryName, stri
"x386" => "x86",
"arm" => $"arm{(platform.variant != "v7" ? platform.variant : "")}",
"arm64" => "arm64",
"ppc64le" => "ppc64le",
"s390x" => "s390x",
_ => null
};

Expand Down

0 comments on commit e36fc30

Please sign in to comment.