fix(packaging): align ghcr image name to ghcr.io/nullislabs/shepherd#220
Open
lgahdl wants to merge 3 commits into
Open
fix(packaging): align ghcr image name to ghcr.io/nullislabs/shepherd#220lgahdl wants to merge 3 commits into
lgahdl wants to merge 3 commits into
Conversation
docker-compose.yml and docs referenced the old ghcr.io/bleu/nullis-shepherd name while the publish workflow pushes to ghcr.io/nullislabs/shepherd (github.repository). Fresh-server docker compose pull was fetching the wrong image or failing. Closes nullislabs#125
The git clone and systemd Documentation= URLs in deployment docs still pointed to the old github.com/bleu/nullis-shepherd location. Fix them to match the canonical nullislabs/shepherd repo used everywhere else. Historical references in m2/m3 runbooks and ADRs are left unchanged.
Collaborator
|
Great catch on the image name. I think there's one spot that was missed though — line 25 in It's commented out, but since this is the operator template (the deployment docs say "cp .env.example .env"), wonder if someone uncommenting it and just changing the SHA tag would still end up pulling from the old path. Maybe update it here too since it's the same class of issue this PR is fixing. |
jean-neiverth
approved these changes
Jul 10, 2026
…v.example Review catch: the docker.md runbook has fresh-server operators cp .env.example .env, and anyone uncommenting SHEPHERD_IMAGE to pin a build pulled the foreign-org image - exactly the failure mode nullislabs#125 describes. Remaining old-name hits are historical PR pointers in ADR-0009 and the M2/M3 runbooks, left intentionally. Also corrects the docker.yml header: tag pushes publish v<tag> + sha-<sha>; `latest` only moves on default-branch pushes per the metadata-action is_default_branch gate.
mfw78
added
the
base: develop
lgahdl
force-pushed
the
fix/125-ghcr-image-name
branch
from
ece6ebd to
0e8660d
Compare
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
3 participants
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Summary
docker-compose.ymlanddocs/deployment/docker.mdreferencedghcr.io/bleu/nullis-shepherdwhile the publish workflow pushes toghcr.io/nullislabs/shepherd(${{ github.repository }}). On a fresh server,docker compose pullwas fetching the wrong image or failing outright — directly breaking the grant's "Docker image runs on a fresh server with minimal configuration" criterion.ghcr.io/nullislabs/shepherdname.Changes
docker-compose.ymlimage:default + comment →ghcr.io/nullislabs/shepherddocs/deployment/docker.md.github/workflows/docker.yml<owner>/nullis-shepherd)Closes #125
Test plan
docker compose pullon a fresh machine will now resolve to the correct imageAI Assistance: Claude Code used for the image-name sweep and this PR body.