Repository navigation
ci: run on GitHub hosted runners instead of Blacksmith - #10
Open
simonostendorf wants to merge 1 commit into
Open
simonostendorf wants to merge 1 commit into
simonostendorf wants to merge 1 commit into
Conversation
This repository is public, where GitHub hosted runners are free and unlimited, while Blacksmith bills per minute. Both bill per job rounded up to a full minute, and this suite is wide and short, so the rounding was most of the bill. The runs-on-runner input is dropped rather than set, because ubuntu-latest is already its default in the reusable workflows. AGENTS.md claimed this repository is private and therefore bills its minutes. That stopped being true when it was published. Signed-off-by: Simon Ostendorf <github@simon-ostendorf.de>
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.
What
blacksmith-2vcpu-ubuntu-2404toubuntu-latest.useblacksmith/setup-docker-builderbecomesdocker/setup-buildx-action, anduseblacksmith/build-push-actionbecomesdocker/build-push-action, both pinned by digest.runs-on-runnerinput is deleted from every reusable workflow call rather than set toubuntu-latest, because that is already the input's default inostendorf-cloud/workflows.describes behaviour that no longer exists.
Why
repositories, while Blacksmith bills per minute, so the spend bought nothing.
of them already public.
wide and short, so the per-job rounding dominated the bill.
Verification
task lint:actionspasses.task lint:yamlpasses.grep -ri blacksmith .github/workflows/ AGENTS.mdprints nothing.ostendorf-cloud/release-me/.github/workflows/publish.yaml. Neither was invented.with:key passed to the docker actions was checked againstaction.ymlat the pinnedcommit.
loadandfilewere confirmed to exist there rather than assumed.Risk
platforms:key is added, so an image that built single-archamd64 still does.
docker/build-push-actionis upstream of the Blacksmith fork, so the inputs carry over, butthe first run on each workflow is where that is proven rather than reasoned about.
which costs time and no money on a public repository.
AGENTS.mdcarried a now-false deviation stating this repository is private and thereforebills its minutes. Removed here. Other stale deviations about CodeQL and licensing remain and
need a separate pass.
Open in [code]smith · Need help on this PR? Tag
@codesmith-botwith what you need.