-
Notifications
You must be signed in to change notification settings - Fork 1.4k
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
support to pull base images from insecure registries. #308
support to pull base images from insecure registries. #308
Conversation
Hi @everpeace. Thanks for your PR. I'm waiting for a GoogleContainerTools member to verify that this patch is reasonable to test. If it is, they should reply with I understand the commands that are listed here. Instructions for interacting with me using PR comments are available here. If you have questions or suggestions related to my behavior, please file an issue against the kubernetes/test-infra repository. |
pkg/options/options.go
Outdated
SnapshotMode string | ||
Bucket string | ||
DockerInsecureSkipTLSVerify bool | ||
DockerInsecureSkipTLSVerifyAtPull bool |
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.
I'd be fine with just using the same DockerInsecureSkipTLSVerify flag for both of these, WDYT?
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.
I'd be fine too. I'll fix it 👍
cad2962
to
739c7bb
Compare
@dlorenc I changed to re-use and I squashed my commits. Would you mind taking a look again? |
d863656
to
fe02e5e
Compare
fe02e5e
to
3abfc63
Compare
Looked like an IO timeout, I'll retry the test run. |
Hey @everpeace, this needs a rebase. |
Hey @everpeace, with #409 kaniko supports pulling and pushing from insecure registries. I'll go ahead and close this PR. |
Thnaks, actually dtaniwaki is my colleague :-) |
Motivation
We operate an insecure docker registry and kubernetes cluster in on-premise environment. kaniko can push container images to the insecure registry. But, can't pull from the insecure registry.
How
make
--insecure-skip-tls-verify
be able to affect when pulling images.Because user should be able to use multi stage build, with this option, it tries to connect to a registry in secure way first. Only when it failed, it will switch to connect insecure mode.