rok commented on code in PR #50047:
URL: https://github.com/apache/arrow/pull/50047#discussion_r3340912811


##########
.github/workflows/cpp.yml:
##########
@@ -149,6 +149,14 @@ jobs:
           sudo sysctl -w vm.mmap_rnd_bits=28
           source ci/scripts/util_enable_core_dumps.sh
           archery docker run ${{ matrix.image }}
+      - name: Save Docker Volumes
+        if: ${{ !cancelled() }}
+        continue-on-error: true
+        uses: 
apache/infrastructure-actions/stash/save@0ba14156c9f4c3cfbe4b0c9f36339ab0f8d81e53
+        with:
+          path: .docker
+          key: ${{ matrix.image }}

Review Comment:
   Good point. I’ll disambiguate it and include the env-derived matrix values 
in the key so this workflow doesn’t share a stash entry with other 
workflows/images that happen to use the same Docker image name.



-- 
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.

To unsubscribe, e-mail: [email protected]

For queries about this service, please contact Infrastructure at:
[email protected]

Reply via email to