Skip to content

ingress: add per-protocol hostNetwork listener ports#44447

Merged
joestringer merged 1 commit intomainfrom
pr/vk/ingress/host/multi/port/support
Feb 24, 2026
Merged

ingress: add per-protocol hostNetwork listener ports#44447
joestringer merged 1 commit intomainfrom
pr/vk/ingress/host/multi/port/support

Conversation

@viktor-kurchenko
Copy link
Copy Markdown
Contributor

@viktor-kurchenko viktor-kurchenko commented Feb 19, 2026

HostNetwork mode previously forced HTTP/HTTPS/TLS passthrough to share a single listener port. Introduce per-protocol overrides so users can configure distinct ports while keeping the shared port as the fallback.

Fixes: #39422

ingress: allow per-protocol port overrides for HTTP, HTTPS and TLS passthrough in the HostNetwork mode.

@viktor-kurchenko viktor-kurchenko added release-note/minor This PR changes functionality that users may find relevant to operating Cilium. area/operator Impacts the cilium-operator component area/servicemesh GH issues or PRs regarding servicemesh feature/k8s-ingress labels Feb 19, 2026
HostNetwork mode previously forced HTTP/HTTPS/TLS passthrough to share a
single listener port. Introduce per-protocol overrides so users can
configure distinct ports while keeping the shared port as the fallback.

Signed-off-by: viktor-kurchenko <viktor.kurchenko@isovalent.com>
@viktor-kurchenko viktor-kurchenko force-pushed the pr/vk/ingress/host/multi/port/support branch from 8f691d7 to c186c0c Compare February 19, 2026 17:21
@viktor-kurchenko
Copy link
Copy Markdown
Contributor Author

/test

@viktor-kurchenko viktor-kurchenko added backport/1.18 This PR represents a backport for Cilium 1.18.x of a PR that was merged to main. backport/1.19 This PR represents a backport for Cilium 1.19.x of a PR that was merged to main. labels Feb 20, 2026
@viktor-kurchenko viktor-kurchenko marked this pull request as ready for review February 20, 2026 10:44
@viktor-kurchenko viktor-kurchenko requested review from a team as code owners February 20, 2026 10:44
Copy link
Copy Markdown
Member

@qmonnet qmonnet left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Doc changes look OK

@maintainer-s-little-helper maintainer-s-little-helper bot added the ready-to-merge This PR has passed all tests and received consensus from code owners to merge. label Feb 23, 2026
@joestringer joestringer added this pull request to the merge queue Feb 24, 2026
Merged via the queue into main with commit cc5a235 Feb 24, 2026
585 of 589 checks passed
@joestringer joestringer deleted the pr/vk/ingress/host/multi/port/support branch February 24, 2026 01:29
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

area/operator Impacts the cilium-operator component area/servicemesh GH issues or PRs regarding servicemesh backport/1.18 This PR represents a backport for Cilium 1.18.x of a PR that was merged to main. backport/1.19 This PR represents a backport for Cilium 1.19.x of a PR that was merged to main. feature/k8s-ingress ready-to-merge This PR has passed all tests and received consensus from code owners to merge. release-note/minor This PR changes functionality that users may find relevant to operating Cilium.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Ingress controller can only listen on a single host port and not on BOTH 80 and 443

6 participants