-
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
remove mirrors from default; document the deprecation of registry.configs and registry.mirrors #5304
Conversation
88d8240
to
645b251
Compare
Build succeeded.
|
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
Not sure a question here is appropriate, is there alternative in |
see example here showing a set of hosts for a given default namespace.. chatting it up with derek the idea is config_path/host_namespace/hosts.toml so config_path/gcr.io/hosts.toml and config_path/docker.io/hosts.toml |
645b251
to
106a3df
Compare
Build succeeded.
|
106a3df
to
0158932
Compare
Build succeeded.
|
a37825f
to
5238a0a
Compare
Build succeeded.
|
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
5238a0a
to
25d03f6
Compare
Build succeeded.
|
25d03f6
to
ed5948f
Compare
Build succeeded.
|
/test pull-containerd-node-e2e |
Signed-off-by: Mike Brown <brownwm@us.ibm.com>
Signed-off-by: Mike Brown <brownwm@us.ibm.com>
ed5948f
to
f9bcf4a
Compare
rebased to pick up the re-enabled cri tests |
Build succeeded.
|
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
we'll probably need to highlight this in release notes or something
Some clean up of the docs around registry config.
A fix to remove old style mirrors config from the default config. Without the fix init will report
WARN[2021-04-07T09:21:29.773381274-05:00] failed to load plugin io.containerd.grpc.v1.cri error="invalid plugin config:
mirrorscannot be set when
config_pathis provided"
Signed-off-by: Mike Brown brownwm@us.ibm.com