Skip to content

ci: pass VERSION_STR to publish job and fix Slack Docker tag#55

Merged
ysibirski merged 1 commit into
mainfrom
fix/ci-release-version-str
Jul 7, 2026
Merged

ci: pass VERSION_STR to publish job and fix Slack Docker tag#55
ysibirski merged 1 commit into
mainfrom
fix/ci-release-version-str

Conversation

@ysibirski

Copy link
Copy Markdown
Contributor

Problem

The v2.0.0-rc1 release failed at the Docker publish step:

Error parsing reference: "index.docker.io/***/vflow-node:" is not a valid repository/tag: invalid reference format

The tag after the colon was empty. check_release_requirements.sh computes VERSION_STR (the v-stripped tag, e.g. 2.0.0-rc1) and correctly detected the RC release, but the workflow never exposed VERSION_STR as a job output nor forwarded it to the publish job. So publish-docker-image.sh read an empty VERSION_STR, producing an empty docker_tag_full and the vflow-node: reference.

Fix

Mirror the zkVerify CI-release.yml wiring:

  • Add version-str output to the check-requirements job.
  • echo "VERSION_STR=..." >> "$GITHUB_OUTPUT" in the check step.
  • Pass VERSION_STR into the publish-docker-image step env.

Also fix a related v-prefix mismatch in the Slack notification: the Docker Hub tag link and docker pull command used github.ref_name (v2.0.0-rc1), but the pushed image tag has no v (2.0.0-rc1). Both now use the v-stripped version-str, matching the actual image. The GitHub release tag_name intentionally keeps github.ref_name (the GitHub tag keeps the v).

Result

v2.0.0-rc1 now publishes as vflow-node:2.0.0-rc1, and the Slack docker pull command references a tag that actually exists.

🤖 Generated with Claude Code

The release workflow computed VERSION_STR in check_release_requirements.sh
but never exposed it as a job output or forwarded it to the publish job, so
publish-docker-image.sh saw an empty tag and failed with "invalid reference
format" (e.g. vflow-node: on the v2.0.0-rc1 release).

Expose VERSION_STR as a check-requirements output and pass it into the
publish step, mirroring the zkVerify workflow. Also switch the Slack Docker
Hub tag link and docker pull command from github.ref_name (v-prefixed) to the
v-stripped version-str, matching the actual pushed image tag.

Co-Authored-By: Claude Opus 4.8 (1M context) <noreply@anthropic.com>
@ysibirski ysibirski requested a review from cronicc July 6, 2026 16:56
@ysibirski ysibirski requested a review from a team as a code owner July 6, 2026 16:56
@ysibirski ysibirski requested review from davehorizen and drgora July 6, 2026 16:56

@drgora drgora 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.

LGTM, thanks

@ysibirski ysibirski merged commit 2e64209 into main Jul 7, 2026
13 checks passed
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.

3 participants