Skip to content
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

Automated backport of #3220: Add TCP port to Lighthouse CoreDNS ClusterIP service #3229

Conversation

tpantelis
Copy link
Contributor

Backport of #3220 on release-0.17.

#3220: Add TCP port to Lighthouse CoreDNS ClusterIP service

For details on the backport process, see the backport requests page.

Define TCP port in ClusterIP service to allow k8s DNS
server to forward TCP connection requests.
Lighthouse CoreDNS pods already support TCP on port 53.
This is needed to support TCP retries after truncation
per RFC1035 and RFC2181.
Signed-off-by: Tony Le <[email protected]>
@submariner-bot
Copy link
Contributor

🤖 Created branch: z_pr3229/tpantelis/automated-backport-of-#3220-upstream-release-0.17
🚀 Full E2E won't run until the "ready-to-test" label is applied. I will add it automatically once the PR has 2 approvals, or you can add it manually.

@tpantelis tpantelis added the release-note-needed Should be mentioned in the release notes label Sep 17, 2024
@tpantelis tpantelis enabled auto-merge (rebase) September 17, 2024 18:31
@submariner-bot submariner-bot added the ready-to-test When a PR is ready for full E2E testing label Sep 23, 2024
@tpantelis tpantelis merged commit 864aef9 into submariner-io:release-0.17 Sep 23, 2024
50 checks passed
@submariner-bot
Copy link
Contributor

🤖 Closed branches: [z_pr3229/tpantelis/automated-backport-of-#3220-upstream-release-0.17]

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
automated-backport ready-to-test When a PR is ready for full E2E testing release-note-handled release-note-needed Should be mentioned in the release notes
Projects
None yet
Development

Successfully merging this pull request may close these issues.

5 participants