Skip to content

Ci fixes#2

Merged
pnc merged 7 commits intopnc:mainfrom
eleostech:ci-fixes
Apr 11, 2026
Merged

Ci fixes#2
pnc merged 7 commits intopnc:mainfrom
eleostech:ci-fixes

Conversation

@pnc
Copy link
Copy Markdown
Owner

@pnc pnc commented Apr 11, 2026

No description provided.

pnc and others added 7 commits April 11, 2026 03:57
GitHub-hosted macOS runners don't support HVF (QEMU aborts with
HV_UNSUPPORTED). DarwinBackend now auto-detects HVF via sysctl and
falls back to TCG, matching LinuxBackend's KVM detection pattern.
QEMU_ACCEL env var overrides detection for CI or debugging.

Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com>
GitHub runners have /dev/kvm but it isn't accessible to the runner
user. Use os.access() with R_OK|W_OK instead of Path.exists() so
QEMU falls back to TCG instead of crashing with "Permission denied."
Also respect QEMU_ACCEL env var like DarwinBackend.

Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com>
actions/checkout@v4 and actions/upload-artifact@v4 use the
deprecated Node.js 20 runtime. v5 uses Node.js 24.

Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com>
Under TCG on CI, cloud-init status can take longer than 15s to
respond while packages are installing. Catch TimeoutExpired and
retry instead of crashing the test. Also bumped the per-poll
timeout from 15s to 30s.

Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com>
Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com>
Poll for up to 3s instead of sleeping 1s to reliably catch fast
failures like port-in-use. Print the actual mitmdump log output
so the user sees the real error.

Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com>
300s is too tight for macOS TCG on GitHub runners — cloud-init
was still installing packages when the deadline expired. The
job-level timeout (20min) is the real safety net.

Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com>
@pnc pnc merged commit 5039c05 into pnc:main Apr 11, 2026
3 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