Thanks to visit codestin.com
Credit goes to github.com

Skip to content

Conversation

ManickaP
Copy link
Member

@ManickaP ManickaP commented Sep 8, 2025

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.
@Copilot Copilot AI review requested due to automatic review settings September 8, 2025 08:38
Copy link
Contributor

@Copilot 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 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
Member Author

ManickaP commented Sep 8, 2025

/azp run runtime-libraries stress-http

Copy link

Azure Pipelines successfully started running 1 pipeline(s).

@ManickaP ManickaP changed the title [HTTP] Stress fix for docker compose [release/8.0-staging][HTTP] Stress fix for docker compose Sep 8, 2025
@ManickaP
Copy link
Member Author

ManickaP commented Sep 8, 2025

/azp run runtime-libraries stress-http

Copy link

Azure Pipelines successfully started running 1 pipeline(s).

@ManickaP
Copy link
Member Author

ManickaP commented Sep 8, 2025

@ManickaP ManickaP requested a review from a team September 8, 2025 14:26
@ManickaP
Copy link
Member Author

ManickaP commented Sep 8, 2025

Test only change, tell-mode.

@ManickaP
Copy link
Member Author

ManickaP commented Sep 9, 2025

/ba-g #117731

@ManickaP ManickaP added the Servicing-approved Approved for servicing release label Sep 9, 2025
@ManickaP ManickaP merged commit 337b406 into release/8.0-staging Sep 9, 2025
103 of 107 checks passed
@jkotas jkotas deleted the ManickaP-patch-1 branch September 22, 2025 23:28
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Servicing-approved Approved for servicing release
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants