Skip to content

Commit f6b8025

Browse files
committed
overlay2: get rid of unnecessary kernel version check
Signed-off-by: Jonas Geiler <[email protected]>
1 parent b2fd67d commit f6b8025

1 file changed

Lines changed: 3 additions & 6 deletions

File tree

daemon/graphdriver/overlay2/check.go

Lines changed: 3 additions & 6 deletions
Original file line numberDiff line numberDiff line change
@@ -12,7 +12,6 @@ import (
1212
"github.com/containerd/containerd/mount"
1313
"github.com/containerd/containerd/pkg/userns"
1414
"github.com/docker/docker/daemon/graphdriver/overlayutils"
15-
"github.com/docker/docker/pkg/parsers/kernel"
1615
"github.com/docker/docker/pkg/system"
1716
"github.com/pkg/errors"
1817
"golang.org/x/sys/unix"
@@ -29,16 +28,14 @@ import (
2928
func doesSupportNativeDiff(d string) error {
3029
userxattr := false
3130
if userns.RunningInUserNS() {
32-
if !kernel.CheckKernelVersion(5, 11, 0) {
33-
return errors.New("running in a user namespace")
34-
}
35-
3631
needed, err := overlayutils.NeedsUserXAttr(d)
3732
if err != nil {
3833
return err
3934
}
4035
if needed {
4136
userxattr = true
37+
} else {
38+
return errors.New("not supported in user namespace, consider updating to kernel 5.11 or later to fix")
4239
}
4340
}
4441

@@ -103,7 +100,7 @@ func doesSupportNativeDiff(d string) error {
103100
return errors.Wrap(err, "failed to read opaque flag on upper layer")
104101
}
105102
if string(xattrOpaque) == "y" {
106-
return errors.New("opaque flag erroneously copied up, consider update to kernel 4.8 or later to fix")
103+
return errors.New("opaque flag erroneously copied up, consider updating to kernel 4.8 or later to fix")
107104
}
108105

109106
// rename "d1" to "d2"

0 commit comments

Comments
 (0)