-
Notifications
You must be signed in to change notification settings - Fork 178
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
Update RFC 0027: Unify load balancing property name #1039
Update RFC 0027: Unify load balancing property name #1039
Conversation
Unify loadbalancing property
@beyhan this is ready to be reviewed. Do we need full TOC approval for such updates? |
@maxmoehl yes, we need. |
Change was proposed here: cloudfoundry/capi-release#482 (comment) |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
The changes to route registrar will alter bosh properties that already exist. This will need to be marked as a breaking change. I'll comment more on specific PRs.
In cloudfoundry/capi-release#482, it was decided to unify the API property and internal state for the per-route load-balancing algorithm feature to
loadbalancing
. This was already introduced into cloud controller in cloudfoundry/cloud_controller_ng#4080The following component PRs accompany this change in the RFC: