Bug: correct release chart ECR Full URL (#242)

* Bug: correct release chart ECR Full URL * removed unused variable in shell script
voiceflow · May 27, 2024 · d9f8f9c · d9f8f9c
1 parent b020d6e
commit d9f8f9c
Showing 1 changed file with 1 addition and 2 deletions.
diff --git a/src/scripts/helm/package-publish-charts-ecr.sh b/src/scripts/helm/package-publish-charts-ecr.sh
@@ -26,7 +26,6 @@ for chart in ${CHARTS?}; do
 
     # Get the chart version from the packaged chart
     packaged_chart="$(ls "$dist")"
-    chart_version=$(helm show chart "$dist/$packaged_chart" | yq --raw-output '.version')
     channel=$(helm show chart "$chart/$chart" | yq --raw-output '.annotations."release-repository"')
     echo "Publishing in $channel channel"
 
@@ -36,7 +35,7 @@ for chart in ${CHARTS?}; do
     fi
 
     # Construct the full ECR URL with the OCI protocol
-    FULL_ECR_URL="oci://$ECR_REPOSITORY_URI/$repo/$chart:$chart_version"
+    FULL_ECR_URL="oci://$ECR_REPOSITORY_URI/$repo"
 
     # Push the chart to ECR using the OCI protocol
     helm push "$dist/$packaged_chart" "$FULL_ECR_URL"