Skip to content

fix(publish): resilient publishing and reordered flow#315

Merged
fslabs-bot[bot] merged 5 commits into
mainfrom
lp/fix-publishing
Mar 20, 2026
Merged

fix(publish): resilient publishing and reordered flow#315
fslabs-bot[bot] merged 5 commits into
mainfrom
lp/fix-publishing

Conversation

@loispostula

Copy link
Copy Markdown
Member

Summary

  • Reorder publish steps so cargo publish runs before nix/docker — nix or docker failures no longer block crate publishing
  • Add --fallback to nix build so it builds from source when cache substituters have network issues
  • Auto-create GitHub Release on 404 instead of failing when no release exists for the tag
  • Clarify release process documentation in README

Context

The publish-all Prow job was failing due to transient network errors downloading from the Nix cache (SSL_ERROR_SYSCALL, curl failures). Because the publish flow short-circuits, this also prevented cargo publish from running. Additionally, report_publish_to_github was failing with "Could not find a release with tag" since HANDLE_TAGS=false means no release is pre-created.

Nix and Docker build failures were preventing cargo publish from
running due to the sequential short-circuit flow.
Build from source when cache substituters have network issues.
Auto-create a GitHub release on 404 instead of failing, only
triggered when the release genuinely doesn't exist.
@fslabs-bot fslabs-bot Bot added the size/XXL Denotes a PR that changes 1000+ lines, ignoring generated files. label Mar 20, 2026

@bonsairobo bonsairobo left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Awesome

Comment thread README.md Outdated
@fslabs-bot

fslabs-bot Bot commented Mar 20, 2026

Copy link
Copy Markdown

[APPROVALNOTIFIER] This PR is APPROVED

This pull-request has been approved by: bonsairobo

The full list of commands accepted by this bot can be found here.

The pull request process is described here

Details Needs approval from an approver in each of these files:

Approvers can indicate their approval by writing /approve in a comment
Approvers can cancel approval by writing /approve cancel in a comment

@fslabs-bot fslabs-bot Bot added the approved Indicates a PR has been approved by an approver from all required OWNERS files. label Mar 20, 2026
@fslabs-bot fslabs-bot Bot added size/L Denotes a PR that changes 100-499 lines, ignoring generated files. and removed size/XXL Denotes a PR that changes 1000+ lines, ignoring generated files. labels Mar 20, 2026
@fslabs-bot fslabs-bot Bot merged commit 4afc6d9 into main Mar 20, 2026
2 checks passed
@fslabs-bot fslabs-bot Bot deleted the lp/fix-publishing branch March 20, 2026 11:42
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

approved Indicates a PR has been approved by an approver from all required OWNERS files. size/L Denotes a PR that changes 100-499 lines, ignoring generated files.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants