Skip to content

test(cli): raise the docker builder test timeout to 60s - #576

Merged
brunomenezes merged 1 commit into
prerelease/v2-alphafrom
fix/integration-test-timeout
Nov 11, 2026
Merged

brunomenezes merged 1 commit into
prerelease/v2-alphafrom
fix/integration-test-timeout

Conversation

@brunomenezes

Copy link
Copy Markdown
Member

Summary

The docker builder integration tests had no timeout of their own, so each case ran under bun's 5 s default. The first case, should build an ext2 drive with a target definition, builds the riscv64 fixture through QEMU and writes a 93 MB ext2 image, which takes about 4 s on a cold CI runner. On a slower runner it crosses 5 s and times out, as it did on the "Version Packages (alpha)" PR run 43079198446, and the killed build then leaves a root.ext2 ENOENT as an unhandled error. The Release workflow runs the same suite before publishing, so the flake can also block a release.

setDefaultTimeout(60000) now covers the whole file, the value the beforeAll already used for pulling the images, and the beforeAll drops its own option. A timeout only caps a failing run, so passing runs are unchanged. No changesets added.

@changeset-bot

changeset-bot Bot commented Nov 11, 2026

Copy link
Copy Markdown

⚠️ No Changeset found

Latest commit: ddb937d

Merging this PR will not cause a version bump for any packages. If these changes should not result in a new version, you're good to go. If these changes should result in a version bump, you need to add a changeset.

This PR includes no changesets

When changesets are added to this PR, you'll see the packages that this PR includes changesets for and the associated semver types

Click here to learn what changesets are, and how to add one.

Click here if you're a maintainer who wants to add a changeset to this PR

@github-actions

github-actions Bot commented Nov 11, 2026 •

Copy link
Copy Markdown
Contributor

Coverage Report

Status Category Percentage Covered / Total
🟢 Lines 91.2% (🎯 0%) 12399 / 13596
🔵 Statements 91.2% 12399 / 13596
🔵 Functions 81.87% 280 / 342
🔵 Branches 0% 0 / 0
📁 File Coverage (20 files)
File Lines Statements Functions Branches Uncovered Lines
apps/cli/src/base.ts 🔴 38.01% 🔴 38.01% 🟡 64.71% 🔴 0% 62, 70-74, 79, 93, 101-174,...
apps/cli/src/builder/directory.ts 🟢 100% 🟢 100% 🟢 100% 🔴 0% -
apps/cli/src/builder/docker.ts 🟢 87.12% 🟢 87.12% 🟡 66.67% 🔴 0% 77-79, 81, 114-116, 174-183
apps/cli/src/builder/empty.ts 🟢 100% 🟢 100% 🟢 100% 🔴 0% -
apps/cli/src/builder/none.ts 🟢 100% 🟢 100% 🟢 100% 🔴 0% -
apps/cli/src/builder/nvram.ts 🟢 97.14% 🟢 97.14% 🟢 100% 🔴 0% 38
apps/cli/src/builder/tar.ts 🟢 100% 🟢 100% 🟢 100% 🔴 0% -
apps/cli/src/commands/foreclose.ts 🔴 49.09% 🔴 49.09% 🔴 50% 🔴 0% 69, 91-145
apps/cli/src/commands/refund.ts 🔴 38.2% 🔴 38.2% 🟡 66.67% 🔴 0% 55-109
apps/cli/src/commands/withdraw.ts 🔴 46.58% 🔴 46.58% 🟡 73.33% 🔴 0% 95-123, 277-278, 301-356, 3...
apps/cli/src/compose/anvil.ts 🟡 79.25% 🟡 79.25% 🟢 100% 🔴 0% 19-29
apps/cli/src/compose/builder.ts 🟢 99.79% 🟢 99.79% 🟢 100% 🔴 0% 228
apps/cli/src/compose/bundler.ts 🔴 4.82% 🔴 4.82% 🔴 0% 🔴 0% 8-40, 44-75, 79-92
apps/cli/src/compose/common.ts 🟢 100% 🟢 100% 🟢 100% 🔴 0% -
apps/cli/src/compose/database.ts 🟢 100% 🟢 100% 🟢 100% 🔴 0% -
apps/cli/src/compose/explorer.ts 🔴 6.67% 🔴 6.67% 🔴 0% 🔴 0% 10-38, 43-55, 59-72
apps/cli/src/compose/node.ts 🟢 100% 🟢 100% 🟢 100% 🔴 0% -
apps/cli/src/compose/passkey.ts 🔴 8.33% 🔴 8.33% 🔴 0% 🔴 0% 9-18, 23-42, 46-59
apps/cli/src/compose/paymaster.ts 🔴 7.69% 🔴 7.69% 🔴 0% 🔴 0% 8-21, 25-44, 48-61
apps/cli/src/compose/proxy.ts 🟢 100% 🟢 100% 🟢 100% 🔴 0% -

@github-project-automation github-project-automation Bot moved this from Todo to Waiting Merge in Rollups SDK Nov 11, 2026
@github-project-automation github-project-automation Bot moved this to 👍 Waiting Merge in Rollups Tooling Nov 11, 2026
@brunomenezes brunomenezes moved this from 👍 Waiting Merge to 👁 In Review in Rollups Tooling Nov 11, 2026
@brunomenezes
brunomenezes merged commit 05d2495 into prerelease/v2-alpha Nov 11, 2026
4 of 6 checks passed
@brunomenezes
brunomenezes deleted the fix/integration-test-timeout branch November 11, 2026 19:58
@github-project-automation github-project-automation Bot moved this from 👁 In Review to 📦 Done in Rollups Tooling Nov 11, 2026
@github-project-automation github-project-automation Bot moved this from Waiting Merge to Done in Rollups SDK Nov 11, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

Status: Done
Status: 📦 Done

Development

Successfully merging this pull request may close these issues.

2 participants