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.

Sign up for GitHub

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

Jump to bottom

Support for Multi-cluster istio setup #437

Closed
Aditya-Chowdhry opened this issue Feb 13, 2020 · 1 comment
Closed

Support for Multi-cluster istio setup #437

Aditya-Chowdhry opened this issue Feb 13, 2020 · 1 comment
Labels
help wanted Extra attention is needed kind/feature Feature request

Comments

@Aditya-Chowdhry
Copy link

Hi,

We have a multi-cluster Istio setup. One host cluster and multiple remote clusters.
We create virtual services for all applications irrespective of the cluster in the host cluster to enable service to service communication. https://istio.io/docs/setup/install/multicluster/shared-vpn/

From Flagger docs, it is assumed that there will be only one cluster where Istio is installed.
Does flagger support multi-cluster setup?
@stefanprodan
Copy link
Member

Does flagger support multi-cluster setup?

No it doesn't. To support Istio multi-cluster, the Flagger instances running on the remote clusters should create/update the Istio objects on the host cluster.

I think these changes are needed:

  • mount the Istio host cluster kubeconfig in Flagger pod
  • make the Istio router clientset use the Istio host cluster kubeconfig

@stefanprodan stefanprodan added kind/feature Feature request help wanted Extra attention is needed labels Feb 13, 2020
viditganpi pushed a commit to viditganpi/flagger that referenced this issue Feb 18, 2020
Subject:- Canary support for istio multi cluster
2cb6ce4 
Body:-
If applied this commit will resolve the following reported issue fluxcd#437
Have added support for consuming kubeconfig of istio host cluster where istio
resources will be created.
@viditganpi viditganpi mentioned this issue Feb 18, 2020
Merged
@stefanprodan stefanprodan mentioned this issue Feb 20, 2020
Merged
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
help wanted Extra attention is needed kind/feature Feature request
Projects
None yet
Milestone
No milestone
Development

No branches or pull requests
2 participants
@stefanprodan @Aditya-Chowdhry