Skip to content

Go: move to standard windows runner #19525

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
May 20, 2025
Merged

Go: move to standard windows runner #19525

merged 1 commit into from
May 20, 2025

Conversation

redsun82
Copy link
Contributor

@redsun82 redsun82 commented May 19, 2025

Seems like windows-latest-xl is not available any more, leading to jobs waiting endlessly (see for example https://github.com/github/codeql/actions/runs/15109425790). This should unblock CI, but longer term we should consider doing what other languages do (i.e. run tests from the internal repo).

Seems like `windows-latest-xl` is not available any more. This should unblock CI, but longer term we should consider doing what other languages do (i.e. run tests from the internal repo).
@Copilot Copilot AI review requested due to automatic review settings May 19, 2025 15:25
@redsun82 redsun82 requested a review from a team as a code owner May 19, 2025 15:25
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

Updates the Windows CI job to use the supported windows-latest runner and removes the previous repository-owner conditional to unblock builds.

  • Switch Windows tests from windows-latest-xl to windows-latest
  • Remove if: github.repository_owner == 'github' guard on the Windows job
Comments suppressed due to low confidence (2)

.github/workflows/go-tests-other-os.yml:28

  • Removing the repository_owner check causes this job to run on all forks, which may expose secrets. Consider re-adding a conditional or scoping permissions if this was unintentional.
test-win:

.github/workflows/go-tests-other-os.yml:30

  • [nitpick] Switching to windows-latest restores runner availability, but verify that it provides equivalent resources and environment for your tests compared to windows-latest-xl.
runs-on: windows-latest

@redsun82 redsun82 merged commit 860b3d7 into main May 20, 2025
8 checks passed
@redsun82 redsun82 deleted the redsun82/go-win branch May 20, 2025 08:07
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.

3 participants