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

feat(kuma-cp) new multicluster flow #1122

Merged
merged 7 commits into from
Nov 3, 2020

Conversation

jakubdyszkiewicz
Copy link
Contributor

@jakubdyszkiewicz jakubdyszkiewicz commented Nov 2, 2020

Summary

This PR introduces a new multicluster flow.

Before

When user want to connect Global with Remote they had to

  • Deploy Global
  • Extract public IP of global and use it in Deploy Remote command
  • Wait for Remote to be deployed
  • Extract Remote Ingress address
  • Apply zone with Ingress address
  • 🎉

After

When user want to connect Global with Remote they have to

  • Deploy Global
  • Extract public IP of global and use it in Deploy Remote command
  • 🎉

Full changelog

  • Introduced publicAddress and publicPort to Ingress data plane that indicates the address and port that is accessible to other clusters
  • Removed flattening of Ingresses, therefore you don't have to have LB for them. Every ingress can have separate public IP
  • On Kubernetes, automatically resolve publicAddress and publicPort. It works with both LoadBalancer services and NodePort.
    When LoadBalancer Service waits for its address, we still have an Ingress synced but without public coordinates, which means it won't be used.
    When LoadBalancer Service receives IP, it's automatically updated by Kubernetes controlled, synced to Global, and synced to other Remotes.
    You can override the address and port with annotations if you wish.
  • Zone is now automatically created when Remote initiate a stream to Global (which is symmetric with DP that connects to CP creating Dataplane definition)
  • Remove Ingress from zones tab in the GUI chore(zones) remove url column kuma-gui#81

Documentation

  • In progress

Signed-off-by: Jakub Dyszkiewicz <[email protected]>
Signed-off-by: Jakub Dyszkiewicz <[email protected]>
@jakubdyszkiewicz jakubdyszkiewicz requested a review from a team as a code owner November 2, 2020 10:52
Signed-off-by: Jakub Dyszkiewicz <[email protected]>
@jakubdyszkiewicz jakubdyszkiewicz force-pushed the feat/ingress-public-address branch from c0134bf to f4f44fd Compare November 3, 2020 11:29
@jakubdyszkiewicz jakubdyszkiewicz merged commit c8f3cd5 into master Nov 3, 2020
@jakubdyszkiewicz jakubdyszkiewicz deleted the feat/ingress-public-address branch November 3, 2020 12:58
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.

3 participants