Skip to content

Update windows open queue to windows11 and VMimage to windows 2025 #4875

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

Open
wants to merge 2 commits into
base: main
Choose a base branch
from

Conversation

LoopedBard3
Copy link
Member

@LoopedBard3 LoopedBard3 commented Jun 16, 2025

Update windows open queue to windows11 and VMimage to windows 2025.

Internal (successful) test run: https://dev.azure.com/dnceng/internal/_build/results?buildId=2731529&view=results

@LoopedBard3 LoopedBard3 requested a review from Copilot June 16, 2025 20:12
@LoopedBard3 LoopedBard3 self-assigned this Jun 16, 2025
@LoopedBard3 LoopedBard3 added enhancement New feature or request lab-infrastructure dependencies Pull requests that update a dependency file labels Jun 16, 2025
Copilot

This comment was marked as outdated.

@LoopedBard3 LoopedBard3 force-pushed the UpdateWindowsOpenQueueToWindows11 branch from d30c867 to 27ddb5b Compare June 16, 2025 20:16
@LoopedBard3
Copy link
Member Author

The internal run was successful.

Copy link

@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 updates the build machine configuration to use the latest Windows images and queue names.

  • Updated vmImage from windows-2019 to windows-2025
  • Updated osVersion from 22H2 to 24H2 for public jobs
  • Updated queue name from Windows.10.Amd64.Client.Open to Windows.11.Amd64.Client.Open

@LoopedBard3
Copy link
Member Author

All of the scenarios failures are due to Process xxxx failed to run. Result: CloseFailed, the sdk_scenario failures are mostly due to Process 5220 failed to run. Result: TimeoutExceeded with some due to PermissionError: [WinError 32] The process cannot access the file because it is being used by another process: 'app'. The internal run on actual machines was successful so I am going to try a public run with windows-2022 instead of 2025 to see if that helps.

@LoopedBard3
Copy link
Member Author

This test run that uses windows-2022 instead of 2025 still hits the same issue. https://dev.azure.com/dnceng-public/public/_build/results?buildId=1070518&view=results. I think it would make sense to take the update so we don't get caught with the old image deprecated and then spend some time investigating these specific cases.

@DrewScoggins
Copy link
Member

DrewScoggins commented Jun 17, 2025

Do we see these same failures on runs that are not a part of this PR?

I just looked and it seems like we are seeing the same failures. I think merging here is fine.

@LoopedBard3
Copy link
Member Author

It seems the performance-ci scheduled run hits the same sdk_scenarios scenarios while the scenarios succeed, but the scenarios issues are not hit. So, it probably makes more sense to keep this until we either have a chance to investigate the scenarios or if we see mass failure due to the images going away.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
dependencies Pull requests that update a dependency file enhancement New feature or request lab-infrastructure
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants