ci/release: Fix vm-builder artifact paths (#1260)

Follow-up to #1259 😞
neondatabase · Feb 13, 2025 · ac88965 · ac88965
1 parent 14bfadb
commit ac88965
Showing 1 changed file with 9 additions and 4 deletions.
diff --git a/.github/workflows/release.yaml b/.github/workflows/release.yaml
@@ -69,8 +69,6 @@ jobs:
         uses: actions/download-artifact@v4
         with:
           pattern: 'vm-builder-*'
-          path: vm-builder
-          merge-multiple: true # place the various vm-builder-$ARCH binaries into this dir
 
       - name: download manifests
         uses: actions/download-artifact@v4
@@ -80,14 +78,21 @@ jobs:
           # See e2e-test.yaml: the individual yamls are flattened inside the artifact.
           path: rendered_manifests
 
+      - name: rename vm-builder files
+        run: |
+          # add the arch-specific suffix to the binary name, so they don't conflict when used as
+          # assets attached to the release.
+          mv vm-builder-amd64/vm-builder vm-builder-amd64/vm-builder-amd64
+          mv vm-builder-arm64/vm-builder vm-builder-arm64/vm-builder-arm64
+
       - name: github release
         if: ${{ needs.get-tag.outputs.dry-run == 'false' }}
         uses: softprops/action-gh-release@v2
         with:
           fail_on_unmatched_files: true
           files: |
-            vm-builder/vm-builder-arm64
-            vm-builder/vm-builder-amd64
+            vm-builder-amd64/vm-builder-amd64
+            vm-builder-arm64/vm-builder-arm64
             rendered_manifests/autoscale-scheduler.yaml
             rendered_manifests/autoscaler-agent.yaml
             rendered_manifests/neonvm.yaml