Skip to content

Conversation

@jkoritzinsky
Copy link
Member

@jkoritzinsky jkoritzinsky commented Apr 18, 2023

We don't run tests on official builds since repo consolidation in .NET 5. We used to run tests on official builds back in dotnet/coreclr, but we discontinued that practice. This YAML was all included in support of doing so. As a result, it's been dead for years and just clutters up our pipeline definitions and creates confusion.

Closes #57007

We don't run tests on official builds since repo consolidation in .NET 5. We used to run tests on official builds back in dotnet/coreclr, but we discontinued that practice. This YAML was all included in support of doing so. As a result, it's been dead for years and just clutters up our pipeline definitions and creates confusion.
@ghost ghost added the needs-area-label An area label is needed to ensure this gets routed to the appropriate area owners label Apr 18, 2023
@ghost ghost assigned jkoritzinsky Apr 18, 2023
@jkoritzinsky jkoritzinsky requested a review from sbomer April 18, 2023 23:03
@jkoritzinsky jkoritzinsky added area-Infrastructure-coreclr and removed needs-area-label An area label is needed to ensure this gets routed to the appropriate area owners labels Apr 18, 2023
@jkoritzinsky jkoritzinsky added this to the 8.0.0 milestone Apr 18, 2023
@ghost
Copy link

ghost commented Apr 18, 2023

Tagging subscribers to this area: @hoyosjs
See info in area-owners.md if you want to be subscribed.

Issue Details

We don't run tests on official builds since repo consolidation in .NET 5. We used to run tests on official builds back in dotnet/coreclr, but we discontinued that practice. This YAML was all included in support of doing so. As a result, it's been dead for years and just clutters up our pipeline definitions and creates confusion.

Author: jkoritzinsky
Assignees: jkoritzinsky
Labels:

area-Infrastructure-coreclr

Milestone: -

Copy link
Member

@sbomer sbomer left a comment

Choose a reason for hiding this comment

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

LGTM overall, just wanted to make sure I follow the reasoning behind some of the removals.

@jkoritzinsky jkoritzinsky merged commit 9a16a9d into dotnet:main Apr 19, 2023
@jkoritzinsky jkoritzinsky deleted the removed-unused-helix-queues branch April 19, 2023 19:30
@ghost ghost locked as resolved and limited conversation to collaborators May 20, 2023
Sign up for free to 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.

CoreCLR Runtime Tests aren't running on outerloop-only queues

2 participants