Skip to content

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

Merged
ManickaP merged 4 commits intodotnet: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 intodotnet:mainfrom
ManickaP:revert-docker-compose-changes-125995

Conversation

@ManickaP
Copy link
Copy Markdown
Member

@ManickaP ManickaP commented Apr 1, 2026

Revert docker-compose command changes from #125995

Note

This PR was AI/Copilot-generated.

Reverts the docker-composedocker compose command changes introduced in #125995, while keeping the Azure Linux 3 image pool swap (Build.Ubuntu.2204.Amd64.Openbuild.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.Openbuild.azurelinux.3.amd64.open

Ref #125995, #125748, #125690

Copilot AI review requested due to automatic review settings April 1, 2026 09:22
@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

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
File Description
eng/pipelines/libraries/enterprise/linux.yml Uses docker-compose for enterprise test environment lifecycle on Azure Linux 3 pool.
eng/pipelines/libraries/stress/http.yml Uses docker-compose for HttpStress container orchestration steps.
eng/pipelines/libraries/stress/ssl.yml Uses docker-compose for SslStress container orchestration steps.
src/libraries/Common/tests/System/Net/StressTests/run-docker-compose.ps1 Switches stress runner back to docker-compose for build/up operations.
src/libraries/Common/tests/System/Net/StressTests/run-docker-compose.sh Switches stress runner back to docker-compose for build/up operations.

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 ManickaP force-pushed the revert-docker-compose-changes-125995 branch from 390c660 to 3f3edaa Compare April 1, 2026 09:33
Copilot AI review requested due to automatic review settings April 1, 2026 09:36
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

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

Copilot AI review requested due to automatic review settings April 1, 2026 09:49
@ManickaP ManickaP force-pushed the revert-docker-compose-changes-125995 branch from bc1703d to e4746d5 Compare April 1, 2026 09:49
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

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

@ManickaP ManickaP force-pushed the revert-docker-compose-changes-125995 branch from e4746d5 to ee7d3f8 Compare April 1, 2026 09:58
@ManickaP
Copy link
Copy Markdown
Member Author

ManickaP commented Apr 1, 2026

/azp run runtime-libraries stress-http

@ManickaP
Copy link
Copy Markdown
Member Author

ManickaP commented Apr 1, 2026

/azp run runtime-libraries stress-ssl

@azure-pipelines
Copy link
Copy Markdown

Azure Pipelines successfully started running 1 pipeline(s).

1 similar comment
@azure-pipelines
Copy link
Copy Markdown

Azure Pipelines successfully started running 1 pipeline(s).

Copilot AI review requested due to automatic review settings April 1, 2026 12:12
@ManickaP
Copy link
Copy Markdown
Member Author

ManickaP commented Apr 1, 2026

/azp run runtime-libraries stress-http

@ManickaP
Copy link
Copy Markdown
Member Author

ManickaP commented Apr 1, 2026

/azp run runtime-libraries stress-ssl

@azure-pipelines
Copy link
Copy Markdown

Azure Pipelines successfully started running 1 pipeline(s).

1 similar comment
@azure-pipelines
Copy link
Copy Markdown

Azure Pipelines successfully started running 1 pipeline(s).

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

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

@ManickaP
Copy link
Copy Markdown
Member Author

ManickaP commented Apr 1, 2026

/azp run runtime-libraries stress-ssl

@ManickaP
Copy link
Copy Markdown
Member Author

ManickaP commented Apr 1, 2026

/azp run runtime-libraries stress-http

@azure-pipelines
Copy link
Copy Markdown

Azure Pipelines successfully started running 1 pipeline(s).

1 similar comment
@azure-pipelines
Copy link
Copy Markdown

Azure Pipelines successfully started running 1 pipeline(s).

@ManickaP
Copy link
Copy Markdown
Member Author

ManickaP commented Apr 2, 2026

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
Copy link
Copy Markdown
Member Author

ManickaP commented Apr 2, 2026

/ba-g unrelated

@ManickaP ManickaP enabled auto-merge (squash) April 2, 2026 08:29
@ManickaP ManickaP merged commit cc1ae2b into dotnet:main Apr 2, 2026
115 of 125 checks passed
@ManickaP ManickaP deleted the revert-docker-compose-changes-125995 branch April 2, 2026 08:39
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants