Skip to content

ci: extend platform checkout fetch timeout#88199

Merged
RomneyDa merged 1 commit into
mainfrom
fix/windows-checkout-retry
May 30, 2026
Merged

ci: extend platform checkout fetch timeout#88199
RomneyDa merged 1 commit into
mainfrom
fix/windows-checkout-retry

Conversation

@RomneyDa

@RomneyDa RomneyDa commented May 30, 2026

Copy link
Copy Markdown
Member

Summary

  • Increase the Windows/macOS CI checkout fetch watchdog from 30s to 90s.
  • Keep the manual checkout bounded, but avoid killing git fetch while index-pack may still be writing/verifying the shallow pack.
  • Add a workflow guard assertion for the platform checkout timeout.

Failure

CI / checks-windows-node-test failed during Checkout, before deps or tests:

  • CHECKOUT_SHA=01650d0673a70361c04fd12c2c57b27367e2ad83
  • fatal: fetch-pack: invalid index-pack output
  • exit 124

The platform checkout helper was terminating fetches after 30s. On slower Windows runner fetches, that can interrupt Git during pack indexing and produce this checkout-level failure. Refs ed36f42.

Verification

  • node scripts/run-vitest.mjs run --config test/vitest/vitest.tooling.config.ts test/scripts/ci-workflow-guards.test.ts --reporter=verbose - passed, 1 file / 9 tests.
  • git diff --check HEAD^ HEAD - passed.
  • actionlint .github/workflows/ci.yml - not run locally; actionlint is not installed in this checkout.

Real behavior proof

Behavior addressed: platform checkout fetches should not be killed after 30s while Git may still be indexing a shallow pack.
Real environment tested: local macOS checkout, workflow parsed by repo tooling tests.
Exact steps or command run after this patch: focused workflow guard Vitest command listed above.
Evidence after fix: guard test confirms platform checkout steps keep bounded fetches and use the named 90s timeout budget.
Observed result after fix: checkout remains bounded but gives Windows/macOS fetches a more realistic window before termination.
What was not tested: live Windows runner checkout; PR CI will exercise workflow syntax and the normal checkout path.

@openclaw-barnacle openclaw-barnacle Bot added size: S maintainer Maintainer-authored PR labels May 30, 2026
@clawsweeper

clawsweeper Bot commented May 30, 2026

Copy link
Copy Markdown
Contributor

ClawSweeper status: review started.

I am starting a fresh review of this pull request: ci: retry platform checkout fetch failures This is item 1/1 in the current shard. Shard 0/1.

This placeholder means the worker is alive and reading the current context. I will edit this same comment with the actual review when the claws are done clicking.

Crustacean status: shell secured, claws on keyboard, evidence pebbles being sorted.

@RomneyDa
RomneyDa force-pushed the fix/windows-checkout-retry branch from 8f5b86f to 5227b2a Compare May 30, 2026 02:13
@RomneyDa RomneyDa changed the title ci: retry platform checkout fetch failures ci: relax platform checkout fetch timeout May 30, 2026
@RomneyDa RomneyDa changed the title ci: relax platform checkout fetch timeout ci: extend platform checkout fetch timeout May 30, 2026
@RomneyDa
RomneyDa merged commit 784fbcf into main May 30, 2026
106 of 111 checks passed
@RomneyDa
RomneyDa deleted the fix/windows-checkout-retry branch May 30, 2026 02:17
github-actions Bot pushed a commit to Desicool/openclaw that referenced this pull request May 30, 2026
SYU8384 pushed a commit to SYU8384/openclaw that referenced this pull request Jun 3, 2026
sablehead pushed a commit to sablehead/openclaw that referenced this pull request Jun 10, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

maintainer Maintainer-authored PR size: XS

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant