Skip to content

fix: CI appVersion mismatch + Helm OCI namespace collision#67

Merged
thepagent merged 1 commit intomainfrom
fix/ci-appversion-oci
Apr 6, 2026
Merged

fix: CI appVersion mismatch + Helm OCI namespace collision#67
thepagent merged 1 commit intomainfrom
fix/ci-appversion-oci

Conversation

@thepagent
Copy link
Copy Markdown
Collaborator

@thepagent thepagent commented Apr 6, 2026

Problem

Two CI design flaws caused 0.3.1-beta.1 deployment to fail (see #66):

  1. appVersion points to non-existent image — bump-chart created a PR to update Chart.yaml; when merged, the new commit SHA had no corresponding Docker image
  2. Helm chart and Docker image pushed to same GHCR repohelm push to oci://ghcr.io/thepagent collided with Docker images at ghcr.io/thepagent/agent-broker

Changes

build.yml

  • bump-chart now pushes directly to main with [skip ci] instead of creating a PR
  • Explicitly captures image SHA from the build commit, ensuring appVersion always matches an existing image

release.yml

  • helm push now targets oci://ghcr.io/<owner>/charts instead of oci://ghcr.io/<owner>
  • Updated OCI install instructions in release notes accordingly

Impact

  • No more manual --set image.tag= override needed
  • helm install default image tag is always correct
  • OCI install path changes to oci://ghcr.io/thepagent/charts/agent-broker

Fixes #66

1. bump-chart: push directly to main with [skip ci] instead of creating
   a PR. This ensures appVersion in Chart.yaml always matches the actual
   image SHA from the build that triggered it. Previously the bump PR
   merge created a new commit SHA that had no corresponding image.

2. release.yml: push Helm charts to oci://ghcr.io/<owner>/charts instead
   of oci://ghcr.io/<owner> to avoid namespace collision with Docker
   images in the same GHCR repo.

Fixes #66
@thepagent thepagent merged commit 1037b6d into main Apr 6, 2026
@thepagent thepagent deleted the fix/ci-appversion-oci branch April 6, 2026 05:25
Reese-max pushed a commit to Reese-max/openab that referenced this pull request Apr 12, 2026
)

1. bump-chart: push directly to main with [skip ci] instead of creating
   a PR. This ensures appVersion in Chart.yaml always matches the actual
   image SHA from the build that triggered it. Previously the bump PR
   merge created a new commit SHA that had no corresponding image.

2. release.yml: push Helm charts to oci://ghcr.io/<owner>/charts instead
   of oci://ghcr.io/<owner> to avoid namespace collision with Docker
   images in the same GHCR repo.

Fixes openabdev#66

Co-authored-by: thepagent <thepagent@users.noreply.github.com>
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.

bug: CI appVersion points to non-existent image + Helm OCI namespace collision with Docker images

2 participants