Fix separate TCP and UDP listeners on the same port for… by 210g · Pull Request #533 · oracle/oci-cloud-controller-manager

Skip to content

Navigation Menu

Sign in

Appearance settings

Provide feedback

We read every piece of feedback, and take your input very seriously.

Saved searches

Use saved searches to filter your results more quickly

Sign up

Appearance settings

Conversation

@210g

Copy link Copy Markdown

@210g 210g commented

Mar 31, 2026

Ref: #532

This PR fixes the issue by conditionally separating the listeners:

  • If NodePorts are different: It creates separate Listeners and BackendSets (e.g., TCP-80 and UDP-80) to route traffic accurately to their respective NodePorts.
  • If NodePorts are the same (or omitted/0): It retains the existing behavior and creates a single TCP_AND_UDP Listener for backward compatibility and resource optimization.
  • To ensure backward compatibility, the logic compares the NodePort of the ServicePorts. The existing ProtocolTypeMixed logic is bypassed only when the NodePorts explicitly differ.
  • Minor optimizations with string processing

Related unit tests passed.

… NLB

Signed-off-by: Minyoung Oh <minyoung.oh23@gmail.com>

@oracle-contributor-agreement oracle-contributor-agreement bot added the OCA Verified

All contributors have signed the Oracle Contributor Agreement.

label

Mar 31, 2026

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

Reviewers

No reviews

Assignees

No one assigned

Labels

OCA Verified

All contributors have signed the Oracle Contributor Agreement.

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

1 participant

@210g