From 159fbc2c5a23884df89fa627782659c1cabbf64f Mon Sep 17 00:00:00 2001
From: hanabi1224 <harlowmoo@gmail.com>
Date: Fri, 10 May 2024 22:53:25 +0800
Subject: [PATCH] simplify api compare test

---
 .github/workflows/forest.yml                 | 23 --------------------
 scripts/tests/api_compare/docker-compose.yml |  3 +++
 2 files changed, 3 insertions(+), 23 deletions(-)

diff --git a/.github/workflows/forest.yml b/.github/workflows/forest.yml
index b1063090cb6..24ff41be1ef 100644
--- a/.github/workflows/forest.yml
+++ b/.github/workflows/forest.yml
@@ -376,20 +376,6 @@ jobs:
       # We use a custom Dockerfile for CI to speed up the build process.
       FOREST_DOCKERFILE_OVERRIDE: scripts/devnet/forest_ci.dockerfile
     steps:
-      - uses: actions/cache@v4
-        id: cache
-        with:
-          path: '${{ env.FIL_PROOFS_PARAMETER_CACHE }}'
-          key: proof-params-keys
-      - name: Load cache into volume
-        if: steps.cache.outputs.cache-hit == 'true'
-        run: |
-          docker volume create api_compare_filecoin-proofs
-          docker run --rm \
-            -v api_compare_filecoin-proofs:/proofs \
-            -v $FIL_PROOFS_PARAMETER_CACHE:/cache \
-            $SHELL_IMAGE \
-            sh -c "cp /cache/* /proofs"
       - uses: actions/checkout@v4
       - uses: actions/download-artifact@v4
         with:
@@ -400,15 +386,6 @@ jobs:
       - name: Dump docker logs
         if: always()
         uses: jwalton/gh-docker-logs@v2
-      - name: Prepare cache folder for uploading
-        if: steps.cache.outputs.cache-hit != 'true'
-        run: |
-          docker run --rm \
-            -v api_compare_filecoin-proofs:/proofs \
-            -v $FIL_PROOFS_PARAMETER_CACHE:/cache \
-            $SHELL_IMAGE \
-            sh -c "cp /proofs/* /cache"
-          sudo chmod -R 755 $FIL_PROOFS_PARAMETER_CACHE
   snapshot-parity-checks:
     needs:
       - build-ubuntu
diff --git a/scripts/tests/api_compare/docker-compose.yml b/scripts/tests/api_compare/docker-compose.yml
index cd97ecb55f4..d6413f101f9 100644
--- a/scripts/tests/api_compare/docker-compose.yml
+++ b/scripts/tests/api_compare/docker-compose.yml
@@ -6,6 +6,7 @@ services:
       dockerfile: aria2-Dockerfile
     volumes:
       - node-data:/data
+      - filecoin-proofs:${FIL_PROOFS_PARAMETER_CACHE}
     networks:
       - api-tests
     environment:
@@ -14,6 +15,8 @@ services:
     command:
       - |
         set -euxo pipefail
+        # fetch parameter files
+        forest-tool fetch-params --keys
         # if there are some files in the data directory, then we don't need to fetch the snapshot
         if [ "$$(ls -A /data/*.car.zst)" ]; then
           echo "Snapshot already fetched"