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

Containers: add mappings for s390x and ppc64le platform architectures to corresponding rids. #33280

Merged
merged 1 commit into from
Jun 21, 2023

Conversation

tmds
Copy link
Member

@tmds tmds commented Jun 15, 2023

Rids for these architectures are defined in the rid graph, and Red Hat provides images for them, like ubi8/dotnet-70-runtime.

@baronfel ptal.

@dotnet-issue-labeler dotnet-issue-labeler bot added Area-Containers Related to dotnet SDK containers functionality untriaged Request triage from a team member labels Jun 15, 2023
@baronfel
Copy link
Member

Can you add this to #33166 and extend the tests that I added in that PR instead? I don't want to lose this and in that PR I extracted out the RID mapping to a separate component for reuse as a build task + easier testing.

@baronfel
Copy link
Member

Also, it's cool as heck that you all have such a broad image baseline!

@tmds
Copy link
Member Author

tmds commented Jun 15, 2023

Can you add this to #33166 and extend the tests that I added in that PR instead? I don't want to lose this and in that PR I extracted out the RID mapping to a separate component for reuse as a build task + easier testing.

Sure, I'll do that once that PR is merged.

@tmds
Copy link
Member Author

tmds commented Jun 16, 2023

Closing in favor of #33166.

@tmds tmds closed this Jun 16, 2023
@tmds
Copy link
Member Author

tmds commented Jun 21, 2023

@baronfel depending on when you think #33166 will land, would you consider merging this?

@baronfel baronfel reopened this Jun 21, 2023
@baronfel baronfel enabled auto-merge June 21, 2023 14:59
@baronfel
Copy link
Member

@tmds yeah we're going to take this now because 8.0.100-p6 snap is today. Should auto-merge once green. Thanks again for adding this :)

@baronfel baronfel merged commit e36fc30 into dotnet:main Jun 21, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Area-Containers Related to dotnet SDK containers functionality untriaged Request triage from a team member
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants