Revert needless changes

apache · Nov 7, 2024 · d2e4700 · d2e4700
1 parent c12be93
commit d2e4700
Showing 1 changed file with 14 additions and 3 deletions.
diff --git a/dev/tasks/python-wheels/github.windows.yml b/dev/tasks/python-wheels/github.windows.yml
@@ -47,9 +47,18 @@ jobs:
           GITHUB_TOKEN: {{ '${{ secrets.GITHUB_TOKEN }}' }}
         run: |
           cd arrow
-          archery docker run ^
-            -e SETUPTOOLS_SCM_PRETEND_VERSION={{ arrow.no_rc_version }} ^
-            python-wheel-windows-vs2019
+          @rem We want to use only
+          @rem   archery docker run -e SETUPTOOLS_SCM_PRETEND_VERSION={{ arrow.no_rc_version }} python-wheel-windows-vs2019
+          @rem but it doesn't use pulled caches.
+          @rem It always build an image from scratch.
+          @rem We can remove this workaround once we find a way to use
+          @rem pulled caches when build an image.
+          echo on
+          archery docker pull --no-ignore-pull-failures python-wheel-windows-vs2019
+          if errorlevel 1 (
+            archery docker build --no-pull python-wheel-windows-vs2019 || exit /B 1
+          )
+          archery docker run --no-build -e SETUPTOOLS_SCM_PRETEND_VERSION={{ arrow.no_rc_version }} python-wheel-windows-vs2019
 
       - uses: actions/upload-artifact@v4
         with:
@@ -66,8 +75,10 @@ jobs:
       {{ macros.github_upload_gemfury("arrow/python/dist/*.whl")|indent }}
       {{ macros.github_upload_wheel_scientific_python("arrow/python/dist/*.whl")|indent }}
 
+      {% if arrow.is_default_branch() %}
       - name: Push Docker Image
         shell: cmd
         run: |
           cd arrow
           archery docker push python-wheel-windows-vs2019
+      {% endif %}