Skip to content

fix(ci): lowercase image owner in ghcr tag - #5

Merged
djwmarcx merged 1 commit into
mainfrom
fix/ghcr-lowercase-owner
Jun 23, 2026
Merged

djwmarcx merged 1 commit into
mainfrom
fix/ghcr-lowercase-owner

Conversation

@djwmarcx

Copy link
Copy Markdown
Member

Problem

The v0.0.3 release deploy failed at build:

buildx failed with: ERROR: failed to build: invalid tag
"ghcr.io/OpenHomeFoundation/web-api:v0.0.3": repository name must be lowercase

github.repository_owner is OpenHomeFoundation (mixed case), and ghcr.io requires lowercase repository names.

Fix

Add a step that lowercases the owner and use it when composing the image tag — the same fix already present in slack-budget-app. The login step's username is left untouched since ghcr login is case-insensitive.

github.repository_owner is read via an env: var rather than inline interpolation, per workflow-injection best practice.

ghcr.io rejects uppercase repository names, so the OpenHomeFoundation
owner produced an invalid tag (ghcr.io/OpenHomeFoundation/web-api:...)
and buildx failed. Lowercase github.repository_owner before composing
the image tag, matching the fix already in slack-budget-app.
Copilot AI review requested due to automatic review settings June 23, 2026 09:54

Copilot AI 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.

Pull request overview

Fixes GHCR image tag generation in the release workflow by ensuring the github.repository_owner portion of the image name is lowercased (GHCR requires lowercase repository names), preventing deploy failures like the v0.0.3 “invalid tag” error.

Changes:

  • Add a workflow step that derives a lowercased repository owner and exposes it via step outputs.
  • Update the Docker build/push step to use the lowercased owner when composing the ghcr.io/... image tag.

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

@djwmarcx
djwmarcx merged commit 8010114 into main Jun 23, 2026
1 check passed
@djwmarcx
djwmarcx deleted the fix/ghcr-lowercase-owner branch June 23, 2026 10:39
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.

2 participants