From cca12b90d005e5f43ec202bf175d3cd1a949e925 Mon Sep 17 00:00:00 2001 From: =?UTF-8?q?Yoann=20Rodi=C3=A8re?= Date: Fri, 25 Oct 2024 12:27:06 +0200 Subject: [PATCH] Do not include the whole Gradle cache in GH Actions caches We don't need the whole cache, since most (all?) of it is stored on Develocity already. Including ./gradle/caches/modules-2 is enough, because that's where Maven dependencies are stored, and *those* are not stored on Develocity. (cherry picked from commit 013bb46470f97285ae57190c9ce5201f31105a21) --- .github/workflows/atlas.yml | 8 ++++---- .github/workflows/contributor-build.yml | 8 ++++---- 2 files changed, 8 insertions(+), 8 deletions(-) diff --git a/.github/workflows/atlas.yml b/.github/workflows/atlas.yml index 5f2dd1c2f4..87fa70d03f 100644 --- a/.github/workflows/atlas.yml +++ b/.github/workflows/atlas.yml @@ -64,7 +64,7 @@ jobs: echo "buildtool-monthly-cache-key=${ROOT_CACHE_KEY}-${CURRENT_MONTH}" >> $GITHUB_OUTPUT echo "buildtool-monthly-branch-cache-key=${ROOT_CACHE_KEY}-${CURRENT_MONTH}-${CURRENT_BRANCH}" >> $GITHUB_OUTPUT echo "buildtool-cache-key=${ROOT_CACHE_KEY}-${CURRENT_MONTH}-${CURRENT_BRANCH}-${CURRENT_DAY}" >> $GITHUB_OUTPUT - - name: Cache Maven/Gradle Local Caches + - name: Cache Maven/Gradle Dependency/Dist Caches id: cache-maven uses: actions/cache@v4 # if it's not a pull request, we restore and save the cache @@ -73,7 +73,7 @@ jobs: path: | ~/.m2/repository/ ~/.m2/wrapper/ - ~/.gradle/caches/ + ~/.gradle/caches/modules-2 ~/.gradle/wrapper/ # A new cache will be stored daily. After that first store of the day, cache save actions will fail because the cache is immutable but it's not a problem. # The whole cache is dropped monthly to prevent unlimited growth. @@ -82,7 +82,7 @@ jobs: restore-keys: | ${{ steps.cache-key.outputs.buildtool-monthly-branch-cache-key }}- ${{ steps.cache-key.outputs.buildtool-monthly-cache-key }}- - - name: Restore Maven/Gradle Local Caches + - name: Restore Maven/Gradle Dependency/Dist Caches uses: actions/cache/restore@v4 # if it a pull request, we restore the cache but we don't save it if: github.event_name == 'pull_request' @@ -90,7 +90,7 @@ jobs: path: | ~/.m2/repository/ ~/.m2/wrapper/ - ~/.gradle/caches/ + ~/.gradle/caches/modules-2 ~/.gradle/wrapper/ key: ${{ steps.cache-key.outputs.buildtool-cache-key }} restore-keys: | diff --git a/.github/workflows/contributor-build.yml b/.github/workflows/contributor-build.yml index 9c7a328a49..d38c4fd82d 100644 --- a/.github/workflows/contributor-build.yml +++ b/.github/workflows/contributor-build.yml @@ -73,7 +73,7 @@ jobs: echo "buildtool-monthly-cache-key=${ROOT_CACHE_KEY}-${CURRENT_MONTH}" >> $GITHUB_OUTPUT echo "buildtool-monthly-branch-cache-key=${ROOT_CACHE_KEY}-${CURRENT_MONTH}-${CURRENT_BRANCH}" >> $GITHUB_OUTPUT echo "buildtool-cache-key=${ROOT_CACHE_KEY}-${CURRENT_MONTH}-${CURRENT_BRANCH}-${CURRENT_DAY}" >> $GITHUB_OUTPUT - - name: Cache Maven/Gradle Local Caches + - name: Cache Maven/Gradle Dependency/Dist Caches id: cache-maven uses: actions/cache@v4 # if it's not a pull request, we restore and save the cache @@ -82,7 +82,7 @@ jobs: path: | ~/.m2/repository/ ~/.m2/wrapper/ - ~/.gradle/caches/ + ~/.gradle/caches/modules-2 ~/.gradle/wrapper/ # A new cache will be stored daily. After that first store of the day, cache save actions will fail because the cache is immutable but it's not a problem. # The whole cache is dropped monthly to prevent unlimited growth. @@ -91,7 +91,7 @@ jobs: restore-keys: | ${{ steps.cache-key.outputs.buildtool-monthly-branch-cache-key }}- ${{ steps.cache-key.outputs.buildtool-monthly-cache-key }}- - - name: Restore Maven/Gradle Local Caches + - name: Restore Maven/Gradle Dependency/Dist Caches uses: actions/cache/restore@v4 # if it a pull request, we restore the cache but we don't save it if: github.event_name == 'pull_request' @@ -99,7 +99,7 @@ jobs: path: | ~/.m2/repository/ ~/.m2/wrapper/ - ~/.gradle/caches/ + ~/.gradle/caches/modules-2 ~/.gradle/wrapper/ key: ${{ steps.cache-key.outputs.buildtool-cache-key }} restore-keys: |