Skip to content

ci: order runs per branch, a version tag no longer republishes the commit image - #26

Merged
lightchainaidev merged 1 commit into
testnetfrom
ci-review-fixes
Oct 13, 2026
Merged

lightchainaidev merged 1 commit into
testnetfrom
ci-review-fixes

Conversation

@lightchainaidev

Copy link
Copy Markdown
Contributor

Two fixes from a review of the workflow.

Runs are ordered per branch. The image job had its own concurrency group, which only ordered image builds among themselves, after each run's checks. A later push whose checks finished first could publish first, leaving :testnet on the older commit. The group is now on the workflow: a later push waits for the earlier run. A run that is still waiting gives way to a newer push, so a commit in the middle of a quick series may get no image of its own.

A version tag no longer republishes the commit image. The runs for v0.0.3-rc.1 and v0.0.3 each pushed worker:0acde8d again, replacing the image the testnet push had published for that commit with a rebuild (same source, different digest). A tag run now publishes the version tag only.

Merging pushes to testnet, so CI publishes worker:testnet and worker:<commit>. It creates no release.

…mmit image

Runs are ordered per branch. The image job had its own concurrency
group, which only ordered image builds among themselves, after each
run's checks: a later push whose checks finished first could publish
first and leave the branch tag on the older commit. The group is now on
the workflow, so a later push waits for the earlier run. A run that is
still waiting gives way to a newer push.

A version tag run published the image under the commit tag as well,
replacing the one the branch push had published for that commit with a
rebuild. A tag run now publishes the version tag only.
@lightchainaidev
lightchainaidev merged commit 888e17c into testnet Oct 13, 2026
3 checks passed
@lightchainaidev
lightchainaidev deleted the ci-review-fixes branch October 13, 2026 13:43
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