Publish from a GitHub-hosted runner, and say why - #17
Merged
Conversation
Moving the publish jobs to Ubicloud looked like closing a gap: every other job in the repository is on Ubicloud, and nothing about downloading a tarball and running `npm publish` wants a particular runner. It is not that. npm signs a provenance statement on publish, and the registry refuses one signed anywhere it reads as self-hosted, which is what an Ubicloud runner is: npm error 422 Error verifying sigstore provenance bundle: Unsupported GitHub Actions runner environment: "self-hosted". Only "github-hosted" runners are supported when publishing with provenance. It fails at the registry rather than at the runner, so nothing local catches it: the release ran, decided, packed a correct 0.2.2 tarball, signed it, and died on the PUT. Nothing was published, and because the decider reads npm rather than the diff, landing this makes the same release go out unchanged. Both publish jobs move back, each with the reason in place so the next reader does not close the same gap again.
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Unblocks the 0.2.2 release. My change in #16 broke it, and this is the revert plus the reason.
What happened
Moving the two
publishjobs to Ubicloud looked like closing a gap — every other job in the repository is on Ubicloud, and nothing about downloading a tarball and runningnpm publishwants a particular runner. That reasoning was wrong.npm signs a provenance statement on publish, and the registry refuses one signed anywhere it reads as self-hosted, which is what an Ubicloud runner is:
It fails at the registry, not at the runner, so nothing before the PUT catches it. The release run decided correctly, packed a correct
solid-layouts-0.2.2.tgz, signed it, and then died.What this does
Both publish jobs move back to
ubuntu-latest, each with the constraint written above it so the next reader does not close the same gap again.release-oxcpublishes with provenance the same way, so it would have failed the same way on its next release — it just had not run yet.Everything else stays on Ubicloud.
release-oxc's build matrix keeps its separate, already-documentedmacos-14exception.After merging
Nothing was published, and the decider reads npm rather than the diff, so landing this re-runs the same release unchanged and
solid-layouts@0.2.2goes out. No version bump needed — 0.2.2 is already onmasterfrom #16.