Skip to content

ci: re-add macos-latest build matrix and fix flaky mount timeout#165

Merged
azuline merged 5 commits into
masterfrom
devin/1784154041-macos-ci
Jul 16, 2026
Merged

ci: re-add macos-latest build matrix and fix flaky mount timeout#165
azuline merged 5 commits into
masterfrom
devin/1784154041-macos-ci

Conversation

@devin-ai-integration

Copy link
Copy Markdown
Contributor

Summary

Re-adds the macos-latest job to the build matrix (removed in no macosi) and makes it pass.

The only thing failing on macOS was the VFS Test step: all 17 failures were the same assertion in start_virtual_fs, which waited at most 2s for the FUSE mount to become visible. macFUSE consistently takes just over 2s to mount (CI showed 2.0–2.17s), so every VFS test tripped assert diff < 2, "timed out waiting for vfs to mount". Linux mounts in ~100ms and was unaffected.

Fix is to raise that wait budget:

# start_virtual_fs()
-    assert diff < 2, "timed out waiting for vfs to mount"
+    assert diff < 10, "timed out waiting for vfs to mount"

Workflow change re-introduces the matrix and the macFUSE install step:

runs-on: ${{ matrix.os }}
strategy:
  fail-fast: false
  matrix:
    os: [ubuntu-latest, macos-latest]
...
- name: Install macFUSE
  if: matrix.os == 'macos-latest'
  run: brew install --cask macfuse

fail-fast: false so a failure on one OS doesn't cancel the other.

Link to Devin session: https://app.devin.ai/sessions/5f1b65c493c34e64ab1e1d49ed52f5c1
Requested by: @azuline

@azuline azuline self-assigned this Jul 15, 2026
@devin-ai-integration

Copy link
Copy Markdown
Contributor Author

🤖 Devin AI Engineer

I'll be helping with this pull request! Here's what you should know:

✅ I will automatically:

  • Address comments on this PR. Add '(aside)' to your comment to have me ignore it.
  • Look at CI failures and help fix them

Note: I can only respond to comments from users who have write access to this repository.

⚙️ Control Options:

  • Disable automatic comment, CI, and merge conflict monitoring

@azuline
azuline merged commit 37267c1 into master Jul 16, 2026
2 checks passed
@azuline
azuline deleted the devin/1784154041-macos-ci branch July 16, 2026 00:02
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