-
Notifications
You must be signed in to change notification settings - Fork 4.9k
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
Use latest released SDK for HttpStress. #104713
Conversation
This PR removes the .1xx suffix on the channel of the installed SDK to allow installing more recent SDKs.
/azp run runtime-libraries stress-http |
Tagging subscribers to this area: @dotnet/ncl |
Azure Pipelines successfully started running 1 pipeline(s). |
Not for ssl stress? runtime/src/libraries/System.Net.Security/tests/StressTests/SslStress/build-local.sh Line 43 in 0d426df
runtime/src/libraries/System.Net.Security/tests/StressTests/SslStress/Build-Local.ps1 Line 41 in 0d426df
|
/azp run runtime-libraries stress-http |
/azp run runtime-libraries stress-ssl |
Azure Pipelines successfully started running 1 pipeline(s). |
1 similar comment
Azure Pipelines successfully started running 1 pipeline(s). |
:( |
The received message is cut:
😢 |
This is weird, according to versions, sha, and other indicators the pulled in runtime should already contain the fix. I will investigate. |
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.
Let's merge it, but I think we should re-open the HTTP stress issue with content mismatch until we know what's going on.
Reopened #76183 |
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.
Let's merge it
OK!
/ba-g Test failures are unrelated. |
This PR removes the .1xx suffix on the channel of the installed SDK to allow installing more recent SDKs.