Skip to content

chore(release): retarget image and chart publish/pull to the llm-d org#317

Merged
jtechapps merged 1 commit into
llm-d:mainfrom
shimib:chore/migrate-ghcr-to-llm-d
Jul 15, 2026
Merged

chore(release): retarget image and chart publish/pull to the llm-d org#317
jtechapps merged 1 commit into
llm-d:mainfrom
shimib:chore/migrate-ghcr-to-llm-d

Conversation

@shimib

@shimib shimib commented Jul 15, 2026

Copy link
Copy Markdown
Member

What does this PR do?

Retargets image and Helm-chart publishing and pulling from ghcr.io/llm-d-incubation to ghcr.io/llm-d, following the repo's transfer into the llm-d org. GitHub redirects the git repo on transfer, but GHCR packages do not follow a repo transfer, so every publish/pull path had to move explicitly.

Changes

Publish (release):

  • .github/workflows/ci-release.yaml — build/push the release image and trivy-scan it under ghcr.io/llm-d.
  • .github/workflows/ci-build-images.yaml — reusable build workflow registry/image.
  • scripts/publish-helm-chart.sh — push the chart to oci://ghcr.io/llm-d/charts.

Pull (consumers):

  • charts/async-processor/values.yaml — default image repository → ghcr.io/llm-d/llm-d-async (this is the one end users hit on helm install), plus the chart unit-test expectations.
  • deploy/install.sh, the e2e default image (test/e2e/e2e_suite_test.go), and docs/guides/e2e-deploy*.

Local/dev: Makefile IMAGE_TAG_BASE.

Out of scope (intentional)

  • Go module paths (github.com/llm-d-incubation/llm-d-async/... in go.mods and imports) are left unchanged. They are self-consistent and resolve via GitHub's repo redirect, so builds and go get keep working; renaming the module path is a separate breaking change for downstream consumers (e.g. the batch-gateway operator) and warrants its own PR.

⚠️ Ops prerequisite (not code)

Merging this is necessary but not sufficient — a release will only publish once an llm-d org admin ensures the release workflow's GHCR_TOKEN has write:packages on the llm-d org and the llm-d-async image + charts/async-processor packages can be created there. Existing ghcr.io/llm-d-incubation/* packages keep working via their old path until deleted, so pinned consumers aren't broken immediately.

How was this tested?

  • helm template → default image renders ghcr.io/llm-d/llm-d-async:0.7.0
  • helm lint clean; helm unittest 49/49 pass
  • go vet -tags e2e ./test/e2e/ ok

Release note

Image and Helm chart are now published to and pulled from ghcr.io/llm-d (was ghcr.io/llm-d-incubation).

The repo moved from llm-d-incubation to the llm-d org, but GHCR packages do not
follow a repo transfer, so every publish/pull path still targeted
ghcr.io/llm-d-incubation. Retarget them to ghcr.io/llm-d:

- ci-release.yaml / ci-build-images.yaml: build, push, and scan images under ghcr.io/llm-d
- scripts/publish-helm-chart.sh: push the chart to oci://ghcr.io/llm-d/charts
- charts/async-processor/values.yaml: default image pull repository -> ghcr.io/llm-d/llm-d-async
  (and the chart unit-test expectations to match)
- Makefile IMAGE_TAG_BASE, deploy/install.sh, the e2e default image, and the e2e-deploy docs

Go module paths (github.com/llm-d-incubation/...) are intentionally left unchanged:
they are self-consistent and resolve via GitHub's repo redirect; renaming them is a
separate breaking change for downstream consumers.

Verified: helm template renders ghcr.io/llm-d/llm-d-async, helm lint clean,
helm unittest 49/49 pass, go vet -tags e2e ok.

Signed-off-by: Shimi Bandiel <shimib@google.com>
@jtechapps
jtechapps merged commit 77d087c into llm-d:main Jul 15, 2026
8 checks passed
@j-mok-dev

Copy link
Copy Markdown
Contributor

@jtechapps @shimib The release workflow (https://github.com/llm-d/llm-d-async/actions/runs/29516135464) is failing on docker-build-and-push after the github.token migration, so ghcr.io/llm-d/llm-d-async:v0.7.4 isn't published yet.
We're trying to consume v0.7.4 downstream (llm-d/llm-d-batch-gateway#590). any ETA on the fix?

@jtechapps

Copy link
Copy Markdown
Collaborator

@j-mok-dev We are working to resolve this ASAP looks like we missed creating package. #327

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.

3 participants