Skip to content

Conversation

j1m-ryan
Copy link

Proposed changes

Fixes #4970

Checklist

Before creating a PR, run through this checklist and mark each as complete.

  • I have read the CONTRIBUTING doc
  • I have added tests that prove my fix is effective or that my feature works
  • I have checked that all unit tests pass after adding my changes
  • I have updated necessary documentation
  • I have rebased my branch onto main
  • I will ensure my PR is targeting the main branch and pulling from my branch from my own fork

@j1m-ryan j1m-ryan requested a review from a team as a code owner April 26, 2024 12:36
@github-actions github-actions bot added bug An issue reporting a potential bug go Pull requests that update Go code labels Apr 26, 2024
@j1m-ryan j1m-ryan enabled auto-merge (squash) April 26, 2024 16:20
@j1m-ryan j1m-ryan merged commit 3b7340b into main Apr 26, 2024
@j1m-ryan j1m-ryan deleted the fix/named-port-use-cluster-ip branch April 26, 2024 16:43
j1m-ryan pushed a commit that referenced this pull request Apr 29, 2024
…r-ip (#5456) (#5470)

Add support for named ports in ingresses which use-cluster-ip (#5456)
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug An issue reporting a potential bug go Pull requests that update Go code
Projects
None yet
Development

Successfully merging this pull request may close these issues.

NIC Fails to Resolve Named Ports with nginx.org/use-cluster-ip Annotation
3 participants