Skip to content

CI: fix Android SDK cache restore permissions on Actions - #338

Merged
ggrell merged 4 commits into
mainfrom
codex/fix-android-sdk-cache-permissions
Mar 30, 2026
Merged

CI: fix Android SDK cache restore permissions on Actions#338
ggrell merged 4 commits into
mainfrom
codex/fix-android-sdk-cache-permissions

Conversation

@ggrell

@ggrell ggrell commented Mar 30, 2026

Copy link
Copy Markdown
Owner

Summary

  • move Android SDK cache to a writable runner path ($RUNNER_TEMP/android-sdk) instead of /usr/local/lib/android/sdk
  • export ANDROID_SDK_ROOT and ANDROID_HOME to that writable location in all CI workflows
  • install required SDK packages only when cache is missed (platform-tools, platforms;android-36, build-tools;36.0.0)
  • bump cache key suffix to v2 to avoid stale cache reuse

Why

actions/cache restore into /usr/local/lib/android/sdk triggered repeated tar metadata errors (Cannot utime / Cannot change mode: Operation not permitted) on GitHub-hosted runners.

Expected impact

  • no more noisy cache restore permission errors
  • Android SDK is still cached between runs
  • cache miss behavior remains explicit and deterministic

@github-actions

github-actions Bot commented Mar 30, 2026

Copy link
Copy Markdown
Messages
📖 JUnit: 27 tests, 0 failures/errors, 0 skipped.
📖 Static analysis summary: Detekt 0, Android Lint 0.

Generated by 🚫 Danger Kotlin against 91f05e4

@codecov

codecov Bot commented Mar 30, 2026

Copy link
Copy Markdown

Codecov Report

✅ All modified and coverable lines are covered by tests.
✅ Project coverage is 88.11%. Comparing base (e48c425) to head (91f05e4).
⚠️ Report is 3 commits behind head on main.

Additional details and impacted files

Impacted file tree graph

@@            Coverage Diff            @@
##               main     #338   +/-   ##
=========================================
  Coverage     88.11%   88.11%           
  Complexity       50       50           
=========================================
  Files             9        9           
  Lines           101      101           
=========================================
  Hits             89       89           
  Misses           12       12           

Continue to review full report in Codecov by Sentry.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update 403df05...91f05e4. Read the comment docs.

🚀 New features to boost your workflow:
  • ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.

@ggrell
ggrell marked this pull request as ready for review March 30, 2026 20:22
@ggrell
ggrell enabled auto-merge (squash) March 30, 2026 20:22
@ggrell
ggrell merged commit b29de1a into main Mar 30, 2026
1 check passed
@ggrell
ggrell deleted the codex/fix-android-sdk-cache-permissions branch March 30, 2026 20:22
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant