Skip to content

Conversation

@TravisEz13
Copy link
Member

Backport of #26489 to release/v7.6

Triggered by @TravisEz13 on behalf of @app/copilot-swe-agent

Original CL Label: CL-BuildPackaging

/cc @PowerShell/powershell-maintainers

Impact

REQUIRED: Choose either Tooling Impact or Customer Impact (or both). At least one checkbox must be selected.

Tooling Impact

  • Required tooling change
  • Optional tooling change (include reasoning)

Fixes intermittent CI failures when multiple test jobs run concurrently and hit GitHub API rate limits (60 req/hr unauthenticated). By adding GITHUB_TOKEN authentication, test actions get 5000 req/hr limit, preventing rate limit errors when fetching DSC releases during test setup.

Customer Impact

  • Customer reported
  • Found internally

Regression

REQUIRED: Check exactly one box.

  • Yes
  • No

This is not a regression.

Testing

Original PR verified by running affected workflows in fork and confirming CI passes. Backport tested by applying clean cherry-pick to updated release/v7.6 branch - no conflicts encountered. The fix adds GitHub token authentication to API calls in test actions, preventing rate limit errors during concurrent CI runs.

Risk

REQUIRED: Check exactly one box.

  • High
  • Medium
  • Low

Medium risk: Changes affect CI/CD test infrastructure across all platforms (Linux, macOS, Windows). However, the change is minimal (adding authentication headers), uses built-in GITHUB_TOKEN (no new secrets), and has been validated in master. Risk is isolated to test execution - does not affect production code.

Co-authored-by: copilot-swe-agent[bot] <198982749+Copilot@users.noreply.github.com>
Co-authored-by: TravisEz13 <10873629+TravisEz13@users.noreply.github.com>
Copilot AI review requested due to automatic review settings November 19, 2025 23:28
@TravisEz13 TravisEz13 requested a review from a team as a code owner November 19, 2025 23:28
@TravisEz13 TravisEz13 added the CL-BuildPackaging Indicates that a PR should be marked as a build or packaging change in the Change Log label Nov 19, 2025
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 backports a fix from #26489 to the release/v7.6 branch to resolve GitHub API rate limit errors in test actions. When multiple test jobs run concurrently, unauthenticated GitHub API requests are limited to 60 requests per hour, causing intermittent CI failures. By adding GITHUB_TOKEN authentication, the rate limit increases to 5,000 requests per hour, eliminating these errors.

Key Changes:

  • Added GITHUB_TOKEN input parameter to test actions for Windows and Unix platforms
  • Modified test actions to include Authorization headers when calling GitHub API
  • Updated all CI workflows (Linux, macOS, Windows) to pass GITHUB_TOKEN to test jobs

Reviewed Changes

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

Show a summary per file
File Description
.github/workflows/windows-ci.yml Added GITHUB_TOKEN parameter to all four Windows test jobs (unelevated/elevated CI and Others)
.github/workflows/macos-ci.yml Added GITHUB_TOKEN parameter to all four macOS test jobs (unelevated/elevated CI and Others)
.github/workflows/linux-ci.yml Added GITHUB_TOKEN parameter to all four Linux test jobs (unelevated/elevated CI and Others)
.github/actions/test/windows/action.yml Added GITHUB_TOKEN input, created Authorization header, and applied it to GitHub API calls for DSC release fetching
.github/actions/test/nix/action.yml Added GITHUB_TOKEN input, created Authorization header, and applied it to GitHub API calls for DSC release fetching

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

@TravisEz13 TravisEz13 merged commit ea0bfb3 into PowerShell:release/v7.6 Nov 20, 2025
40 checks passed
@TravisEz13 TravisEz13 deleted the backport/release/v7.6/26489-6928efb4c branch November 20, 2025 00:38
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

CL-BuildPackaging Indicates that a PR should be marked as a build or packaging change in the Change Log

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants