updates top chart to use oci images (#663)

nucleuscloud · Nov 27, 2023 · e38f4fa · e38f4fa
1 parent 6367e44
commit e38f4fa
Showing 1 changed file with 10 additions and 12 deletions.
diff --git a/.github/workflows/artifact-release.yml b/.github/workflows/artifact-release.yml
@@ -339,9 +339,9 @@ jobs:
   helm-top:
     runs-on: ubuntu-latest
     needs:
-      - docker-backend
-      - docker-frontend
-      - docker-worker
+      - helm-backend
+      - helm-frontend
+      - helm-worker
     steps:
       - name: Checkout
         uses: actions/checkout@v4
@@ -365,15 +365,13 @@ jobs:
               yq -i '.appVersion = strenv(CHART_VERSION)' "$filepath"
             done
 
-            # the top level helm chart references local copies which are set to version: 0
-            # as such, we don't want to set their versions because then helm will error saying it can't find them when we run a "helm dep up"
-            # in the future we can uncomment this if we want the top level chart to reference the real oci artifacts instead of local
-            # for filepath in $(find charts/*/Chart.yaml -type f -name 'Chart.yaml') ; do
-            #   has_deps=$(yq -r '.dependencies[0].version' "$filepath")
-            #   if [ $has_deps != null ]; then
-            #     yq -i '.dependencies[].version = strenv(CHART_VERSION)' "$filepath"
-            #   fi
-            # done
+            for filepath in $(find charts/*/Chart.yaml -type f -name 'Chart.yaml') ; do
+              has_deps=$(yq -r '.dependencies[0].version' "$filepath")
+              if [ $has_deps != null ]; then
+                yq -i '.dependencies[].version = strenv(CHART_VERSION)' "$filepath"
+                yq -i '.dependencies[].repository = oci://ghcr.io/strenv(GITHUB_REPOSITORY)/helm' "$filepath"
+              fi
+            done
 
       - name: Install Helm
         uses: azure/setup-helm@v3