Skip to content

Conversation

@mjethwa-msft
Copy link
Contributor

This pull request introduces significant enhancements and new features to the load balancer and network interface protocol buffer definitions, as well as updates to the Go module dependencies. The main focus is on expanding support for advanced load balancer configurations, including outbound NAT rules, probes, frontend IP configurations, and backend address pools, and ensuring these are reflected in both proto and generated Go code. Additionally, some dependency versions have been updated.

Load Balancer Feature Expansion

  • Added new message types and enums to moc_cloudagent_loadbalancer.proto to support outbound NAT rules (LoadbalancerOutboundNatRule), load distribution strategies (LoadDistribution), probe protocols (ProbeProtocol), and resource tagging. Also introduced new messages for Probe, FrontEndIpConfiguration, and BackendAddressPool, and expanded the LoadBalancer message to include these new fields and repeated lists for advanced configuration. [1] [2]

  • Included an ApiVersion field in LoadBalancerRequest to support versioning for API requests.

Network Interface Feature Expansion

  • Added a repeated field loadBalancerAddressPoolsRef to the IpConfiguration message in both the proto file (moc_cloudagent_networkinterface.proto) and the generated Go code, allowing IP configurations to reference multiple backend address pools. [1] [2]
  • Implemented a corresponding getter method in Go for LoadBalancerAddressPoolsRef to retrieve these references from IpConfiguration.

Dependency Updates

  • Updated several indirect Go module dependencies to earlier versions in go.mod, including golang.org/x/net, golang.org/x/sys, golang.org/x/text, google.golang.org/genproto/googleapis/rpc, and google.golang.org/protobuf, likely for compatibility with proto changes.

Generated Code Synchronization

  • Regenerated the Go file descriptor for moc_cloudagent_networkinterface.pb.go to reflect the new and updated proto definitions, increasing the descriptor size and syncing with the new proto schema.

@mjethwa-msft
Copy link
Contributor Author

/azp run

@azure-pipelines
Copy link

Azure Pipelines successfully started running 1 pipeline(s).

@mjethwa-msft mjethwa-msft force-pushed the user/mjethwa-msft/slb-v2 branch from 9d1926e to f919bb4 Compare September 30, 2025 19:11
@mjethwa-msft
Copy link
Contributor Author

/azp run

@azure-pipelines
Copy link

Azure Pipelines successfully started running 1 pipeline(s).

@mjethwa-msft mjethwa-msft marked this pull request as ready for review September 30, 2025 23:47
madhanrm
madhanrm previously approved these changes Sep 30, 2025
@mjethwa-msft mjethwa-msft force-pushed the user/mjethwa-msft/slb-v2 branch from f919bb4 to 2fc7eb1 Compare October 27, 2025 17:27
@mjethwa-msft
Copy link
Contributor Author

/azp run

@azure-pipelines
Copy link

Azure Pipelines successfully started running 1 pipeline(s).

@mjethwa-msft
Copy link
Contributor Author

/azp run

@azure-pipelines
Copy link

Azure Pipelines successfully started running 1 pipeline(s).

@mjethwa-msft mjethwa-msft merged commit cefc792 into main Nov 6, 2025
7 checks passed
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.

6 participants