CI: cache pipenv dependencies (#174)

crowdsecurity · Sep 20, 2024 · 2488cbe · 2488cbe
1 parent d2a205e
commit 2488cbe
Show file tree

Hide file tree

Showing 2 changed files with 23 additions and 18 deletions.
diff --git a/.github/workflows/tests.yml b/.github/workflows/tests.yml
@@ -38,22 +38,24 @@ jobs:
       env:
         RICHGO_FORCE_COLOR: 1
 
-    - name: Cache virtualenvs
-      id: cache-pipenv
-      uses: actions/cache@v4
+    - name: "Setup Python"
+      uses: actions/setup-python@v5
       with:
-        path: ~/.local/share/virtualenvs
-        key: ${{ runner.os }}-pipenv-${{ hashFiles('**/Pipfile.lock') }}
+        python-version: "3.x"
+        cache: 'pipenv'
 
-    - name: Install functional test dependencies
+    - name: "Install python dependencies"
       run: |
-        docker network create net-test
-        python3 -m pip install --upgrade pipenv wheel
+        python -m pip install --upgrade pipenv wheel
         pipenv install --deploy
         # some tests need root, so we have to install pytest twice
         sudo python3 -m pip install --upgrade pipenv wheel
         sudo pipenv install --deploy
 
+    - name: Install functional test dependencies
+      run: |
+        docker network create net-test
+
     - name: Run functional tests
       env:
         CROWDSEC_TEST_VERSION: dev

diff --git a/.github/workflows/tests_deb.yml b/.github/workflows/tests_deb.yml
@@ -24,24 +24,27 @@ jobs:
     - name: Set up Go
       uses: actions/setup-go@v5
       with:
-        go-version: "1.22"
+        go-version: '1.22'
 
-    - name: Cache virtualenvs
-      id: cache-pipenv
-      uses: actions/cache@v4
+    - name: "Setup Python"
+      uses: actions/setup-python@v5
       with:
-        path: ~/.local/share/virtualenvs
-        key: ${{ runner.os }}-pipenv-${{ hashFiles('**/Pipfile.lock') }}
+        python-version: "3.x"
+        cache: 'pipenv'
+
+    - name: "Install python dependencies"
+      run: |
+        python -m pip install --upgrade pipenv wheel
+        pipenv install --deploy
+        # some tests need root, so we have to install pytest twice
+        sudo python3 -m pip install --upgrade pipenv wheel
+        sudo pipenv install --deploy
 
     - name: Install functional test dependencies
       run: |
         sudo apt update
         sudo apt install -y build-essential debhelper devscripts fakeroot lintian
         docker network create net-test
-        python3 -m pip install --upgrade pipenv wheel
-        pipenv install --deploy
-        sudo python3 -m pip install --upgrade pipenv wheel
-        sudo pipenv install --deploy
 
     - name: Run functional tests
       env: