integration tests: `actions/cache/save` should use more specific key

[aldy505]

Problem Statement

Seeing this on CI:

Run BYK/docker-volume-cache-action/save@be89365902126f508dcae387a32ec3712df6b1cd
Run PATHS=""
Run actions/cache/save@v4
/usr/bin/tar --posix -cf cache.tzst --exclude cache.tzst -P -C /home/runner/work/self-hosted/self-hosted --files-from manifest.txt --use-compress-program zstdmt
Failed to save: Unable to reserve cache with key db-volumes-sentry-v2-8d4bf906ddda2e2f87dab4e73bebbb9a, another job may be creating this cache.
Warning: Cache save failed.
Run sudo mv -f /usr/bin/tar.orig /usr/bin/tar
Run BYK/docker-volume-cache-action/save@be89365902126f508dcae387a32ec3712df6b1cd
Run PATHS=""
Run actions/cache/save@v4
/usr/bin/tar --posix -cf cache.tzst --exclude cache.tzst -P -C /home/runner/work/self-hosted/self-hosted --files-from manifest.txt --use-compress-program zstdmt
Failed to save: Unable to reserve cache with key db-volumes-snuba-v2-d65bd66aadf37349f0ed70e1005ec4f6, another job may be creating this cache.
Warning: Cache save failed.
Run sudo mv -f /usr/bin/tar.orig /usr/bin/tar
Run BYK/docker-volume-cache-action/save@be89365902126f508dcae387a32ec3712df6b1cd
Run PATHS=""
Run actions/cache/save@v4
/usr/bin/tar --posix -cf cache.tzst --exclude cache.tzst -P -C /home/runner/work/self-hosted/self-hosted --files-from manifest.txt --use-compress-program zstdmt
Failed to save: Unable to reserve cache with key db-volumes-kafka-v1-8d4bf906ddda2e2f87dab4e73bebbb9a-d65bd66aadf37349f0ed70e1005ec4f6, another job may be creating this cache.
Warning: Cache save failed.

Not sure if we need to change @BYK's GitHub Action, or we can provide better cache key.

Solution Brainstorm

No response

[BYK]

Is this posing a real issue? We may wanna add the architecture to the cache key as otherwise this just indicates a job with the same parameters is running and trying to save the cache?

[aldy505]

We may wanna add the architecture to the cache key as otherwise this just indicates a job with the same parameters is running and trying to save the cache?

Yes, this should work. Otherwise, nothing will be cached. This is likely the reason integration tests have increased to 10 minutes in length, from the previous 6–8 minutes.

[BYK]

You should be able to add it in the repo, right? Do you wanna give it a shot?

[aldy505]

I should be, but I'll backlog this. I haven't got the time.