Skip to content

[release/6.0] Use OutputRid in test projects - #58340

Merged
jeffschwMSFT merged 1 commit into
release/6.0from
backport/pr-58166-to-release/6.0
Aug 31, 2021
Merged

[release/6.0] Use OutputRid in test projects#58340
jeffschwMSFT merged 1 commit into
release/6.0from
backport/pr-58166-to-release/6.0

Conversation

@github-actions

@github-actionsgithub-actionsBot commented Aug 30, 2021

Copy link
Copy Markdown
Contributor

Properly pass the RID to the CORE_ROOT generation logic.

Backport of #58166 to release/6.0

/cc @hoyosjs@am11

Customer Impact

Without this, testing arm variants in outerloop for servicing is not possible.

Testing

CI tests

Risk

Really low, Test only impact.

@ghost

Copy link
Copy Markdown

I couldn't figure out the best area label to add to this PR. If you have write-permissions please help me learn by adding exactly one area label.

@ghost

Copy link
Copy Markdown

Tagging subscribers to this area: @dotnet/runtime-infrastructure
See info in area-owners.md if you want to be subscribed.

Issue Details

Backport of #58166 to release/6.0

/cc @hoyosjs@am11

Customer Impact

Testing

Risk

Author:github-actions[bot]
Assignees:-
Labels:

area-Infrastructure

Milestone:-

@hoyosjs

Copy link
Copy Markdown
Member

/azp run runtime-coreclr outerloop

@azure-pipelines

Copy link
Copy Markdown
Azure Pipelines successfully started running 1 pipeline(s).

@jeffschwMSFTjeffschwMSFT left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

Approved. Please request a code review and once we have a green CI we can merge.

@jeffschwMSFT
jeffschwMSFT merged commit 9f6ad57 into release/6.0Aug 31, 2021
@akoeplinger
akoeplinger deleted the backport/pr-58166-to-release/6.0 branch September 9, 2021 11:26
@ghostghost locked as resolved and limited conversation to collaborators Oct 9, 2021
Sign up for freeto subscribe to this conversation on GitHub. Already have an account? Sign in.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

5 participants

@hoyosjs@akoeplinger@jeffschwMSFT@danmoseley@am11