Merge pull request #21 from OpenAstroTech/feature/js/19-optimize-windows-ci-build-upload-artifact

GHA faster upload/download

Author: julianneswinoga

.github/workflows/build-and-release.yaml

@@ -76,14 +76,18 @@ jobs:
     - name: Remove smoke test artifacts
       run: |
         rm -rv ./dist/logs
-    # Work around stupid issue that GHA can't store file permissions on artifacts
-    - name: Store which files are executable
-      run: pushd dist/ && find . -type f -executable -print > restore_executable_permissions.txt && popd
+        rm -rv ./dist/OATFWGUI/__pycache__
+    - name: Set artifact name
+      run: echo "ARTIFACT_ZIP_NAME=OATFWGUI_${{ env.OATFWGUI_VERSION }}_${{ runner.os }}_${{ runner.arch }}" >> $GITHUB_ENV
+    - name: Rename and Zip artifacts
+      run: mv dist $ARTIFACT_ZIP_NAME && 7z a -tzip $ARTIFACT_ZIP_NAME.zip $ARTIFACT_ZIP_NAME/
+      env:
+        ARTIFACT_ZIP_NAME: ${{ env.ARTIFACT_ZIP_NAME }}
     - name: Upload artifact
       uses: actions/upload-artifact@v3
       with:
-        name: OATFWGUI_${{ env.OATFWGUI_VERSION }}_${{ runner.os }}_${{ runner.arch }}
-        path: dist/*
+        name: ${{ env.ARTIFACT_ZIP_NAME }}
+        path: ${{ env.ARTIFACT_ZIP_NAME }}.zip
 
   build-virtualenv:
     runs-on: 'ubuntu-20.04'
@@ -125,16 +129,19 @@ jobs:
       run: |
         rm -rv ./dist/logs
         rm -rv ./dist/.venv_*
-    # Work around stupid issue that GHA can't store file permissions on artifacts
-    - name: Store which files are executable
-      run: pushd dist/ && find . -type f -executable -print > restore_executable_permissions.txt && popd
-
+        rm -rv ./dist/OATFWGUI/__pycache__
+    - name: Set artifact name
+      run: echo "ARTIFACT_ZIP_NAME=OATFWGUI_${{ env.OATFWGUI_VERSION }}_${{ runner.os }}_${{ runner.arch }}" >> $GITHUB_ENV
+    - name: Rename and Zip artifacts
+      run: mv dist $ARTIFACT_ZIP_NAME && 7z a -tzip $ARTIFACT_ZIP_NAME.zip $ARTIFACT_ZIP_NAME/
+      env:
+        ARTIFACT_ZIP_NAME: ${{ env.ARTIFACT_ZIP_NAME }}
     - name: Upload artifact
       if: ${{ matrix.py_version }} == "3.10" # Only publish one artifact
       uses: actions/upload-artifact@v3
       with:
-        name: OATFWGUI_${{ env.OATFWGUI_VERSION }}_${{ runner.os }}_${{ runner.arch }}
-        path: dist/*
+        name: ${{ env.ARTIFACT_ZIP_NAME }}
+        path: ${{ env.ARTIFACT_ZIP_NAME }}.zip
 
   publish-release:
     # only publish release on tags
@@ -150,41 +157,17 @@ jobs:
         uses: actions/download-artifact@v3
         with:
           path: artifacts
-      - name: Restore executable permissions
-        run: |
-          set -e
-          shopt -s nullglob
-          PERM_FILE='restore_executable_permissions.txt'
-          for dir in artifacts/*/; do
-            echo "Entering $dir"
-            pushd "$dir"
-            if [ ! -f "$PERM_FILE" ]; then
-              echo "Could not find permissions file ${PERM_FILE}!"
-              continue
-            fi
-            while IFS= read -r f_path || [ -n "$f_path" ]; do
-              echo "Restoring execution on $f_path"
-              chmod +x "$f_path"
-            done < "$PERM_FILE"
-            rm -v "$PERM_FILE"
-            popd
-          done
-      - name: Zip artifacts
+      - name: Show artifact contents
         run: |
-          set -e
-          shopt -s nullglob
-          pushd artifacts/
-          for dir in ./*/; do
-            out_zip="../$(basename $dir).zip"
-            echo "Zipping $dir into $out_zip"
-            zip -8 --recurse-paths "$out_zip" "$dir"
+          tree .
+          for zip_dist in artifacts/*/*.zip; do
+            echo "$zip_dist contains:"
+            zip -sf $zip_dist
           done
-          popd
       - name: Publish GitHub release
         uses: softprops/action-gh-release@v1
         with:
           name: ${{ github.ref_name }}
           generate_release_notes: true
           fail_on_unmatched_files: true
-          files: |
-            *.zip
+          files: artifacts/*/*.zip