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

Send AZ in StartActualLRP/EvacuateRunningActualLRP requests to BBS #48

Merged
merged 1 commit into from
Nov 9, 2023

Conversation

mariash
Copy link
Member

@mariash mariash commented Oct 25, 2023

What is this change about?

Rep will pass availability zone it is configured with in BBS calls StartActualLRP and EvacuateRunningActualLRP.

What problem it is trying to solve?

This will allow BBS to store and pass through availability zone to the Route Emitter which will advertise in Registry Message for the Gorouter so that it can make decisions based on balancing algorithm.

How should this change be described in diego-release release notes?

[Feature] Diego advertises the availability zone of an LRP to enable clever intra-AZ routing

Please provide any contextual information.

tracker story
issue

Thank you!

@mariash mariash requested a review from a team as a code owner October 25, 2023 19:16
@MarcPaquette
Copy link
Contributor

Is this okay to merge in?

@jrussett jrussett merged commit b2c8f5c into main Nov 9, 2023
@jrussett jrussett deleted the az-local-routing branch November 10, 2023 01:08
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