Skip to content

setup_builder: wait for non-zero device size before formatting sticky disk (release/v2) - #119

Merged
piob-io merged 1 commit into
release/v2from
devin/1784222940-wait-device-size-v2
Jul 16, 2026
Merged

setup_builder: wait for non-zero device size before formatting sticky disk (release/v2)#119
piob-io merged 1 commit into
release/v2from
devin/1784222940-wait-device-size-v2

Conversation

@piob-io

@piob-io piob-io commented Jul 16, 2026

Copy link
Copy Markdown
Contributor

Summary

Backport of #118 to release/v2: newly exposed sticky disk devices can momentarily report size 0 (the expose/resize race), and formatting at that instant produces a tiny/broken filesystem. setupStickyDisk now polls blockdev --getsize64 until the device reports a non-zero size (bounded wait) before mkfs/mount, failing with a clear error if the device never becomes ready.

Source change cherry-picked cleanly from main (1bee150); dist/ rebuilt against this branch's dependencies.

Link to Devin session: https://app.devin.ai/sessions/ba61e842e479410d95116fe1dfe77b0f
Requested by: @piob-io


View with Codesmith Autofix with Codesmith
Need help on this PR? Tag /codesmith with what you need. Autofix is disabled.

… disk

Co-Authored-By: Devin AI <158243242+devin-ai-integration[bot]@users.noreply.github.com>
@piob-io piob-io self-assigned this Jul 16, 2026
@devin-ai-integration

Copy link
Copy Markdown
Contributor

🤖 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

@piob-io
piob-io requested a review from adityamaru July 16, 2026 17:38
@piob-io
piob-io merged commit 33803e6 into release/v2 Jul 16, 2026
8 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.

2 participants