Skip to content

packaging: canonicalise the ghcr image name to nullislabs/shepherd#476

Open
mfw78 wants to merge 1 commit into
feat/m5-ci-sccache-fail-openfrom
feat/m5-ghcr-image-align
Open

packaging: canonicalise the ghcr image name to nullislabs/shepherd#476
mfw78 wants to merge 1 commit into
feat/m5-ci-sccache-fail-openfrom
feat/m5-ghcr-image-align

Conversation

@mfw78

@mfw78 mfw78 commented Jul 18, 2026

Copy link
Copy Markdown
Contributor

What

Aligns .env.example, docker-compose.yml and docs/deployment/docker.md to the ghcr.io/nullislabs/shepherd image name actually pushed by .github/workflows/docker.yml (ghcr.io/${{ github.repository }}).

Why

Compose and docs still referenced the stale ghcr.io/bleu/nullis-shepherd name, so docker compose pull on a fresh server per the deployment runbook fails or fetches a foreign-org image.

Closes #125

Testing

Manual grep for the old bleu/nullis-shepherd name across the repo; none remain.

AI Assistance

Drafted with AI assistance under review.

The publish workflow pushes ghcr.io/${{ github.repository }}, so the
compose default, the runbook, and the .env.example sample now name
ghcr.io/nullislabs/shepherd; the stale ghcr.io/bleu/nullis-shepherd
references made docker compose pull fail on a fresh server.

@lgahdl lgahdl left a comment

Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Another duplicate-effort case: PR #220 (mine, still open) fixes the same issue (#125) and covers everything here plus one file this PR misses — docs/production.md's systemd unit Documentation= field, which still points at https://github.com/bleu/nullis-shepherd. Worth folding that fix in here too (or merging #220's version of docs/production.md alongside this), otherwise the systemd docs URL stays stale after this lands. Everything else checked matches exactly — verified no bleu/nullis-shepherd reference remains in any of the four files this PR does touch. Approving, but flagging the gap.

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