Skip to content

ci: move dev-smoketest to GitHub-hosted ubuntu runner#209

Merged
islandbitcoin merged 3 commits into
mainfrom
ci/smoketest-hosted-runner
Jul 7, 2026
Merged

ci: move dev-smoketest to GitHub-hosted ubuntu runner#209
islandbitcoin merged 3 commits into
mainfrom
ci/smoketest-hosted-runner

Conversation

@islandbitcoin

Copy link
Copy Markdown
Contributor

Why

The smoketest ran on a self-hosted Mac runner (MacMax), which caused a recurring failure catalog: Docker disk pressure (the TLS-timeout/Pending-pod/kafka-timeout trio), boot debris after mid-run shutdowns, macOS keychain lockups on image pulls, and stranded queued runs when the host powered off. Separately, a self-hosted runner on a public repo lets outside PR code execute on that personal machine.

Public-repo GitHub-hosted runners are 4 vCPU / 16GB / 150GB, ephemeral, and free — more capacity than the job needs, and ephemerality removes the entire workaround catalog.

What

  • runs-on: self-hostedubuntu-latest
  • Install k3d (the only missing tool; helm/kubectl/jq/terraform/docker are preinstalled)
  • Drop the now-dead persistent-runner workarounds: disk-hygiene prune step, k3d boot-debris self-heal, macOS keychain / docker-socket shims in the mirror step

Validation

This PR's own dev-smoketest run executes the migrated workflow on the hosted runner.

Follow-ups (not in this PR)

  • After merge: deregister the MacMax runner from repo settings and disable its auto-start on the Mac
  • If Docker Hub anonymous rate limits (429s) ever flake the mirror step on shared runner IPs, add a docker login with DOCKERHUB_USERNAME/DOCKERHUB_TOKEN secrets

🤖 Generated with Claude Code

Dread added 3 commits July 7, 2026 09:29
strfry exits at startup when its configured NOFILE setrlimit exceeds the
container hard limit (65536 on GitHub-hosted runners vs ~1M on Docker
Desktop). Default render verified byte-identical; dev deploy sets 0.
Also: smoketest diagnostics now catch CrashLoopBackOff pods (phase stays
Running, so the old phase filter skipped their logs).
@islandbitcoin islandbitcoin merged commit 211c1ae into main Jul 7, 2026
5 of 6 checks passed
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