Skip to content

Conversation

@LoopedBard3
Copy link
Member

@LoopedBard3 LoopedBard3 commented Jan 26, 2026

Remove the Tiger SDK jobs (Windows and Ubuntu) as we have gotten a bit of overlap with the Viper machines. Also reenable previously disabled Windows CI jobs.

@LoopedBard3 LoopedBard3 self-assigned this Jan 26, 2026
@LoopedBard3 LoopedBard3 changed the title Remove Tiger SDK jobs. Remove Tiger SDK jobs + Reenable Windows ci Jan 26, 2026
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 claims to "Remove the Tiger SDK jobs (Windows and Ubuntu) as we have gotten a bit of overlap with the Viper machines," but the actual code changes contradict this description. The changes primarily enable Tiger SDK jobs in the public jobs section by uncommenting previously disabled machine configurations, while also cleaning up commented-out Tiger SDK references in the private jobs section where Viper machine equivalents already exist.

Changes:

  • Uncommented/enabled multiple Tiger SDK job configurations (win-x64, win-x86, ubuntu-x64) in the public jobs section
  • Removed commented-out Tiger SDK references from private jobs section where Viper machines are already configured

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

jobTemplate: /eng/pipelines/templates/run-scenarios-job.yml
buildMachines:
#- win-x64
- win-x64
Copy link

Copilot AI Jan 26, 2026

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

The PR description states "Remove the Tiger SDK jobs" but this change actually enables/uncomments the win-x64 job. This appears to be re-enabling Tiger SDK jobs rather than removing them. Please verify if the PR description is accurate or if the code changes need to be revised.

Copilot uses AI. Check for mistakes.
Copy link
Member

@DrewScoggins DrewScoggins left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM

@LoopedBard3
Copy link
Member Author

Almost all failures are related to net10.0, will remove net10.0 builds in another PR shortly at which point we can more easily fix any other errors. Looking at the non-net10 micro windows error the run ran successfully and it was an issue with the send to helix connection. Merging.

@LoopedBard3 LoopedBard3 merged commit cc06d31 into dotnet:main Jan 27, 2026
72 of 103 checks passed
@LoopedBard3 LoopedBard3 deleted the ReenableWindowsVerificationRuns branch January 27, 2026 23:18
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants