Skip to content

switch winget PR creation from Octo STS to PAT - #2235

Merged
pritt20 merged 1 commit into
mainfrom
fix_winget_release_workflow
Jul 31, 2026
Merged

switch winget PR creation from Octo STS to PAT#2235
pritt20 merged 1 commit into
mainfrom
fix_winget_release_workflow

Conversation

@pritt20

@pritt20 pritt20 commented Jul 31, 2026

Copy link
Copy Markdown
Contributor

Description

The update-winget job was failing at PR creation with Resource not accessible by integration. Root cause: Octo STS mints GitHub App installation tokens, and a GitHub App can only open PRs where it is installed on the base repo. Since we can't install our App on microsoft/winget-pkgs, PR creation always fails.

Switching to a user-scoped PAT resolves issue where any authenticated user can open PRs to public upstream repos from a fork.

🎟 Issue(s)

Related #XXX

🧪 Functional Testing

List the functional testing steps to confirm this feature or fix.

📸 Screenshots

Add screenshots to illustrate the validity of these changes.

📋 Checklist

  • Rebased from the main (or release if patching) branch (before testing)
  • Ran make test before taking out of draft
  • Ran make lint before taking out of draft
  • Added/updated applicable tests
  • Tested against Astro-API (if necessary).
  • Tested against Houston-API and Astronomer (if necessary).
  • Communicated to/tagged owners of respective clients potentially impacted by these changes.
  • Updated any related documentation

@pritt20
pritt20 requested a review from a team as a code owner July 31, 2026 12:30
@coveralls-official

Copy link
Copy Markdown

Coverage Report for CI Build 30630853371

Coverage remained the same at 43.896%

Details

  • Coverage remained the same as the base build.
  • Patch coverage: No coverable lines changed in this PR.
  • No coverage regressions found.

Uncovered Changes

No uncovered changes found.

Coverage Regressions

No coverage regressions found.


Coverage Stats

Coverage Status
Relevant Lines: 59030
Covered Lines: 25912
Line Coverage: 43.9%
Coverage Strength: 8.66 hits per line

💛 - Coveralls

@pritt20
pritt20 merged commit 58e75d1 into main Jul 31, 2026
7 checks passed
@pritt20
pritt20 deleted the fix_winget_release_workflow branch July 31, 2026 12:49
pritt20 added a commit that referenced this pull request Jul 31, 2026
Co-authored-by: Pritesh Arora <pritt@Priteshs-MacBook-Pro.local>
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