Skip to content

ci : skip container re-tagging when Require Docker is disabled - #30008

Merged
ggerganov merged 1 commit into
masterfrom
gg/make-release-docker-retag
Oct 5, 2026
Merged

ggerganov merged 1 commit into
masterfrom
gg/make-release-docker-retag

Conversation

@ggerganov

Copy link
Copy Markdown
Member

Overview

Follow-up to #29989: the require_docker flag disabled the container image check, but the "Re-tag container images with release version" step still ran unconditionally and aborted the job when the nightly images did not exist (see run 37344417952, which failed after the release was already created).

Changes:

  • .github/workflows/make-release.yml: skip the re-tag step when require_docker is false; when it runs, keep going on a missing image and report all missing tags at the end instead of dying on the first one
  • scripts/make-release-checks.sh: clarify that REQUIRE_DOCKER=false also skips re-tagging

Requirements

Assisted-by: pi:llama.cpp/Qwen3.8-Flash-Next
@github-actions github-actions Bot added the devops improvements to build systems and github actions label Oct 5, 2026
@ggerganov
ggerganov marked this pull request as ready for review October 5, 2026 17:50
@ggerganov
ggerganov requested a review from a team as a code owner October 5, 2026 17:50
@ggerganov
ggerganov merged commit c250304 into master Oct 5, 2026
3 checks passed
edwardyoon pushed a commit to edwardyoon/focus-llama that referenced this pull request Oct 8, 2026
…org#30008)

Assisted-by: pi:llama.cpp/Qwen3.8-Flash-Next
(cherry picked from commit c250304)
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

devops improvements to build systems and github actions

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants