Skip to content

fix: lowercase the latest-minimal GHCR tag#3

Merged
davidfic merged 1 commit intomainfrom
fix/docker-lowercase-ghcr-tag
Apr 16, 2026
Merged

fix: lowercase the latest-minimal GHCR tag#3
davidfic merged 1 commit intomainfrom
fix/docker-lowercase-ghcr-tag

Conversation

@davidfic
Copy link
Copy Markdown
Contributor

Unblocks pilot's main-branch Docker job, which was failing on:

```
ERROR: failed to build: invalid tag
"ghcr.io/Beacon-Stack/pilot:latest-minimal":
repository name must be lowercase
```

The `Build and push (latest — alpine with ffprobe)` step escapes this because it uses `docker/metadata-action` outputs which auto-lowercase. The minimal step hardcoded `ghcr.io/${{ github.repository }}:latest-minimal` which preserves the org casing (`Beacon-Stack/pilot`) — GHCR rejects uppercase. Spelling the repo explicitly as `ghcr.io/beacon-stack/pilot` is the minimal fix.

Test plan

  • PR CI green
  • Main CI green after merge
  • Both `ghcr.io/beacon-stack/pilot:latest` and `:latest-minimal` publish

The Build and push (minimal — scratch, no ffprobe) step in ci.yml was
hardcoding:

    tags: ghcr.io/${{ github.repository }}:latest-minimal

github.repository preserves the case of the org in the URL, which is
"Beacon-Stack/pilot" — GHCR strictly requires lowercase image names
and rejects the push with:

    ERROR: failed to build: invalid tag
    "ghcr.io/Beacon-Stack/pilot:latest-minimal":
    repository name must be lowercase

The sibling "Build and push (latest)" step escapes this because it
uses docker/metadata-action outputs, which auto-lowercase. The
minimal step hardcodes the tag directly and misses that treatment.

Spell the repo explicitly as ghcr.io/beacon-stack/pilot instead of
using the expression. The literal never needs to change and matches
the canonical org slug GitHub resolves case-insensitively.

Co-Authored-By: Claude Opus 4.6 (1M context) <noreply@anthropic.com>
@davidfic davidfic merged commit 3e99847 into main Apr 16, 2026
7 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.

1 participant