Skip to content

fix: grant contents:write to release flow buildassets job#732

Merged
thomasrockhu-codecov merged 1 commit into
mainfrom
th/fix-release-flow-permissions
Jul 8, 2026
Merged

fix: grant contents:write to release flow buildassets job#732
thomasrockhu-codecov merged 1 commit into
mainfrom
th/fix-release-flow-permissions

Conversation

@thomasrockhu-codecov

Copy link
Copy Markdown
Contributor

Summary

  • The Build and Publish CLI Release workflow (release_flow.yml) failed with startup_failure (e.g. run 28972104407).
  • Root cause: commit 987d53c bumped build_assets.yml's workflow-level permission to contents: write (needed so upload-release-action can attach release assets via GITHUB_TOKEN). A reusable workflow cannot request more permission than the caller grants, but release_flow.yml calls it from the buildassets job with only the default contents: read, so the workflow failed to compile and no jobs ran.
  • Fix: grant contents: write on the buildassets job so the reusable workflow can compile and successfully upload assets.

Test plan

  • Merge to main, then re-cut the v11.3.0 release/tag so GitHub loads the corrected workflow files for the release: created event.
  • Confirm the Build and Publish CLI Release run starts (no startup_failure) and the buildassets job uploads release assets.

Made with Cursor

The build_assets.yml reusable workflow now declares contents:write (needed
to attach release assets via upload-release-action). A called workflow cannot
request more permission than the caller grants, so release_flow.yml calling
it with only contents:read caused a startup_failure. Grant contents:write on
the buildassets job so the reusable workflow can compile and upload assets.

Co-authored-by: Cursor <cursoragent@cursor.com>
@codecov

codecov Bot commented Jul 8, 2026

Copy link
Copy Markdown

❌ 1 Tests Failed:

Tests completed Failed Passed Skipped
723 1 722 1
View the full list of 1 ❄️ flaky test(s)
api.temp.calculator.test_calculator::test_divide

Flake rate in main: 100.00% (Passed 0 times, Failed 982 times)

Stack Traces | 0.001s run time
def
                test_divide():
                > assert Calculator.divide(1, 2) == 0.5
                E assert 1.0 == 0.5
                E + where 1.0 = <function Calculator.divide at 0x104c9eb90>(1, 2)
                E + where <function Calculator.divide at 0x104c9eb90> = Calculator.divide
                .../temp/calculator/test_calculator.py:30: AssertionError

To view more test analytics, go to the Test Analytics Dashboard
📋 Got 3 mins? Take this short survey to help us improve Test Analytics.

@thomasrockhu-codecov thomasrockhu-codecov merged commit ceec285 into main Jul 8, 2026
22 checks passed
@thomasrockhu-codecov thomasrockhu-codecov deleted the th/fix-release-flow-permissions branch July 8, 2026 20:17
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