Skip to content

[HTTP/SSL] Revert docker-compose command changes from PR #125995 - #126399

Merged
ManickaP merged 4 commits into
dotnet:mainfrom
ManickaP:revert-docker-compose-changes-125995
Apr 2, 2026
Merged

[HTTP/SSL] Revert docker-compose command changes from PR #125995#126399
ManickaP merged 4 commits into
dotnet:mainfrom
ManickaP:revert-docker-compose-changes-125995

Conversation

@ManickaP

@ManickaPManickaP commented Apr 1, 2026

Copy link
Copy Markdown
Member

Revert docker-compose command changes from #125995

Note

This PR was AI/Copilot-generated.

Reverts the docker-compose → docker compose command changes introduced in #125995, while keeping the Azure Linux 3 image pool swap (Build.Ubuntu.2204.Amd64.Open → build.azurelinux.3.amd64.open).

What is reverted

All docker-compose (hyphenated) → docker compose (space-separated) changes across 5 files:

  • eng/pipelines/libraries/enterprise/linux.yml
  • eng/pipelines/libraries/stress/http.yml
  • eng/pipelines/libraries/stress/ssl.yml
  • src/libraries/Common/tests/System/Net/StressTests/run-docker-compose.ps1
  • src/libraries/Common/tests/System/Net/StressTests/run-docker-compose.sh

What is kept

The image pool migration in the 3 pipeline YAML files:
Build.Ubuntu.2204.Amd64.Open → build.azurelinux.3.amd64.open

Ref #125995, #125748, #125690

CopilotAI review requested due to automatic review settings April 1, 2026 09:22
@dotnet-policy-service

dotnet-policy-serviceBot commented Apr 1, 2026

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.

CopilotAI reviewed Apr 1, 2026
View reviewed changes

CopilotAI left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Pull request overview

This PR reverts the docker compose (v2-style) invocations back to docker-compose (hyphenated) for the HTTP/SSL stress and enterprise networking pipelines and helper scripts, while preserving the move to the Azure Linux 3 public build pool image.

Changes:

  • Revert docker compose back to docker-compose in enterprise + stress pipeline YAMLs.
  • Revert docker compose back to docker-compose in shared stress test runner scripts (.sh / .ps1).
  • Keep the updated Linux pool image (build.azurelinux.3.amd64.open) in the affected pipelines.

Reviewed changes

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

Show a summary per file
FileDescription
eng/pipelines/libraries/enterprise/linux.ymlUses docker-compose for enterprise test environment lifecycle on Azure Linux 3 pool.
eng/pipelines/libraries/stress/http.ymlUses docker-compose for HttpStress container orchestration steps.
eng/pipelines/libraries/stress/ssl.ymlUses docker-compose for SslStress container orchestration steps.
src/libraries/Common/tests/System/Net/StressTests/run-docker-compose.ps1Switches stress runner back to docker-compose for build/up operations.
src/libraries/Common/tests/System/Net/StressTests/run-docker-compose.shSwitches stress runner back to docker-compose for build/up operations.

Comment threadsrc/libraries/Common/tests/System/Net/StressTests/run-docker-compose.sh Outdated
Comment threadsrc/libraries/Common/tests/System/Net/StressTests/run-docker-compose.ps1
@ManickaP
Revert docker-compose command changes from PRdotnet#125995
3f3edaa
Reverts the docker-compose -> docker compose command changes introduced
in dotnet#125995, while keeping the Azure Linux 3 image pool swap.
Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com>
@ManickaP
ManickaPforce-pushed the revert-docker-compose-changes-125995 branch from 390c660 to 3f3edaaCompareApril 1, 2026 09:33
CopilotAI review requested due to automatic review settings April 1, 2026 09:36
CopilotAI reviewed Apr 1, 2026
View reviewed changes

CopilotAI left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Pull request overview

Copilot reviewed 5 out of 5 changed files in this pull request and generated 1 comment.

Comment threadeng/pipelines/libraries/stress/ssl.yml Outdated
CopilotAI review requested due to automatic review settings April 1, 2026 09:49
@ManickaP
ManickaPforce-pushed the revert-docker-compose-changes-125995 branch from bc1703d to e4746d5CompareApril 1, 2026 09:49
CopilotAI reviewed Apr 1, 2026
View reviewed changes

CopilotAI left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Pull request overview

Copilot reviewed 5 out of 5 changed files in this pull request and generated 4 comments.

Comment threadeng/pipelines/libraries/stress/http.yml Outdated
Comment threadeng/pipelines/libraries/stress/http.yml
Comment threadeng/pipelines/libraries/stress/ssl.yml Outdated
Comment threadeng/pipelines/libraries/stress/ssl.yml
@ManickaP
ManickaPforce-pushed the revert-docker-compose-changes-125995 branch from e4746d5 to ee7d3f8CompareApril 1, 2026 09:58
@ManickaP

ManickaP commented Apr 1, 2026

Copy link
Copy Markdown
MemberAuthor

/azp run runtime-libraries stress-http

@ManickaP

ManickaP commented Apr 1, 2026

Copy link
Copy Markdown
MemberAuthor

/azp run runtime-libraries stress-ssl

@azure-pipelines

azure-pipelinesBot commented Apr 1, 2026

Copy link
Copy Markdown
Azure Pipelines successfully started running 1 pipeline(s).

1 similar comment
@azure-pipelines

azure-pipelinesBot commented Apr 1, 2026

Copy link
Copy Markdown
Azure Pipelines successfully started running 1 pipeline(s).

CopilotAI review requested due to automatic review settings April 1, 2026 12:12
@ManickaP

ManickaP commented Apr 1, 2026

Copy link
Copy Markdown
MemberAuthor

/azp run runtime-libraries stress-http

@ManickaP

ManickaP commented Apr 1, 2026

Copy link
Copy Markdown
MemberAuthor

/azp run runtime-libraries stress-ssl

@azure-pipelines

azure-pipelinesBot commented Apr 1, 2026

Copy link
Copy Markdown
Azure Pipelines successfully started running 1 pipeline(s).

1 similar comment
@azure-pipelines

azure-pipelinesBot commented Apr 1, 2026

Copy link
Copy Markdown
Azure Pipelines successfully started running 1 pipeline(s).

CopilotAI reviewed Apr 1, 2026
View reviewed changes

CopilotAI left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Pull request overview

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

Comment threadeng/pipelines/libraries/stress/http.yml
Comment threadeng/pipelines/libraries/stress/ssl.yml
Comment threadsrc/libraries/Common/tests/System/Net/StressTests/run-docker-compose.ps1
@ManickaP

ManickaP commented Apr 1, 2026

Copy link
Copy Markdown
MemberAuthor

/azp run runtime-libraries stress-ssl

@ManickaP

ManickaP commented Apr 1, 2026

Copy link
Copy Markdown
MemberAuthor

/azp run runtime-libraries stress-http

@azure-pipelines

azure-pipelinesBot commented Apr 1, 2026

Copy link
Copy Markdown
Azure Pipelines successfully started running 1 pipeline(s).

1 similar comment
@azure-pipelines

azure-pipelinesBot commented Apr 1, 2026

Copy link
Copy Markdown
Azure Pipelines successfully started running 1 pipeline(s).

This was referenced Apr 1, 2026
Open
Open
Closed
Closed
Open
@ManickaP
ManickaP requested a review from a teamApril 2, 2026 08:27
@ManickaP

ManickaP commented Apr 2, 2026

Copy link
Copy Markdown
MemberAuthor

Both stress pipeline are green. I'm also trying to address/fix the PR trigger, which for some reason did not trigger on this PR.

@ManickaP

ManickaP commented Apr 2, 2026

Copy link
Copy Markdown
MemberAuthor

/ba-g unrelated

@ManickaP
ManickaP enabled auto-merge (squash) April 2, 2026 08:29
@ManickaP
ManickaP merged commit cc1ae2b into dotnet:mainApr 2, 2026
115 of 125 checks passed
@ManickaP
ManickaP deleted the revert-docker-compose-changes-125995 branch April 2, 2026 08:39
This was referenced Apr 2, 2026
Closed
Merged
@github-actionsgithub-actionsBot locked and limited conversation to collaborators May 2, 2026
Sign up for freeto subscribe to this conversation on GitHub. Already have an account? Sign in.

Reviewers

Copilot code reviewCopilot
Copilot review effort, defaults to Lite
Applies to this pull request for everyone.Learn more about Copilot code review.
Copilot left review comments
@cincuranetcincuranetcincuranet approved these changes

Assignees

@ManickaPManickaP

Labels

area-System.Net.Http

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

3 participants

@ManickaP@cincuranet