Skip to content

Conversation

MihaZupan
Copy link
Member

Fixes the build error described in #42211 (comment)

#114285 changes the apphost location variable, and this PR disables that for our stress builds since we get it from a locally downloaded nightly build instead.

@MihaZupan MihaZupan added this to the 10.0.0 milestone May 7, 2025
@MihaZupan MihaZupan self-assigned this May 7, 2025
@Copilot Copilot AI review requested due to automatic review settings May 7, 2025 01:46
Copy link
Contributor

@Copilot Copilot AI left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Copilot wasn't able to review any files in this pull request.

Files not reviewed (2)
  • src/libraries/System.Net.Http/tests/StressTests/HttpStress/Directory.Build.props: Language not supported
  • src/libraries/System.Net.Security/tests/StressTests/SslStress/Directory.Build.props: Language not supported

@dotnet-policy-service
Copy link
Contributor

Tagging subscribers to this area: @dotnet/ncl
See info in area-owners.md if you want to be subscribed.

@MihaZupan
Copy link
Member Author

/azp run runtime-libraries stress-http

@azure-pipelines
Copy link

Azure Pipelines successfully started running 1 pipeline(s).

@MihaZupan
Copy link
Member Author

/azp run runtime-libraries stress-ssl

@azure-pipelines
Copy link

Azure Pipelines successfully started running 1 pipeline(s).

Copy link
Member

@wfurt wfurt left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM

@MihaZupan MihaZupan merged commit fadf883 into dotnet:main May 7, 2025
90 of 93 checks passed
@github-actions github-actions bot locked and limited conversation to collaborators Jun 7, 2025
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants