Skip to content

Commit 992e8b7

Browse files
fix(ci): publish vendored chart under distinct OCI name
The fork's chart was pushing to ghcr.io/<owner>/charts/trigger, which collides with an existing package in the GovSignals org owned by a different repo — push was 403'd on the blob HEAD probe. Rename the published artifact to vendored-upstream-trigger by rewriting Chart.yaml's name field at package time. CHART_NAME env now drives both the local filename and the OCI path, and the rename only affects the published artifact (the on-disk chart still says name: trigger so upstream contracts and lint/test stay unchanged). Result: ghcr.io/<owner>/charts/vendored-upstream-trigger:<version>. Consumers reference it via dependencies.name=vendored-upstream-trigger (or alias: trigger for backwards-compatible subchart values). Co-authored-by: Cursor <cursoragent@cursor.com>
1 parent b184cf5 commit 992e8b7

2 files changed

Lines changed: 15 additions & 4 deletions

File tree

.github/workflows/helm-prerelease.yml

Lines changed: 7 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -24,7 +24,10 @@ concurrency:
2424

2525
env:
2626
REGISTRY: ghcr.io
27-
CHART_NAME: trigger
27+
# Published OCI chart name. Distinct from upstream `trigger` so the
28+
# GovSignals fork's vendored chart doesn't collide with the upstream
29+
# package under ghcr.io/govsignals/charts/trigger.
30+
CHART_NAME: vendored-upstream-trigger
2831

2932
jobs:
3033
lint-and-test:
@@ -127,9 +130,10 @@ jobs:
127130
echo "version=$PRERELEASE_VERSION" >> $GITHUB_OUTPUT
128131
echo "Prerelease version: $PRERELEASE_VERSION"
129132
130-
- name: Update Chart.yaml with prerelease version
133+
- name: Update Chart.yaml with prerelease version and rename chart
131134
run: |
132135
sed -i "s/^version:.*/version: ${{ steps.version.outputs.version }}/" ./hosting/k8s/helm/Chart.yaml
136+
sed -i "s/^name:.*/name: ${{ env.CHART_NAME }}/" ./hosting/k8s/helm/Chart.yaml
133137
134138
- name: Override appVersion
135139
if: github.event_name == 'workflow_dispatch' && inputs.app_version != ''
@@ -188,7 +192,7 @@ jobs:
188192
**Install:**
189193
```bash
190194
helm upgrade --install trigger \
191-
oci://ghcr.io/${{ steps.owner.outputs.lc }}/charts/trigger \
195+
oci://ghcr.io/${{ steps.owner.outputs.lc }}/charts/${{ env.CHART_NAME }} \
192196
--version "${{ steps.version.outputs.version }}"
193197
```
194198

.github/workflows/release-helm.yml

Lines changed: 8 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -19,7 +19,10 @@ on:
1919

2020
env:
2121
REGISTRY: ghcr.io
22-
CHART_NAME: trigger
22+
# Published OCI chart name. Distinct from upstream `trigger` so the
23+
# GovSignals fork's vendored chart doesn't collide with the upstream
24+
# package under ghcr.io/govsignals/charts/trigger.
25+
CHART_NAME: vendored-upstream-trigger
2326

2427
jobs:
2528
lint-and-test:
@@ -119,6 +122,10 @@ jobs:
119122
fi
120123
echo "✅ Chart.yaml version matches release version."
121124
125+
- name: Rename chart for vendored-upstream OCI publish
126+
run: |
127+
sed -i "s/^name:.*/name: ${{ env.CHART_NAME }}/" ./hosting/k8s/helm/Chart.yaml
128+
122129
- name: Package Helm Chart
123130
run: |
124131
helm package ./hosting/k8s/helm/ --destination /tmp/

0 commit comments

Comments
 (0)