Skip to content
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

The referenced k8s service(s) must be in namespace #379

Merged
merged 1 commit into from May 19, 2016

Conversation

errm
Copy link
Contributor

@errm errm commented May 18, 2016

By design k8s ingress is only designed to ballance services from within
the namespace of the ingress.

This is disscuessed a little in
kubernetes/kubernetes#17088.

For now traefik should only reference the services in the current
namespace. For me this was a confusing change of behaviour
from the reference implimentations, as I have services
with the same name in each namespace.

@errm errm force-pushed the k8s-services-from-same-namespace branch 2 times, most recently from 4d621f6 to 5923d22 Compare May 18, 2016 16:38
By design k8s ingress is only designed to ballance services from within
the namespace of the ingress.

This is disscuessed a little in
kubernetes/kubernetes#17088.

For now traefik should only reference the services in the current
namespace. For me this was a confusing change of behaviour
from the reference implimentations, as I have services
with the same name in each namespace.
@emilevauge
Copy link
Member

Thanks @errm
LGTM
/cc @vdemeester

@vdemeester vdemeester merged commit 6bacbf6 into traefik:master May 19, 2016
@errm errm deleted the k8s-services-from-same-namespace branch September 28, 2017 15:29
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

3 participants