Skip to content

[release/10.0] [NET] Fix docker compose.#126473

Merged
wfurt merged 2 commits intodotnet:release/10.0from
ManickaP:fix-stress
Apr 24, 2026
Merged

[release/10.0] [NET] Fix docker compose.#126473
wfurt merged 2 commits intodotnet:release/10.0from
ManickaP:fix-stress

Conversation

@ManickaP
Copy link
Copy Markdown
Member

@ManickaP ManickaP commented Apr 2, 2026

Fixes broken stress by recent changes in used images.
Test infra only change, no product code affected.

Customer Impact

  • Customer reported
  • Found internally

Regression

  • Yes
  • No

Testing

Stress pipelines ran and passed.

Risk

Low, only test infra change, no product code change.

Copilot AI review requested due to automatic review settings April 2, 2026 14:59
@dotnet-policy-service
Copy link
Copy Markdown
Contributor

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

Copy link
Copy Markdown
Contributor

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.

Pull request overview

Updates System.Net stress-test Docker Compose runner scripts to address infra changes affecting compose invocation.

Changes:

  • Switch the bash runner to use docker compose (Compose v2 plugin) for build and up.
  • Update the PowerShell runner to use docker-compose (Compose v1) for build and up (including log/error text).

Reviewed changes

Copilot reviewed 2 out of 2 changed files in this pull request and generated 3 comments.

File Description
src/libraries/Common/tests/System/Net/StressTests/run-docker-compose.sh Uses docker compose for compose build/run.
src/libraries/Common/tests/System/Net/StressTests/run-docker-compose.ps1 Uses docker-compose for compose build/run and updates related messaging.

Comment thread src/libraries/Common/tests/System/Net/StressTests/run-docker-compose.ps1 Outdated
Comment thread src/libraries/Common/tests/System/Net/StressTests/run-docker-compose.ps1 Outdated
@karelz karelz added this to the 11.0.0 milestone Apr 14, 2026
@ManickaP ManickaP modified the milestones: 11.0.0, 10.0.x Apr 15, 2026
@ManickaP
Copy link
Copy Markdown
Member Author

@ManickaP
Copy link
Copy Markdown
Member Author

/ba-g unrelated, this touches only stress test infra

@ManickaP
Copy link
Copy Markdown
Member Author

Test only change -> Tell Mode -> Servicing approved.

@ManickaP ManickaP added the Servicing-approved Approved for servicing release label Apr 16, 2026
@ManickaP ManickaP requested review from a team and karelz April 16, 2026 12:35
Copy link
Copy Markdown
Member

@karelz karelz left a comment

Choose a reason for hiding this comment

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

Needed to unblock Stress tests in .NET 10 branch

@karelz karelz added the test-bug Problem in test source code (most likely) label Apr 21, 2026
@ManickaP
Copy link
Copy Markdown
Member Author

Gentle ping on this @wfurt

@wfurt wfurt merged commit 1b55544 into dotnet:release/10.0 Apr 24, 2026
109 of 114 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

area-System.Net.Http Servicing-approved Approved for servicing release test-bug Problem in test source code (most likely)

Projects

None yet

Development

Successfully merging this pull request may close these issues.

6 participants