quintilesims / layer0

Build, Manage, and Deploy Your Applications
Apache License 2.0
44 stars 20 forks source link

642: Cross-Zone Load Balancing Management #643

Closed tlake closed 5 years ago

tlake commented 5 years ago

What does this pull request do? Introduces a new feature to load balancers that allows configuration of cross-zone load balancing.

How should this be tested? Spin up a new Layer0 dev API and:

Checklist

closes #642

corresponding issue in JIRA: https://jiraims.rm.imshealth.com/browse/OCEMCM-5776

sesh-kebab commented 5 years ago

Nice work and the quick turn around time! @zpatrick would be impressed.

Initial pass over the code looks fine. But doing a more thorough review pass and QA your RC, will take a bit more time given the current workload. Just wanted to let you know this hasn't been forgotten about.