Skip to content

chore(dockerhub): migrate namespace zkcoin → zkcoins - #96

Merged
TaprootFreak merged 1 commit into
developfrom
chore/dockerhub-namespace-zkcoins
May 25, 2026
Merged

chore(dockerhub): migrate namespace zkcoin → zkcoins#96
TaprootFreak merged 1 commit into
developfrom
chore/dockerhub-namespace-zkcoins

Conversation

@TaprootFreak

Copy link
Copy Markdown
Contributor

Why

The Docker Hub namespace was `zkcoin/` (singular) while the rest of
the project identity uses the plural form:

  • GitHub org: `zk-coins`
  • Domain: `zkcoins.app`
  • Cargo package: `node` in workspace `zk-coins/node`
  • API service id: `zkcoins-node`
  • Compose service / container: `zkcoins-node`

`zkcoin/` was a leftover. Migrating now while we're already doing the
identity rename minimises churn.

What

Replaces every `zkcoin/node` and `zkcoin/server` reference in this
repo with `zkcoins/node`:

  • `.github/workflows/deploy-dev.yaml`: `DOCKER_TAGS` + buildx cache refs
  • `.github/workflows/deploy-prd.yaml`: same
  • `Dockerfile`: comment header (build / run examples)
  • `README.md`, `CONTRIBUTING.md`, `ROADMAP.md`, `MIGRATION_RESEARCH.md`: image refs

Repo secrets `DOCKER_USERNAME` / `DOCKER_PASSWORD` already point at the
new `zkcoins` Docker Hub account (set ahead of this PR — that's why
the post-#93 `Deploy DEV` failed with `insufficient_scope` on
`zkcoin/node:beta`).

Coordinated follow-ups

  • DFXServer/server: `infrastructure/dfx{dev,prd}/zkcoins/docker-compose.yaml`
    `image:` lines + READMEs — pushed by me immediately after this PR merges.
  • Old `zkcoin/node` Docker Hub repo stays as a deprecated read-only
    pointer; final deprecation notice + last-tag freeze is Phase 9 of
    the rename rollout.

Test

  • `Lint & Build` should pass (no Rust code changed).
  • After merge: `Deploy DEV` will push `zkcoins/node:beta` to the new
    namespace (creds match), then SSH to dfxdev runs `zkcoins-node`,
    `deploy.sh` accepts it, but `docker compose pull` needs the
    DFXServer/server compose update (separate push) to point at the new
    image — which I'll push as soon as the new image lands on Docker Hub.

Aligns Docker Hub namespace with the rest of the project identity:
GitHub org `zk-coins`, domain `zkcoins.app`, Cargo workspace member
`node` already use the plural form; the singular `zkcoin/*` namespace
was a holdover from the early days.

Touched:
- .github/workflows/deploy-{dev,prd}.yaml — DOCKER_TAGS + cache refs
- Dockerfile — comment header (build/run examples)
- README, CONTRIBUTING, ROADMAP, MIGRATION_RESEARCH — image refs

Coordinated DFXServer/server compose update follows immediately after
merge so dfxdev / dfxprd pull the new `zkcoins/node:{beta,latest}`.
The legacy `zkcoin/node` repo on Docker Hub stays as a deprecated
read-only mirror.
@TaprootFreak TaprootFreak added the ci:full Trigger heavy CI jobs (Server + Shared Tests + Coverage Gate, ~60-90 min on M3 Ultra) label May 25, 2026
@TaprootFreak
TaprootFreak merged commit 643fd79 into develop May 25, 2026
5 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

ci:full Trigger heavy CI jobs (Server + Shared Tests + Coverage Gate, ~60-90 min on M3 Ultra)

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant