Skip to content

Fix s390x rootfs build - #7609

Merged
janvorli merged 1 commit into
dotnet:mainfrom
janvorli:fix-s390x-rootfs-build
Jul 12, 2021
Merged

Fix s390x rootfs build#7609
janvorli merged 1 commit into
dotnet:mainfrom
janvorli:fix-s390x-rootfs-build

Conversation

@janvorli

Copy link
Copy Markdown
Member

Ubuntu for s390x architecture doesn't have OMP libraries. Remove them from rootfs build for s390x. Support for this architecture is in a very experimental stage and the OMP is needed for ML .NET only.

Ubuntu for s390x architecture doesn't have OMP libraries.
@janvorli
janvorli requested a review from michaelgsharp July 12, 2021 11:34
@janvorli janvorli self-assigned this Jul 12, 2021
@janvorli

Copy link
Copy Markdown
Member Author

The Linux leg failure is unrelated to this change. It seems that it is trying to use image that was recently removed. @mthalman is that one of those?

2021-07-12T11:34:45.8207074Z ##[command]/usr/bin/docker network prune --force --filter "label=b93516"
2021-07-12T11:34:45.9350721Z ##[command]/usr/bin/docker pull microsoft/dotnet-buildtools-prereqs:ubuntu-14.04-cross-0cd4667-20170319080304
2021-07-12T11:34:46.3766430Z Error response from daemon: pull access denied for microsoft/dotnet-buildtools-prereqs, repository does not exist or may require 'docker login': denied: requested access to the resource is denied

@janvorli

Copy link
Copy Markdown
Member Author

Ah, I can see that a pending PR #7608 is fixing that issue.

@michaelgsharp michaelgsharp left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM. :shipit:

@janvorli janvorli closed this Jul 12, 2021
@janvorli janvorli reopened this Jul 12, 2021
@janvorli
janvorli merged commit b59fb36 into dotnet:main Jul 12, 2021
@janvorli
janvorli deleted the fix-s390x-rootfs-build branch July 12, 2021 20:07
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