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

Cluster local services #1837

Closed
sbarzowski opened this issue Aug 10, 2018 · 3 comments
Closed

Cluster local services #1837

sbarzowski opened this issue Aug 10, 2018 · 3 comments
Labels
area/networking kind/feature Well-understood/specified features, ready for coding.

Comments

@sbarzowski
Copy link

sbarzowski commented Aug 10, 2018

Can we have Services which are reachable only from within the cluster? It is sometimes useful to have a helper service and exposing it to the outside world may be a security issue. It may be also useful for putting a proxy service in front of the "real services".

@tcnghia

BTW I'd be happy to help with this, but I'm sort of new to k8s world, and very new to knative / istio stuff, so I would need some guidance on how to approach it.

@knative-prow-robot knative-prow-robot added area/networking kind/feature Well-understood/specified features, ready for coding. labels Aug 10, 2018
@sbarzowski
Copy link
Author

My understanding is that it mostly boils down to not including knative-shared-gateway.knative-serving.svc.cluster.local in VirtualService associated with the service. Probably also it's worth it to remove the public hostname for a cluster-local service for clarity.

Can someone confirm?

@tcnghia
Copy link
Contributor

tcnghia commented Aug 16, 2018

@sbarzowski that would be part of it, which will make it:
(1) possible to connect to route.ns.svc.cluster.local from a pod with Istio sidecar injected.
(2) not possible to connect through knative-ingressgateway public IP.

however, that also (3) make it not possible to connect to route.ns.svc.cluster.local from a pod without sidecar injected. we currently make those requests go through knative-ingressgateway through the use of an ExternalName service.

I think we could implement this first by doing what you suggest and find a way to solve (3) after.

@tcnghia
Copy link
Contributor

tcnghia commented Nov 6, 2018

This is now tracked in #2127, which is planned to be part of Knative 0.3

@tcnghia tcnghia closed this as completed Nov 6, 2018
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
area/networking kind/feature Well-understood/specified features, ready for coding.
Projects
None yet
Development

No branches or pull requests

3 participants