Skip to content

Retry release asset uploads in CI - #576

Merged
jserv merged 1 commit into
mainfrom
fix
Nov 2, 2026
Merged

jserv merged 1 commit into
mainfrom
fix

Conversation

@jserv

@jserv jserv commented Nov 2, 2026 •

Copy link
Copy Markdown
Contributor

The desktop workflow's rolling job failed on main (run 41514226247) with HTTP 408: Upload body timed out while uploading one installer to the staged draft release, although all three platform builds had succeeded. The same single-attempt upload exists in the latest release job of check.yml.

Each upload is now tried up to three times with a growing pause, and --clobber replaces an asset a cut-off attempt left behind instead of refusing it. The size check after the upload is unchanged, so a truncated or missing asset still fails the job.

Verified: both workflow files parse as YAML, and codex and claude reviewed the loop read-only and found nothing wrong under bash -e -o pipefail, including the EXIT trap that deletes the draft after a third failure. The upload paths only run on main, so the retry itself has not been exercised; whether --clobber sees a half-written asset is unconfirmed until a real timeout hits it.


Summary by cubic

Retries GitHub release asset uploads in the desktop and check workflows so a transient HTTP 408: Upload body timed out no longer fails the job even though all builds succeeded. Each upload is tried up to three times with a growing pause, and --clobber replaces an asset a cut-off attempt left behind. The size check that follows still fails the job on a truncated or missing asset.

Written for commit 61f5858. Summary will update on new commits.

View guided diff Turn on auto-fix

The rolling jobs of the desktop and check workflows stage a draft
release and upload the installers in one attempt. GitHub answered one
upload with HTTP 408 (body timed out), which failed the desktop job
although every build had succeeded.

Try each upload up to three times with a growing pause, and pass
--clobber so an asset a cut-off attempt left behind is replaced rather
than refused. The size check that follows still fails the job on a
truncated or missing asset.

Change-Id: I2d9092d7df3f297850ed3c33ce017e30a3de31f4
cubic-dev-ai[bot]

This comment was marked as resolved.

@jserv
jserv merged commit eb4bc27 into main Nov 2, 2026
19 checks passed
@jserv
jserv deleted the fix branch November 2, 2026 20:51
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.

1 participant