From d8a6953eb23fe74bd940c9550f60c7552ac60ba7 Mon Sep 17 00:00:00 2001
From: Ryo Onodera <ryoqun@gmail.com>
Date: Wed, 8 Mar 2023 22:54:19 +0900
Subject: [PATCH] Disable cache if retried 3 times (#30638)

* Disable cache if retired 3 times

* Add more comments

* Reduce if nesting to avoid maintenance pitfall by DISABLE_CACHE

* Well /home was required...

* Adjust to align with third time's the charm

* Adjust if arrangement a bit with tweaks
---
 ci/docker-run.sh | 26 ++++++++++++++++++--------
 1 file changed, 18 insertions(+), 8 deletions(-)

diff --git a/ci/docker-run.sh b/ci/docker-run.sh
index a7b94f902..b1748c80e 100755
--- a/ci/docker-run.sh
+++ b/ci/docker-run.sh
@@ -46,14 +46,24 @@ if [[ -n $CI ]]; then
   ARGS+=(--volume "$HOME:/home")
 
   if [[ -n $BUILDKITE ]]; then
-    # sccache
-    ARGS+=(
-      --env "RUSTC_WRAPPER=/home/.cargo/bin/sccache"
-      --env AWS_ACCESS_KEY_ID
-      --env AWS_SECRET_ACCESS_KEY
-      --env SCCACHE_BUCKET
-      --env SCCACHE_REGION
-    )
+    # I hate buildkite-esque echo is leaking into this generic shell wrapper.
+    # but it's easiest to notify to users, and properly guarded under $BUILDKITE_ env
+    # (2 is chosen for third time's the charm).
+    if [[ $BUILDKITE_RETRY_COUNT -ge 2 ]]; then
+      # Disable sccache to create a clean-room environment to preclude any
+      # sccache-related bugs
+      echo "--- $0 ... (with sccache being DISABLED due to many (${BUILDKITE_RETRY_COUNT}) retries)"
+    else
+      echo "--- $0 ... (with sccache enabled)"
+      # sccache
+      ARGS+=(
+        --env "RUSTC_WRAPPER=/home/.cargo/bin/sccache"
+        --env AWS_ACCESS_KEY_ID
+        --env AWS_SECRET_ACCESS_KEY
+        --env SCCACHE_BUCKET
+        --env SCCACHE_REGION
+      )
+    fi
   fi
 else
   # Avoid sharing ~/.cargo when building locally to avoid a mixed macOS/Linux