-
Notifications
You must be signed in to change notification settings - Fork 4.1k
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
Update .NET 9 runtime for single machine CI job #75889
Conversation
575ba40
to
fe015c6
Compare
@@ -33,7 +33,7 @@ jobs: | |||
displayName: 'Install .NET 9 Runtime' | |||
inputs: | |||
packageType: runtime | |||
version: 9.0.0-preview.3.24172.9 | |||
version: 9.0.0-rc.2.24473.5 |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Man I wish it were more straightforward to flow the sdk version from global.json here. This is obviously prone to fall out of date, but, I'm not sure we can justify the investigation effort at this point either.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
There is a useGlobalJson task parameter but surely no use if global.json is not on disk on the test machine.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Good point, I might look into this later (perhaps when arcade bumps SDK and I will need to do this again). But for now, the tests are actually failing on some assert, so I'm going to prioritize investigating that.
@sharwell @tmat it looks like the test
And it only fails on the Single_Machine leg. There aren't any details about the error. Any ideas what to do about it? The last build where it worked is https://dev.azure.com/dnceng-public/public/_build/results?buildId=866953. It was running on a machine with .NET runtime 9.0.0-preview.3.24172.9 and VS 17.11.0 Preview 2.1. Then the CI machine updated to VS 17.12.0 Preview 4 it seems and so I needed to update the .NET runtime to 9.0.100-rc.2.24474.11 (see the error in this PR's description). |
* Update .NET 9 runtime for single machine CI job * Use regular pool
Fixes this error in the single machine CI leg (which runs only in main branch):