Skip to content

[release/6.0-staging] Remove Windows 7 helix queues #100988

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Merged
merged 1 commit into from
Apr 19, 2024

Conversation

jkoritzinsky
Copy link
Member

Backport of #100981 to release/6.0-staging

Customer Impact

  • Customer reported
  • Found internally

Due to changing internal requirements, Helix is decomissioning the Win7 queues.

As a result, we need to stop sending work to them before they are turned off (as any build that uses them after that point will fail.

Regression

  • Yes
  • No

Testing

Infrastructure-only removal of a queue, no testing necessary.

Risk

Low: Simple removal of a Helix queue. We do lose some test coverage, but we can't keep the queue around.

IMPORTANT: If this backport is for a servicing release, please verify that:

  • The PR target branch is release/X.0-staging, not release/X.0.

  • If the change touches code that ships in a NuGet package, you have added the necessary package authoring and gotten it explicitly reviewed.

Backport of #100981 to release/6.0-staging

## Customer Impact

- [ ] Customer reported
- [X] Found internally

Due to changing internal requirements, Helix is decomissioning the Win7 queues.

As a result, we need to stop sending work to them before they are turned off (as any build that uses them after that point will fail.

## Regression

- [ ] Yes
- [X] No

## Testing

Infrastructure-only removal of a queue, no testing necessary.

## Risk

Low: Simple removal of a Helix queue. We do lose some test coverage, but we can't keep the queue around.

**IMPORTANT**: If this backport is for a servicing release, please verify that:

- The PR target branch is `release/X.0-staging`, not `release/X.0`.

- If the change touches code that ships in a NuGet package, you have added the necessary [package authoring](https://github.com/dotnet/runtime/blob/main/docs/project/library-servicing.md) and gotten it explicitly reviewed.
Copy link
Contributor

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

@rbhanda rbhanda added Servicing-approved Approved for servicing release and removed Servicing-consider Issue for next servicing release review labels Apr 16, 2024
@rbhanda rbhanda added this to the 6.0.31 milestone Apr 16, 2024
@jkoritzinsky jkoritzinsky merged commit 626a28d into release/6.0-staging Apr 19, 2024
106 of 117 checks passed
@jkotas jkotas deleted the jkoritzinsky-patch-1 branch April 27, 2024 22:49
@github-actions github-actions bot locked and limited conversation to collaborators May 28, 2024
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants