-
Notifications
You must be signed in to change notification settings - Fork 10.3k
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
Fix some issues causing aspnetcore-helix-matrix failures #26840
Conversation
I manually triggered the aspnetcore-helix-matrix pipeline on the halter73/fix-helix-matrix PR branch and now it's showing up under the PR checks. I wonder if GitHub will block merging now if it fails 🤔 |
@@ -43,7 +43,7 @@ | |||
<ItemGroup Condition="'$(TargetArchitecture)' == 'arm64' AND '$(IsHelixDaily)' == 'true' AND '$(IsWindowsOnlyTest)' != 'true'"> | |||
<HelixAvailableTargetQueue Include="(Debian.9.Arm64.Open)Ubuntu.1804.Armarch.Open@mcr.microsoft.com/dotnet-buildtools/prereqs:debian-9-helix-arm64v8-a12566d-20190807161036" Platform="Linux" /> | |||
</ItemGroup> | |||
<ItemGroup Condition="'$(TargetArchitecture)' == 'arm64' AND '$(IsHelixDaily)' == 'true'"> | |||
<ItemGroup Condition="'$(IsWindowsOnlyTest)' != 'true' AND '$(TargetArchitecture)' == 'arm64' AND '$(IsHelixDaily)' == 'true'"> |
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.
Is there an issue tracking this part? The initial error looked like an easy fix in TestMatrix.
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.
I don't believe so, @halter73 could you please confirm? If not, I'll create a new one.
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.
Thanks @TanayParikh, but I got it. #26872
> [0.019s] [Microsoft.AspNetCore.Server.Kestrel.Https.Internal.HttpsConnectionMiddleware] [Information] HTTP/2 over TLS is not supported on Windows versions older than Windows 10 and Windows Server 2016 due to incompatible ciphers or missing ALPN support. Falling back to HTTP/1.1 instead.
…ficate on Windows.10.Arm64.Open
3242ec7
to
30cc292
Compare
The recent aspnetcore-helix-matrix failures are related to https://github.com/dotnet/core-eng/issues/10962. I'm merging this since all the PR checks passed. |
Thanks @halter73! |
Example failures:
https://dev.azure.com/dnceng/public/_build/results?buildId=848189&view=ms.vss-test-web.build-test-results-tab