Skip to content

Conversation

@bacongobbler
Copy link
Contributor

closes #1046

@k8s-ci-robot k8s-ci-robot added the cncf-cla: yes Indicates the PR's author has signed the CNCF CLA. label Jan 26, 2017
@minikube-bot
Copy link
Collaborator

Can one of the admins verify this patch?

@bacongobbler bacongobbler force-pushed the document-insecure-registry branch from 25f319e to bd2e1c0 Compare January 26, 2017 21:35
@codecov-io
Copy link

codecov-io commented Jan 26, 2017

Current coverage is 34.65% (diff: 100%)

Merging #1047 into master will not change coverage

@@             master      #1047   diff @@
==========================================
  Files            43         43          
  Lines          2000       2000          
  Methods           0          0          
  Messages          0          0          
  Branches          0          0          
==========================================
  Hits            693        693          
  Misses         1170       1170          
  Partials        137        137          

Powered by Codecov. Last update 2cbec00...bd2e1c0

README.md Outdated
`minikube start` command to enable insecure communication between the docker engine and registries listening to reqeusts from the CIDR range.

One nifty hack is to allow the kubelet running in minikube to talk to registries deployed inside a pod in the cluster without backing them
with TLS certificates. Because the service cluster IP range is known to be available at 10.0.0.0/24, users can pull images from registries
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

small nit: I think its 10.0.0.1?

DefaultServiceClusterIP = "10.0.0.1"

Copy link
Contributor Author

@bacongobbler bacongobbler Jan 26, 2017

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Nice catch. I'll fix that/make it more clear :)

@bacongobbler bacongobbler force-pushed the document-insecure-registry branch from bd2e1c0 to 63bd5d8 Compare January 26, 2017 23:26
Copy link
Contributor

@r2d4 r2d4 left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM

@dlorenc dlorenc merged commit 5d19af6 into kubernetes:master Jan 27, 2017
@bacongobbler bacongobbler deleted the document-insecure-registry branch January 27, 2017 19:02
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

cncf-cla: yes Indicates the PR's author has signed the CNCF CLA.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

support $KUBE_ENABLE_INSECURE_REGISTY

7 participants