Skip to content

Commit 7b998f3

Browse files
vigneshrajsbclaude
andcommitted
fix(nativeHelm): support OCI repoUrl override for ORG_CHART deploys
constructHelmCommand now detects OCI URLs in the ORG_CHART branch and uses the full chartRepoUrl as the chart reference instead of the bare chart name. generateHelmInstallScript extended to run helm repo add for ORG_CHART when a non-OCI custom repoUrl is present, matching the existing PUBLIC chart behavior. Co-Authored-By: Claude Sonnet 4.6 <noreply@anthropic.com>
1 parent 10bb198 commit 7b998f3

File tree

1 file changed

+8
-3
lines changed

1 file changed

+8
-3
lines changed

src/server/lib/nativeHelm/utils.ts

Lines changed: 8 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -226,7 +226,12 @@ export function constructHelmCommand(
226226
command += ` ${chartPath}`;
227227
}
228228
} else {
229-
command += ` ${chartPath}`;
229+
const isOciChart = chartRepoUrl?.startsWith('oci://');
230+
if (isOciChart) {
231+
command += ` ${chartRepoUrl}`;
232+
} else {
233+
command += ` ${chartPath}`;
234+
}
230235
}
231236

232237
command += ` --namespace ${namespace}`;
@@ -305,11 +310,11 @@ ls -la
305310
`;
306311
}
307312

308-
if (chartType === ChartType.PUBLIC) {
313+
if (chartType === ChartType.PUBLIC || chartType === ChartType.ORG_CHART) {
309314
const isOciChart = chartRepoUrl?.startsWith('oci://');
310315

311316
if (!isOciChart) {
312-
if (chartPath.includes('/')) {
317+
if (chartType === ChartType.PUBLIC && chartPath.includes('/')) {
313318
const [repoName] = chartPath.split('/');
314319
const repoUrl = getRepoUrl(repoName);
315320
script += `

0 commit comments

Comments
 (0)