Skip to content

[release/9.0-staging][HTTP] Stress fix for docker compose - #119455

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

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

Conversation

@ManickaP

@ManickaPManickaP commented Sep 8, 2025

Copy link
Copy Markdown
Member

Backport of #119274 to release/9.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:40

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 adds docker compose cleanup commands to HTTP stress testing pipeline to prevent interference between individual test runs. It's a backport from main branch to the release/10.0 branch to fix infrastructure issues in CI stress runs.

Key changes:

  • Adds docker compose down commands before each HTTP stress test run
  • Ensures clean state between HTTP 2.0 and HTTP 1.1 stress tests

@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/9.0-staging][HTTP] Stress fix for docker composeSep 8, 2025
Removed 'docker-compose down' command from HTTP 1.1 and 2.0 stress test configurations.
@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

/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

HTTP stress is running, the failure in https://dev.azure.com/dnceng-public/public/_build/results?buildId=1142024&view=results is known issue #109121

@ManickaP

Copy link
Copy Markdown
MemberAuthor

Test only change, tell-mode.

@ManickaPManickaP added the Servicing-approved Approved for servicing release label Sep 8, 2025
@ManickaP
ManickaP requested a review from a teamSeptember 8, 2025 14:27
@ManickaP
ManickaP merged commit 981a610 into release/9.0-stagingSep 9, 2025
80 of 83 checks passed
@ManickaP
ManickaP deleted the ManickaP-patch-2 branch September 9, 2025 06:28
@github-actionsgithub-actionsBot locked and limited conversation to collaborators Oct 9, 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