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

[BUG] Always route to local instance of a service from istio-ingressgateway #62

Closed
aattuluri opened this issue Jan 26, 2020 · 0 comments
Assignees
Labels
bug Something isn't working
Milestone

Comments

@aattuluri
Copy link
Contributor

Describe the bug
When a service name is accessed in a cluster, it's possible for it to bounce between clusters.

Steps To Reproduce
Deploy Service 1 to Cluster 1 and Cluster 2
Deploy Service 2 to Cluster 2
Make a call from Service 2 to Service 1 using global name created by admiral
Notice that some of the calls made from Service 2 to Service 1 take very long time. This is because at the istio-ingressgateway they might be routed back to the original cluster where the request is originating from as that's also a possible destination for that service.

Expected behavior
Service calls using global names should not bounce between clusters.

Proposed solution:
Create a virtual service that always routes calls to the local instance of a service and attach it to istio-ingressgateway gateway
Example:

apiVersion: networking.istio.io/v1alpha3
kind: VirtualService
metadata:
  name: default.greeting.global-default-vs
  namespace: admiral-sync
spec:
  exportTo:
  - '*'
  gateways:
  - istio-multicluster-ingressgateway
  hosts:
  - default.greeting.global
  http:
  - route:
    - destination:
        host: greeting.sample.svc.cluster.local
        port:
          number: 8080
@aattuluri aattuluri added the bug Something isn't working label Jan 26, 2020
@aattuluri aattuluri changed the title [BUG] Always route to local instance of a service from ingress-gateway [BUG] Always route to local instance of a service from istio-ingressgateway Jan 26, 2020
@aattuluri aattuluri added this to the v0.1 milestone Jan 26, 2020
@aattuluri aattuluri self-assigned this Apr 14, 2020
Mengying-Li pushed a commit that referenced this issue Apr 20, 2020
…rvice instance (#91)

* Route traffic to cluster local service instance from ingress-gateway

* Override kustomize admiral image tag for releases with the release tag

* Minor documentation edits

Fixes #90
Fixes #62
Fixes #39

Signed-off-by: Madeline <madeline_li@intuit.com>
asushanthk pushed a commit to asushanthk/admiral that referenced this issue Apr 28, 2020
…rvice instance (istio-ecosystem#91)

* Route traffic to cluster local service instance from ingress-gateway

* Override kustomize admiral image tag for releases with the release tag

* Minor documentation edits

Fixes istio-ecosystem#90
Fixes istio-ecosystem#62
Fixes istio-ecosystem#39

Signed-off-by: sa <sushanth_a@intuit.com>
itsLucario pushed a commit to itsLucario/admiral that referenced this issue Aug 9, 2022
…tem#62)

* MESH-000 Add logs for debugging missing service entries
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Something isn't working
Projects
None yet
Development

No branches or pull requests

1 participant