[CI] Add cachepot -- part2: use cache

quasar-finance · Aug 23, 2024 · b08c48a · b08c48a
1 parent 88802b1
commit b08c48a
Show file tree

Hide file tree

Showing 4 changed files with 67 additions and 49 deletions.
diff --git a/.github/workflows/all_checks_osmosis.yml b/.github/workflows/all_checks_osmosis.yml
@@ -3,6 +3,10 @@ name: Test Tube (osmosis)
 on:
   workflow_call:
     inputs:
+      unittest:
+        required: false
+        default: false
+        type: boolean
       proptest:
         required: false
         default: false
@@ -25,20 +29,22 @@ jobs:
         uses: actions/checkout@v3
       - name: Install Rust
         uses: dtolnay/rust-toolchain@stable
+      - name: Disk Usage
+        run: df -h
       - name: Cleanup runner
-        run: sudo rm -rf /usr/local/share/boost && sudo rm -rf /usr/share/dotnet && sudo rm -rf /usr/local/lib/android && sudo rm -rf /opt/ghc && sudo rm -rf /opt/hostedtoolcache/CodeQL && sudo docker image prune --all --force
+        run: sudo rm -rf /usr/local/share/boost && sudo rm -rf /usr/share/dotnet && sudo rm -rf /usr/local/lib/android && sudo rm -rf /opt/ghc && sudo rm -rf /opt/hostedtoolcache/CodeQL && sudo docker system prune --all --force
+      - name: Disk Usage
+        run: df -h
       - name: Restore dependencies
-        id: restore-cache
+        id: restore_cache
         uses: actions/cache/restore@v4
         with:
           path: |
-            ~/.cache/cachepot
             ~/.cargo
             ~/go
-            ~/target
-          key: ${{ runner.os }}-cargo-osmosis-${{github.sha}}
-          restore-keys: ${{ runner.os }}-cargo-osmosis
-      - name: Update PATH and RUSTC_WRAPPER
+          key: ${{ runner.os }}-cache-osmosis-${{ github.sha }}
+          restore-keys: ${{ runner.os }}-cache-osmosis
+      - name: Update PATH
         run: echo "$HOME/.cargo/bin" >> $GITHUB_PATH
       - name: Install cachepot
         run: test -e $HOME/.cargo/bin/cachepot && echo "cachepot found -- skipping install" || cargo install --git https://github.com/paritytech/cachepot
@@ -47,41 +53,40 @@ jobs:
       - name: Rust lint
         run:  cargo clippy --features test-tube --all-targets -- -D warnings
         working-directory: smart-contracts/osmosis
-      - name: Unit tests
-        run:  cargo unit-test
-        working-directory: smart-contracts/osmosis
-      - name: Build merkle-incentives
-        run:  cargo test-tube-build
-        working-directory: smart-contracts/osmosis/contracts/merkle-incentives
-      - name: Test merkle-incentives
-        run:  cargo test-tube
-        working-directory: smart-contracts/osmosis/contracts/merkle-incentives
-      - name: Build dex-router-osmosis
-        run:  cargo test-tube-build
-        working-directory: smart-contracts/osmosis/contracts/dex-router-osmosis
-      - name: Test dex-router-osmosis
-        run:  cargo test-tube
-        working-directory: smart-contracts/osmosis/contracts/dex-router-osmosis
-      - name: Build cl-vault
-        run:  cargo test-tube-build
-        working-directory: smart-contracts/osmosis/contracts/cl-vault
-      - name: Test cl-vault
-        run:  cargo test-tube
-        working-directory: smart-contracts/osmosis/contracts/cl-vault
-      - if: inputs.proptest
-        name: Run prop-test
-        run:  cargo prop-test
-        env:
-          PROPTEST_CASES: 10
-        working-directory: smart-contracts/osmosis/contracts/cl-vault
-      - name: Store dependencies
-        if: inputs.store_deps
+      # - if: inputs.unittest
+      #   name: Unit tests
+      #   run:  cargo unit-test
+      #   working-directory: smart-contracts/osmosis
+      # - name: Build merkle-incentives
+      #   run:  cargo test-tube-build
+      #   working-directory: smart-contracts/osmosis/contracts/merkle-incentives
+      # - name: Test merkle-incentives
+      #   run:  cargo test-tube
+      #   working-directory: smart-contracts/osmosis/contracts/merkle-incentives
+      # - name: Build dex-router-osmosis
+      #   run:  cargo test-tube-build
+      #   working-directory: smart-contracts/osmosis/contracts/dex-router-osmosis
+      # - name: Test dex-router-osmosis
+      #   run:  cargo test-tube
+      #   working-directory: smart-contracts/osmosis/contracts/dex-router-osmosis
+      # - name: Build cl-vault
+      #   run:  cargo test-tube-build
+      #   working-directory: smart-contracts/osmosis/contracts/cl-vault
+      # - name: Test cl-vault
+      #   run:  cargo test-tube
+      #   working-directory: smart-contracts/osmosis/contracts/cl-vault
+      # - if: inputs.proptest
+      #   name: Run prop-test
+      #   run:  cargo prop-test
+      #   env:
+      #     PROPTEST_CASES: 10
+      #   working-directory: smart-contracts/osmosis/contracts/cl-vault
+      - if: ${{ (inputs.store_deps) && (steps.restore_cache.outputs.cache-hit == false) }}
+        name: Store dependencies
         uses: actions/cache/save@v4
         with:
           path: |
-            ~/.cache/cachepot
             ~/.cargo
             ~/go
-            ~/target
-          key: ${{ runner.os }}-cargo-osmosis-${{github.sha}}
+          key: ${{ runner.os }}-cache-osmosis-${{ github.sha }}
 
diff --git a/.github/workflows/cargo_build_cache.yml b/.github/workflows/cargo_build_cache.yml
@@ -7,6 +7,9 @@ on:
         required: true
         type: string
 
+env:
+  CARGO_TARGET_DIR: ~/target
+
 jobs:
   store-build-cache:
     runs-on: ubuntu-latest
@@ -22,11 +25,11 @@ jobs:
             ~/.cache/cachepot
             ~/.cargo
             ~/go
-            smart-contracts/${{ inputs.workspace }}/**/target
+            ~/target
           key: ${{ runner.os }}-cargo-${{ inputs.workspace }}-${{github.sha}}
           restore-keys: ${{ runner.os }}-cargo-${{ inputs.workspace }}
       - name: Install cachepot
-        run: test -e $HOME/.cargo/bin/cachepot && echo "cachepot found -- skipping install" || cargo install --git https://github.com/paritytech/cachepot
+        run: test -e ~/.cargo/bin/cachepot && echo "cachepot found -- skipping install" || cargo install --git https://github.com/paritytech/cachepot
       - name: Rust check
         run:  cargo check --all-targets --features test-tube
         working-directory: smart-contracts/${{ inputs.workspace }}
@@ -39,5 +42,5 @@ jobs:
             ~/.cache/cachepot
             ~/.cargo
             ~/go
-            smart-contracts/${{ inputs.workspace }}/**/target
+            ~/target
           key: ${{ runner.os }}-cargo-${{ inputs.workspace }}-${{github.sha}}
diff --git a/.github/workflows/rust_basic.yml b/.github/workflows/rust_basic.yml
@@ -14,6 +14,10 @@ on:
         default: true
         type: boolean
 
+env:
+  RUSTC_WRAPPER: cachepot
+  CARGO_TARGET_DIR: ~/target
+
 jobs:
   checks:
     runs-on: ubuntu-latest
@@ -23,21 +27,23 @@ jobs:
       - name: Install Rust
         uses: dtolnay/rust-toolchain@stable
       - name: Restore dependencies
+        id: restore-cache
         uses: actions/cache/restore@v4
         with:
           path: |
             ~/.cargo
             ~/go
-            smart-contracts/${{ inputs.workspace }}/**/target
-          key: ${{ runner.os }}-cargo-${{ inputs.workspace }}-$GITHUB_SHA
-          restore-keys: ${{ runner.os }}-cargo-${{ inputs.workspace }}
+          key: ${{ runner.os }}-cache-osmosis-${{ github.sha }}
+          restore-keys: ${{ runner.os }}-cache-osmosis
+      - name: Update PATH
+        run: echo "$HOME/.cargo/bin" >> $GITHUB_PATH
       - name: Rust lint
-        run: cargo lint
-        working-directory: smart-contracts/${{ inputs.workspace }}/${{ inputs.target }}
+        run:  cargo clippy --features test-tube --all-targets -- -D warnings
+        working-directory: smart-contracts/${{ inputs.workspace }}
       - name: Rust format check
         run: cargo fmt --all -- --check
         working-directory: smart-contracts/${{ inputs.workspace }}/${{ inputs.target }}
-      - name: Run unit-tests
-        if: inputs.unit_tests
+      - if: inputs.unit_tests
+        name: Run unit-tests
         run: cargo unit-test
         working-directory: smart-contracts/${{ inputs.workspace }}/${{ inputs.target }}
diff --git a/.github/workflows/test_tube.yml b/.github/workflows/test_tube.yml
@@ -24,5 +24,9 @@ jobs:
     uses: ./.github/workflows/all_checks_osmosis.yml
     if: github.event_name == 'push'
     with:
+      unittest: true
       proptest: true
       store_deps: true
+  pre-commit:
+    uses: ./.github/workflows/all_checks_osmosis.yml
+    if: github.event_name == 'pull_request'