Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Revert "Update perf run yaml with changes for crank on helix (#90368)" #90424

Merged
merged 1 commit into from
Aug 11, 2023

Conversation

radical
Copy link
Member

@radical radical commented Aug 11, 2023

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

…90368)"

This reverts commit bbe40f8.

This caused perf pipeline runs to break.

Issue: dotnet#90420
@radical radical added the arch-wasm WebAssembly architecture label Aug 11, 2023
@ghost ghost assigned radical Aug 11, 2023
@ghost
Copy link

ghost commented Aug 11, 2023

Tagging subscribers to 'arch-wasm': @lewing
See info in area-owners.md if you want to be subscribed.

Issue Details

This reverts commit bbe40f8.

This caused perf pipeline runs to break.

Issue: #90420

Author: radical
Assignees: -
Labels:

arch-wasm

Milestone: -

@radical
Copy link
Member Author

radical commented Aug 11, 2023

Staging this, in case the proper fix will take time.

@ghost
Copy link

ghost commented Aug 11, 2023

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

Issue Details

This reverts commit bbe40f8.

This caused perf pipeline runs to break.

Issue: #90420

Author: radical
Assignees: radical
Labels:

area-Infrastructure

Milestone: -

@radical radical added the perf-pipeline Issues with dotnet-runtime-perf, or runtime-wasm-perf pipelines label Aug 11, 2023
@caaavik-msft
Copy link
Contributor

Reverting this change will break as well unless we revert the changes in the performance repository too, I will work on a proper fix right now

@radical radical added the NO-MERGE The PR is not ready for merge yet (see discussion for detailed reasons) label Aug 11, 2023
@radical radical removed the NO-MERGE The PR is not ready for merge yet (see discussion for detailed reasons) label Aug 11, 2023
@caaavik-msft
Copy link
Contributor

I have made a PR to performance that removes the dependency on dataclasses library so that we can continue with this PR and revert the changes: dotnet/performance#3251
Once both PRs are merged things should be working again

@radical
Copy link
Member Author

radical commented Aug 11, 2023

This will not get tested by CI here. Merging it.

@radical radical merged commit 0916f7b into dotnet:main Aug 11, 2023
10 of 71 checks passed
@radical radical deleted the revert-to-fix-perf-pipeline branch August 11, 2023 19:16
@ghost ghost locked as resolved and limited conversation to collaborators Sep 11, 2023
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
area-Infrastructure perf-pipeline Issues with dotnet-runtime-perf, or runtime-wasm-perf pipelines
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants