Skip to content

Conversation

@caaavik-msft
Copy link
Contributor

As part of the work to get crank on helix in the performance repository, we need to install the dataclasses pip package on the agents. This PR needs to be merged in after the Crank runner for Helix change is merged

@ghost
Copy link

ghost commented Aug 11, 2023

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

Issue Details

As part of the work to get crank on helix in the performance repository, we need to install the dataclasses pip package on the agents. This PR needs to be merged in after the Crank runner for Helix change is merged

Author: caaavik-msft
Assignees: caaavik-msft
Labels:

area-Infrastructure-coreclr

Milestone: -

Copy link
Member

@DrewScoggins DrewScoggins left a comment

Choose a reason for hiding this comment

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

LGTM

@caaavik-msft caaavik-msft merged commit bbe40f8 into dotnet:main Aug 11, 2023
radical added a commit to radical/runtime that referenced this pull request Aug 11, 2023
…90368)"

This reverts commit bbe40f8.

This caused perf pipeline runs to break.

Issue: dotnet#90420
radical added a commit that referenced this pull request Aug 11, 2023
#90424)

This reverts commit bbe40f8.

This caused perf pipeline runs to break. But dotnet/performance#3251 from @caaavik-msft, along with this revert will fix the issue.

Issue: #90420
@ghost ghost locked as resolved and limited conversation to collaborators Sep 10, 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.

3 participants