-
Notifications
You must be signed in to change notification settings - Fork 4.8k
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
Wasm perf runs are completely broken due to recent changes #86001
Comments
Tagging subscribers to this area: @dotnet/runtime-infrastructure Issue DetailsThis has been broken since May 8, like https://dev.azure.com/dnceng/internal/_build/results?buildId=2176141&view=results .
This seems to be broken since #85915 .
|
Thank you for making this issue to track this, I just merged something to hopefully fix this: b6f6dd0. I will watch to make sure that this fix also fixes the wasm runs, but it will take a little bit to get to that point in the current run. |
That latest update b92c1d2 seems to have fixed the prereq install issues. Waiting for the run to finish to see if there are any downstream issues. |
@LoopedBard3 iOS perf measurements are still broken. Here is one of recent test runs - https://helixri107v0xdeko0k025g8.blob.core.windows.net/dotnet-runtime-refs-heads-main-d59c35a3d23442aea5/SOD%20-%20iOS%20HelloWorld%20.app%20Size/1/console.fc85fc43.log?sv=2021-08-06&se=2023-08-08T06%3A47%3A56Z&sr=c&sp=rl&sig=7bOj249carhodQa0xoi5YALZ9%2BCYkt3LAdJDb3ZJvcI%3D.
|
Added tracking specific for the iOS scenarios to this issue: dotnet/performance#3010. This should allow us to close this issue if there are no more broken WASM runs that I am missing, the main ones appear to be fixed. |
@radical I suggest we close this issue as the issue has been fixed. |
This has been broken since May 8, like https://dev.azure.com/dnceng/internal/_build/results?buildId=2176141&view=results .
This seems to be broken since #85915 .
cc @LoopedBard3 @kotlarmilos @lewing
Known Issue Error Message
Fill the error message using known issues guidance.
Report
Summary
The text was updated successfully, but these errors were encountered: