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

Figure out multi namespace / multi control plane / mesh configuration #632

Closed
ldemailly opened this issue Aug 31, 2017 · 9 comments
Closed

Comments

@ldemailly
Copy link
Member

ldemailly commented Aug 31, 2017

For 0.3 we need to solve the bootstrap configuration issue for the side cars

For testing (at least and also probably (tbd?) for production use cases) we want to support

multiple sets of multiple namespaces using 1 istio control plane and config in the same cluster
(and also multi cluster meshes eventually)

Right now the way the sidecar finds pilot and mixer is by assuming there is a mounted configmap coming from the namespace - that doesn't scale well with dozens of namespaces

See istio/old_pilot_repo#1153 and istio/old_pilot_repo#1154 for some 0.2 background

Note: even if we don't want to support multiple meshes in prod, the bootstrap can use streamlining and shouldn't require copying a configmap in each namespace (imo)

@andraxylia
Copy link
Contributor

This is not a scale problem with dozens or event hundred of namespaces unless proven otherwise. It happens as a namespace is created.
This issue jus makes an opinion, and does not propose an alternate viable and better proposal. I will leave it open for tracking for the first few weeks of 0.3, and close it if nothing better is proposed.

@andraxylia
Copy link
Contributor

This should also be moved to pilot.

@ldemailly ldemailly removed their assignment Sep 4, 2017
@ldemailly
Copy link
Member Author

There is relevant discussion in newly filled istio/old_pilot_repo#1272 but this issue is istio wide not only pilot related

@ldemailly ldemailly added the Epic label Sep 22, 2017
mandarjog pushed a commit to mandarjog/istio that referenced this issue Oct 30, 2017
)

* Documented but unimplemented routes should return 501 not 404

* Re-order routes and name ruleid parameter


Former-commit-id: ce5f06848652e1dd8d7c5533b497bde0f8aec1c3
mandarjog pushed a commit that referenced this issue Oct 31, 2017
* Documented but unimplemented routes should return 501 not 404

* Re-order routes and name ruleid parameter


Former-commit-id: 228338f654228b11060551f29cd6fb9f78aba6c6
@andraxylia andraxylia modified the milestones: Istio 0.3, Nebulous Future Oct 31, 2017
mandarjog pushed a commit that referenced this issue Oct 31, 2017
* New ?codes= option to simulate HTTP errors from echo server

* Don't change BUILD, it isn't needed

* Restore BUILD
@andraxylia
Copy link
Contributor

This is no longer an issue.

@ldemailly
Copy link
Member Author

it's only no longer an issue if we don't want to support multiple control planes in a cluster

@andraxylia
Copy link
Contributor

No concrete plans for multiple control planes, so let's re-open if/when needed, so we can focus on the ones we are working on.

@ldemailly
Copy link
Member Author

sgtm / agreed

@msogrin
Copy link

msogrin commented Jan 30, 2018

This is no longer an issue.

@andraxylia Could you please elaborate on the solution/conclusion/workaround/change_of_direction/etc?

@ldemailly
Copy link
Member Author

@msogrin we fixed the unnecessary config map copying part of the issue (the bug/perf/scale issue)

and for now we are assuming istio watches all your namespaces in any given namespace

@rlenglet rlenglet removed this from the Nebulous Future milestone Jul 9, 2019
howardjohn pushed a commit to howardjohn/istio that referenced this issue Jan 12, 2020
* Support customized labels for istio-ingress

Signed-off-by: Chun Lin Yang <clyang@cn.ibm.com>

* update istio-egress
howardjohn pushed a commit to howardjohn/istio that referenced this issue Jan 12, 2020
* Remove unnecessary env variables

* update goldens
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

5 participants