-
Notifications
You must be signed in to change notification settings - Fork 2.6k
[release/3.1] Enable additional CI testing #28149
Conversation
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.
LGTM
It looks good, and it looks like the tests ran, but can you be sure? The Windows arm64 tests don't show up on the "Tests" results tab: https://dev.azure.com/dnceng/public/_build/results?buildId=999073&view=ms.vss-test-web.build-test-results-tab Can you find one/some console logs in Kusto (or elsewhere)? |
It looks like Windows arm64 Pri-0 tests don't do R2R? Maybe R2R was limited in 3.1 testing? |
I had exactly the same question earlier today and had already validated that. I should've mentioned this in the PR. I copied output from work items corresponding to JIT.Regression test collection that ran on the new Helix queue Test Pri0 Windows_NT arm64 checked
Test Pri1 Windows_NT arm64
Test Pri1 R2R Windows_NT arm64 checked
I believe the reason why we can't see the test results in AzDO Tests tab is due to
I opened https://github.com/dotnet/core-eng/issues/12223 |
Not sure (remember) why it was configured that way. |
Thanks for double checking they are running; looks good.
yeah, that's fine. |
Found out that gcstress, r2r and corefx* pipelines don't use the new queue - updated a bunch of yml files to fix this. |
Given this is an infrastructure only change, we can take via tell mode. Please coordinate with Matt Mitchel and Anirudh Agnihotry once we are ready to merge and they can help us with timing. Post merging we can send a tell mode notification. Thanks @echesakovMSFT for prepping this change. |
It's either going to be March 9th or early next week. |
@leecow I am going to mark this as servicing-approved just so that we do not get missed on the next opportunity to merge. If there is a concern, please feel free to remove from this infra only change. |
There are failing tests, re-run... |
@aik-jahoda You would observe the same test failures when running against release/3.1 branch. Note that the change doesn't affect platforms other than win-arm64. |
Merging this one in respone to
|
Enable additional CI testing using new Helix queue Windows.10.Arm64v8.Open.
Customer Impact
None
Testing
Validated the change by manually triggering two pipelines coreclr-ci#20210217.2 and coreclr-outerloop#20210217.1
Risk
None - the change affects testing infrastructure only
cc @BruceForstall @JulieLeeMSFT @jeffschwMSFT