Skip to content

Conversation

@rzikm
Copy link
Member

@rzikm rzikm commented Nov 19, 2025

Customer Impact

None, test-only change to enable regular stress test runs on release/10.0 branch

Regression

  • Yes
  • No

Testing

Stress test will be triggered for this PR

Risk

Low, test-only change.

Copilot AI review requested due to automatic review settings November 19, 2025 17:16
@github-actions github-actions bot added the needs-area-label An area label is needed to ensure this gets routed to the appropriate area owners label Nov 19, 2025
Copilot finished reviewing on behalf of rzikm November 19, 2025 17:18
Copy link
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 updates the branch filters for networking stress test pipelines to run on .NET major release branches instead of staging branches. The pattern is changed from release/*-staging to release/*.0 to align with the actual release branch naming convention.

Key Changes:

  • Updated scheduled build branch filters in HTTP and SSL stress test pipelines
  • Changed pattern from release/*-staging to release/*.0 to match major release branches (e.g., release/10.0)

Reviewed Changes

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

File Description
eng/pipelines/libraries/stress/ssl.yml Updated scheduled build branch filter to target major release branches
eng/pipelines/libraries/stress/http.yml Updated scheduled build branch filter to target major release branches

@rzikm rzikm requested a review from a team November 19, 2025 17:19
@rzikm rzikm added area-System.Net and removed needs-area-label An area label is needed to ensure this gets routed to the appropriate area owners labels Nov 19, 2025
@rzikm
Copy link
Member Author

rzikm commented Nov 19, 2025

/azp run runtime-libraries stress-http

@rzikm
Copy link
Member Author

rzikm commented Nov 19, 2025

/azp run runtime-libraries stress-ssl

@azure-pipelines
Copy link

Azure Pipelines successfully started running 1 pipeline(s).

1 similar comment
@azure-pipelines
Copy link

Azure Pipelines successfully started running 1 pipeline(s).

@dotnet-policy-service
Copy link
Contributor

Tagging subscribers to this area: @dotnet/ncl
See info in area-owners.md if you want to be subscribed.

@rzikm
Copy link
Member Author

rzikm commented Nov 20, 2025

Stress test runs pass, other failures are unrelated

@rzikm
Copy link
Member Author

rzikm commented Nov 20, 2025

/ba-g Stress test runs pass, other failures are unrelated

@karelz karelz added test-enhancement Improvements of test source code Servicing-consider Issue for next servicing release review labels Nov 20, 2025
@karelz karelz added this to the 10.0.x milestone Nov 20, 2025
@karelz
Copy link
Member

karelz commented Nov 20, 2025

@artl93 this is test-only / test-infra change only. It is ready for servicing. Now that 10 is GA, will you be merging these? Should we tag you? Or should I run ti by Tactics?

@artl93
Copy link
Member

artl93 commented Nov 21, 2025

infra changes are tell mode.
I'll approve and merge -- though I think we need a new flow.

@artl93 artl93 added Servicing-approved Approved for servicing release and removed Servicing-consider Issue for next servicing release review labels Nov 21, 2025
@artl93 artl93 merged commit eef1961 into dotnet:release/10.0 Nov 21, 2025
102 of 110 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

area-System.Net Servicing-approved Approved for servicing release test-enhancement Improvements of test source code

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants