-
Notifications
You must be signed in to change notification settings - Fork 3.3k
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
[release/1.6] Migrate errdefs
package to github.com/containerd/errdefs
#10267
[release/1.6] Migrate errdefs
package to github.com/containerd/errdefs
#10267
Conversation
This updates the errdefs package to be an alias for the new errdefs module. This helps transitioning consumers to the new module, and makes sure that containerd v2 and v1 use the same definitions. Signed-off-by: Sebastiaan van Stijn <github@gone.nl>
Signed-off-by: Sebastiaan van Stijn <github@gone.nl>
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LGTM
It's to make sure projects using containerd as a module don't break; the aliases will make sure that they use the same type(s) as containerd is using, which come from the new errdefs module. Eventually all uses should go away, but at least currently there's still uses, including some in containerd's dependency tree;
Same was done for the
We can't update those yet, until this change is in, and released, otherwise they would be using a different type than containerd itself (so errors wouldn't be matched), but once this is released, we can update the minimum required version of containerd in those repositories, and switch to using the other module. |
errdefs
package to github.com/containerd/errdefs
github.com/containerd/errdefs
#10266This updates the errdefs package to be an alias for the new errdefs module. This helps transitioning consumers to the new module, and makes sure that containerd v2 and v1 use the same definitions.