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.

Sign up for GitHub

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

Jump to bottom

Fix LoadBalancer port protocol assignment #415

Merged

afritzler merged 1 commit into main from fix/port_protocol_assignment

May 1, 2024

Contributor

ushabelgur commented Apr 29, 2024

Proposed Changes

The update modifies the way the protocol of each LoadBalancerPort is assigned. Instead of directly referencing the service's port protocol, a new variable is created for it. This change improves code clarity and helps avoid unexpected behavior due to mutable references.

Fixes #

ushabelgur requested a review from a team as a code owner

April 29, 2024 11:19

github-actions bot added size/XS bug labels


          fix LoadBalancer port protocol assignment

972df2f

ushabelgur force-pushed the fix/port_protocol_assignment branch from be9b778 to 972df2f Compare

April 29, 2024 11:46

afritzler approved these changes

View reviewed changes

afritzler merged commit 3941fad into main

9 checks passed

afritzler deleted the fix/port_protocol_assignment branch

May 1, 2024 08:26

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels