Skip to content

[release/2.2] Fix image defaults on Darwin to usable configuration#12544

Merged
AkihiroSuda merged 5 commits intocontainerd:release/2.2from
k8s-infra-cherrypick-robot:cherry-pick-12533-to-release/2.2
Nov 21, 2025
Merged

[release/2.2] Fix image defaults on Darwin to usable configuration#12544
AkihiroSuda merged 5 commits intocontainerd:release/2.2from
k8s-infra-cherrypick-robot:cherry-pick-12533-to-release/2.2

Conversation

@k8s-infra-cherrypick-robot

This is an automated cherry-pick of #12533

/assign dmcgowan

Use the Linux default rather than the block size from the local macOS
system. The local macOS block size is not relevant as the erofs file
will not be mounted directly on macOS.

Signed-off-by: Derek McGowan <[email protected]>
Non-Linux hosts are not expected to be able to directly mount erofs
snapshotters on the host system. Non-Linux hosts should use block mode
by default with a reasonably set default block size.

Signed-off-by: Derek McGowan <[email protected]>
Use the erofs differ by default on darwin. This could be default for all
Unix platforms but limit the default changes to fix broken cases for backports.

Signed-off-by: Derek McGowan <[email protected]>
Fix the default unpack configuration on darwin to a usable
configuration.

Signed-off-by: Derek McGowan <[email protected]>
Match the defaults set by the transfer service which will configure
linux by default on darwin hosts.

Signed-off-by: Derek McGowan <[email protected]>
@hsiangkao
Copy link
Member

=== FAIL: core/mount/manager TestLoopbackMount (0.07s)
    log_hook.go:47: time="2025-11-19T00:23:15.557630830Z" level=debug msg="activating mount" func="manager.(*mountManager).Activate" file="/home/runner/work/containerd/containerd/core/mount/manager/manager.go:134" mounts="[{loop /tmp/TestLoopbackMount581986403/001/fs-1352655250  []} {format/ext4 {{ mount 0 }}  []}]" name=id1 testcase=TestLoopbackMount
    helpers.go:100: unmount /tmp/TestLoopbackMount581986403/001/test-mount-3644850342
    manager_linux_test.go:80: 
        	Error Trace:	/home/runner/work/containerd/containerd/core/mount/manager/manager_linux_test.go:80
        	            				/home/runner/work/containerd/containerd/core/mount/manager/manager_linux_test.go:105
        	Error:      	Received unexpected error:
        	            	failed to get loop device info: no such device or address
        	Test:       	TestLoopbackMount

ENXIO came from https://github.com/torvalds/linux/blob/v6.17/drivers/block/loop.c#L1291
It seems the most possible reason is fd is invalid (maybe closed by others), but not sure why it could happen.

@github-project-automation github-project-automation bot moved this from Needs Triage to Review In Progress in Pull Request Review Nov 21, 2025
@AkihiroSuda AkihiroSuda merged commit b0731e8 into containerd:release/2.2 Nov 21, 2025
128 of 132 checks passed
@github-project-automation github-project-automation bot moved this from Review In Progress to Done in Pull Request Review Nov 21, 2025
@dmcgowan dmcgowan added impact/changelog area/distribution Image Distribution and removed area/distribution Image Distribution labels Dec 11, 2025
smallprogram added a commit to smallprogram/openwrt_packages that referenced this pull request Jan 29, 2026
Container Runtime Interface (CRI)
 - Redact all query parameters in CRI error logs (containerd/containerd#12546)

Image Distribution
 - Fix image defaults on Darwin to usable configuration (containerd/containerd#12544)
 - Fix possible panic from WithMediaTypeKeyPrefix (containerd/containerd#12516)

Runtime
 - Update runc binary to v1.3.4 (containerd/containerd#12593)
 - Fix parsing of hugetlb..events files (containerd/cgroups#379)

Signed-off-by: David Mandy <[email protected]>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

Archived in project

Development

Successfully merging this pull request may close these issues.

5 participants