-
Notifications
You must be signed in to change notification settings - Fork 10.5k
Add Windows.10.Amd64.Server20H1.Open helix queue #25627
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
Conversation
.azure/pipelines/helix-matrix.yml
Outdated
| pr: none | ||
| # Run PR validation on all branches | ||
| pr: | ||
| autoCancel: true | ||
| branches: |
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.
TODO: Revert
|
/azp run |
|
Azure Pipelines successfully started running 3 pipeline(s). |
dougbu
left a comment
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.
First commit works for me. Careful w/ "green" requirements though -- failures in the aspnetcore-helix-matrix pipeline won't prevent merging and blocked labels are just a hint to the user 😺
|
Hmm, Windows.10.Amd64.Server20H1.Open timed out. Apparently it needs more capacity. |
|
/azp run |
|
Azure Pipelines successfully started running 3 pipeline(s). |
|
/azp run |
|
Azure Pipelines successfully started running 3 pipeline(s). |
|
Skipped one set of tests: |
|
Windows.10.Amd64.Server20H1.Open is now passing with the one skipped diagnostics project. I've reverted the change to run the full helix pass on PRs. This should be ready to merge after a fresh CI run. |
|
@Pilchie Please approve/merge this test change for rc2. |
|
I'm going to force merge this, given that it shouldn't affect these tests. |
Let's see how well we work on new windows versions...