Skip to content

Commit df350f3

Browse files
authored
Merge pull request moby#5194 from thaJeztah/fix_wrong_errdefs
snapshot/containerd: fix wrong errdefs package import
2 parents 59a3ce5 + c6745c3 commit df350f3

File tree

4 files changed

+5
-54
lines changed

4 files changed

+5
-54
lines changed

.golangci.yml

Lines changed: 2 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -56,6 +56,8 @@ linters-settings:
5656
desc: Use github.com/moby/sys/user/userns instead.
5757
- pkg: "github.com/containerd/containerd/platforms"
5858
desc: The containerd platforms package was migrated to a separate module. Use github.com/containerd/platforms instead.
59+
- pkg: "github.com/containerd/nydus-snapshotter/pkg/errdefs"
60+
desc: You probably meant to use github.com/containerd/errdefs
5961
- pkg: "github.com/opencontainers/runc/libcontainer/userns"
6062
desc: Use github.com/moby/sys/user/userns instead.
6163
- pkg: "io/ioutil"

snapshot/containerd/content.go

Lines changed: 3 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -5,7 +5,7 @@ import (
55

66
"github.com/containerd/containerd/content"
77
"github.com/containerd/containerd/namespaces"
8-
"github.com/containerd/nydus-snapshotter/pkg/errdefs"
8+
cerrdefs "github.com/containerd/errdefs"
99
digest "github.com/opencontainers/go-digest"
1010
ocispecs "github.com/opencontainers/image-spec/specs-go/v1"
1111
"github.com/pkg/errors"
@@ -103,7 +103,7 @@ var _ content.Store = &nsFallbackStore{}
103103
func (c *nsFallbackStore) Info(ctx context.Context, dgst digest.Digest) (content.Info, error) {
104104
info, err := c.main.Info(ctx, dgst)
105105
if err != nil {
106-
if errdefs.IsNotFound(err) {
106+
if cerrdefs.IsNotFound(err) {
107107
return c.fb.Info(ctx, dgst)
108108
}
109109
}
@@ -137,7 +137,7 @@ func (c *nsFallbackStore) Abort(ctx context.Context, ref string) error {
137137
func (c *nsFallbackStore) ReaderAt(ctx context.Context, desc ocispecs.Descriptor) (content.ReaderAt, error) {
138138
ra, err := c.main.ReaderAt(ctx, desc)
139139
if err != nil {
140-
if errdefs.IsNotFound(err) {
140+
if cerrdefs.IsNotFound(err) {
141141
return c.fb.ReaderAt(ctx, desc)
142142
}
143143
}

vendor/github.com/containerd/nydus-snapshotter/pkg/errdefs/errors.go

Lines changed: 0 additions & 50 deletions
This file was deleted.

vendor/modules.txt

Lines changed: 0 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -383,7 +383,6 @@ github.com/containerd/log
383383
## explicit; go 1.19
384384
github.com/containerd/nydus-snapshotter/pkg/converter
385385
github.com/containerd/nydus-snapshotter/pkg/converter/tool
386-
github.com/containerd/nydus-snapshotter/pkg/errdefs
387386
github.com/containerd/nydus-snapshotter/pkg/label
388387
# github.com/containerd/platforms v0.2.1
389388
## explicit; go 1.20

0 commit comments

Comments
 (0)