You signed in with another tab or window. Reload to refresh your session.You signed out in another tab or window. Reload to refresh your session.You switched accounts on another tab or window. Reload to refresh your session.Dismiss alert
Move local ccache storage to B: drive on Windows runners — current CACHE_DIR is on C: (OS disk, 30GB) which may cause disk space issues with high cache miss rates. Should use the temp disk on B: like the build directory.
Ref: Enable ccache + bazel-remote caching for Windows builds #4161 (comment)
Fix set vs export shell detection in setup_ccache.py — Windows CI uses bash (not cmd), so set might be wrong. Detect shell type instead of OS, or remove the output since workflows set CCACHE_CONFIGPATH as an env var directly.
Ref: Enable ccache + bazel-remote caching for Windows builds #4161 (comment)
Follow-up items from #4161 review comments.
Tasks
Move local ccache storage to B: drive on Windows runners — current
CACHE_DIRis on C: (OS disk, 30GB) which may cause disk space issues with high cache miss rates. Should use the temp disk on B: like the build directory.Ref: Enable ccache + bazel-remote caching for Windows builds #4161 (comment)
Fix
setvsexportshell detection insetup_ccache.py— Windows CI uses bash (not cmd), sosetmight be wrong. Detect shell type instead of OS, or remove the output since workflows setCCACHE_CONFIGPATHas an env var directly.Ref: Enable ccache + bazel-remote caching for Windows builds #4161 (comment)
Consider Windows-compatible
compiler_check— current POSIX check usesldd/sha256sum; a Windows equivalent could improve cache robustness beyond the defaultmtimecheck.Ref: Enable ccache + bazel-remote caching for Windows builds #4161 (comment)
Refine ccache preset selection for multi-arch workflows — split
presubmit/postsubmit/releasepresets based on trigger type (pull_requestvspushvsschedule) when multi-arch replaces old CI pipelines.Ref: Enable ccache + bazel-remote caching for Windows builds #4161 (review)
Monitor GHA cache usage — with
actions/cacheremoved from Windows builds, check https://github.com/ROCm/TheRock/actions/caches to confirm reduced pressure on the 10GB limit.Ref: Enable ccache + bazel-remote caching for Windows builds #4161 (review)