Skip to content

fix(ci): use github-script for wheel pruning instead of gh CLI#354

Merged
pimlock merged 1 commit intomainfrom
fix/debug-wheel-pruning
Mar 16, 2026
Merged

fix(ci): use github-script for wheel pruning instead of gh CLI#354
pimlock merged 1 commit intomainfrom
fix/debug-wheel-pruning

Conversation

@pimlock
Copy link
Collaborator

@pimlock pimlock commented Mar 16, 2026

Summary

Root Cause

The release-devel job runs on build-amd64 without a container, unlike most other jobs which use ghcr.io/nvidia/openshell/ci:latest. The gh CLI is only available inside that container. The pruning step's guard clause (gh release view ... > /dev/null 2>&1) swallowed the error and always hit the "No existing devel release found" bail-out.

Verification

Tested with a temporary github-script debug workflow on this branch (run #23135749664) that confirmed:

  • The Octokit approach successfully queries the devel release
  • 25 assets found: 21 stale wheels across 7 versions + 4 non-wheel assets
  • Only the latest 3 wheels should be kept; the other 18 are stale

Changes

  • .github/workflows/release-dev.yml: Replaced the Prune stale wheel assets from devel release step from a bash script using gh CLI to an actions/github-script@v7 step using Octokit

Testing

  • Verified github-script can list devel release assets from the same runner environment
  • Will verify pruning works on next merge to main

The gh CLI is not available on the bare build-amd64 runner (only in the
CI container image), causing 'gh release view' to fail silently and skip
pruning every time. Switch to actions/github-script which uses the
built-in Octokit client and needs no external CLI tools.

Tested via a temporary debug workflow on this branch that confirmed the
github-script approach successfully lists all 25 devel release assets
(21 stale wheels across 7 versions).
@pimlock pimlock requested a review from drew March 16, 2026 09:05
@drew drew force-pushed the fix/debug-wheel-pruning branch from 5240a2c to 5aa0035 Compare March 16, 2026 13:55
@pimlock pimlock self-assigned this Mar 16, 2026
@pimlock pimlock merged commit ee40fc8 into main Mar 16, 2026
9 checks passed
@pimlock pimlock deleted the fix/debug-wheel-pruning branch March 16, 2026 14:53
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