Skip to content
Merged
Changes from 1 commit
Commits
File filter

Filter by extension

Filter by extension

Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
4 changes: 2 additions & 2 deletions eng/pipelines/common/templates/pipeline-with-resources.yml
Original file line number Diff line number Diff line change
Expand Up @@ -15,9 +15,9 @@ resources:
ROOTFS_DIR: /crossrootfs/armv6

- container: linux_arm64
image: mcr.microsoft.com/dotnet-buildtools/prereqs:ubuntu-18.04-cross-arm64
image: mcr.microsoft.com/dotnet-buildtools/prereqs:ubuntu-20.04-cross-arm64
env:
ROOTFS_DIR: /crossrootfs/arm64
ROOTFS_DIR: /crossrootfs
Copy link
Member

@am11 am11 Jan 21, 2023

Choose a reason for hiding this comment

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

This convention is documented in number of places. Lets not introduce a break because everyone using these prereq images outside this CI (for years) remember the default location of ROOTFS_DIR.

eng/pipelines/common/templates/pipeline-with-resources.yml:        ROOTFS_DIR: /crossrootfs/arm
eng/pipelines/common/templates/pipeline-with-resources.yml:        ROOTFS_DIR: /crossrootfs/armv6
eng/pipelines/common/templates/pipeline-with-resources.yml:        ROOTFS_DIR: /crossrootfs/arm64
eng/pipelines/common/templates/pipeline-with-resources.yml:        ROOTFS_DIR: /crossrootfs/arm
eng/pipelines/common/templates/pipeline-with-resources.yml:        ROOTFS_DIR: /crossrootfs/arm64
eng/pipelines/common/templates/pipeline-with-resources.yml:        ROOTFS_DIR: /crossrootfs/x86
eng/pipelines/common/templates/pipeline-with-resources.yml:        ROOTFS_DIR: /crossrootfs/s390x
eng/pipelines/common/templates/pipeline-with-resources.yml:        ROOTFS_DIR: /crossrootfs/ppc64le
eng/pipelines/common/templates/pipeline-with-resources.yml:        ROOTFS_DIR: /crossrootfs/x64
eng/pipelines/common/templates/pipeline-with-resources.yml:        ROOTFS_DIR: /crossrootfs/armel

Copy link
Member Author

Choose a reason for hiding this comment

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

When I updated the build images, the change to the rootfs dir was unintentional. Fixing it in dotnet/dotnet-buildtools-prereqs-docker#781.

Copy link
Member Author

Choose a reason for hiding this comment

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

I will update dotnet/runtime once the fix is rolled out.


- container: linux_musl_x64
image: mcr.microsoft.com/dotnet-buildtools/prereqs:alpine-3.13-WithNode
Expand Down