- 
                Notifications
    You must be signed in to change notification settings 
- Fork 21.9k
Revise backend pool names and add second backend pool #127786
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
base: main
Are you sure you want to change the base?
Conversation
In this scenario, 2nd backend pool is needed, as in current scenario, both backend VMs respond to both frontend IPs. This effectively breaks DHCP relay HA scenario, as requests to both Frontend Addresses may go to the same backend server, breaking HA provided by Load Balancer. Also, taking into consideration that source port of DHCP RElay is always 67, load balancing will not work as you would expece for ordinary connections (both Src IP and Port, and DST IP And port are always the same), this means connections will stick to some servers. In situation, where DHCP Servers on VMs may be also configured in HA LB mode, this can fully break DHCP functionality.
| @microsoft-github-policy-service agree company="Microsoft" | 
| Learn Build status updates of commit 11fbac5: ✅ Validation status: passed
 For more details, please refer to the build report. | 
| @mobileha : Thanks for your contribution! The author(s) and reviewer(s) have been notified to review your proposed change. | 
| Can you review the proposed changes? IMPORTANT: When the changes are ready for publication, adding a  #label:"aq-pr-triaged" | 
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.
Pull Request Overview
This PR addresses a critical DHCP relay high availability issue where both backend VMs were responding to both frontend IPs, breaking the HA scenario. The changes introduce a second backend pool to ensure proper traffic distribution and prevent DHCP functionality issues caused by sticky connections.
Key changes:
- Renamed the original backend pool from backend-pooltobackend-pool-1for consistency
- Added a second backend pool (backend-pool-2) with configuration instructions
- Updated load balancer rules to reference the appropriate backend pools
💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.
Co-authored-by: Copilot <[email protected]>
Co-authored-by: Copilot <[email protected]>
| Learn Build status updates of commit b896e57: ✅ Validation status: passed
 For more details, please refer to the build report. | 
| Learn Build status updates of commit c98077e: ✅ Validation status: passed
 For more details, please refer to the build report. | 
In this scenario, 2nd backend pool is needed, as in current scenario, both backend VMs respond to both frontend IPs. This effectively breaks DHCP relay HA scenario, as requests to both Frontend Addresses may go to the same backend server, breaking HA provided by Load Balancer. Also, taking into consideration that source port of DHCP RElay is always 67, load balancing will not work as you would expece for ordinary connections (both Src IP and Port, and DST IP And port are always the same), this means connections will stick to some servers. In situation, where DHCP Servers on VMs may be also configured in HA LB mode, this can fully break DHCP functionality.
VMs LB template that included into this document also needs reviewing or adding here as static text.