Skip to content
This repository has been archived by the owner on Sep 16, 2020. It is now read-only.

Refactor loadbalancer #135

Open
wants to merge 3 commits into
base: master
Choose a base branch
from

Conversation

vchrisb
Copy link
Contributor

@vchrisb vchrisb commented Apr 9, 2019

this is related to #134
This PR is making the load_balancer module more flexible by:

  • allowing to have multiple global lbs
  • making tcp router optional (default is true)
  • using count straight forward

Variables url_map_name, http_proxy_name, https_proxy_name, http_forwarding_rule_name and https_forwarding_rule_name were introduced to keep it backwards compatible.

Therefore all existing objects and names are kept, hence no update to existing deployments is required!

if merged, I would make consecutive PRs for adding an optional LB for service mesh (service mesh branch) and refactor isolation segment module to also use the load_balancer module (isoseg branch).

vchrisb added 2 commits April 9, 2019 10:44
making the load_balancer module more flexible by:
* allowing to have multiple globe lbs
* making tcp router optional
* using count straight forward

this isn't introducing and changes for existing objects, hence this is backwards compatible
`global` and `global_lb`are bools
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants