Skip to content

[release/8.0-staging][HTTP] Stress fix for docker compose - #119454

Merged
ManickaP merged 2 commits into
release/8.0-stagingfrom
ManickaP-patch-1
Sep 9, 2025
Merged

[release/8.0-staging][HTTP] Stress fix for docker compose#119454
ManickaP merged 2 commits into
release/8.0-stagingfrom
ManickaP-patch-1

Conversation

@ManickaP

@ManickaPManickaP commented Sep 8, 2025

Copy link
Copy Markdown
Member

Backport of #119274 to release/8.0-staging
/cc @ManickaP

Customer Impact

  • Customer reported
  • Found internally

Wind down docker compose between individual runs in HTTP stress.

Regression

  • Yes
  • No

Infra update.

Testing

CI stress runs.

Risk

Low. Test only change.

Backport of #119274 to release/10.0
/cc @ManickaP
## Customer Impact
- [ ] Customer reported
- [x] Found internally
Wind down docker compose between individual runs in HTTP stress.
## Regression
- [x] Yes
- [ ] No
Infra update.
## Testing
CI stress runs.
## Risk
Low. Test only change.
CopilotAI review requested due to automatic review settings September 8, 2025 08:38

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 fixes a stress testing issue by adding proper cleanup between HTTP stress test runs. It ensures that Docker Compose is properly wound down before starting each test iteration to prevent resource conflicts and ensure clean test environments.

Key changes:

  • Added docker compose down commands before each HTTP stress test run
  • Applied to both HTTP 2.0 and HTTP 1.1 test scenarios

@ManickaP

Copy link
Copy Markdown
MemberAuthor

/azp run runtime-libraries stress-http

@azure-pipelines

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

@ManickaPManickaP changed the title [HTTP] Stress fix for docker compose[release/8.0-staging][HTTP] Stress fix for docker composeSep 8, 2025
@ManickaP

Copy link
Copy Markdown
MemberAuthor

/azp run runtime-libraries stress-http

@azure-pipelines

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

@ManickaP

Copy link
Copy Markdown
MemberAuthor

@ManickaP
ManickaP requested a review from a teamSeptember 8, 2025 14:26
@ManickaP

Copy link
Copy Markdown
MemberAuthor

Test only change, tell-mode.

@ManickaP

Copy link
Copy Markdown
MemberAuthor

/ba-g #117731

This was referenced Oct 22, 2025
@github-actionsgithub-actionsBot locked and limited conversation to collaborators Oct 23, 2025
Sign up for freeto subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

Servicing-approvedApproved for servicing release

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants

@ManickaP@liveans