-
Notifications
You must be signed in to change notification settings - Fork 4.9k
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
Ingress controller addon #611
Comments
Sounds like a good idea. I think we should enable it by default, but also allow it to be disabled or swapped on startup through our |
Great, let me know if you run into roadblocks. I believe people are already running it on local-up-cluster so it should just work, in theory: https://github.com/kubernetes/contrib/tree/master/ingress/controllers/nginx#local-cluster |
The nginx ingress controller does work with minikube, I already tested that successfully :) |
I actually used it in a demo on tuesday 👍, I'm waiting for #639 to enable it by default though |
Now that #639 is merged, can we get the ingress addon out of the box? I have a bunch of developers starting to use minikube. Setting up an ingress is a bit of pain - it would be really nice if it was out of the box |
We're waiting on kubernetes-retired/contrib#1879 (comment) I've left another friendly ping. Once that gets merged and released I have a working branch I can merge to minikube |
Automatic merge from submit-queue Make map_hash_bucket_size configurable I was getting an error while trying to run the nginx controller in minikube. This allows this nginx configuration option to be passed in through a configmap. The default value depends on the processor's cache line size (32 | 64 | 128), however ServerNameHashBucketSize is determined similarly, so I've set it to the same default (64). Fixes #1817 ref kubernetes/minikube#611 cc @bprashanth
is it possible to enable this by default now that kubernetes-retired/contrib#1879 was merged? |
Not sure if it's in an image yet. If you need a new image pushed please upload a pr with the version bump to the makefile. |
…size Automatic merge from submit-queue Make map_hash_bucket_size configurable I was getting an error while trying to run the nginx controller in minikube. This allows this nginx configuration option to be passed in through a configmap. The default value depends on the processor's cache line size (32 | 64 | 128), however ServerNameHashBucketSize is determined similarly, so I've set it to the same default (64). Fixes kubernetes-retired#1817 ref kubernetes/minikube#611 cc @bprashanth
I'm not sure which version I should bump in the makefile. |
@danielepolencic take a look at kubernetes-retired/contrib#2015 (comment) I am contemplating building our own image as it seems easy enough. I reopened that PR for now.. Would love to get this in minikube |
@r2d4 I'd love to get this in minikube too. I'm not an expert by any means, but I'm happy to help. Is there anything I can do? |
This should be in the next version of minikube with
|
Thanks! |
For the implementation, you can see https://github.com/kubernetes/minikube/tree/master/deploy/addons/ingress The controller image is |
I'm not having much luck with this. When I enabled the addon initially, The service was created but the two rcs and the configmap weren't ( the namespace logs in the dashboard show there was a failure at ImagePull, possibly a transient network issue, maybe IPv6 routing ) or any time while I waited thereafter.
I tried I'm gonna try recreating my VM with the addon enabled Any other debug suggestions? / what logs etc might be helpful? |
I would try running the YAMLs themselves slightly modified. You will need to remove the cluster-service annotation (this is used by the addon-manager) and you may want to change the namespace from kube-system to default for easier debugging. The yaml files used in minikube are here. There are no major differences from the example ones used in contrib/ingress https://github.com/kubernetes/minikube/tree/master/deploy/addons/ingress Which version of minikube are you running? You may want to check that your addon-manager is at least version 5.1. Older ones might not support creating the configmap. It might also help to get the output of |
I just rebuilt minikube from master and blew away Addon manager was enabled (by default) and image is running There is no sign that the addon-manager is aware of either the configmap or the rc.yaml
but they do seem to be in its dir and have the right content
(also cat'ed but seems uneeded here) |
looks indeed like ConfigMap support didn't come in til 5.2: https://github.com/kubernetes/kubernetes/blob/master/cluster/addons/addon-manager/CHANGELOG.md#version-52-wed-october-26-2016-zihong-zheng-zihongzgooglecom I can try updating the addon-manager in place. What version were you running to test against? |
Ah, thats probably the issue. I ran some tests against 6.1, but thought I had also ran them against 5.1 also. 6.1 looks like it accepts all resources. |
Initial feature request mentioned
Was this documented? I couldn't find it. |
Looks like helm |
Can we deploy the nginx controller (https://github.com/kubernetes/contrib/tree/master/ingress/controllers/nginx) as an addon so it works out of the box, and document how to swap it with one of the several other implementations out there?
The text was updated successfully, but these errors were encountered: