Skip to content

fix: allow probing disks smaller than max magic size#142

Merged
talos-bot merged 2 commits intosiderolabs:v2from
smira:fix/small-probe
Dec 24, 2025
Merged

fix: allow probing disks smaller than max magic size#142
talos-bot merged 2 commits intosiderolabs:v2from
smira:fix/small-probe

Conversation

@smira
Copy link
Copy Markdown
Member

@smira smira commented Dec 23, 2025

At the moment, max magic size is 64k, which doesn't allow probing
filesystems smaller than that size.

Skip magics which are bigger than the probe size.

Add a regression test.

At the moment, max magic size is 64k, which doesn't allow probing
filesystems smaller than that size.

Skip magics which are bigger than the probe size.

Add a regression test.

Signed-off-by: Andrey Smirnov <andrey.smirnov@siderolabs.com>
@smira
Copy link
Copy Markdown
Member Author

smira commented Dec 23, 2025

See siderolabs/talos#12477

@github-project-automation github-project-automation bot moved this from In Review to Approved in Planning Dec 23, 2025
Update Go modules.

Signed-off-by: Andrey Smirnov <andrey.smirnov@siderolabs.com>
@smira
Copy link
Copy Markdown
Member Author

smira commented Dec 24, 2025

/m

@talos-bot talos-bot merged commit ee8cbdd into siderolabs:v2 Dec 24, 2025
15 checks passed
@github-project-automation github-project-automation bot moved this from Approved to Done in Planning Dec 24, 2025
smira added a commit to smira/talos that referenced this pull request Dec 24, 2025
Fixes siderolabs#12477

The PR in go-blockdevice: siderolabs/go-blockdevice#142

Signed-off-by: Andrey Smirnov <andrey.smirnov@siderolabs.com>
shanduur pushed a commit to shanduur/talos that referenced this pull request Jan 2, 2026
Fixes siderolabs#12477

The PR in go-blockdevice: siderolabs/go-blockdevice#142

Signed-off-by: Andrey Smirnov <andrey.smirnov@siderolabs.com>
(cherry picked from commit f09ae1e)
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

Archived in project

Development

Successfully merging this pull request may close these issues.

3 participants