Skip to content

ci: stop running test-android on main (self-hosted runner offline)#380

Merged
kraenhansen merged 1 commit into
mainfrom
kh/disable-android-on-main
Jul 19, 2026
Merged

ci: stop running test-android on main (self-hosted runner offline)#380
kraenhansen merged 1 commit into
mainfrom
kh/disable-android-on-main

Conversation

@kraenhansen

Copy link
Copy Markdown
Collaborator

Why

The test-android job runs on ubuntu-self-hosted, which is currently offline. On pushes to main the job sits in queued indefinitely (observed 6h+), so the Check run never completes and main can never report green. It's the sole remaining blocker after #374 / #375 / #377 fixed the CMake failures.

What

  • Gate test-android to PRs labeled Android 🤖 only, removing it from main pushes (same pattern as test-macos).

This is a temporary stopgap. Re-enabling on main — once the self-hosted runner is back, or after migrating to a GitHub-hosted ubuntu-latest runner with KVM — is tracked in #379.

Test plan

  • Not runnable here (this PR is unlabeled, so test-android is skipped by design — that's the point).
  • After merge, the Check run on main completes without hanging on Android.

🤖 Generated with Claude Code

The test-android job runs on ubuntu-self-hosted, which is currently
offline. On pushes to main the job sits in `queued` indefinitely, so the
Check run never completes and main can never report green. Gate the job
to PRs labeled `Android 🤖` only (matching the test-macos pattern),
removing it from main pushes.

Temporary; re-enabling on main is tracked in #379.

Co-Authored-By: Claude Opus 4.8 (1M context) <noreply@anthropic.com>
Claude-Session: https://claude.ai/code/session_01UCN2h9xbyn4yhxfVZyMzAm
@kraenhansen kraenhansen self-assigned this Jul 19, 2026
@kraenhansen
kraenhansen merged commit c77f968 into main Jul 19, 2026
15 checks passed
@kraenhansen
kraenhansen deleted the kh/disable-android-on-main branch July 19, 2026 19:06
kraenhansen pushed a commit that referenced this pull request Jul 19, 2026
Reconcile the pnpm-migrated check.yml with two changes that landed on main
while this branch was in review:

- #382 fixed the CMake 4.2 framework-HEADERS root cause in
  weak-node-api/CMakeLists.txt (included via the rebase) and removed the
  now-redundant "Install compatible CMake version" pin from all five macOS
  jobs. Drop those steps here too; CMAKE_VERSION is retained since
  test-android still uses it to select the Android SDK cmake package.
- #380 gated test-android to labeled PRs only (the ubuntu-self-hosted runner
  is offline and otherwise leaves the job queued forever on main).

With this, check.yml differs from main purely by the pnpm conversion.

Co-Authored-By: Claude Opus 4.8 <noreply@anthropic.com>
Claude-Session: https://claude.ai/code/session_01DnwodAoNbqPec77191HVXn
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